diff --git a/src/Composer/DependencyResolver/RuleSetGenerator.php b/src/Composer/DependencyResolver/RuleSetGenerator.php index afd735e7f..d415dd14e 100644 --- a/src/Composer/DependencyResolver/RuleSetGenerator.php +++ b/src/Composer/DependencyResolver/RuleSetGenerator.php @@ -223,7 +223,7 @@ class RuleSetGenerator if ($platformRequirementFilter->isIgnored($link->getTarget())) { continue; } elseif ($platformRequirementFilter instanceof IgnoreListPlatformRequirementFilter) { - $constraint = $platformRequirementFilter->filterConstraint($link->getTarget(), $constraint); + $constraint = $platformRequirementFilter->filterConstraint($link->getTarget(), $constraint, false); } $conflicts = $this->pool->whatProvides($link->getTarget(), $constraint); diff --git a/src/Composer/Filter/PlatformRequirementFilter/IgnoreListPlatformRequirementFilter.php b/src/Composer/Filter/PlatformRequirementFilter/IgnoreListPlatformRequirementFilter.php index 700795832..43cc33469 100644 --- a/src/Composer/Filter/PlatformRequirementFilter/IgnoreListPlatformRequirementFilter.php +++ b/src/Composer/Filter/PlatformRequirementFilter/IgnoreListPlatformRequirementFilter.php @@ -60,13 +60,16 @@ final class IgnoreListPlatformRequirementFilter implements PlatformRequirementFi return Preg::isMatch($this->ignoreRegex, $req); } - public function filterConstraint(string $req, ConstraintInterface $constraint): ConstraintInterface + /** + * @param bool $allowUpperBoundOverride For conflicts we do not want the upper bound to be skipped + */ + public function filterConstraint(string $req, ConstraintInterface $constraint, bool $allowUpperBoundOverride = true): ConstraintInterface { if (!PlatformRepository::isPlatformPackage($req)) { return $constraint; } - if (!Preg::isMatch($this->ignoreUpperBoundRegex, $req)) { + if (!$allowUpperBoundOverride || !Preg::isMatch($this->ignoreUpperBoundRegex, $req)) { return $constraint; } diff --git a/tests/Composer/Test/Fixtures/installer/update-ignore-platform-package-requirement-list-upper-bounds.test b/tests/Composer/Test/Fixtures/installer/update-ignore-platform-package-requirement-list-upper-bounds.test index 0c7e776de..62d8fbb4b 100644 --- a/tests/Composer/Test/Fixtures/installer/update-ignore-platform-package-requirement-list-upper-bounds.test +++ b/tests/Composer/Test/Fixtures/installer/update-ignore-platform-package-requirement-list-upper-bounds.test @@ -7,13 +7,15 @@ Update with ignore-platform-req list ignoring upper bound of a dependency "type": "package", "package": [ { "name": "a/a", "version": "1.0.1", "require": { "ext-foo-bar": "3.*" } }, - { "name": "b/b", "version": "1.0.1", "require": { "ext-foo-bar": "10.*" } } + { "name": "b/b", "version": "1.0.1", "require": { "ext-foo-bar": "10.*" } }, + { "name": "c/c", "version": "1.0.1", "conflict": { "ext-foo-bar": "4.0.0 - 4.0.2", "php": "3.0.*" } } ] } ], "require": { "a/a": "1.0.*", "b/b": "1.0.*", + "c/c": "1.0.*", "php": "^4.3", "ext-foo-baz": "9.0.0" },