Anomalie #455: autoplay: erreur affichage favoris

src/Muzich/CoreBundle/Controller/ElementController.php

@@ -1106,8 +1106,13 @@ class ElementController extends Controller
       $display_comments      = false;
     }
     
-    if (!($element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
-      ->findOneById($element_id)))
+    // On prépare la récupèration de l'élèment
+    $es = new ElementSearcher();
+    $es->init(array(
+      'ids' => array($element_id)
+    ));
+    
+    if (!($element = $es->getElements($this->getDoctrine(), $this->getUserId(), 'single')))
     {
       throw $this->createNotFoundException('Not found');
     }

src/Muzich/CoreBundle/Entity/Element.php

@@ -712,14 +712,27 @@ class Element
     }
   }
   
-  /**
-   * Retourne le nombre de fois que cet élément a été msi en favoris 
-   * 
-   * @return int
-   */
-  public function getCountFavorite()
+//  /**
+//   * Retourne le nombre de fois que cet élément a été msi en favoris 
+//   * 
+//   * @return int
+//   */
+//  public function getCountFavorite()
+//  {
+//    return count($this->elements_favorites);
+//  }
+  
+  public function hasFavoriteUser($user_id)
   {
-    return count($this->elements_favorites);
+    foreach ($this->elements_favorites as $favorite)
+    {
+      if ($favorite->getUser()->getId() == $user_id)
+      {
+        return true;
+      }
+    }
+    
+    return false;
   }
   
   public function setGroupToId()

src/Muzich/CoreBundle/Resources/views/SearchElement/element.html.twig

@@ -239,13 +239,13 @@
       </li>
       <li>
 
-      {% if element.getCountFavorite %}
+      {% if element.hasFavoriteUser(app.user.id) %}
         <a class="favorite_link" href="{{ path('favorite_remove', { 'id': element.id, 'token': app.user.personalHash }) }}" >
-          <img src="{{ asset('bundles/muzichcore/img/favorite.png') }}" title="{{ 'element.favorite.remove'|trans({}, 'elements') }}" alt="{{ 'element.favorite.remove'|trans({}, 'elements') }}"/>
+          <img id="favorite_{{ element.id }}_is" src="{{ asset('bundles/muzichcore/img/favorite.png') }}" title="{{ 'element.favorite.remove'|trans({}, 'elements') }}" alt="{{ 'element.favorite.remove'|trans({}, 'elements') }}"/>
         </a>
       {% else %}
         <a class="favorite_link" href="{{ path('favorite_add', { 'id': element.id, 'token': app.user.personalHash }) }}" >
-          <img src="{{ asset('bundles/muzichcore/img/favorite_bw.png') }}" title="{{ 'element.favorite.add'|trans({}, 'elements') }}" alt="{{ 'element.favorite.add'|trans({}, 'elements') }}" />
+          <img id="favorite_{{ element.id }}_isnot" src="{{ asset('bundles/muzichcore/img/favorite_bw.png') }}" title="{{ 'element.favorite.add'|trans({}, 'elements') }}" alt="{{ 'element.favorite.add'|trans({}, 'elements') }}" />
         </a>
       {% endif %}
 

src/Muzich/CoreBundle/Searcher/ElementSearcher.php

@@ -329,6 +329,10 @@ class ElementSearcher extends Searcher implements SearcherInterface
         return count($query->getArrayResult());
       break;
     
+      case 'single':
+        return $query->getSingleResult();
+      break;
+    
       default :
         throw new \Exception('Mode de récupération des Elements non supporté.');
       break;

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

@@ -76,6 +76,41 @@ class FavoriteControllerTest extends FunctionalTest
   }
   
   /**
+   * Test vérifiant que l'étoile apparait bien lorsque un élement est en favoris
+   * Test d'affichage dom en somme
+   */
+  public function testStarFavorites()
+  {
+    $this->client = self::createClient();
+    $this->connectUser('bux', 'toor');
+    
+    $bux = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')
+      ->findOneByUsername('bux')
+    ;
+    
+    // On se rend sur sa page des favoris de bux
+    $this->crawler = $this->client->request('GET', $this->generateUrl('favorite_user_list', array('slug' => 'bux')));
+           
+    $elements = $this->getDoctrine()->getEntityManager()->createQuery("
+      SELECT e FROM MuzichCoreBundle:Element e
+      LEFT JOIN e.elements_favorites ef
+      WHERE ef.user = :uid
+    ")->setParameter('uid', $bux->getId())
+      ->getResult()
+    ;
+    
+    $this->assertTrue(!is_null($elements));
+    
+    if (count($elements))
+    {
+      foreach ($elements as $element)
+      {
+        $this->exist('img#favorite_'.$element->getId().'_is');
+      }
+    }
+  }
+  
+  /**
    * Test d'ajout en favori un element, puis son retrait
    * Ce test dépend actuellement du fait que l'élément testé se truove sur la 
    * page du groupe en question