hotline #101969 fix <br /> in holds popups

VERSIONS_HOTLINE/101969

+ - ticket #101969 : Suppression d'une balise html <br /> pouvant apparaître dans les messages liés aux réservations
\ No newline at end of file

application/modules/opac/controllers/AbonneController.php

@@ -469,7 +469,7 @@ class AbonneController extends ZendAfi_Controller_Action {
     $cards = new Class_User_Cards($this->_user);
 
     if ($delete = $this->_getParam('id_delete')) {
-      $title = $cards->getHoldTitle($delete);
+      $title = strip_tags($cards->getHoldTitle($delete));
       $result = $cards->cancelHold($delete);
       $message = array_key_exists('erreur', $result) && $result['erreur']
         ? $result['erreur']

application/modules/opac/controllers/RechercheController.php

@@ -856,11 +856,13 @@ class RechercheController extends ZendAfi_Controller_Action {
         || !($notice = $item->getNotice()))
       return '';
 
+    $title = strip_tags($notice->getTitrePrincipal());
+
     if(!$notice->getAuteurPrincipal())
-      return sprintf(" '%s'",$notice->getTitrePrincipal());
+      return sprintf(" '%s'", $title);
 
     return sprintf(" '%s / %s'",
-                   $notice->getTitrePrincipal(),
+                   $title,
                    $notice->getAuteurPrincipal());
   }
 

tests/application/modules/opac/controllers/RechercheControllerReservationTest.php

@@ -497,18 +497,18 @@ class RechercheControllerReservationWithWebServiceKohaTest
 
 
   /** @test */
-  public function withMatchingHoldAndRecordMessageShouldContainsPickupLocationAndRecordTitleAuthor() {
+  public function withMatchingHoldAndRecordWithBRInTitleMessageShouldContainsPickupLocationAndRecordTitleAuthor() {
     $exemplaire = $this->fixture('Class_Exemplaire',
                                  ['id' => 456,
                                   'code_barres' => 123,
                                   'id_int_bib' => 1,
                                   'notice' => $this->fixture('Class_Notice',
                                                              ['id' => 8890,
-                                                              'titre_principal' => 'Arcadia',
+                                                              'titre_principal' => 'Albator <br /> Arcadia',
                                                               'auteur_principal' => 'Matsumoto'])]);
     $this->_dispatchWithEmprunteurAndAssertContentEquals(
       $this->_prepareEmprunteurHolding($exemplaire, 'Valensole'),
-      'Votre réservation est enregistrée.<br>Nous vous informerons quand le document \'Arcadia / Matsumoto\' sera disponible pour être retiré à : Valensole');
+      'Votre réservation est enregistrée.<br>Nous vous informerons quand le document \'Albator  Arcadia / Matsumoto\' sera disponible pour être retiré à : Valensole');
   }
 
 

tests/scenarios/MultiCards/MultiCardsTest.php

@@ -459,7 +459,7 @@ abstract class Scenario_AbonneMultiCardsAccountOfEwokTestCase extends Scenario_A
       ->getExemplaire()->setTitre('Alice');
 
     $arcadia = (new Class_WebService_SIGB_Reservation('78',
-                                                      (new Class_WebService_SIGB_Exemplaire(44))->setTitre('Arcadia')))
+                                                      (new Class_WebService_SIGB_Exemplaire(44))->setTitre('Albator <br /> Arcadia')))
       ->setWaitingToBePulled();
 
     $emprunteur = (new Class_WebService_SIGB_Emprunteur('2', 'luke'))
@@ -703,10 +703,18 @@ class Scenario_AbonneMultiCardsAccountOfEwokCancelReservationsActionTest
   public function shouldCallCancelArcadiaHoldOfLuke() {
     $this->assertTrue($this->_comm->methodHasBeenCalledWithParams('supprimerReservation', [Class_Users::find(2), 78]));
   }
+
+
+  /** @test */
+  public function popupShouldContainsMessageVotreReservationDuDocumentAlbatorArcadiaWithoutBrABienEteSupprime() {
+    $this->assertFlashMessengerContentContains('Votre réservation du document Albator  Arcadia a bien été supprimée',
+                                               $this->_response->getBody());
+  }
 }
 
 
 
+
 class Scenario_AbonneMultiCardsLoginNotificationTest
   extends Scenario_AbonneMultiCardsTestCase {