bug #1049 Don't copy constraint from framework-bundle when installing a new symfony package (nicolas-grekas)

nicolas-grekas · nicolas-grekas · commit ccc4d2dc15e9 · 2025-05-21T09:23:28.000+02:00
This PR was merged into the 2.x branch. Discussion ---------- Don't copy constraint from framework-bundle when installing a new symfony package Fix #989 Commits ------- 39ff4d5 Dont copy constraint from framework-bundle when installing a new symfony package
diff --git a/src/PackageResolver.php b/src/PackageResolver.php
@@ -68,11 +68,11 @@ public function parseVersion(string $package, string $version, bool $isRequire):
             try {
                 $config = @json_decode(file_get_contents(Factory::getComposerFile()), true);
             } finally {
-                if (!$isRequire || !(isset($config['extra']['symfony']['require']) || isset($config['require']['symfony/framework-bundle']))) {
+                if (!$isRequire || !isset($config['extra']['symfony']['require'])) {
                     return '';
                 }
             }
-            $version = $config['extra']['symfony']['require'] ?? $config['require']['symfony/framework-bundle'];
+            $version = $config['extra']['symfony']['require'];
         } elseif ('dev' === $version) {
             $version = '^'.$versions['dev-name'].'@dev';
         } elseif ('next' === $version) {

Original file line number	Diff line number	Diff line change
`@@ -68,11 +68,11 @@ public function parseVersion(string $package, string $version, bool $isRequire):`
`68`	`68`	`try {`
`69`	`69`	`$config = @json_decode(file_get_contents(Factory::getComposerFile()), true);`
`70`	`70`	`} finally {`
`71`		`- if (!$isRequire \|\| !(isset($config['extra']['symfony']['require']) \|\| isset($config['require']['symfony/framework-bundle']))) {`
	`71`	`+ if (!$isRequire \|\| !isset($config['extra']['symfony']['require'])) {`
`72`	`72`	`return '';`
`73`	`73`	`}`
`74`	`74`	`}`
`75`		`- $version = $config['extra']['symfony']['require'] ?? $config['require']['symfony/framework-bundle'];`
	`75`	`+ $version = $config['extra']['symfony']['require'];`
`76`	`76`	`} elseif ('dev' === $version) {`
`77`	`77`	`$version = '^'.$versions['dev-name'].'@dev';`
`78`	`78`	`} elseif ('next' === $version) {`