Merge branch 'hotline#12742_affichage_reservations' into 'hotline_6.41'

Hotline#12742 Affichage Reservations

VERSIONS

+ticket #12742: SIGB Pergame: affichage de la bibliothèque dans la liste des réservations
 ticket #13072: Correction de l'affichage de la liste des abonnés à une lettre d'information lorsque les nom, prenom, identifiant, mail sont trés longs
 ticket #13140: Correction de la réaffectation des auteurs aux avis sur les notices aprés un réimport total
 ticket #13172: Correction de l'affichage des kiosques si le titre de la boîte contient un caractère slash

library/Class/WebService/SIGB/Pergame/Service.php

@@ -87,15 +87,17 @@ class Class_WebService_SIGB_Pergame_Service extends Class_WebService_SIGB_Abstra
 
 
 	public function getReservationsOf($emprunteur) {
-
-		$user = Class_Users::getLoader()->find($emprunteur->getId());
-		$reservations_db = Class_Reservation::getLoader()->findAllBy(array('IDABON' => $user->getIdabon(),
-																																			 'ORDREABON' => $user->getOrdreabon()));
+		$user = Class_Users::find($emprunteur->getId());
+		$reservations_db = Class_Reservation::findAllBy(['IDABON' => $user->getIdabon(),
+																										 'ORDREABON' => $user->getOrdreabon()]);
 		$reservations = array();
 		foreach($reservations_db as $reservation) {
+			$bib = Class_Bib::find($reservation->getIdSite());
+
 			$reservations []= Class_WebService_SIGB_Reservation::newInstanceWithEmptyExemplaire()
 				->setId($reservation->getId())
 				->setNoticeOPAC($reservation->getNotice())
+				->setBibliotheque($bib ? $bib->getLibelle() : '')
 				->setRang($reservation->getRang())
 				->setEtat($reservation->getEtat());
 		}

tests/library/Class/WebService/SIGB/PergameTest.php

@@ -114,24 +114,34 @@ class PergameServiceGetEmprunteurTest extends PergameServiceTestCase {
 
 		Storm_Test_ObjectWrapper::onLoaderOfModel('Class_Reservation')
 			->whenCalled('findAllBy')
-			->with(array('IDABON' => 23, 'ORDREABON' => 2))
-			->answers(array(Class_Reservation::getLoader()
-											->newInstanceWithId(76)
-											->setIdNoticeOrigine('1HP')
-											->setIdPergame('1HP')
-											->setDateResa('2011-12-25')
-											->setIdSite(2)
-											->setIdabon(23)
-											->setOrdreabon(2)))
+			->with(['IDABON' => 23, 'ORDREABON' => 2])
+			->answers([
+									Class_Reservation::newInstanceWithId(76,
+																											 ['id_notice_origine' => '1HP',
+																												'id_pergame' => '1HP',
+																												'date_resa' => '2011-12-25',
+																												'id_site' => 2,
+																												'idabon' => 23,
+																												'ordreabon' => 2]),
+
+									Class_Reservation::newInstanceWithId(77,
+																											 ['id_notice_origine' => '3HP',
+																												'id_pergame' => '3HP',
+																												'date_resa' => '2011-12-25',
+																												'id_site' => 0,
+																												'idabon' => 23,
+																												'ordreabon' => 2]),
+									])
 			->whenCalled('countBy')
-			->with(array('ID_NOTICE_ORIGINE' => 1, 
-									 'where' => sprintf('DATE_RESA<"%s"', '2011-12-25')))
+			->with(['ID_NOTICE_ORIGINE' => 1, 
+							'where' => sprintf('DATE_RESA<"%s"', '2011-12-25')])
 			->answers(2);
 
-		$jc = Class_Users::newInstanceWithId(23)
-			->setLogin('jc')
-			->setIdabon(23)
-			->setOrdreabon(2);
+		$jc = Class_Users::newInstanceWithId(23, 
+																				 ['login' => 'jc',
+																					'idabon' => 23,
+																					'ordreabon' => 2]);
+
 		$this->emprunteur_jc = $this->_service_cran->getEmprunteur($jc);
 		$this->_first_emprunt = array_first($this->emprunteur_jc->getEmprunts());
 		$this->_first_reservation = array_first($this->emprunteur_jc->getReservations());
@@ -214,9 +224,23 @@ class PergameServiceGetEmprunteurTest extends PergameServiceTestCase {
 	public function firstReservationEtatShouldBeEnPret() {
 		$this->assertEquals('En prêt', $this->_first_reservation->getEtat());
 	}
+
+
+	/** @test */
+	public function firstReservationBibliothequeShouldBeCranGevrier() {
+		$this->assertEquals('Cran-Gevrier', $this->_first_reservation->getBibliotheque());
+	}
+
+	/** @test */
+	public function secondeReservationBibliothequeShouldBeEmpty() {
+		$resa = $this->emprunteur_jc->getReservations()[1];
+		$this->assertEmpty($resa->getBibliotheque());
+	}
 }
 
 
+
+
 class PergameServiceGetExemplairePotterTest extends PergameServiceTestCase {
 	public function setUp() {
 		parent::setUp();