hotline #14462 - Fix pergame holds error message

library/Class/Systeme/PergameService.php

@@ -210,12 +210,12 @@ class Class_Systeme_PergameService {
 		// controle quota par carte
 		$nb = Class_Reservation::countBy(['idabon' => $id_abon]);
 		if ($nb >= $regles['Max_par_carte']) 
-			return ['popup' => 'La réservation est impossible car vous avez atteint le nombre maximum de réservations sur votre carte.'];
+			return ['erreur' => 'La réservation est impossible car vous avez atteint le nombre maximum de réservations sur votre carte.'];
 		
 		// controle quota par document
 		$nb = Class_Reservation::countBy(['id_notice_origine' => $id_origine]);
 		if ($nb >= $regles['Max_par_document'])
-			return ['popup' => 'La réservation est impossible car le nombre maximum de réservations pour ce document a été atteint (' . $regles["Max_par_document"] . ').'];
+			return ['erreur' => 'La réservation est impossible car le nombre maximum de réservations pour ce document a été atteint (' . $regles["Max_par_document"] . ').'];
 		
 		if (!$id_bib) 
 			$id_bib = 1;

tests/library/Class/Systeme/PergameServiceTest.php

@@ -264,7 +264,7 @@ extends PergameServiceReserverExemplaireValidUserTestCase {
 
 	/** @test */
 	public function shouldReturnMaxHoldError() {
-		$this->assertPopupContains('nombre maximum de réservations');
+		$this->assertEquals(['erreur' => 'La réservation est impossible car vous avez atteint le nombre maximum de réservations sur votre carte.'], $this->response);
 	}
 }
 
@@ -288,7 +288,7 @@ extends PergameServiceReserverExemplaireValidUserTestCase {
 
 	/** @test */
 	public function shouldReturnMaxHoldError() {
-		$this->assertPopupContains('nombre maximum de réservations pour ce document');
+		$this->assertEquals(['erreur' => 'La réservation est impossible car le nombre maximum de réservations pour ce document a été atteint (3).'], $this->response);
 	}
 }