Symfony PHP Event Listener not working

Question

I have a system where a user can create an order. As soon as the order is created another entity table should be updated with the current user who created the order, the time he created it, and some other information.

Basically a Log.

Here is my Listener class;

namespace Qi\Bss\BaseBundle\Lib\PurchaseModule;

use Qi\Bss\BaseBundle\Entity\Business\PurchaseModule\PmodLog;
use Qi\Bss\BaseBundle\Entity\Business\PurchaseModule\PmodOrder;

use Doctrine\ORM\Event\LifecycleEventArgs;

/**
 * Class OrderLogger. Purchase Module activity logger.
 */
class OrderLogger
{
    /**
     * Service container
     * @var type 
     */
    private $serviceContainer;

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

        if ($order instanceof PmodOrder) {
            $logRecord = new PmodLog();

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

            $logRecord->setCreatedBy($user);
            $logRecord->setAction('Gemaak');
            $logRecord->setCreatedAt(new \DateTime());
            $logRecord->setOrder($order);
            $logRecord->setDepartment($user->getDepartment());
        }
    }

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

I don't get what I'm doing wrong and that the log table in my database doesn't get updated whatsoever.

Here is my service as well;

bss.pmod.logger:
    class: Qi\Bss\BaseBundle\Lib\PurchaseModule\OrderLogger
    calls:
    - [ setServiceContainer, [@service_container] ]
    tags:
        - { name: doctrine.event_listener, event: postPersist }

I don't get an error, the order is perfectly created but the log database remain empty.

dlondero · Accepted Answer · 2016-07-11 09:34:11Z

1

Well don't you need to persist and flush $logRecord? You can get the entity manager from $args.

answered Jul 11, 2016 at 9:34

dlondero

2,6051 gold badge25 silver badges33 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

user6565137 Over a year ago

Ahh! Thank you! I was aware of that persist and flush and tried $em = $this->getDoctrine()->getManager(); but that will only work in controllers, right? I didn't knew that I can get the entity manager from $args.

dlondero Over a year ago

See doctrine-project.org/api/orm/2.1/….

BentCoder · Accepted Answer · 2016-07-11 09:41:26Z

1

You can adapt this to yours. Right after someone creates a User entity, a new UserLog entity/record gets created by inserting some data from User entity as well.

Controller

public function createAction()
{
    $user = new User();
    $user->setUsername('username');
    $user->setPassword('password');

    $this->entityManager->persist($user);
    $this->entityManager->flush();
}

Services.yml

services:
    application_backend.event_listener.user_entity:
        class: Application\BackendBundle\EventListener\UserEntityListener
        tags:
                - { name: doctrine.event_listener, event: postPersist }

UserEntityListener.php

namespace Application\BackendBundle\EventListener;

use Application\BackendBundle\Entity\User;
use Application\BackendBundle\Entity\UserLog;
use Doctrine\ORM\Event\LifecycleEventArgs;

class UserEntityListener
{
    public function postPersist(LifecycleEventArgs $args)
    {
        $entity = $args->getEntity();

        if ($entity instanceof User) {
            $log = new UserLog();
            $log->setUserId($entity->getId());
            $log->setMessage('postPersist at ' . date('d/m/Y H:i:s'));

            $em = $args->getEntityManager();
            $em->persist($log);
            $em->flush();
        }
    }
}

answered Jul 11, 2016 at 9:41

BentCoder

12.8k23 gold badges100 silver badges173 bronze badges

2 Comments

user6565137 Over a year ago

Thank you! I got it to work from @dlondero's answer, but yours would have done the trick as well. Have my upvote (will show when I get 15 rep).

VaN Over a year ago

I think you shouldn't flush things inside an event listener. Right now, u're flushing twice, first in the listener, bound to postPersist, then in the controller (but this one is flushing nothing). I would remove the flush() (but keep the persist) in the listener, and let the controller handle the flush().

Collectives™ on Stack Overflow

Symfony PHP Event Listener not working

2 Answers 2

2 Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related