如何让服务在生命周期事件侦听器中工作

Question

我在项目的不同位置使用了服务。在我的控制器中它完美地工作。

我在prePersist生命周期事件侦听器中需要它，但在那里调用服务不想工作。当我尝试它时，我得到以下错误;

试图调用方法 “获得” 上 类 “XX \的xxx \ XxxxBundle \ LIB \ yyyy年\ OrderUserListener”。

我认为我明白我必须在我的事件监听器中注入我的服务，但我不明白该怎么做。

这就是OrderUserListener和OrderLogger服务分别在service.yml文件中的样子;

bss.pmod.current_user_id: 
    class: Xx\Xxx\XxxxBundle\Lib\Yyyy\OrderUserListener 
    calls: 
    - [ setServiceContainer, [@service_container] ] 
    tags: 
     - { name: doctrine.event_listener, event: prePersist } 

bss.pmod.order_logger: 
    class: Xx\Xxx\XxxxBundle\Lib\Yyyy\OrderLogger 
    arguments: [ "@doctrine.orm.entity_manager", "@security.token_storage" ] 

这是我的OrderLogger函数，我想在我的服务中注入;

class OrderLogger { 

    private $em; 
    private $tokenStorage; 

    /** 
    * Constructor. 
    * 
    * @param EntityManager $em 
    * @param TokenStorage $securityTokenStorage 
    */ 
    public function __construct(EntityManager $em, TokenStorage $securityTokenStorage) 
    { 
     $this->em = $em; 
     $this->tokenStorage = $securityTokenStorage; 
    } 

    /** 
    * Log an order action. 
    * 
    * @param string $text 
    */ 
    public function log($order, $action) 
    { 
     $logRecord = new PmodLog(); 
     if (is_object($this->tokenStorage->getToken())) { 
      $user = $this->tokenStorage->getToken()->getUser(); 
      if (is_object($user)) { 
       $logRecord->setUser($user); 
      } 
     } 
     $logRecord->setOrder($order); 
     $logRecord->setAction($action); 
     $logRecord->setTime(new \DateTime()); 

     $this->em->persist($logRecord); 
     $this->em->flush(); 
    } 

} 

我的事件监听器看起来像这样;

class OrderUserListener 
{ 

    /** 
    * Service container 
    * @var type 
    */ 
    private $serviceContainer; 

    /** 
    * Performs tasks before destruction 
    * @ORM\PrePersist 
    */ 
    public function prePersist(LifecycleEventArgs $args) 
    { 
     $order = $args->getEntity(); 

     if ($order instanceof PmodOrder) { 
      $user = $this->serviceContainer->get('security.token_storage')->getToken()->getUser(); 

      if ($user) { 
       $order->setCreatedBy($user); 
       $order->setCreatedAt(new \DateTime(date('Y-m-d H:i:s'))); 
       $order->setDepartment($user->getDepartment()); 
       $order->setStatus(PmodOrder::STATUS_AWAITING_APPROVAL); 

       //$this->get('bss.pmod.order_logger')->log($order, 'Order Created');  // This is then clearly wrong. 
      } 
     } 
    } 

    /** 
    * Sets the sales order exporter object 
    * @param type $serviceContainer 
    */ 
    public function setServiceContainer($serviceContainer) 
    { 
     $this->serviceContainer = $serviceContainer; 
    } 
} 

如果有人可以向我解释如何用我的示例做到这一点，我会非常感激它。

Answer 1

首先，看起来像你混合ContainerAwareInterface使用的Symfony的Controller基类（现赞成不赞成ContainerAwareTrait）。 ContainerAwareInterface旨在区分需要注入服务容器的类，并且控制器会自动注入它。由于看起来这条线是有效的：

$user = $this->serviceContainer->get('security.token_storage')->getToken()->getUser(); 

，看起来容器被正确注入。

您尝试从服务中调用的get()方法实际上让我想起Controller's get() method。但是你的班级不是Controller的后裔，而且我也不能说它实际上是一个控制者。

你应该做的，而不是调用get()这里什么是调用容器：

$logger = $this->serviceContainer->get('bss.pmod.order_logger')->log($order, 'Order Created'); 

然而，ContainerAware服务通常被认为是不好的做法。您可以改为直接通过构造函数注入服务：

# Service definition 
bss.pmod.current_user_id: 
    class: Xx\Xxx\XxxxBundle\Lib\Yyyy\OrderUserListener 
    arguments: 
     - "@security.token_storage" 
     - "@bss.pmod.order_logger" 
    tags: 
     - { name: doctrine.event_listener, event: prePersist } 

class OrderUserListener 
{ 
    private $tokenStorage; 

    private $logger; 

    public function __construct($tokenStorage, $logger) 
    { 
     $this->tokenStorage = $tokenStorage; 
     $this->logger = $logger; 
    } 

    // ... 

    public function prePersist(LifecycleEventArgs $args) 
    { 
     // ... 

     // Here you can call the injected services: 
     $user = $this->tokenStorage->getToken()->getUser(); 

     $this->logger->log($order, 'Created'); 
    } 
} 

Answer 2

变化$this->get('bss.pmod.order_logger')->log($order, 'Order Created');到$this->serviceContainer->get('bss.pmod.order_logger')->log($order, 'Order Created');