search

mogilvie / EncryptBundle

Encryption bundle
82 stars 27 forks source link

Add doctrine/annotations to require #33

Closed achterin closed 1 year ago

achterin commented 1 year ago

As Doctrine\Common\Annotations\Reader is part of the doctrine/annotations package, this is a hard requirement for this package as DoctrineEncryptSubscriber uses it.

Without this i got the following error on a fresh symfony install:

Executing script cache:clear [KO] [KO] Script cache:clear returned with error code 1 !!
!! In DefinitionErrorExceptionPass.php line 51: !!
!! Cannot autowire service "SpecShaper\EncryptBundle\Subscribers\DoctrineEncry
!! ptSubscriberInterface": argument "$annReader" of method "SpecShaper\Encrypt
!! Bundle\Subscribers\DoctrineEncryptSubscriber::__construct()" has type "Doct
!! rine\Common\Annotations\Reader" but this class was not found.
!!
!!
!!
Script @auto-scripts was called via post-install-cmd

mogilvie commented 1 year ago

Thanks @achterin Although, next major release of this bundle will be dropping annotations in favour of attributes, so will soon remove the requirement and the autowired constuctor in the DoctrineEncryptSubscriber.