Commit 975f7adc authored by Patrick Barroca's avatar Patrick Barroca 🐧

Merge branch 'hotline#101969_presence_d_une_balise_html' into 'hotline'

hotline #101969 fix <br /> in holds popups

See merge request !3451
parents 01292c1e 7517ac3c
Pipeline #9767 passed with stage
in 47 minutes and 35 seconds
- 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
...@@ -469,7 +469,7 @@ class AbonneController extends ZendAfi_Controller_Action { ...@@ -469,7 +469,7 @@ class AbonneController extends ZendAfi_Controller_Action {
$cards = new Class_User_Cards($this->_user); $cards = new Class_User_Cards($this->_user);
if ($delete = $this->_getParam('id_delete')) { if ($delete = $this->_getParam('id_delete')) {
$title = $cards->getHoldTitle($delete); $title = strip_tags($cards->getHoldTitle($delete));
$result = $cards->cancelHold($delete); $result = $cards->cancelHold($delete);
$message = array_key_exists('erreur', $result) && $result['erreur'] $message = array_key_exists('erreur', $result) && $result['erreur']
? $result['erreur'] ? $result['erreur']
......
...@@ -856,11 +856,13 @@ class RechercheController extends ZendAfi_Controller_Action { ...@@ -856,11 +856,13 @@ class RechercheController extends ZendAfi_Controller_Action {
|| !($notice = $item->getNotice())) || !($notice = $item->getNotice()))
return ''; return '';
$title = strip_tags($notice->getTitrePrincipal());
if(!$notice->getAuteurPrincipal()) if(!$notice->getAuteurPrincipal())
return sprintf(" '%s'",$notice->getTitrePrincipal()); return sprintf(" '%s'", $title);
return sprintf(" '%s / %s'", return sprintf(" '%s / %s'",
$notice->getTitrePrincipal(), $title,
$notice->getAuteurPrincipal()); $notice->getAuteurPrincipal());
} }
......
...@@ -497,18 +497,18 @@ class RechercheControllerReservationWithWebServiceKohaTest ...@@ -497,18 +497,18 @@ class RechercheControllerReservationWithWebServiceKohaTest
/** @test */ /** @test */
public function withMatchingHoldAndRecordMessageShouldContainsPickupLocationAndRecordTitleAuthor() { public function withMatchingHoldAndRecordWithBRInTitleMessageShouldContainsPickupLocationAndRecordTitleAuthor() {
$exemplaire = $this->fixture('Class_Exemplaire', $exemplaire = $this->fixture('Class_Exemplaire',
['id' => 456, ['id' => 456,
'code_barres' => 123, 'code_barres' => 123,
'id_int_bib' => 1, 'id_int_bib' => 1,
'notice' => $this->fixture('Class_Notice', 'notice' => $this->fixture('Class_Notice',
['id' => 8890, ['id' => 8890,
'titre_principal' => 'Arcadia', 'titre_principal' => 'Albator <br /> Arcadia',
'auteur_principal' => 'Matsumoto'])]); 'auteur_principal' => 'Matsumoto'])]);
$this->_dispatchWithEmprunteurAndAssertContentEquals( $this->_dispatchWithEmprunteurAndAssertContentEquals(
$this->_prepareEmprunteurHolding($exemplaire, 'Valensole'), $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');
} }
......
...@@ -459,7 +459,7 @@ abstract class Scenario_AbonneMultiCardsAccountOfEwokTestCase extends Scenario_A ...@@ -459,7 +459,7 @@ abstract class Scenario_AbonneMultiCardsAccountOfEwokTestCase extends Scenario_A
->getExemplaire()->setTitre('Alice'); ->getExemplaire()->setTitre('Alice');
$arcadia = (new Class_WebService_SIGB_Reservation('78', $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(); ->setWaitingToBePulled();
$emprunteur = (new Class_WebService_SIGB_Emprunteur('2', 'luke')) $emprunteur = (new Class_WebService_SIGB_Emprunteur('2', 'luke'))
...@@ -703,10 +703,18 @@ class Scenario_AbonneMultiCardsAccountOfEwokCancelReservationsActionTest ...@@ -703,10 +703,18 @@ class Scenario_AbonneMultiCardsAccountOfEwokCancelReservationsActionTest
public function shouldCallCancelArcadiaHoldOfLuke() { public function shouldCallCancelArcadiaHoldOfLuke() {
$this->assertTrue($this->_comm->methodHasBeenCalledWithParams('supprimerReservation', [Class_Users::find(2), 78])); $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 class Scenario_AbonneMultiCardsLoginNotificationTest
extends Scenario_AbonneMultiCardsTestCase { extends Scenario_AbonneMultiCardsTestCase {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment