Closed dbu closed 10 years ago
wouterj
commented
10 years ago The Testing component requires it when loading the sonata bundle set: https://github.com/symfony-cmf/Testing/blob/master/src/Symfony/Cmf/Component/Testing/HttpKernel/TestKernel.php#L50
After looking at sonata's composer.json files, I can't discover any knp package in there. However, the code requires the KnpMenuBundle. It looks like this is a problem in sonata, @rande
wouterj
commented
10 years ago OK, seems like an undocumented regression in the SonataAdminBundle. The dependencies for KnpMenu 2.0 were added 11 days ago in https://github.com/sonata-project/SonataAdminBundle/commit/6fc4f0400ea6efe27ecc997105d5f2228c94c6ae and 5 days ago, all knp requirements where removed in https://github.com/sonata-project/SonataAdminBundle/commit/fb0bd9c9dd2a3a863a3ed425f38400bec7feec22 and an error message saying:
Boom! you are living on the edge ;) The AdminBundle requires the KnpMenuBundle! Please add either
"knplabs/knp-menu-bundle": "~1.1",if you want the stable version or"knplabs/knp-menu-bundle": "~2.0@dev"and"knplabs/knp-menu": "~2.0@dev",into your composer.json file if you want the MopaBootstrapBundle-compatible version and add the KnpMenuBundle into the AppKernel');
Was added. This means all bundles requiring the SonataAdminBundle for their tests needs to be updated to also require "knplabs/knp-menu-bundle": "~1.1".
rande
commented
10 years ago We can reintroduce the dependency to support both version: "knplabs/knp-menu-bundle": "*"
On Tue, Jan 7, 2014 at 10:12 PM, Wouter J notifications@github.com wrote:
OK, seems like an undocumented regression in the SonataAdminBundle. The dependencies for KnpMenu 2.0 were added 11 days ago in sonata-project/SonataAdminBundle@6fc4f04https://github.com/sonata-project/SonataAdminBundle/commit/6fc4f0400ea6efe27ecc997105d5f2228c94c6aeand 5 days ago, all knp requirements where removed in sonata-project/SonataAdminBundle@fb0bd9chttps://github.com/sonata-project/SonataAdminBundle/commit/fb0bd9c9dd2a3a863a3ed425f38400bec7feec22and an error message saying:
Boom! you are living on the edge ;) The AdminBundle requires the KnpMenuBundle! Please add either "knplabs/knp-menu-bundle": "~1.1", if you want the stable version or "knplabs/knp-menu-bundle": "~2.0@dev" and "knplabs/knp-menu": "~2.0@dev", into your composer.json file if you want the MopaBootstrapBundle-compatible version and add the KnpMenuBundle into the AppKernel');
Was added. This means all bundles requiring the SonataAdminBundle for their tests needs to be updated to also require "knplabs/knp-menu-bundle": "~1.1".
— Reply to this email directly or view it on GitHubhttps://github.com/symfony-cmf/BlockBundle/issues/167#issuecomment-31779822 .
Thomas Rabaix http://rabaix.net | http://sonata-project.org http://rabaix.net
dbu
commented
10 years ago @rande did you bump the requirement to KnpMenuBundle 2.0? or can you work with either? then the correct thing would be knplabs/knp-menu-bundle: ">=1.1.0,<3.0.0" afaik
rande
commented
10 years ago The situation with some bundles is not very good. KnpMenuBundle and FosUserBundle have been on 2.0-dev for more than 12 months or so with no stable release. Also some widely used bundles (MopaBootstrapBundle, and many others) are using them with their non stable version.
What should we do ? We try to support them both version. For KnpMenuBundle this is quite easy and the AdminBundle support 1.X and 2.X branch. So I will update the composer.json to avoid issue on third party bundles.
The situation is more complex with FosUserBundle and we don't have a solution yet.
As a side note, we try to reduce the amount of code in the AdminBundle:
On Wed, Jan 8, 2014 at 7:46 AM, David Buchmann notifications@github.comwrote:
@rande https://github.com/rande did you bump the requirement to KnpMenuBundle 2.0? or can you work with either? then the correct thing would be knplabs/knp-menu-bundle: ">=1.1.0,<3.0.0" afaik
— Reply to this email directly or view it on GitHubhttps://github.com/symfony-cmf/BlockBundle/issues/167#issuecomment-31809415 .
Thomas Rabaix http://rabaix.net | http://sonata-project.org http://rabaix.net
rande
commented
10 years ago dbu
commented
10 years ago limit cmf block bundle to sonata-block-bundle <2.2.8 and sonata-cache-bundle <2.1.6 (cache did not start to depend on corebundle, but we want to be on the safe side) also both in 1.0 and master
tests on master are failing. seems that Testing expects KnpMenuBundle to be available, but it isn't. i thought that is a requirement of sonata admin, so either something got wrong there, or something else is wrong.
locally i can't get composer to install the deps of this bundle, i get
Problem 1