Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support Doctrine/ORM 3.x & PHP 8.2 #58

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Support Doctrine/ORM 3.x & PHP 8.2 #58

wants to merge 14 commits into from

Conversation

Lpsd
Copy link

@Lpsd Lpsd commented Jul 20, 2024

All other support is dropped, minimum Doctrine 3.x, Symfony 7, PHP 8.2.

  • Removes annotations
  • Use Doctrine listener instead of subscriber
    • EncryptEventSubscriber => EncryptEventListener
    • DoctrineEncryptSubscriber => DoctrineEncryptListener
    • subscriber_class => listener_class

Tested lightly, appears to be working fine.

Resolves #57 #55 #50

@Lpsd Lpsd changed the title Support Doctrine 3.x & PHP 8.2 Support Doctrine/ORM 3.x & PHP 8.2 Jul 21, 2024
@Seros
Copy link

Seros commented Oct 9, 2024

This looks good but one question, could we have support for Symfony ^6.4 as well as it's quite often in use?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for Doctrine/ORM 3.x
2 participants