Skip to content

Commit 912733d

Browse files
DanielSiepmannDanielSiepmann
committed
[FEATURE] Add support for composer.json only extensions
The internal array always uses the extension key. Not having an ext_emconf.php with dependencies to extension keys, but only a composer.json with dependencies to package names will always result in: UnexpectedValueException: The package "extension_key" depends on "composer/package-key" which is not present in the system. This is a common situation in modern TYPO3 projects where multiple extensions are given which declare their internal dependencies to have proper TYPO3 loading order. A fallback from the dependency (which might be the composer name) to the actual extension key is added. Resolves: #541
1 parent ee62a11 commit 912733d

File tree

6 files changed

+164
-5
lines changed

6 files changed

+164
-5
lines changed

‎Classes/Core/PackageCollection.php‎

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,8 @@ protected function convertConfigurationForGraph(array $allPackageConstraints, ar
162162
];
163163
if (isset($allPackageConstraints[$packageKey]['dependencies'])) {
164164
foreach ($allPackageConstraints[$packageKey]['dependencies'] as $dependentPackageKey) {
165-
if (!in_array($dependentPackageKey, $packageKeys, true)) {
165+
$extensionKey = $this->composerPackageManager->getPackageInfo($dependentPackageKey)?->getExtensionKey() ?? '';
166+
if (!in_array($dependentPackageKey, $packageKeys, true) && !in_array($extensionKey, $packageKeys, true)) {
166167
if ($this->isComposerDependency($dependentPackageKey)) {
167168
// The given package has a dependency to a Composer package that has no relation to TYPO3
168169
// We can ignore those, when calculating the extension order
@@ -174,7 +175,7 @@ protected function convertConfigurationForGraph(array $allPackageConstraints, ar
174175
1519931815
175176
);
176177
}
177-
$dependencies[$packageKey]['after'][] = $dependentPackageKey;
178+
$dependencies[$packageKey]['after'][] = $extensionKey ?: $dependentPackageKey;
178179
}
179180
}
180181
if (isset($allPackageConstraints[$packageKey]['suggestions'])) {
@@ -255,15 +256,18 @@ protected function getDependencyArrayForPackage(PackageInterface $package, array
255256
foreach ($dependentPackageConstraints as $constraint) {
256257
if ($constraint instanceof PackageConstraint) {
257258
$dependentPackageKey = $constraint->getValue();
259+
$extensionKey = $this->composerPackageManager->getPackageInfo($dependentPackageKey)?->getExtensionKey() ?? '';
258260
if (!in_array($dependentPackageKey, $dependentPackageKeys, true) && !in_array($dependentPackageKey, $trace, true)) {
259-
$dependentPackageKeys[] = $dependentPackageKey;
261+
$dependentPackageKeys[] = $extensionKey ?: $dependentPackageKey;
260262
}
261-
if (!isset($this->packages[$dependentPackageKey])) {
263+
264+
if (!isset($this->packages[$dependentPackageKey]) && !isset($this->packages[$extensionKey])) {
262265
if ($this->isComposerDependency($dependentPackageKey)) {
263266
// The given package has a dependency to a Composer package that has no relation to TYPO3
264267
// We can ignore those, when calculating the extension order
265268
continue;
266269
}
270+
267271
throw new Exception(
268272
sprintf(
269273
'Package "%s" depends on package "%s" which does not exist.',
@@ -273,7 +277,7 @@ protected function getDependencyArrayForPackage(PackageInterface $package, array
273277
1695119749
274278
);
275279
}
276-
$this->getDependencyArrayForPackage($this->packages[$dependentPackageKey], $dependentPackageKeys, $trace);
280+
$this->getDependencyArrayForPackage($this->packages[$dependentPackageKey] ?? $this->packages[$extensionKey], $dependentPackageKeys, $trace);
277281
}
278282
}
279283
return array_reverse($dependentPackageKeys);

‎Tests/Unit/Core/Fixtures/Packages/PackageStates.php‎

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
<?php
2+
return [
3+
'packages' => [
4+
'core' => [
5+
'packagePath' => '.Build/vendor/typo3/cms-core/',
6+
],
7+
'extbase' => [
8+
'packagePath' => '.Build/vendor/typo3/cms-extbase/',
9+
],
10+
'fluid' => [
11+
'packagePath' => '.Build/vendor/typo3/cms-fluid/',
12+
],
13+
'backend' => [
14+
'packagePath' => '.Build/vendor/typo3/cms-backend/',
15+
],
16+
'frontend' => [
17+
'packagePath' => '.Build/vendor/typo3/cms-frontend/',
18+
],
19+
'package0' => [
20+
'packagePath' => 'Tests/Unit/Core/Fixtures/Packages/package0/',
21+
],
22+
'package2' => [
23+
'packagePath' => 'Tests/Unit/Core/Fixtures/Packages/package2/',
24+
],
25+
'package1' => [
26+
'packagePath' => 'Tests/Unit/Core/Fixtures/Packages/package1/',
27+
],
28+
],
29+
'version' => 5,
30+
];

‎Tests/Unit/Core/Fixtures/Packages/package0/composer.json‎

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
{
2+
"name": "typo3/testing-framework-package-0",
3+
"description": "Package 0",
4+
"type": "typo3-cms-extension",
5+
"license": [
6+
"GPL-2.0-or-later"
7+
],
8+
"require": {
9+
"php": "*",
10+
"typo3/cms-core": "*"
11+
},
12+
"extra": {
13+
"typo3/cms": {
14+
"extension-key": "package0"
15+
}
16+
}
17+
}

‎Tests/Unit/Core/Fixtures/Packages/package1/composer.json‎

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
{
2+
"name": "typo3/testing-framework-package-1",
3+
"description": "Package 1, with replace entry",
4+
"type": "typo3-cms-extension",
5+
"license": [
6+
"GPL-2.0-or-later"
7+
],
8+
"require": {
9+
"php": "*",
10+
"typo3/cms-core": "*"
11+
},
12+
"replace": {
13+
"typo3-ter/package1": "self.version"
14+
},
15+
"extra": {
16+
"typo3/cms": {
17+
"extension-key": "package1"
18+
}
19+
}
20+
}

‎Tests/Unit/Core/Fixtures/Packages/package2/composer.json‎

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
{
2+
"name": "typo3/testing-framework-package-2",
3+
"description": "Package 2 depending on package 1",
4+
"type": "typo3-cms-extension",
5+
"license": [
6+
"GPL-2.0-or-later"
7+
],
8+
"require": {
9+
"php": "*",
10+
"typo3/testing-framework-package-1": "*",
11+
"typo3/testing-framework-package-0": "*",
12+
"typo3/cms-core": "*"
13+
},
14+
"extra": {
15+
"typo3/cms": {
16+
"extension-key": "package2"
17+
}
18+
}
19+
}

‎Tests/Unit/Core/PackageCollectionTest.php‎

Lines changed: 69 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,69 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
/*
6+
* Copyright (C) 2024 Daniel Siepmann <coding@daniel-siepmann.de>
7+
*
8+
* This program is free software; you can redistribute it and/or
9+
* modify it under the terms of the GNU General Public License
10+
* as published by the Free Software Foundation; either version 2
11+
* of the License, or (at your option) any later version.
12+
*
13+
* This program is distributed in the hope that it will be useful,
14+
* but WITHOUT ANY WARRANTY; without even the implied warranty of
15+
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
16+
* GNU General Public License for more details.
17+
*
18+
* You should have received a copy of the GNU General Public License
19+
* along with this program; if not, write to the Free Software
20+
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
21+
* 02110-1301, USA.
22+
*/
23+
24+
namespace Typo3\TestingFramework\Tests\Unit\Core;
25+
26+
use TYPO3\CMS\Core\Package\PackageManager;
27+
use TYPO3\CMS\Core\Service\DependencyOrderingService;
28+
use PHPUnit\Framework\TestCase;
29+
use TYPO3\TestingFramework\Composer\ComposerPackageManager;
30+
use TYPO3\TestingFramework\Core\PackageCollection;
31+
32+
final class PackageCollectionTest extends TestCase
33+
{
34+
/**
35+
* @test
36+
*/
37+
public function sortsComposerPackages(): void
38+
{
39+
$packageStates = require __DIR__ . '/Fixtures/Packages/PackageStates.php';
40+
$packageStates = $packageStates['packages'];
41+
$basePath = realpath(__DIR__ . '/../../../');
42+
43+
$composerPackageManager = new ComposerPackageManager();
44+
// That way it knows about the extensions, this is done by TestBase upfront.
45+
$composerPackageManager->getPackageInfoWithFallback(__DIR__ . '/Fixtures/Packages/package0');
46+
$composerPackageManager->getPackageInfoWithFallback(__DIR__ . '/Fixtures/Packages/package1');
47+
$composerPackageManager->getPackageInfoWithFallback(__DIR__ . '/Fixtures/Packages/package2');
48+
49+
$subject = PackageCollection::fromPackageStates(
50+
$composerPackageManager,
51+
new PackageManager(
52+
new DependencyOrderingService(),
53+
__DIR__ . '/Fixtures/Packages/PackageStates.php',
54+
$basePath
55+
),
56+
$basePath,
57+
$packageStates
58+
);
59+
60+
$result = $subject->sortPackageStates(
61+
$packageStates,
62+
new DependencyOrderingService()
63+
);
64+
65+
self::assertSame(7, array_search('package2', array_keys($result)), 'Package 2 is not stored at loading order 7.');
66+
self::assertSame(6, array_search('package1', array_keys($result)), 'Package 1 is not stored at loading order 6.');
67+
self::assertSame(5, array_search('package0', array_keys($result)), 'Package 0 is not stored at loading order 5.');
68+
}
69+
}

0 commit comments

Comments
 (0)