Merge pull request #441 from metafacture/440-doNotTrimControlField

Do not trim control field values in MARCXML handler.

Author: blackwinter

metafacture-biblio/src/main/java/org/metafacture/biblio/marc21/MarcXmlHandler.java

@@ -126,7 +126,7 @@ else if (DATAFIELD.equals(localName)) {
             getReceiver().endEntity();
         }
         else if (CONTROLFIELD.equals(localName)) {
-            getReceiver().literal(currentTag, builder.toString().trim());
+            getReceiver().literal(currentTag, builder.toString());
         }
         else if (RECORD.equals(localName) && checkNamespace(uri)) {
             getReceiver().endRecord();

metafacture-biblio/src/test/java/org/metafacture/biblio/marc21/MarcXmlHandlerTest.java

@@ -77,6 +77,20 @@ public void shouldFindTagAttributeAtSecondPositionInControlFieldElement()
         verify(receiver).literal("001", fieldValue);
     }
 
+    @Test
+    public void issue440_shouldNotRemoveWhitespaceFromControlFields() throws SAXException {
+        final AttributesImpl attributes = new AttributesImpl();
+        attributes.addAttribute(NAMESPACE, "tag", "tag", "CDATA", "008");
+
+        final String fieldValue = "      t20202020au |||||||||||| ||||ger d";
+
+        marcXmlHandler.startElement(NAMESPACE, CONTROLFIELD, "", attributes);
+        marcXmlHandler.characters(fieldValue.toCharArray(), 0, fieldValue.length());
+        marcXmlHandler.endElement(NAMESPACE, CONTROLFIELD, "");
+
+        verify(receiver).literal("008", fieldValue);
+    }
+
     @Test
     public void issue233ShouldNotRemoveWhitespaceFromLeader()
             throws SAXException {