Merge branch 'hotline#87090_lunel_pb_de_disponibilite_de_certains_documents' into 'hotline'

hotline #87090 WS Carthame : filter barcodes sent by WS to get only real barcodes

See merge request !3014

VERSIONS_HOTLINE/87090

+ - ticket #87090 : Web services SIGB Carthame : contournement des erreurs du web service qui peut renvoyer plusieurs valeurs différentes pour le code barres exemplaire.
\ No newline at end of file

library/Class/WebService/SIGB/Carthame/RecordResponseReader.php

@@ -23,6 +23,7 @@ class Class_WebService_SIGB_Carthame_RecordResponseReader {
   const STATUT_RESERVE = 3;
   const STATUT_PRETE = 4;
   const STATUT_EXCLU = 6;
+  const BARCODE_MIN_LEN = 4;
 
   /** @var Class_WebService_XMLParser */
   protected $_xml_parser;
@@ -67,6 +68,10 @@ class Class_WebService_SIGB_Carthame_RecordResponseReader {
 
 
   public function endSFa($data) {
+    //@see http://forge.afi-sa.fr/issues/87090
+    if (strlen(trim($data)) < static::BARCODE_MIN_LEN)
+      return;
+
     $this->_current_exemplaire->setCodeBarre((string)$data);
   }
 

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

@@ -134,10 +134,18 @@ class CarthameKtmNoticeAlbatorTest extends CarthameTestCase {
     $copies = $this->albator->getExemplaires();
     $this->assertEquals('Disponible', $copies[0]->getDisponibilite());
   }
+
+
+  /** @test */
+  public function secondCopyBarCodeShouldBe88446464646() {
+    $copies = $this->albator->getExemplaires();
+    $this->assertEquals('88446464646', $copies[1]->getCodeBarre());
+  }
 }
 
 
 
+
 abstract class CarthameOperationTestCase extends CarthameTestCase {
   public function setUp() {
     parent::setUp();
@@ -988,6 +996,7 @@ class CarthameTestFixtures {
             <SFv>Quetigny</SFv>
             <SFo>3</SFo>
             <SFq>20111111</SFq>
+            <SFA>T</SFA>
           </F941>
         </Unimarc>
       </Notice>