that's needed after all
[migration-tools.git] / fingerprinter
index c0a8741..f8ba533 100755 (executable)
@@ -74,7 +74,7 @@ sub populate_marc {
 
     # date1, date2
     my $my_008 = $record->field('008');
-    $marc{tag008} = $my_008->as_string();# if ($my_008);
+    $marc{tag008} = $my_008->as_string() if ($my_008);
     if (defined $marc{tag008}) {
         unless (length $marc{tag008} == 40) {
             $marc{tag008} = $marc{tag008} . ('|' x (40 - length($marc{tag008})));
@@ -364,16 +364,16 @@ sub dump_exception {
         print XF "Undefined record at line $count; likely bad XML\n";
         return;
     }
-    print XF "Record ", $marc->{id}, " did not make the cut: ";
-    print XF "Missing item_form. " unless ($marc->{item_form});
+    print XF "Record ", $marc->{id}, " excluded: ";
+    print XF "missing item_form; " unless ($marc->{item_form});
     unless (defined $marc->{date1})
-      { print XF "Missing date1. " }
+      { print XF "missing date1; " }
     else
-      { print XF "Invalid date1: ", $marc->{date1}, " "
+      { print XF "invalid date1: '", $marc->{date1}, "'; "
           unless ($marc->{date1} =~ /\d{4}/); }
-    print XF "Missing record_type. " unless ($marc->{record_type});
-    print XF "Missing bib_lvl. " unless ($marc->{bib_lvl});
-    print XF "Missing title. " unless ($marc->{title});
+    print XF "missing record_type; " unless ($marc->{record_type});
+    print XF "missing bib_lvl; " unless ($marc->{bib_lvl});
+    print XF "missing title " unless ($marc->{title});
     print XF "\n";
 }