Evolution #143: Commentaires sur un élément

app/Resources/translations/elements.fr.yml

@@ -36,9 +36,10 @@ element:
     add_cancel:         Annuler
     edit_submit:        Envoyer
     edit_cancel:        Annuler
-    add_error:
+    errors:
       min:              Le commentaire doit être de %limit% caractéres minimum
       max:              Le commentaire doit être de %limit% caractéres maximum
+      unknow:           Une erreur est survenue
   
 comment:
   edit:

src/Muzich/CommentBundle/Controller/CommentController.php

@@ -32,7 +32,7 @@ class CommentController extends Controller
       return $this->jsonResponse(array(
         'status' => 'error',
         'errors' => array($this->trans(
-          'element.comments.add_error.min', 
+          'element.comments.errors.min', 
           array(
             '%limit%' => $this->container->getParameter('comment_add_min_length')
           ), 
@@ -45,7 +45,7 @@ class CommentController extends Controller
       return $this->jsonResponse(array(
         'status' => 'error',
         'errors' => array($this->trans(
-          'element.comments.add_error.max', 
+          'element.comments.errors.max', 
           array(
             '%limit%' => $this->container->getParameter('comment_add_max_length')
           ), 
@@ -94,7 +94,20 @@ class CommentController extends Controller
     
     // On met a jour les commentaires
     $cm = new CommentsManager($element->getComments());
-    $cm->delete($this->getUserId(), $date);
+    
+    
+    if (!$cm->delete($this->getUserId(), $date))
+    {
+      return $this->jsonResponse(array(
+        'status' => 'error',
+        'errors' => array($this->trans(
+          'element.comments.errors.unknow', 
+          array(), 
+          'elements'
+        )
+      )));
+    }
+    
     $element->setComments($cm->get());
       
     $this->getDoctrine()->getEntityManager()->persist($element);
@@ -162,7 +175,7 @@ class CommentController extends Controller
         'status' => 'error',
         'dom_id' => $dom_id,
         'errors' => array($this->trans(
-          'element.comments.add_error.min', 
+          'element.comments.errors.min', 
           array(
             '%limit%' => $this->container->getParameter('comment_add_min_length')
           ), 
@@ -176,7 +189,7 @@ class CommentController extends Controller
         'status' => 'error',
         'dom_id' => $dom_id,
         'errors' => array($this->trans(
-          'element.comments.add_error.max', 
+          'element.comments.errors.max', 
           array(
             '%limit%' => $this->container->getParameter('comment_add_max_length')
           ), 
@@ -192,13 +205,26 @@ class CommentController extends Controller
 
     $this->getDoctrine()->getEntityManager()->persist($element);
     $this->getDoctrine()->getEntityManager()->flush();
-
+    
+    if (!($comment_index = $cm->getIndex($this->getUserId(), $date)))
+    {
+      return $this->jsonResponse(array(
+        'status' => 'error',
+        'dom_id' => $dom_id,
+        'errors' => array($this->trans(
+          'element.comments.errors.unknow', 
+          array(), 
+          'elements'
+        )
+      )));
+    }
+    
     // On récupère le html du li avec le comment pour la réponse
     $html = $this->render('MuzichCommentBundle:Comment:comment.html.twig', array(
-      'comment'     => $cm->get($cm->getIndex($this->getUserId(), $date)),
+      'comment'     => $cm->get($comment_index),
       'element_id'  => $element->getId()
     ))->getContent();
-
+    
     return $this->jsonResponse(array(
       'status' => 'success',
       'dom_id' => $dom_id,

src/Muzich/CoreBundle/Managers/CommentsManager.php

@@ -72,11 +72,19 @@ class CommentsManager
         $comments[] = $comment;
       }
     }
+    
     $this->comments = $comments;
   }
   
+  /**
+   *
+   * @param int $user_id
+   * @param string $date
+   * @return boolean 
+   */
   public function delete($user_id, $date)
   {
+    $found = false;
     $comments = array();
     foreach ($this->comments as $comment)
     {
@@ -84,8 +92,13 @@ class CommentsManager
       {
         $comments[] = $comment;
       }
+      else
+      {
+        $found = true;
+      }
     }
     $this->comments = $comments;
+    return $found;
   }
   
   public function getIndex($user_id, $date)

src/Muzich/CoreBundle/Tests/Controller/CommentControllerTest.php

@@ -0,0 +1,210 @@
+<?php
+
+namespace Muzich\CoreBundle\Tests\Controller;
+
+use Muzich\CoreBundle\lib\FunctionalTest;
+use Muzich\CoreBundle\Managers\CommentsManager;
+
+/**
+ * Test des commentaires. Doit couvrir:
+ * 
+ * * La consultation de commentaires en fixtures
+ * * L'ajout de commentaire
+ * * La modification d'un commentaire
+ * * La suppression d'un commentaire
+ * 
+ */
+class CommentControllerTest extends FunctionalTest
+{
+ 
+  public function testView()
+  {
+    $this->client = self::createClient();
+    $this->connectUser('paul', 'toor');
+    
+    // On est sur la page home, d'après les fixtures on a des coms en dom
+    $this->exist('div.comments:contains("C\'est trop bon hein ?")');
+    $this->exist('div.comments:contains("C\'est pas mal en effet")');
+    $this->exist('li.element a.display_comments');
+    $this->exist('li.element a.hide_comments');
+    $this->exist('li.element a.add_comment');
+    $this->exist('div.comments a.add_comment');
+  }
+  
+  public function testAddEditDelete()
+  {
+    $this->client = self::createClient();
+    $this->connectUser('paul', 'toor');
+    
+    $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
+      ->findOneByName('Ed Cox - La fanfare des teuffeurs (Hardcordian)')
+    ;
+    
+    // On va ajouter un commentaire a la toute dernière musique posté par bux
+    $this->crawler = $this->client->request(
+      'POST', 
+      $this->generateUrl('ajax_add_comment', array(
+        'element_id' => $element->getId(),
+        'token'      => $this->getUser()->getPersonalHash()
+      )), 
+      array(
+          'comment' => "J'ai réécouté et ouaa je kiff BrOOO"
+        
+      ), 
+      array(), 
+      array('HTTP_X-Requested-With' => 'XMLHttpRequest')
+    );
+    
+    $this->isResponseSuccess();
+    
+    $response = json_decode($this->client->getResponse()->getContent(), true);
+    $this->assertEquals($response['status'], 'success');
+    
+    // On ré-affiche la page home pour voir si le commentaire y est
+    $this->crawler = $this->client->request('GET', $this->generateUrl('home'));
+    $this->exist('div.comments:contains("J\'ai réécouté et ouaa je kiff BrOOO")');
+    
+    $extract = $this->crawler->filter('div.comments li:contains("J\'ai réécouté et ouaa je kiff BrOOO")')
+      ->extract(array('id'));
+    $id = $extract[0];
+    
+    // Faut que l'on récupère la date
+    $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
+      ->findOneByName('Ed Cox - La fanfare des teuffeurs (Hardcordian)')
+    ;
+    $cm = new CommentsManager($element->getComments());
+    $comment = $cm->getLast();
+    
+    $this->assertEquals($comment['c'], "J'ai réécouté et ouaa je kiff BrOOO");
+        
+    // On effectue une modification de ce commentaire
+    $this->crawler = $this->client->request(
+      'POST', 
+      $this->generateUrl('ajax_update_comment', array(
+        'element_id' => $element->getId(),
+        'date'       => $comment['d'],
+        'dom_id'     => $id,
+        'token'      => $this->getUser()->getPersonalHash()
+      )),
+      array(
+          'comment' => "Je me modifie mon com kwaa"
+      ), 
+      array(), 
+      array('HTTP_X-Requested-With' => 'XMLHttpRequest')
+    );
+    
+    $this->isResponseSuccess();
+    
+    $response = json_decode($this->client->getResponse()->getContent(), true);
+    $this->assertEquals($response['status'], 'success');
+    
+    $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
+      ->findOneByName('Ed Cox - La fanfare des teuffeurs (Hardcordian)')
+    ;
+    $cm = new CommentsManager($element->getComments());
+    $comment = $cm->getLast();
+    
+    $this->assertEquals($comment['c'], "Je me modifie mon com kwaa");
+    // Il y a une date d'edition
+    $this->assertTrue(array_key_exists('e', $comment));
+    
+    // On ré-affiche la page home pour voir si le commentaire modifié y est
+    $this->crawler = $this->client->request('GET', $this->generateUrl('home'));
+    $this->exist('div.comments:contains("Je me modifie mon com kwaa")');
+    
+    // maintenant on le supprime
+    $this->crawler = $this->client->request(
+      'GET', 
+      $this->generateUrl('ajax_delete_comment', array(
+        'element_id' => $element->getId(),
+        'date'       => $comment['d'],
+        'token'      => $this->getUser()->getPersonalHash()
+      )),
+      array(), 
+      array(), 
+      array('HTTP_X-Requested-With' => 'XMLHttpRequest')
+    );
+    
+    $this->isResponseSuccess();
+    
+    $response = json_decode($this->client->getResponse()->getContent(), true);
+    $this->assertEquals($response['status'], 'success');
+    
+    $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
+      ->findOneByName('Ed Cox - La fanfare des teuffeurs (Hardcordian)')
+    ;
+    $cm = new CommentsManager($element->getComments());
+    $comment = $cm->getLast();
+    
+    $this->assertNotEquals($comment['c'], "Je me modifie mon com kwaa");
+    
+    // On ré-affiche la page home pour voir si le commentaire modifié y est
+    $this->crawler = $this->client->request('GET', $this->generateUrl('home'));
+    $this->notExist('div.comments:contains("Je me modifie mon com kwaa")');
+  }
+  
+  /**
+   * On test ici la sécurité nous empêchant de modifier / supprimer le comm
+   * d'un autre.
+   */
+  public function testFails()
+  {
+    $this->client = self::createClient();
+    $this->connectUser('paul', 'toor');
+    
+    $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
+      ->findOneByName('Babylon Pression - Des Tasers et des Pauvres')
+    ;
+    
+    $cm = new CommentsManager($element->getComments());
+    // D'après fixtures: Ce dernier commentaire est à bux
+    $comment = $cm->getLast();
+    
+    $this->assertEquals($comment['c'], "Je répond 13");
+    
+    // On récupère l'id dom
+    $extract = $this->crawler->filter('div.comments li:contains("Je répond 13")')
+      ->extract(array('id'));
+    $id = $extract[0];
+    
+    // On essaie de le modifier
+    $this->crawler = $this->client->request(
+      'POST', 
+      $this->generateUrl('ajax_update_comment', array(
+        'element_id' => $element->getId(),
+        'date'       => $comment['d'],
+        'dom_id'     => $id,
+        'token'      => $this->getUser()->getPersonalHash()
+      )),
+      array(
+          'comment' => "Je répond 13 HACKED"
+      ), 
+      array(), 
+      array('HTTP_X-Requested-With' => 'XMLHttpRequest')
+    );
+    
+    $this->isResponseSuccess();
+    
+    $response = json_decode($this->client->getResponse()->getContent(), true);
+    $this->assertEquals($response['status'], 'error');
+    
+    // On essaie de le supprimer
+    $this->crawler = $this->client->request(
+      'GET', 
+      $this->generateUrl('ajax_delete_comment', array(
+        'element_id' => $element->getId(),
+        'date'       => $comment['d'],
+        'token'      => $this->getUser()->getPersonalHash()
+      )),
+      array(), 
+      array(), 
+      array('HTTP_X-Requested-With' => 'XMLHttpRequest')
+    );
+    
+    $this->isResponseSuccess();
+    
+    $response = json_decode($this->client->getResponse()->getContent(), true);
+    $this->assertEquals($response['status'], 'error');
+  }
+  
+}