Fix initiation of Bib object from SRU response

Alma started using namespaces quite some time ago. One of our xpath
queries didn't reflect that, so the MMS id wasn't set on the Bib object.

Author: danmichaelo

CHANGELOG.md

@@ -6,7 +6,9 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
-(nothing yet)
+### Fixed
+
+- Set MMS id on Bib object when initiated from SRU response.
 
 ## [0.7.2] - 2018-11-09
 

spec/Bibs/BibsSpec.php

@@ -59,7 +59,7 @@ public function it_provides_lookup_by_isbn(AlmaClient $client, SruClient $sru)
             ->shouldBeCalled()
             ->willReturn([SruRecord::make(
                 1,
-                '<record><controlfield tag="001">990114012304702201</controlfield></record>'
+                '<record xmlns="http://www.loc.gov/MARC21/slim"><controlfield tag="001">990114012304702201</controlfield></record>'
             )]);
 
         $bib = $this->fromIsbn('123');

src/Bibs/Bib.php

@@ -70,7 +70,7 @@ public function setMarcRecord($xml)
     public static function fromSruRecord(SruRecord $record, Client $client = null)
     {
         $record->data->registerXPathNamespace('marc', 'http://www.loc.gov/MARC21/slim');
-        $mmsId = $record->data->text('.//controlfield[@tag="001"]');
+        $mmsId = $record->data->text('.//marc:controlfield[@tag="001"]');
 
         return (new self($client, $mmsId))
             ->setMarcRecord($record->data->asXML());