public-mirrors
/
composer
mirror of https://github.com/composer/composer
1
0
Fork 0
Code

Add failing test and fix attempt

pull/11798/head
Jordi Boggiano 2024-01-17 11:28:13 +01:00
parent 4e5be9ee7d
commit 05d5db9475
No known key found for this signature in database
GPG Key ID: 7BBD42C429EC80BC
2 changed files with 69 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
src/Composer/DependencyResolver/RuleSetGenerator.php
View File

@ -17,6 +17,7 @@ use Composer\Filter\PlatformRequirementFilter\PlatformRequirementFilterFactory;
use Composer\Filter\PlatformRequirementFilter\PlatformRequirementFilterInterface;
use Composer\Package\BasePackage;
use Composer\Package\AliasPackage;
use Composer\Package\RootPackageInterface;
/**
* @author Nils Adermann <naderman@naderman.de>
@ -203,6 +204,16 @@ class RuleSetGenerator
$this->addRule(RuleSet::TYPE_PACKAGE, $this->createRequireRule($package, $possibleRequires, Rule::RULE_PACKAGE_REQUIRES, $link));
foreach ($possibleRequires as $require) {
$requirers = [];
foreach ($this->pool->getPackages() as $pkg) {
if (isset($pkg->getRequires()[$require->getName()])) {
$requirers[] = $pkg;
}
}
if ($requirers) {
$this->addRule(RuleSet::TYPE_PACKAGE, $this->createRequireRule($package, $requirers, Rule::RULE_PACKAGE_REQUIRES, null));
}
$workQueue->enqueue($require);
}
}
@ -279,6 +290,16 @@ class RuleSetGenerator
if ($packages) {
foreach ($packages as $package) {
$this->addRulesForPackage($package, $platformRequirementFilter);
$requirers = [];
foreach ($this->pool->getPackages() as $pkg) {
if (isset($pkg->getRequires()[$package->getName()])) {
$requirers[] = $pkg;
}
}
if ($requirers) {
$this->addRule(RuleSet::TYPE_PACKAGE, $this->createRequireRule($package, $requirers, Rule::RULE_PACKAGE_REQUIRES, null));
}
}
$rule = $this->createInstallOneOfRule($packages, Rule::RULE_ROOT_REQUIRE, [

48
tests/Composer/Test/Fixtures/installer/provider-packages-are-not-installed-if-not-selected.test Normal file
View File

@ -0,0 +1,48 @@
--TEST--
Test that providers are not installed unless really required by something
--COMPOSER--
{
"repositories": [
{
"type": "package",
"package": [
{
"name": "foo/polyfill",
"provide": {
"foo/requirement": "1.0.0"
},
"version": "1.0.0"
},
{
"name": "foo/indirect",
"require": {
"foo/polyfill": "1.0.0"
},
"version": "1.0.0"
},
{
"name": "foo/indirect",
"version": "1.0.1"
},
{
"name": "foo/requirement",
"require": {
"foo/does-not-exist": "1.0.0"
},
"version": "1.0.0"
}
]
}
],
"require": {
"foo/requirement": "1.0.0",
"foo/indirect": "^1.0"
}
}
--RUN--
update
--EXPECT--
Installing foo/polyfill (1.0.0)
Installing foo/indirect (1.0.0)