Skip to content

Commit a34a1a2

Browse files
committed
Fixing "composer require orm orm" bug
1 parent 5bc34bd commit a34a1a2

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

src/DependencyBuilder.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ private function getRequiredDependencyNames(array $dependencies)
113113
$packages[] = $package['name'];
114114
}
115115

116-
return $packages;
116+
return array_unique($packages);
117117
}
118118

119119
private function calculateMissingDependencies(array $dependencies)
@@ -134,6 +134,6 @@ private function calculateMissingDependencies(array $dependencies)
134134
return [];
135135
}
136136

137-
return array_merge($missingPackages, $missingOptionalPackages);
137+
return array_unique(array_merge($missingPackages, $missingOptionalPackages));
138138
}
139139
}

tests/DependencyBuilderTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ public function testGetAllRequiredDependencies()
1414
$depBuilder->addClassDependency('Bar', 'bar-package');
1515
$depBuilder->addClassDependency('DevStuff', 'dev-stuff-package', true, true);
1616
$depBuilder->addClassDependency('DevStuff2', 'dev-stuff2-package', true, true);
17+
$depBuilder->addClassDependency('DevStuff3', 'dev-stuff2-package', true, true);
1718

1819
$this->assertSame(['foo-package', 'bar-package'], $depBuilder->getAllRequiredDependencies());
1920
$this->assertSame(['dev-stuff-package', 'dev-stuff2-package'], $depBuilder->getAllRequiredDevDependencies());

0 commit comments

Comments
 (0)