Внедрение SecurityContext в прослушиватель prePersist или preUpdate в Symfony2, чтобы получить пользователя в createBy или updatedBy, вызывая ошибку циркулярной ссылки

41

Я настраиваю класс слушателя, где я установлю столбец ownerid на любой доктрине prePersist. Мой файл services.yml выглядит так...

services:
my.listener:
    class: App\SharedBundle\Listener\EntityListener
    arguments: ["@security.context"]
    tags:
        - { name: doctrine.event_listener, event: prePersist }

и мой класс выглядит так...

use Doctrine\ORM\Event\LifecycleEventArgs;
use Symfony\Component\Security\Core\SecurityContextInterface;

class EntityListener
{

protected $securityContext;

public function __construct(SecurityContextInterface $securityContext)
{
    $this->securityContext = $securityContext;
}


/**
 *
 * @param LifecycleEventArgs $args 
 */
public function prePersist(LifecycleEventArgs $args)
{

    $entity = $args->getEntity();
    $entityManager = $args->getEntityManager();

    $entity->setCreatedby();

}
}

В результате возникает следующая ошибка.

ServiceCircularReferenceException: обнаружена циркулярная ссылка для службы "doctrine.orm.default_entity_manager", путь: "doctrine.orm.default_entity_manager → doctrine.dbal.default_connection → my.listener → security.context → security.authentication.manager → fos_user.user_manager".

Мое предположение заключается в том, что контекст безопасности уже был внедрен где-то в цепочке, но я не знаю, как получить к нему доступ. Есть идеи?

Теги:
doctrine2

4 ответа

69
Лучший ответ

У меня были подобные проблемы, и единственным обходным решением было передать весь контейнер в конструкторе (arguments: ['@service_container']).

use Doctrine\ORM\Event\LifecycleEventArgs;
use Symfony\Component\DependencyInjection\ContainerInterface;

class MyListener
{
    protected $container;

    public function __construct(ContainerInterface $container)
    {
        $this->container = $container;
    }

    // ...

    public function prePersist(LifeCycleEventArgs $args)
    {
        $securityContext = $this->container->get('security.context');

        // ...
    }
}
  • 0
    Хороший звонок, я закончил пробовать все виды вещей в тот день, когда задал вопрос, и, в конце концов, начал работать после того, как прошел в Контейнер. Оглядываясь назад, ошибка имеет немного больше смысла.
  • 1
    @ Джереми: Я так и не понял, почему нужно получать исключение по циклической ссылке, передавая отдельные услуги. Почему это имеет смысл?
Показать ещё 9 комментариев
34

Начиная с Symfony 2.6 эта проблема должна быть исправлена. Запрос на растяжение только что был принят в мастер. Ваша проблема описана здесь. https://github.com/symfony/symfony/pull/11690

Как и в Symfony 2.6, вы можете ввести security.token_storage в ваш слушатель. Эта служба будет содержать токен, используемый SecurityContext в <= 2.5. В 3.0 эта служба заменит SecurityContext::getToken() вообще. Вы можете увидеть основной список изменений здесь: http://symfony.com/blog/new-in-symfony-2-6-security-component-improvements#deprecated-the-security-context-service

Пример использования в версии 2.6:

Ваша конфигурация:

services:
    my.listener:
        class: App\SharedBundle\Listener\EntityListener
        arguments:
            - "@security.token_storage"
        tags:
            - { name: doctrine.event_listener, event: prePersist }


Ваш слушатель

namespace App\SharedBundle\Listener;

use Doctrine\ORM\Event\LifecycleEventArgs;
use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;

class EntityListener
{
    private $token_storage;

    public function __construct(TokenStorageInterface $token_storage)
    {
        $this->token_storage = $token_storage;
    }

    public function prePersist(LifeCycleEventArgs $args)
    {
        $entity = $args->getEntity();
        $entity->setCreatedBy($this->token_storage->getToken()->getUsername());
    }
}


Для хорошего примера created_by вы можете использовать https://github.com/hostnet/entity-blamable-component/blob/master/src/Listener/BlamableListener.php для вдохновения. Он использует компонент hostnet/entity-tracker, который предоставляет специальное событие, которое запускается, когда объект изменяется во время вашего запроса. Там также набор для настройки в Symfony2

  • 0
    Хорошо, я рад, что больше нет необходимости вводить весь контейнер.
  • 2
    А как насчет службы, которая не является маркером безопасности?
Показать ещё 6 комментариев
0

В этом потоке есть большой ответ, но все меняется. Теперь в Doctrine есть классы слушателей сущностей: http://docs.doctrine-project.org/projects/doctrine-orm/en/latest/reference/events.html#entity-listeners-class

Итак, вы можете добавить аннотацию к своей сущности, например:

/**
 * @ORM\EntityListeners({"App\Entity\Listener\PhotoListener"})
 * @ORM\Entity(repositoryClass="App\Repository\PhotoRepository")
 */
class Photo 
{
    // Entity code here...
}

Создайте класс следующим образом:

class PhotoListener
{        
    private $container;

    function __construct(ContainerInterface $container)
    {
        $this->container = $container;
    }

    /** @ORM\PreRemove() */
    public function preRemoveHandler(Photo $photo, LifecycleEventArgs $event): void
    {
         // Some code here...
    }
}

Также вы должны определить этот прослушиватель в services.yml следующим образом:

photo_listener:
  class: App\Entity\Listener\PhotoListener
  public: false
  autowire: true
  tags:
    - {name: doctrine.orm.entity_listener}
0

Я использую файлы конфигурации доктрины для установки методов preUpdate или prePersist:

Project\MainBundle\Entity\YourEntity:
    type: entity
    table: yourentities
    repositoryClass: Project\MainBundle\Repository\YourEntitytRepository
    fields:
        id:
            type: integer
            id: true
            generator:
                strategy: AUTO

    lifecycleCallbacks:
        prePersist: [methodNameHere]
        preUpdate: [anotherMethodHere]

И методы объявляются в сущности, так что вам не нужен прослушиватель, и если вам нужен более общий метод, вы можете сделать BaseEntity, чтобы сохранить этот метод и расширить другие полномочия от этого. Надеюсь, это поможет!

  • 3
    Как насчет использования security.context в этом случае?

Ещё вопросы

Сообщество Overcoder
Наверх
Меню