Merge pull request #166 from LD4P/partNumber-order

Move 245 n and p before c
LD4P · Dec 9, 2021 · 39ca49a · 39ca49a
2 parents 113a12a + a33f72e
commit 39ca49a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/rdf2marc/model2marc/field_245.rb b/lib/rdf2marc/model2marc/field_245.rb
@@ -13,9 +13,10 @@ def build
         field.indicator2 = model.nonfile_characters.to_s
         append('a', model.title)
         append('b', model.remainder_of_title)
-        append('c', model.statement_of_responsibility)
+        # Subfields n and p should come before c.
         append_repeatable('n', model.part_numbers)
         append_repeatable('p', model.part_names)
+        append('c', model.statement_of_responsibility)
       end
     end
   end

diff --git a/spec/rdf2marc/model2marc/field_245_spec.rb b/spec/rdf2marc/model2marc/field_245_spec.rb
@@ -18,7 +18,7 @@
   end
 
   let(:expected_fields) do
-    ['245 10 $a title1 $b remainder1 $c responsibility1 $n part_numbers1 $n part_numbers2 $p part_name1 $p part_name2']
+    ['245 10 $a title1 $b remainder1 $n part_numbers1 $n part_numbers2 $p part_name1 $p part_name2 $c responsibility1']
   end
 
   include_examples 'fields', '245'