added event before storing entity

Author: Tito Costa

.gitignore

@@ -0,0 +1,2 @@
+/vendor
+/.idea

DependencyInjection/Configuration.php

@@ -23,6 +23,9 @@ public function getConfigTreeBuilder()
         $rootNode
             ->children()
                 ->scalarNode('logger')->isRequired()->cannotBeEmpty()->end()
+            ->end()
+            ->children()
+                ->scalarNode('db_entity_class')->defaultValue('Oh\FormErrorLogBundle\Entity\FormErrorLog')->cannotBeEmpty()->end()
             ->end();
 
         return $treeBuilder;

DependencyInjection/OhFormErrorLogExtension.php

@@ -23,6 +23,7 @@ public function load(array $configs, ContainerBuilder $container)
         $config = $this->processConfiguration($configuration, $configs);
 
         $container->setAlias('oh_form_error_log.logger.manager', $config['logger']);
+        $container->setParameter('oh_form_error_log.db.entity.class', $config['db_entity_class']);
 
         $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('services.yml');

Entity/FormErrorLogEntityInterface.php

@@ -9,5 +9,6 @@ public function setFormName($formName);
     public function setField($field);
 
     public function setError($error);
-    
+
+    public function setValue($value);
 }

Event/Events.php

@@ -0,0 +1,8 @@
+<?php
+
+namespace Oh\FormErrorLogBundle\Event;
+
+final class Events
+{
+    const PRE_PERSIST = 'oh_form_error_log.entity.pre_persist';
+}

Event/PrePersistEntityEvent.php

@@ -0,0 +1,28 @@
+<?php
+
+namespace Oh\FormErrorLogBundle\Event;
+
+use Oh\FormErrorLogBundle\Entity\FormErrorLogEntityInterface;
+use Symfony\Component\EventDispatcher\Event;
+
+class PrePersistEntityEvent extends Event
+{
+    /** @var FormErrorLogEntityInterface */
+    private $entity;
+
+    /**
+     * @param FormErrorLogEntityInterface $entity
+     */
+    public function __construct(FormErrorLogEntityInterface $entity)
+    {
+        $this->entity = $entity;
+    }
+
+    /**
+     * @return FormErrorLogEntityInterface
+     */
+    public function getEntity()
+    {
+        return $this->entity;
+    }
+}

Logger/DatabaseLogger.php

@@ -2,40 +2,55 @@
 
 namespace Oh\FormErrorLogBundle\Logger;
 
-use Oh\FormErrorLogBundle\Logger\ErrorLogInterface;
+use Doctrine\ORM\EntityManagerInterface;
+use Oh\FormErrorLogBundle\Entity\FormErrorLogEntityInterface;
+use Oh\FormErrorLogBundle\Event\Events;
+use Oh\FormErrorLogBundle\Event\PrePersistEntityEvent;
 use Symfony\Component\DependencyInjection\Exception\InvalidArgumentException;
+use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 
 class DatabaseLogger implements ErrorLogInterface
 {
     private $em;
+
     private $entityClass;
-    
-    public function __construct($em, $entityClass)
+
+    private $eventDispatcher;
+
+    /**
+     * @param EntityManagerInterface $em
+     * @param $entityClass
+     * @param EventDispatcherInterface $eventDispatcher
+     */
+    public function __construct(EntityManagerInterface $em, $entityClass, EventDispatcherInterface $eventDispatcher)
     {
         $this->em = $em;
         $this->entityClass = $entityClass;
+        $this->eventDispatcher = $eventDispatcher;
     }
-    
+
     public function log($formName, $key, $error, $value = '', $uri = '')
     {
-        if($this->entityClass == 'Oh\FormErrorLogBundle\Entity\FormErrorLogEntityInterface') {
+        if ($this->entityClass == 'Oh\FormErrorLogBundle\Entity\FormErrorLogEntityInterface') {
             throw new InvalidArgumentException('You need to update your %oh_form_error_log.db.entity.class% parameter to your own class. See the README for help.');
         }
+
+        /** @var FormErrorLogEntityInterface $entity */
         $entity = new $this->entityClass;
-        
+
         $entity->setFormName($formName);
         $entity->setField($key);
         $entity->setError($error);
         $entity->setValue($value);
         // for BC
-        if(method_exists($entity, 'setUri')) {
+        if (method_exists($entity, 'setUri')) {
             $entity->setUri($uri);
         }
-        
+
+        $this->eventDispatcher->dispatch(Events::PRE_PERSIST, new PrePersistEntityEvent($entity));
+
         $this->em->persist($entity);
-        
+
         $this->em->flush($entity);
-        
     }
-    
 }

Resources/config/services.yml

@@ -1,24 +1,23 @@
 parameters:
-    oh_form_error_log.listener.class:       Oh\FormErrorLogBundle\EventListener\ErrorLogSubscriber
-    oh_form_error_log.logger.class:	    Oh\FormErrorLogBundle\Logger\Logger
-    oh_form_error_log.db.class:	            Oh\FormErrorLogBundle\Logger\DatabaseLogger
-    oh_form_error_log.db.entity.class:	    Oh\FormErrorLogBundle\Entity\FormErrorLog
+    oh_form_error_log.listener.class: Oh\FormErrorLogBundle\EventListener\ErrorLogSubscriber
+    oh_form_error_log.logger.class:	Oh\FormErrorLogBundle\Logger\Logger
+    oh_form_error_log.db.class:	Oh\FormErrorLogBundle\Logger\DatabaseLogger
 
 services:
     oh_form_error_log.logger.db:
-        class:            "%oh_form_error_log.db.class%"
-        arguments:        ["@doctrine.orm.default_entity_manager", "%oh_form_error_log.db.entity.class%"]
+        class: "%oh_form_error_log.db.class%"
+        arguments: ["@doctrine.orm.default_entity_manager", "%oh_form_error_log.db.entity.class%", "@event_dispatcher"]
     oh_form_error_log.logger:
-        class:            "%oh_form_error_log.logger.class%"
-        arguments:        ["@logger"]
+        class: "%oh_form_error_log.logger.class%"
+        arguments: ["@logger"]
         tags:
             - { name: monolog.logger, channel: formerror }
     oh_form_error_log.listener.db:
-        class:            "%oh_form_error_log.listener.class%"
-        arguments:        ["@oh_form_error_log.logger.db", "@request_stack"]
+        class: "%oh_form_error_log.listener.class%"
+        arguments: ["@oh_form_error_log.logger.db", "@request_stack"]
     oh_form_error_log.listener:
-        class:            "%oh_form_error_log.listener.class%"
-        arguments:        ["@oh_form_error_log.logger", "@request_stack"]
+        class: "%oh_form_error_log.listener.class%"
+        arguments: ["@oh_form_error_log.logger", "@request_stack"]
     oh_form_subscriber:
         class: Oh\FormErrorLogBundle\EventListener\ErrorLogSubscriber
         arguments:

composer.json

@@ -1,24 +1,27 @@
 {
-    "name": "lendable/form-error-log-bundle",
-    "type": "symfony-bundle",
-    "description": "Log form errors",
-    "keywords": ["form", "Symfony2"],
-    "homepage": "https://github.com/ollieLtd/OhFormErrorLogBundle",
-    "license": "MIT",
-    "authors": [
-        {
-            "name": "Ollie Harridge",
-            "email": "code@oll.ie"
-        }
-    ],
-    "require": {
-        "php": ">=5.3.2",
-        "symfony/framework-bundle": "~2.1|~3.0"
-    },
-    "autoload": {
-        "psr-0": { 
-	    "Oh\\FormErrorLogBundle": ""
-        }
-    },
-    "target-dir": "Oh/FormErrorLogBundle"
+  "name": "lendable/form-error-log-bundle",
+  "type": "symfony-bundle",
+  "description": "Log form errors",
+  "keywords": [
+    "form",
+    "Symfony2"
+  ],
+  "homepage": "https://github.com/ollieLtd/OhFormErrorLogBundle",
+  "license": "MIT",
+  "authors": [
+    {
+      "name": "Ollie Harridge",
+      "email": "code@oll.ie"
+    }
+  ],
+  "require": {
+    "php": ">=5.3.2",
+    "symfony/framework-standard-edition": "~2.1|~3.0"
+  },
+  "autoload": {
+    "psr-0": {
+      "Oh\\FormErrorLogBundle": ""
+    }
+  },
+  "target-dir": "Oh/FormErrorLogBundle"
 }