Refactorisation de addTag

src/Muzich/CoreBundle/Controller/CoreController.php

@@ -13,8 +13,8 @@ use Muzich\CoreBundle\Entity\Element;
 use Symfony\Component\HttpFoundation\RedirectResponse;
 use Muzich\CoreBundle\Form\Search\ElementSearchForm;
 use Symfony\Component\Routing\Exception\ResourceNotFoundException;
-use Muzich\CoreBundle\Util\StrictCanonicalizer;
 use Muzich\CoreBundle\Entity\Tag;
+use Muzich\CoreBundle\Managers\TagManager;
 
 class CoreController extends Controller
 {
@@ -370,69 +370,14 @@ class CoreController extends Controller
       }
     }
     
-    $canonicalizer = new StrictCanonicalizer();
-    $name_canonicalized = $canonicalizer->canonicalize($name);
+    $tagManager = new TagManager();
+    $tag = $tagManager->addTag($this->getDoctrine(), $name, $this->getUser(), $arguments);
     
-    // Check avant de commencer: On regarde si ce tag n'existe pas déjà en tag
-    // public (en cas de gruge)
-    if (($tag = $this->getDoctrine()->getRepository('MuzichCoreBundle:Tag')
-      ->findOneBy(array(
-        'slug'       => $name_canonicalized,
-        'tomoderate' => false
-      ))))
-    {
-      // Si il existe déjà pas besoin de l'ajouter on retourne l'id et ne nom
-      return $this->jsonResponse(array(
-        'status'   => 'success',
-        'tag_id'   => $tag->getId(),
-        'tag_name' => $tag->getName()
-      ));
-    }
-    
-    // Première étape, on regarde en base si quelqu'un a pas déjà ajouté ce tag
-    if (($tag = $this->getDoctrine()->getRepository('MuzichCoreBundle:Tag')
-      ->findOneBy(array(
-        'slug'       => $name_canonicalized,
-        'tomoderate' => true
-      ))))
-    {
-      // Si il existe déjà pas besoin de l'ajouter on retourne l'id et ne nom
-      // après avoir ajouté cet utilisateurs a la liste de ceux pouvant le voir
-      $privatesids = json_decode($tag->getPrivateids());
-      if (!in_array($this->getUserId(), $privatesids))
-      {
-        $privatesids[] = (string)$this->getUserId();
-      }
-      $tag->setPrivateids(json_encode($privatesids));
-      $tag->setArguments($tag->getArguments(). " ****" . $this->getUser()->getName()."****: " .$arguments);
-      
-      $this->getDoctrine()->getEntityManager()->persist($tag);
-      $this->getDoctrine()->getEntityManager()->flush();
-      
-      return $this->jsonResponse(array(
-        'status'   => 'success',
-        'tag_id'   => $tag->getId(),
-        'tag_name' => $tag->getName()
-      ));
-    }
-    else
-    {
-      // Sinon on l'ajoute en base
-      $tag = new Tag();
-      $tag->setName(ucfirst(strtolower($name)));
-      $tag->setTomoderate(true);
-      $tag->setPrivateids(json_encode(array((string)$this->getUserId())));
-      $tag->setArguments(" ****" . $this->getUser()->getName()."****: " .$arguments);
-      
-      $this->getDoctrine()->getEntityManager()->persist($tag);
-      $this->getDoctrine()->getEntityManager()->flush();
-      
-      return $this->jsonResponse(array(
-        'status'   => 'success',
-        'tag_id'   => $tag->getId(),
-        'tag_name' => $tag->getName()
-      ));
-    }
+    return $this->jsonResponse(array(
+      'status'   => 'success',
+      'tag_id'   => $tag->getId(),
+      'tag_name' => $tag->getName()
+    ));
   }
   
 }

src/Muzich/CoreBundle/Managers/TagManager.php

@@ -6,6 +6,8 @@ use Muzich\CoreBundle\Entity\Tag;
 use Doctrine\ORM\EntityManager;
 use Symfony\Component\DependencyInjection\Container;
 use FOS\UserBundle\Util\CanonicalizerInterface;
+use Symfony\Bundle\DoctrineBundle\Registry;
+use Muzich\CoreBundle\Util\StrictCanonicalizer;
 
 /**
  * 
@@ -17,9 +19,16 @@ class TagManager
   
   protected $nameCanonicalizer;
   
-  public function __construct(CanonicalizerInterface $nameCanonicalizer)
+  public function __construct(CanonicalizerInterface $nameCanonicalizer = null)
   {
-    $this->nameCanonicalizer = $nameCanonicalizer;
+    if ($nameCanonicalizer)
+    {
+      $this->nameCanonicalizer = $nameCanonicalizer;
+    }
+    else
+    {
+      $this->nameCanonicalizer = new StrictCanonicalizer();
+    }
   }
   
   public function updateSlug(Tag $tag)
@@ -27,4 +36,58 @@ class TagManager
     $tag->setSlug($this->nameCanonicalizer->canonicalize($tag->getName()));
   }
   
+  public function addTag(Registry $doctrine, $name, $user, $arguments)
+  {
+    $name_canonicalized = $this->nameCanonicalizer->canonicalize(trim($name));
+    
+    // Check avant de commencer: On regarde si ce tag n'existe pas déjà en tag
+    // public (en cas de gruge)
+    if (($tag = $doctrine->getRepository('MuzichCoreBundle:Tag')
+      ->findOneBy(array(
+        'slug'       => $name_canonicalized,
+        'tomoderate' => false
+      ))))
+    {
+      // Si il existe déjà pas besoin de l'ajouter
+      return $tag;
+    }
+    
+    // Première étape, on regarde en base si quelqu'un a pas déjà ajouté ce tag
+    if (($tag = $doctrine->getRepository('MuzichCoreBundle:Tag')
+      ->findOneBy(array(
+        'slug'       => $name_canonicalized,
+        'tomoderate' => true
+      ))))
+    {
+      // Si il existe déjà pas besoin de l'ajouter ou le retourne
+      // après avoir ajouté cet utilisateurs a la liste de ceux pouvant le voir
+      $privatesids = json_decode($tag->getPrivateids());
+      if (!in_array($user->getId(), $privatesids))
+      {
+        $privatesids[] = (string)$user->getId();
+      }
+      $tag->setPrivateids(json_encode($privatesids));
+      $tag->setArguments($tag->getArguments(). " ****" . $user->getName()."****: " .$arguments);
+      
+      $doctrine->getEntityManager()->persist($tag);
+      $doctrine->getEntityManager()->flush();
+      
+      return $tag;
+    }
+    else
+    {
+      // Sinon on l'ajoute en base
+      $tag = new Tag();
+      $tag->setName(ucfirst(strtolower($name)));
+      $tag->setTomoderate(true);
+      $tag->setPrivateids(json_encode(array((string)$user->getId())));
+      $tag->setArguments(" ****" . $user->getName()."****: " .$arguments);
+      
+      $doctrine->getEntityManager()->persist($tag);
+      $doctrine->getEntityManager()->flush();
+      
+      return $tag;
+    }
+  }
+  
 }