Check additional 245 subfield(s). Might need more too... subfield a i…

…sn't sufficient.
fixlr · Nov 18, 2009 · c3ab53e · c3ab53e
1 parent b197276
commit c3ab53e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/lib/wcapi/record.rb b/lib/wcapi/record.rb
@@ -26,7 +26,16 @@ def link
     end
 
     def title
-      @title ||= xpath_get_text(xpath_first(@doc, "datafield[@tag='245']/subfield[@code='a']"))
+      unless @title
+        @title = []
+        xpath_all(@doc, "datafield[@tag='245']/subfield[@code='a']").each do |i|
+          @title << xpath_get_text(i)
+        end
+        xpath_all(@doc, "datafield[@tag='245']/subfield[@code='b']").each do |i|
+          @title << xpath_get_text(i)
+        end
+      end
+      return @title
     end
 
     def authors

diff --git a/spec/wcapi/get_record_response_spec.rb b/spec/wcapi/get_record_response_spec.rb
@@ -37,7 +37,7 @@
     end
 
     it "should have a title" do
-      @record.title.should == 'Harry Potter and the Half-Blood Prince /'
+      @record.title.should == ['Harry Potter and the Half-Blood Prince /']
     end
 
     it "should have a link" do