Sylius/PayPalPlugin

[Bug] Wrong dependency

Nek- opened this issue · 0 comments

Nek- commented

SyliusPayPalPlugin version affected: 1.4.x

Description

I may be wrong but I think using a dependency to mock the container in the standard require section of the composer.json file sounds wrong:

"require": {
"php": "^7.4 || ^8.0",
"polishsymfonycommunity/symfony-mocker-container": "^1.0",
"sylius/sylius": "^1.10",
"phpseclib/phpseclib": "^2.0",
"sylius-labs/doctrine-migrations-extra-bundle": "^0.1.3",
"doctrine/doctrine-migrations-bundle": "^3.0"
},

I guess it's not super-super safe to have it in production as well? (didn't check anything, but since it's testing stuff...)

Possible Solution

Moving it to require-dev section.