FriendsOfSymfony / FOSHttpCacheBundle

Use the FOSHttpCache library in your Symfony projects
http://foshttpcachebundle.readthedocs.io
Other
430 stars 80 forks source link

[DRAFT]: Remove ExtraFrameworkBundle #599

Closed toxicity1985 closed 11 months ago

toxicity1985 commented 11 months ago

Hello,

I have working on the way to remove the dependencies to ExtraFrameworkBundle. I create an AnnotationListener to get the annotation from the controller and get a few stuff from ExtraFrameworkBundle. I have some issue with the test but i have no idea why. I know that's related to the AbstractRuleListener. The matchRule is not working as expected. If you have some idea what am i wrong ? It's a draft version.

image

dbu commented 11 months ago

oh snap. i thought we could get rid of the explicit references to FrameworkExtraBundle but keep using it for the annotations if it is available. but thats because i recently upgraded to attributes in a library that used the doctrine annotations implementation.

i think porting the annotation things from extra bundle into this bundle is adding too much unrelated legacy code. i think after all we should do a new major version that simply drops annotations support. i created the 3.x branch for a new major version.

sorry for the work i made you do - i hope it was at least interesting to see how it works :see_no_evil:

can you target 3.x with the fixes for the specific request matchers? we don't need the class_exists after all, we should simply make the 3.x version only allow symfony 6.4 and 7. and attributes should be simply deleting the annotation things and they will only have the #[\Attribute... information.

toxicity1985 commented 11 months ago

No worries about the double work. It was ok to go deeper inside this bundle. So il will remove all the stuff related to annotation and make a new pull request for that. I will also make a new one for the request matcher issue.

dbu commented 11 months ago

thank you very much @toxicity1985 !

toxicity1985 commented 11 months ago

i can close this one.