Skip to content

Commit decd0cf

Browse files
committed
Avoid doing noop package transfers, refs #1609
1 parent d8a93d5 commit decd0cf

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

src/Command/TransferOwnershipCommand.php

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -165,9 +165,18 @@ private function outputPackageTable(OutputInterface $output, array $packages, ar
165165
*/
166166
private function transferOwnership(array $packages, array $maintainers): void
167167
{
168+
$normalizedMaintainers = array_values(array_map(fn (User $user) => $user->getId(), $maintainers));
169+
sort($normalizedMaintainers, SORT_NUMERIC);
170+
168171
foreach ($packages as $package) {
169172
$oldMaintainers = $package->getMaintainers()->toArray();
170173

174+
$normalizedOldMaintainers = array_values(array_map(fn (User $user) => $user->getId(), $oldMaintainers));
175+
sort($normalizedOldMaintainers, SORT_NUMERIC);
176+
if ($normalizedMaintainers === $normalizedOldMaintainers) {
177+
continue;
178+
}
179+
171180
$package->getMaintainers()->clear();
172181
foreach ($maintainers as $maintainer) {
173182
$package->addMaintainer($maintainer);

0 commit comments

Comments
 (0)