#13394 Koha: if reserve_id tag is present, use it for CancelHold

VERSIONS

+  - ticket #13394: SIGB Koha: prise en compte de la balise <reserve_id> de la nouvelle version de Koha pour supprimer les réservations
+
 30/04/2014 - v6.42.2
   - ticket #13394: SIGB Koha: récupère le site de retrait dans la liste des réservations (ILSDI élement <branchcode>)
   - ticket #13383: liste des prêts et réservations: pas de retour à la ligne pour l'en-tête de la première colonne (n°)

library/Class/WebService/SIGB/Koha/PatronInfoReader.php

@@ -72,6 +72,14 @@ class Class_WebService_SIGB_Koha_PatronInfoReader extends Class_WebService_SIGB_
 
 		$this->_current_operation->getExemplaire()->setBibliotheque($site->getLibelle());
 	}
+
+
+	/**
+	 * @param string $data
+	 */
+	public function endReserve_id($data) {
+		$this->_currentHold->setId((int)$data);
+	}
 }
 
 ?>
\ No newline at end of file

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

@@ -638,6 +638,21 @@ class KohaFixtures {
 									<itemnumber>24428</itemnumber>
 									<title>Harry Potter et la coupe de feu</title>
 							</hold>
+							<hold>
+								<priority>1</priority>
+								<item></item>
+								<reservedate>2014-05-02</reservedate>
+								<timestamp>2014-05-02 15:40:35</timestamp>
+								<biblionumber>235572</biblionumber>
+								<borrowernumber>17448</borrowernumber>
+								<branchcode>MPU</branchcode>
+								<branchname>MEDIATHEQUE PUBLIQUE ET UNIVERSITAIRE - VALENCE</branchname>
+								<constrainttype>a</constrainttype>
+								<lowestPriority>0</lowestPriority>
+								<suspend>0</suspend>
+								<title>Harry Potter et le prisonnier d\'Azkaban</title>
+								<reserve_id>1960</reserve_id>
+							</hold>
 					</holds>
 					<B_country></B_country>
 					<borrowernumber>572</borrowernumber>

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

@@ -358,8 +358,8 @@ class KohaGetEmprunteurLaureAfondTest extends KohaTestCase {
 
 
 	/** @test */
-	function nbReservationsShouldReturnTwo() {
-		$this->assertEquals(2, $this->laurent->getNbReservations());
+	function nbReservationsShouldReturnThree() {
+		$this->assertEquals(3, $this->laurent->getNbReservations());
 	}
 
 
@@ -414,6 +414,18 @@ class KohaGetEmprunteurLaureAfondTest extends KohaTestCase {
 	function secondReservationBibliothequeShouldBeMontmedy() {
 		$this->assertEquals('Montmedy', $this->laurent->getReservationAt(1)->getBibliotheque());
 	}
+
+
+	/** @test */
+	function thirdReservationNoticeIdShouldBe235572() {
+		$this->assertEquals(235572, $this->laurent->getReservationAt(2)->getNoNotice());
+	}
+
+	/** @test */
+	function thirdReservationIdShouldBe1960() {
+		$this->assertEquals(1960, $this->laurent->getReservationAt(2)->getId());
+	}
+
 }