Drop SonataEasyExtends in Sonata bundles

[jordisala1991]

Same issue like SonataCoreBundle for SonataEasyExtends. The bundle is deprecated since Symfony flex provides and easier path to install our bundles. The usages are on every bundle that defines Entities because it has a Doctrine Compiler pass to create the linking for the entities (associations, etc..)

The same code is on the https://github.com/sonata-project/sonata-doctrine-extensions The SonataEasyExtends is used by: https://github.com/sonata-project/SonataEasyExtendsBundle/network/dependents?dependent_type=PACKAGE&package_id=UGFja2FnZS01NDMwMzc3MjY%3D

The process is:

• [x] STEP 1: Make sonata doctrine extensions work: https://github.com/sonata-project/sonata-doctrine-extensions/pull/184
• [x] STEP 2: Make EasyExtends optional on Sonata 3
  • [x] sonata-project/comment-bundle
  • [x] sonata-project/article-bundle
  • [x] sonata-project/user-bundle
  • [x] sonata-project/timeline-bundle
  • [x] sonata-project/page-bundle
  • [x] sonata-project/notification-bundle
  • [x] sonata-project/news-bundle
  • [x] sonata-project/media-bundle
  • [x] sonata-project/dashboard-bundle
  • [x] sonata-project/classification-bundle
  • [x] sonata-project/ecommerce
  • [x] sonata-project/sandbox
• [ ] STEP 3: Remove EasyExtends BC code on Sonata 4
  • [ ] sonata-project/comment-bundle
  • [ ] sonata-project/article-bundle
  • [ ] sonata-project/user-bundle
  • [ ] sonata-project/timeline-bundle
  • [ ] sonata-project/page-bundle
  • [ ] sonata-project/notification-bundle
  • [ ] sonata-project/news-bundle
  • [ ] sonata-project/media-bundle
  • [ ] sonata-project/dashboard-bundle
  • [ ] sonata-project/classification-bundle
  • [ ] sonata-project/ecommerce

[jordisala1991]

There are 2 bundles that were already started:

• ClassificationBundle have an open PR from @franmomu . Since it is open from february I opened a new one. Do you mind if I continue you work? I would like to pick the documentation part of the PR.

• NotificationBundle have a PR merged already. IMO it could be considered a BC break because you might not have DoctrineExtensions enabled, we should throw a deprecation and use EasyExtendsCollector, I can create a new PR and add the compatibility layer necessary. WDYT @sonata-project/contributors .

[jordisala1991]

From now , all PRs are created and the changes on the code are applied, but I marked all draft because it is missing the docs change.

[jordisala1991]

@phansys @core23 can you check any of the PR with and without this fix? https://github.com/sonata-project/sonata-doctrine-extensions/pull/184

[jordisala1991]

All the bundles got the first iteration of the PR ready. There is still the problem with this PR, sonata-project/sonata-doctrine-extensions#184

There are 2 bundles that suffer directly from this problem in their PRs, because have functional tests: https://github.com/sonata-project/SonataPageBundle/pull/1162 and https://github.com/sonata-project/SonataMediaBundle/pull/1753

ping @core23 @phansys

[jordisala1991]

Step 1 ready and released. Step 2 all PR ready to review.

[VincentLanglet]

I think we can close this issue @jordisala1991.

Since the SonataEasyExtends-related code is marked as deprecated it will be removed for sure in 4.0

[jordisala1991]

Agree