Skip to content

Commit 795cd6f

Browse files
committed
OP-550 - Remove php 8.4 and symfony 6.4 support
1 parent aa51620 commit 795cd6f

File tree

13 files changed

+93
-178
lines changed

13 files changed

+93
-178
lines changed

Diff for: .github/workflows/build.yml

+6-2
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,8 @@ jobs:
2121
strategy:
2222
fail-fast: false
2323
matrix:
24-
php: [ "8.3" ]
25-
symfony: [ "^6.4", "^7.0" ]
24+
php: [ "8.2", "8.3" ]
25+
symfony: [ "^7.1" ]
2626
sylius: [ "^2.0" ]
2727
node: [ "20.x" ]
2828
mysql: [ "8.0" ]
@@ -159,6 +159,10 @@ jobs:
159159
name: Run PHPUnit
160160
run: vendor/bin/phpunit --colors=always
161161

162+
-
163+
name: Cache clear
164+
run: (cd tests/Application && rm -rf var/cache)
165+
162166
-
163167
name: Run Behat
164168
run: vendor/bin/behat --colors --strict -vvv --no-interaction || vendor/bin/behat --colors --strict -vvv --no-interaction --rerun

Diff for: .github/workflows/coding_standard.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@ jobs:
1818
strategy:
1919
fail-fast: false
2020
matrix:
21-
php: [ "8.3" ]
22-
symfony: [ "^6.4", "^7.0" ]
21+
php: [ "8.2", "8.3" ]
22+
symfony: [ "^7.1" ]
2323
sylius: [ "^2.0" ]
2424
node: [ "20.x" ]
2525

Diff for: UPGRADE-1.3.md

-28
This file was deleted.

Diff for: UPGRADE-1.4.md

-86
This file was deleted.

Diff for: composer.json

+9-9
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
"description": "Product bundle for Sylius.",
66
"license": "MIT",
77
"require": {
8-
"php": "^8.2",
8+
"php": "~8.2.0 || ~8.3.0",
99
"sylius/sylius": "~2.0.0",
1010
"dompdf/dompdf": "^2.0",
1111
"sylius/twig-hooks": "^0.5",
@@ -31,7 +31,7 @@
3131
"friends-of-behat/symfony-extension": "^2.6",
3232
"friends-of-behat/variadic-extension": "^1.6",
3333
"gedmo/doctrine-extensions": "^3.9",
34-
"lchrusciel/api-test-case": "^4.1 || ^5.0",
34+
"lchrusciel/api-test-case": "^5.3",
3535
"league/flysystem-bundle": "^3.3",
3636
"nelmio/alice": "^3.10",
3737
"nyholm/psr7": "^1.8",
@@ -42,13 +42,13 @@
4242
"sylius-labs/suite-tags-extension": "~0.2",
4343
"sylius/mailer-bundle": "^1.8 || ^2.0@beta",
4444
"sylius/sylius-rector": "^2.0",
45-
"symfony/browser-kit": "^6.4 || ^7.1",
46-
"symfony/debug-bundle": "^6.4 || ^7.1",
47-
"symfony/dependency-injection": "^6.4 || ^7.1",
48-
"symfony/dotenv": "^6.4 || ^7.1",
49-
"symfony/http-client": "^6.4 || ^7.1",
50-
"symfony/intl": "^6.4 || ^7.1",
51-
"symfony/web-profiler-bundle": "^6.4 || ^7.1"
45+
"symfony/browser-kit": "^7.1",
46+
"symfony/debug-bundle": "^7.1",
47+
"symfony/dependency-injection": "^7.1",
48+
"symfony/dotenv": "^7.1",
49+
"symfony/http-client": "^7.1",
50+
"symfony/intl": "^7.1",
51+
"symfony/web-profiler-bundle": "^7.1"
5252
},
5353
"autoload": {
5454
"psr-4": {

Diff for: src/Component/Product/AddToCartFormComponent.php

+19-19
Original file line numberDiff line numberDiff line change
@@ -62,32 +62,32 @@ final class AddToCartFormComponent extends BaseAddToCartFormComponent
6262
* @param ProductVariantRepositoryInterface<ProductVariantInterface> $productVariantRepository
6363
*/
6464
public function __construct(
65-
protected readonly FormFactoryInterface $formFactory,
66-
protected readonly ObjectManager $manager,
67-
protected readonly RouterInterface $router,
68-
protected readonly RequestStack $requestStack,
69-
protected readonly EventDispatcherInterface $eventDispatcher,
70-
protected readonly CartContextInterface $cartContext,
71-
protected readonly AddToCartCommandFactoryInterface $addToCartCommandFactory,
72-
protected readonly CartItemFactoryInterface $cartItemFactory,
73-
protected readonly string $formClass,
65+
FormFactoryInterface $formFactory,
66+
ObjectManager $manager,
67+
RouterInterface $router,
68+
RequestStack $requestStack,
69+
EventDispatcherInterface $eventDispatcher,
70+
CartContextInterface $cartContext,
71+
AddToCartCommandFactoryInterface $addToCartCommandFactory,
72+
CartItemFactoryInterface $cartItemFactory,
73+
string $formClass,
7474
ProductRepositoryInterface $productRepository,
7575
ProductVariantRepositoryInterface $productVariantRepository,
76-
private readonly AddProductBundleToCartDtoFactory $addProductBundleToCartDtoFactory,
76+
protected readonly AddProductBundleToCartDtoFactory $addProductBundleToCartDtoFactory,
7777
) {
7878
$this->initializeProduct($productRepository);
7979
$this->initializeProductVariant($productVariantRepository);
8080

8181
parent::__construct(
82-
$this->formFactory,
83-
$this->manager,
84-
$this->router,
85-
$this->requestStack,
86-
$this->eventDispatcher,
87-
$this->cartContext,
88-
$this->addToCartCommandFactory,
89-
$this->cartItemFactory,
90-
$this->formClass,
82+
$formFactory,
83+
$manager,
84+
$router,
85+
$requestStack,
86+
$eventDispatcher,
87+
$cartContext,
88+
$addToCartCommandFactory,
89+
$cartItemFactory,
90+
$formClass,
9191
$productRepository,
9292
$productVariantRepository,
9393
);

Diff for: src/Resources/config/services/product.xml

-2
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,7 @@
1414
<argument type="service" id="sylius.context.cart" />
1515
<argument type="service" id="sylius.factory.add_to_cart_command" />
1616
<argument type="service" id="sylius.factory.order_item" />
17-
1817
<argument>BitBag\SyliusProductBundlePlugin\Form\Type\AddProductBundleToCartType</argument>
19-
2018
<argument type="service" id="sylius.repository.product" />
2119
<argument type="service" id="sylius.repository.product_variant" />
2220

Diff for: tests/Api/AdminJsonApiTestCase.php

+1-9
Original file line numberDiff line numberDiff line change
@@ -11,19 +11,11 @@
1111

1212
namespace Tests\BitBag\SyliusProductBundlePlugin\Api;
1313

14-
use Composer\InstalledVersions;
15-
1614
abstract class AdminJsonApiTestCase extends JsonApiTestCase
1715
{
1816
public function getAuthToken(string $email, string $password): string
1917
{
20-
$syliusVersion = InstalledVersions::getVersion('sylius/sylius');
21-
22-
if (version_compare($syliusVersion, '1.13.0', '>=')) {
23-
$endpoint = '/api/v2/admin/administrators/token';
24-
} else {
25-
$endpoint = '/api/v2/admin/authentication-token';
26-
}
18+
$endpoint = '/api/v2/admin/administrators/token';
2719

2820
$this->client->request(
2921
'POST',

Diff for: tests/Application/.gitignore

-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
/node_modules
1111

1212
/config/encryption
13-
/config/jwt
1413

1514
###> symfony/framework-bundle ###
1615
/.env.*.local

Diff for: tests/Application/Kernel.php

-4
Original file line numberDiff line numberDiff line change
@@ -112,9 +112,5 @@ private function registerBundlesFromFile(string $bundlesFile): iterable
112112
private function getConfigurationDirectories(): iterable
113113
{
114114
yield $this->getProjectDir() . '/config';
115-
$symfonyConfigDir = $this->getProjectDir() . '/config/symfony/' . BaseKernel::MAJOR_VERSION . '.x';
116-
if (is_dir($symfonyConfigDir)) {
117-
yield $symfonyConfigDir;
118-
}
119115
}
120116
}

Diff for: tests/Application/config/jwt/private.pem

+54
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,54 @@
1+
-----BEGIN ENCRYPTED PRIVATE KEY-----
2+
MIIJrTBXBgkqhkiG9w0BBQ0wSjApBgkqhkiG9w0BBQwwHAQIDbthk+aF5EACAggA
3+
MAwGCCqGSIb3DQIJBQAwHQYJYIZIAWUDBAEqBBA3DYfh2mXByUxFNke/Wf5SBIIJ
4+
UBckIgXeXBWPLQAAq07pN8uNFMUcUirFuEvbmxVe1PupCCAqriNxi1DqeSu/M7c1
5+
h66y0BqKZu/0G9SVTg63iCKDEiRAM3hLyD2CsjYg8h2LAaqQ9dFYGV0cHRhCXagZ
6+
Sdt9YTfn2rarRbxauMSt0z9zwCaiUrBU4JwSM3g+tD7W0lxAm9TeaqBZek5DIX+j
7+
3Gom5tPYQe8jvfGMGdMPuanoEwH4WbWzGcqypWriy4JwaggwKCQ4ituWfa9kqMMC
8+
8HRmBBDg0gtafmQP910RZh18JL2ewF5Pl7GDsLtOj5gNLNuAiQxDCcYRnD4/Cdsl
9+
bH91btmGX1nUVIFViUTW93eBsjBgdgqOMRVxUKkSSX6CmIZWlE3AazgwSbvOvNrN
10+
JGa8X21UwfuS/JHLmfRmgdti0YxRjJkBYLPpcd3ILsi+MMhSHy0uycAM/dB80Q1B
11+
vkW1UXGbCw/PzA5yHrzULzAl69E3Tt5nTVMIIcBGxw2rf+ej+AVjsuOl7etwecdC
12+
gnA90ViNlGOACLVnhsjd4WVF9Oircosf0UYoblwcT6gw1GSVF9pWuu7k5hy/7Pt/
13+
o1BvonUgz/4VHG+K58qvtnlto+JE0XWzPvukNUyggtekTLyoQCI3ZKge6ui3qLax
14+
N6whHpzFnFVF3GJAisTk5naHFawHNvH7t85pmc+UnjNUUmyl9RStl9LMYDSBKNlR
15+
LzPlJK27E5SLhhyJCni4+UYjH6PdlJuKXJ0365fufJ+5ajHRatwt039xLnK0W+oa
16+
L35NxCuXrn8YxOgJIomt7IrkV3AuxoWxcx4lRFoM0WCdn9SWZVtfFFiyX/Xr1qDg
17+
dUysw3/bePEkOKr5JWx09hT0OKDpkwLFo2Ljtvjln4EMXYEvvVqFciKw0kqF73Dw
18+
NyoSubwR4qs6FQclKW1TAP6UW4B6ffq1iagKOCTZ5bBtsPBZk8UGCJb57q4fUj4P
19+
nJy0hnSdlOH4Am+US4HF4ayOGuaV1Be1taurdJnt5cNnUYRah0wg4nG+wVdG5HJk
20+
f4dJ4nih9d6WA/8LfxdpB7NCwdR+KK6lky+GgLSdhmIT9lzjj2GDsU4lBf29TkBn
21+
lyt98/LWGrgCQgZAQ/obxLT8CZtY+tNejGoMppY+ub8DIaLBFID+fcz13kgA9x7a
22+
TeVB8RPok+S3yHXP9a4WSFe9DGjjN+m7EnRtte7MEjyMoekXVnT04gNbTMoGAjNb
23+
lrR4g3ICygZtsoGSB2VEu7o3azAspXNBMOuJfRCuC0LDXcjH3TbvjX0da5wHBoK9
24+
clRxu+CDo9A849HMkmSje8wED7ysZnkvSX0OdPjXahVd4t1tDRI6jSlzFo9fGcjp
25+
S8Ikm9iMrHXaWcDdtcq4C63CjSynIBr4mNIxe/f2e9nynm3AIv+aOan891RWHqrd
26+
DdpSSPShtzATI9PbB+b+S0Gw58Y8fpO7yoZ87VW1BMpadmFZ87YY78jdB7BwInNI
27+
JqtnivinM6qCsvbdMoGinUyL6PUcfQGiEAibouKr3zNRDC4aesBZZmj7w0dnf+HK
28+
YC905aR0cddlc6DBo/ed3o9krMcZ6oY/vruemPTc5G7Cg3t4H3mInRgURw22X1wo
29+
FsioU1yOdkK+MYxvmGsQvQuSJhp7h1Uz37t/olkPRafZgy2nEtw6DQO0Dm4UfSsD
30+
nysq6dn1WeZPkOipGBRgQmY1FTRzwPoCxi7+/EuHhD8hr962rHOglSuNqPG89J8r
31+
wdbTDr8kgXj2A9p+jI3TVKEX+h6FEhrCHW9SHUqATOZ7RiNL6hKld9j0U4D9gQwZ
32+
dflA0TxpVsHXm7pd1idkr46jIFgw7HA89Erm0Ty7RolfHkqlRca805AVmsKkviIz
33+
sbF5uv4WzIE3ViO8P1KMUhCyElm72mpyNTXBhkxkup9hJ4fQieaN6pET6dQ2xyjs
34+
SBIvQoXI0JQKpespcyAdoh88ULQjRUXEOaNFfN7q+itTcocwmPZfzW2nXORJT2p8
35+
SXLqSE73nYZdqzSYFq1hLcnlubJ7yPBYYG1fI0IydjSGKfnjtB0DReR32OToRZ7m
36+
laduZ8O+IaBUY4Sp6QdYcVbGGpG/wsPmTQyScc/O2bfSI7AiPnL9EnwebI9sPSWQ
37+
R0t0QMXZOSSqNY6jkYjsOCxeekRIdY6havo2Y52Ywti0QNrkT4BQ+175VVTmRMdy
38+
LNaMFeEq6ehSEdaHaozvjHvP50HQT43tCK+RJiL+Gf9FqawoQRt693yO5LFbQsuw
39+
QsUSMi41txpINMa+HEc2K5FvGoPr7FmajLK7X2fr+3c/yZ4fahoMKEAVFWl5kRYx
40+
Fe1smlw1Vxl/qNQ32LFWsBIK+XnYBteYmlpVyYrTgXyjnp1rK2zz0118DPFuYiAP
41+
O0r6nnBz0NbwnSKb7S4CjxBKDvDbWTzP35Q5L/vySnO2zRbM64Gw7sjeLiJittWS
42+
gQfbFpEk9k8KVndKM4H50Jp0WznmYpm1Tman8hUOiCvmq0qdI3bJ5Bnj0K+q2zFV
43+
+noGpMFdq1+8WaUFLQFGCPM+yJgCqDgT1RAgfsGcomckGcmenDtHaTbcSFabEdpM
44+
Tsa2qLdg/Kju+7JyGrkmobXl/azuyjYTHfRvSZrvO5WUDFzhChrJpIL4nA3ZGRlS
45+
gvy+OzyyBh4sRyHwLItwUwE81aya3W4llAkhQ7OycmqniJgjtJzLwnxv2RQsB8bF
46+
pyoqQdKVxkqHdbUFeh9igI4ffRAK+8xDER5J+RUoZ4mO8qJebxar54XTb6I/Lepc
47+
g8ITX8bJ/GH+M6JdP7tLCikDTSGS+i1ReMQXE5XuEajYOVbzQdyWU5jleZIx0f6X
48+
mTa4WvMEGNyNxKZZXsy9FAaBkZqrNzEv8k0uFgFMNWQcMMtiqbei86yACdqe+jiW
49+
HqHv8wfoBHR+eIARub2itOJ/cI+oKv96d4it4FqQ9Lml8RUFFZj7Hrd6EjDb6Nq4
50+
P9ti7eku/xZvS0saBNChvv44GhP6FZJS0i/gidVffLna7Wua98tPZEAXp57k+XUL
51+
PzsRJ4a+hFuQjkyXFoz/v8YuUdyCFUSVVr9ArVu0v4+4euFWpQLav5sXv0Gh9X58
52+
Ek1KIf7Z/tZAJnSjTjFuSbDX/AoTMTxpRBKKnFW6zY0Nw2pjTVMtTVDkv9xkBpBK
53+
wod7FPD5f0T7y9YOARVZnBxVRSkkcYpEJFy5pLNeadg9
54+
-----END ENCRYPTED PRIVATE KEY-----
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
fos_rest:
2-
exception:
3-
debug: true
2+
exception:
3+
debug: true

Diff for: tests/Application/config/symfony/6.x/bundles.php

-14
This file was deleted.

0 commit comments

Comments
 (0)