Update 115309

VERSIONS_HOTLINE/115309

- - ticket #115309 : Magasin de themes: Liste des avis sur la page d'accueil Le lien sur l'image de la notice renvoie directemnt vers l'avis.
\ No newline at end of file
+ - ticket #115309 : Magasin de themes: Liste des avis sur la page d'accueil Le lien sur l'image de la notice renvoie directement vers l'avis.
\ No newline at end of file

library/Class/AvisNotice.php

@@ -223,8 +223,7 @@ class AvisNoticeLoader extends Storm_Model_Loader {
         $reviews_by_records [$record->getId()] = ['Record' => $record,
                                                   'Reviews' => []];
 
-      $reviews_by_records [$record->getId()]['Reviews'] = array_merge($reviews_by_records [$record->getId()]['Reviews'],
-                                                                      [$review]);
+      $reviews_by_records [$record->getId()]['Reviews'][] = $review;
     }
 
     $grouped = [];

library/templates/Intonation/Library/View/Wrapper/ReviewInRecord.php

@@ -25,21 +25,4 @@ class Intonation_Library_View_Wrapper_ReviewInRecord extends Intonation_Library_
   public function getActions() {
     return [];
   }
-
-  public function getMainLink() {
-
-    return new Intonation_Library_Link(['Url' => array_merge($this->_context_params,
-                                                             ['controller' => 'record',
-                                                              'action' => 'reviews',
-                                                              'id' => $this->_model->getIdNotice()]),
-                                        'Image' => Class_Template::current()->getIco($this->_view,
-                                                                                     'read-document',
-                                                                                     'library'),
-                                        'Text' => $this->_('Voir'),
-                                        'Title' => $this->_('Voir le document %s de %s de type %s',
-                                                            $this->getMainTitle(),
-                                                            $this->getSecondaryTitle(),
-                                                            $this->getDocTypeLabel())]);
-    ;
-  }
 }
\ No newline at end of file

library/templates/Intonation/Library/View/Wrapper/ReviewsByRecord.php

@@ -61,7 +61,15 @@ class Intonation_Library_View_Wrapper_ReviewsByRecord extends Intonation_Library
 
 
   public function getMainLink() {
-    return $this->_record->getMainLink();
+    $nav_url = (new Intonation_Library_View_Wrapper_Record_RichContent_Reviews)->getNavUrl();
+    $nav_url ['id'] = $this->_record->getModel()->getId();
+    return new Intonation_Library_Link(['Url' => $nav_url,
+                                        'Image' => Class_Template::current()->getIco($this->_view,
+                                                                                     'read-review',
+                                                                                     'library'),
+                                        'Text' => $this->_('Lire les avis'),
+                                        'Title' => $this->_('Lire les avis sur %s',
+                                                            $this->getMainTitle())]);
   }
 
 

tests/scenarios/Templates/TemplatesReviewsTest.php

-V<?php
+<?php
 /**
  * Copyright (c) 2012-2020, Agence Française Informatique (AFI). All rights reserved.
  *
@@ -22,7 +22,7 @@ V<?php
 require_once('TemplatesTest.php');
 
 
-class TemplatesAddReviewTest extends TemplatesIntonationTestCase {
+class TemplatesReviewsAddTest extends TemplatesIntonationTestCase {
 
   protected $_review;
 
@@ -68,7 +68,7 @@ class TemplatesAddReviewTest extends TemplatesIntonationTestCase {
 }
 
 
-class TemplatesAddReviewAjaxTest extends TemplatesIntonationTestCase {
+class TemplatesReviewsAddAjaxTest extends TemplatesIntonationTestCase {
 
   protected $_review;
 
@@ -137,7 +137,7 @@ class TemplatesReviewsWidgetTest extends TemplatesIntonationTestCase {
                      'link_to_all' => 1,
                      'embeded_code'=> 1,
                      'IntonationShowFooter' => 1,
-                     'rendering' => 'card',
+                     'rendering' => 'card-overlay',
                      'layout' => 'multiple_carousel'])
       ->updateProfile();
 
@@ -155,15 +155,15 @@ class TemplatesReviewsWidgetTest extends TemplatesIntonationTestCase {
                     'clef_oeuvre' => 'PSYKO',
                    ]);
 
-    $this->dispatch('/opac/index/index/id_profil/72', true);
-    $this->assertXPathContentContains('//div[@id = "boite_19"]', 'Le Roi');
-    $this->assertXPath('//a[@href="/record/reviews/id_profil/72/id/2"]');
+    $this->dispatch('/opac/index/index/id_profil/72');
+    $this->assertXPath('//div[@id="boite_19"]//a[contains(@href, "/record/reviews/id_profil/72/id/2")]');
   }
 }
 
 
 
-class TemplatesBlogReviewsActionTest extends TemplatesIntonationTestCase {
+
+class TemplatesReviewsBlogActionTest extends TemplatesIntonationTestCase {
   protected $_storm_default_to_volatile = true;
 
 
@@ -219,7 +219,7 @@ class TemplatesBlogReviewsActionTest extends TemplatesIntonationTestCase {
 
 
 
-class TemplatesBlogReadMoreActionTest extends TemplatesIntonationTestCase {
+class TemplatesReviewsBlogReadMoreActionTest extends TemplatesIntonationTestCase {
   protected $_storm_default_to_volatile = true;
 
 
@@ -253,7 +253,7 @@ class TemplatesBlogReadMoreActionTest extends TemplatesIntonationTestCase {
 
 
 
-class TemplatesRecordReviewsActionTest extends TemplatesIntonationTestCase {
+class TemplatesReviewsActionTest extends TemplatesIntonationTestCase {
   protected $_storm_default_to_volatile = true;
 
 
@@ -297,7 +297,7 @@ class TemplatesRecordReviewsActionTest extends TemplatesIntonationTestCase {
 
 
 
-class TemplatesRecordEditReviewsActionTest extends TemplatesIntonationTestCase {
+class TemplatesReviewsEditActionTest extends TemplatesIntonationTestCase {
   protected $_storm_default_to_volatile = true;
 
 
@@ -332,7 +332,7 @@ class TemplatesRecordEditReviewsActionTest extends TemplatesIntonationTestCase {
 
 
 
-class TemplatesRecordDeleteReviewsActionTest extends TemplatesIntonationTestCase {
+class TemplatesReviewsDeleteReviewsActionTest extends TemplatesIntonationTestCase {
   protected $_storm_default_to_volatile = true;