@natlibfi/marc-record-validators-melinda 11.1.1 → 11.2.0-alpha.1

package/src/cyrillux-usemarcon-replacement.js

@@ -8,6 +8,7 @@
 import clone from 'clone';
 import {MarcRecord} from '@natlibfi/marc-record';
 import {default as fix33X} from './fix-33X';
+import {default as add041} from './addMissingField041';
 import {default as add336} from './addMissingField336';
 import {default as add337} from './addMissingField337';
 import {default as add338} from './addMissingField338';
@@ -68,6 +69,7 @@ export default function () {
     f028.forEach(f => sortAdjacentSubfields(f));
 
     fixField040(record); // All $b values are changed to 'mul'. As a side effect 33X$b=>$a mappings are in Finnish! Ok in this domain!
+    add041().fix(record);
 
     fixRelatorTerms().fix(record);
 
@@ -338,6 +340,8 @@ function expandEnglishAbbreviations(value) {
     replace(/col\. ill\./ui, 'color illustrations').
     replace(/diagrs\./ui, 'diagrams').
     replace(/\bhbk\.\b/gui, 'hardcover').replace(/\bhbk\b/gui, 'hardcover'). // expand to MTS-compliant form
+    replace(/\b1 hr\./gui, '1 hour').
+    replace(/\bhr\./gui, 'hours').
     replace(/\bill\./gui, 'illustrated'). // or illustrations (or Swedish "illustrerad" or...)
     replace(/\billus\./gui, 'illustrated'). // or illustrations
     replace(/incl\./gui, 'includes').
@@ -346,9 +350,10 @@ function expandEnglishAbbreviations(value) {
     // replace(/\bmin\b/gu, 'minutes').
     replace(/\bp\.\b/gui, 'pages').replace(/\bp\b/gu, 'pages').
     replace(/\bpbk\.\b/gui, 'paperback').replace(/\bpbk\b/gui, 'paperback'). // expand to MTS-compliant form
+    replace(/\bpdf\b/gui, 'PDF').
     replace(/\bports\./gui, 'portraits').
     replace('sd., col.', 'sound, color').
-    replace(/ *\((?:chiefly col\.|chiefly color|some cols)\)/gui, '').
+    replace(/ *\((?:chiefly col\.|chiefly color|some col[s.])\)/gui, '').
     replace(/\b1 hr\./gui, '1 hour');
 }
 
@@ -375,33 +380,51 @@ function finnishTranslationsAndMappings(value) {
     replace('approximately', 'noin').
     replace('audio discs', 'äänilevyä').
     replace('black and white', 'mustavalkoinen').
+    replace(/\bbilaga\b/gui, 'liite').
+    replace(/\bbilagor\b/gui, 'liitettä').
     replace(/\bcharts\b/gui, 'kaavioita').
     replace('chiefly color illustrations', 'pääosin värikuvitettu').
     replace('chiefly', 'pääosin').
+    replace(/\bcirca\b/gui, 'noin').
+    replace(/coil[- ]?bound/gui, 'kierreselkä').
     replace('color illustrations', 'värikuvitus').
+    replace(/comb[- ]?bound/gui, 'kierreselkä').
     replace(/\bdigital\b/gui, 'digitaalinen').
     replace(/\belectronic book\b/gui, 'verkkoaineisto').
+    replace('(flera nummersviter)', '(useita numerointijaksoja)').
     replace(/\bfolded sheet\b/gui, 'taitelehti').
-    replace(/\bhardback\b/gui, 'kovakantinen').
+    replace(/\bfärgillustratione[nr]\b/gui, 'värikuvitus').
+    replace(/\bhard(?:back|cover)\b/gui, 'kovakantinen').
     replace(/\bhours\b/gui, 'tuntia').
+    replace(/\bi flera nummersviter/gui, 'useina numerointijaksoina').
     replace('illustrated', 'kuvitettu').
-    replace(/illustration\b/gui, 'kuvitus'). // kuvitus/kuvitettu variation is taken from usemarcon-cyrillux...
-    replace('illustrations', 'kuvitettu').
+    replace(/illustrations?\b/gui, 'kuvitettu'). // Based on usemacron-bookwhere (NB! usemarcon-cyrillux had kuvitus/kuvitettu)
+    replace(/\binbunden\b/gui, 'kovakantinen'). // swe
     replace(/\binsert\b/gui, 'liite').
     replace(/\binserts\b/gui, 'liitteitä').
+    replace(/\bin various pagings/gui, 'useina numerointijaksoina').
     replace('leaves of plates', 'kuvalehteä').
+    replace(/\bljudskiva\b/gui, 'äänilevy').
+    replace(/\bljudskivor\b/gui, 'äänilevyä').
     replace(/\bmap\b/gui, 'kartta').
     replace(/\bmaps\b/gui, 'karttoja'). // or karttaa?
     replace('minutes', 'minuuttia').
+    replace('mjuka pärmar', 'pehmeäkantinen').
     replace('online resource', 'verkkoaineisto').
+    replace('onlineresurs', 'verkkoaineisto').
+    replace('onumrerade', 'numeroimatonta').
     replace('pages of plates', 'kuvalehteä').
     replace(/\bpages\b/gui, 'sivua').
     replace(/\bpaperback\b/gui, 'pehmeäkantinen'). // MTS alt
+    replace(/\bSeiten\b/gu, 'sivua').
     replace(/\bsoftcover\b/gui, 'pehmeäkantinen'). // MTS pref
     replace('sound, color', 'äänellinen, värillinen').
     replace('sound cassettes', 'äänikasettia').replace('sound cassette', 'äänikasetti').
-    replace('sound discs', 'äänilevyä').replace('sound disc', 'äänilevy').
+    replace('sound discs', 'äänilevyä').replace(/sound disc\b/gui, 'äänilevy').
+    replace(/(?:spiral[- ]?bound|spiralrygg)/gui, 'kierreselkä').
+    replace('svartvit', 'mustavalkoinen').
     replace('unnumbered', 'numeroimatonta').
+    replace('(various pagings)', '(useita numerointijaksoja)').
     replace(/\bverkkojulkaisu\b/gui, 'verkkoaineisto').
     replace('videodiscs', 'videolevyä').
     replace('videodisc', 'videolevy').

package/src/punctuation2.js

@@ -86,7 +86,7 @@ export function fieldNeedsModification(field, add = true) {
 
 
 //const stripCrap = / *[-;:,+]+$/u;
-const defaultNeedsPuncAfter = /(?:[a-z0-9A-Z]|å|ä|ö|Å|Ä|Ö)$/u;
+const needsPuncAfterAlphanumeric = /(?:[a-z0-9A-Z]|å|ä|ö|Å|Ä|Ö)$/u;
 const defaultNeedsPuncAfter2 = /(?:[\]a-zA-Z0-9)]|ä|å|ö|Å|Ä|Ö)$/u;
 const doesNotEndInPunc = /[^!?.:;,]$/u; // non-punc for pre-240/700/XXX $, note that '.' comes if preceded by ')'
 const blocksPuncRHS = /^(?:\()/u;
@@ -105,7 +105,6 @@ const cleanX00aDot = {'code': 'abcde', 'followedBy': 'cdegj', 'context': dotIsPr
 const cleanCorruption = {'code': 'abcdefghijklmnopqrstuvwxyz', 'remove': / \.$/u};
 // These $e dot removals are tricky: before removing the comma, we should know that it ain't an abbreviation such as "esitt."...
 const cleanX00eDot = {'code': 'e', 'followedBy': 'egj#', 'context': /(?:[ai]ja|jä)[.,]$/u, 'remove': /\.$/u};
-
 const removeCommaBeforeLanguageSubfieldL = {'followedBy': 'l', 'remove': /,$/u};
 const removeCommaBeforeTitleSubfieldT = {'followedBy': 't', 'remove': /,$/u};
 
@@ -113,17 +112,16 @@ const X00RemoveDotAfterBracket = {'code': 'cq', 'context': /\)\.$/u, 'remove': /
 // 390, 800, 810, 830...
 const cleanPuncBeforeLanguage = {'code': 'atvxyz', 'followedBy': 'l', 'context': puncIsProbablyPunc, 'remove': / *[.,:;]$/u};
 
-
 const addX00aComma = {'add': ',', 'code': 'abcqej', 'followedBy': 'cdeg', 'context': doesNotEndInPunc, 'contextRHS': allowsPuncRHS};
 const addX00dComma = {'name': 'X00$d ending in "-" does not get comma', 'add': ',', 'code': 'd', 'followedBy': 'cdeg', 'context': /[^-,.!]$/u, 'contextRHS': allowsPuncRHS};
 const addX00aComma2 = {'add': ',', 'code': 'abcdej', 'followedBy': 'cdeg', 'context': /(?:[A-Z]|Å|Ä|Ö)\.$/u, 'contextRHS': allowsPuncRHS};
-const addX00Dot = {'add': '.', 'code': 'abcdetv', 'followedBy': '#fklptu', 'context': defaultNeedsPuncAfter};
+const addX00Dot = {'add': '.', 'code': 'abcdetv', 'followedBy': '#fklptu', 'context': needsPuncAfterAlphanumeric};
 
 
-//const addX10iaComma = {'name': 'Punctuate relationship information', 'code': 'i', 'followedBy': 'a', 'context': defaultNeedsPuncAfter2};
-const addX10bDot = {'name': 'Add X10 pre-$b dot', 'add': '.', 'code': 'ab', 'followedBy': 'b', 'context': defaultNeedsPuncAfter};
-const addX10eComma = {'add': ',', 'code': 'abe', 'followedBy': 'e', 'context': defaultNeedsPuncAfter};
-const addX10Dot = {'name': 'Add X10 final dot', 'add': '.', 'code': 'abet', 'followedBy': 'tu#', 'context': defaultNeedsPuncAfter};
+const addX10iColon = {'name': 'Punctuate relationship information', 'code': 'i', 'context': defaultNeedsPuncAfter2};
+const addX10bDot = {'name': 'Add X10 pre-$b dot', 'add': '.', 'code': 'ab', 'followedBy': 'b', 'context': needsPuncAfterAlphanumeric};
+const addX10eComma = {'add': ',', 'code': 'abe', 'followedBy': 'e', 'context': defaultNeedsPuncAfter2};
+const addX10Dot = {'name': 'Add X10 final dot', 'add': '.', 'code': 'abet', 'followedBy': 'tu#', 'context': needsPuncAfterAlphanumeric};
 const addColonToRelationshipInformation = {'name': 'Add \':\' to 7X0 $i relationship info', 'add': ':', 'code': 'i', 'context': defaultNeedsPuncAfter2};
 
 const addDotBeforeLanguageSubfieldL = {'name': 'Add dot before $l', 'add': '.', 'code': 'abepst', 'followedBy': 'l', 'context': doesNotEndInPunc};
@@ -248,6 +246,7 @@ const cleanValidPunctuationRules = {
   '245': clean24X,
   '246': clean24X,
   '260': [
+    {'code': 'abc', 'followedBy': 'a', 'remove': / ;$/u},
     {'code': 'a', 'followedBy': 'b', 'remove': / :$/u},
     {'code': 'b', 'followedBy': 'c', 'remove': /,$/u},
     {'code': 'c', 'followedBy': '#', 'remove': /\.$/u},
@@ -291,29 +290,29 @@ const addToAllEntryFields = [addDotBeforeLanguageSubfieldL, addSemicolonBeforeVo
 
 
 const addX00 = [addX00aComma, addX00aComma2, addX00Dot, addX00dComma, ...addToAllEntryFields];
-const addX10 = [addX10bDot, addX10eComma, addX10Dot, ...addToAllEntryFields];
+const addX10 = [addX10iColon, addX10bDot, addX10eComma, addX10Dot, ...addToAllEntryFields];
 const addX11 = [...addToAllEntryFields];
 const addX30 = [...addToAllEntryFields];
 
 const add24X = [
-  {'code': 'i', 'followedBy': 'a', 'add': ':', 'context': defaultNeedsPuncAfter},
-  {'code': 'a', 'followedBy': 'b', 'add': ' :', 'context': defaultNeedsPuncAfter},
-  {'code': 'abk', 'followedBy': 'f', 'add': ',', 'context': defaultNeedsPuncAfter},
-  {'code': 'abfnp', 'followedBy': 'c', 'add': ' /', 'context': defaultNeedsPuncAfter},
+  {'code': 'i', 'followedBy': 'a', 'add': ':', 'context': needsPuncAfterAlphanumeric},
+  {'code': 'a', 'followedBy': 'b', 'add': ' :', 'context': needsPuncAfterAlphanumeric},
+  {'code': 'abk', 'followedBy': 'f', 'add': ',', 'context': needsPuncAfterAlphanumeric},
+  {'code': 'abfnp', 'followedBy': 'c', 'add': ' /', 'context': needsPuncAfterAlphanumeric},
   addDotBeforeLanguageSubfieldL
 ];
 
 const add245 = [
   ...add24X,
   // Blah! Also "$a = $b" and "$a ; $b" can be valid... But ' :' is better than nothing, I guess...
-  {'code': 'ab', 'followedBy': 'n', 'add': '.', 'context': defaultNeedsPuncAfter},
-  {'code': 'n', 'followedBy': 'p', 'add': ',', 'context': defaultNeedsPuncAfter},
-  {'code': 'abc', 'followedBy': '#', 'add': '.', 'context': defaultNeedsPuncAfter} // Stepping on "punctuation validator's" toes
+  {'code': 'ab', 'followedBy': 'n', 'add': '.', 'context': needsPuncAfterAlphanumeric},
+  {'code': 'n', 'followedBy': 'p', 'add': ',', 'context': defaultNeedsPuncAfter2},
+  {'code': 'abc', 'followedBy': '#', 'add': '.', 'context': needsPuncAfterAlphanumeric} // Stepping on "punctuation validator's" toes
 ];
 
 const addSeriesTitle = [ // 490 and 830
   {'code': 'a', 'followedBy': 'a', 'add': ' =', 'context': defaultNeedsPuncAfter2},
-  {'code': 'axyz', 'followedBy': 'xy', 'add': ',', 'context': defaultNeedsPuncAfter},
+  {'code': 'axyz', 'followedBy': 'xy', 'add': ',', 'context': defaultNeedsPuncAfter2},
   addSemicolonBeforeVolumeDesignation //  eg. 490$axyz-$v
 ];
 
@@ -328,7 +327,7 @@ const addPairedPunctuationRules = {
   '246': add24X,
   '260': [
     {'code': 'a', 'followedBy': 'b', 'add': ' :', 'context': defaultNeedsPuncAfter2},
-    {'code': 'b', 'followedBy': 'c', 'add': ',', 'context': defaultNeedsPuncAfter2},
+    {'code': 'ab', 'followedBy': 'c', 'add': ',', 'context': defaultNeedsPuncAfter2},
     {'code': 'abc', 'followedBy': 'a', 'add': ' ;', 'context': defaultNeedsPuncAfter2},
     {'code': 'e', 'followedBy': 'f', 'add': ' :', 'context': defaultNeedsPuncAfter2},
     {'code': 'f', 'followedBy': 'g', 'add': ',', 'context': defaultNeedsPuncAfter2}
@@ -338,7 +337,7 @@ const addPairedPunctuationRules = {
     {'code': 'b', 'followedBy': 'c', 'add': ',', 'context': defaultNeedsPuncAfter2},
     // NB! The $c rule messes dotless exception "264 #4 $c p1983" up
     // We'll need to add a hacky postprocessor for this? Add 'hasInd1': '0123' etc?
-    {'code': 'c', 'followedBy': '#', 'add': '.', 'context': defaultNeedsPuncAfter, 'ind2': ['0', '1', '2', '3']}
+    {'code': 'c', 'followedBy': '#', 'add': '.', 'context': needsPuncAfterAlphanumeric, 'ind2': ['0', '1', '2', '3']}
   ],
   '300': [
     {'code': 'a', 'followedBy': 'b', 'add': ' :', 'context': defaultNeedsPuncAfter2},
@@ -360,7 +359,7 @@ const addPairedPunctuationRules = {
   '810': addX10,
   '811': addX11,
   '830': [...addX30, ...addSeriesTitle],
-  '946': [{'code': 'i', 'followedBy': 'a', 'add': ':', 'context': defaultNeedsPuncAfter}]
+  '946': [{'code': 'i', 'followedBy': 'a', 'add': ':', 'context': defaultNeedsPuncAfter2}]
 };
 
 /*

package/src/sortSubfields.js

@@ -66,6 +66,7 @@ export default function (tagPattern) {
 
 
 // X00, X10, X11 and X130 could also for their own sets...
+// (ouch! sometimes $c comes after $d...): LoC: 100 0# ‡a Black Foot, ‡c Chief, ‡d d. 1877 ‡c (Spirit)
 const sortOrderForX00 = ['6', 'i', 'a', 'b', 'c', 'q', 'd', 'e', 't', 'u', 'l', 'f', 'x', 'y', 'z', '0', '5', '9']; // skip $g. Can't remember why, though...
 const sortOrderForX10 = ['6', 'i', 'a', 'b', 't', 'n', 'e', 'v', 'w', 'x', 'y', 'z', '0', '5', '9']; // somewhat iffy
 const sortOrderFor7XX = ['8', '6', '7', 'i', 'a', 's', 't', 'b', 'c', 'd', 'm', 'h', 'k', 'o', 'x', 'z', 'g', 'q', 'w'];

package/test-fixtures/addMissingField041/f01/expectedResult.json

@@ -0,0 +1,7 @@
+{
+  "_validationOptions": {},
+  "leader": "12345cam a22123454i 4500",
+  "fields": [
+    { "tag": "008", "value": "012345s7890    fi ||||             mul |"}
+  ]
+}

package/test-fixtures/addMissingField041/f01/metadata.json

@@ -0,0 +1,5 @@
+{
+  "description": "Fixer 01: no action required, won't add 'mul'",
+  "only": false,
+  "fix": true
+}

package/test-fixtures/addMissingField041/f01/record.json

@@ -0,0 +1,6 @@
+{
+  "leader": "12345cam a22123454i 4500",
+  "fields": [
+    { "tag": "008", "value": "012345s7890    fi ||||             mul |"}
+  ]
+}

package/test-fixtures/addMissingField041/f02/expectedResult.json

@@ -0,0 +1,8 @@
+{
+  "_validationOptions": {},
+  "leader": "12345cam a22123454i 4500",
+  "fields": [
+    { "tag": "008", "value": "012345s7890    fi ||||             fin |"},
+    { "tag": "041", "ind1": " ", "ind2": " ", "subfields": [{ "code": "a", "value": "fin"}]}
+  ]
+}

package/test-fixtures/addMissingField041/f02/metadata.json

@@ -0,0 +1,6 @@
+{
+  "description": "Fixer 02: Add field 041$a fin",
+  "comment": "Create a 041$a",
+  "only": false,
+  "fix": true
+}

package/test-fixtures/addMissingField041/f02/record.json

@@ -0,0 +1,6 @@
+{
+  "leader": "12345cam a22123454i 4500",
+  "fields": [
+    { "tag": "008", "value": "012345s7890    fi ||||             fin |"}
+  ]
+}

package/test-fixtures/addMissingField041/f03/expectedResult.json

@@ -0,0 +1,8 @@
+{
+  "_validationOptions": {},
+  "leader": "12345cam a22123454i 4500",
+  "fields": [
+    { "tag": "008", "value": "012345s7890    fi ||||             fin |"},
+    { "tag": "041", "ind1": " ", "ind2": " ", "subfields": [{ "code": "a", "value": "fin"}]}
+  ]
+}

package/test-fixtures/addMissingField041/f03/metadata.json

@@ -0,0 +1,6 @@
+{
+  "description": "Fixer 03: Don't add a secord 041$a field",
+  "comment": "Create a 041$a",
+  "only": false,
+  "fix": true
+}

package/test-fixtures/addMissingField041/f03/record.json

@@ -0,0 +1,7 @@
+{
+  "leader": "12345cam a22123454i 4500",
+  "fields": [
+    { "tag": "008", "value": "012345s7890    fi ||||             fin |"},
+    { "tag": "041", "ind1": " ", "ind2": " ", "subfields": [{ "code": "a", "value": "fin"}]}
+  ]
+}

package/test-fixtures/addMissingField041/f04/expectedResult.json

@@ -0,0 +1,8 @@
+{
+  "_validationOptions": {},
+  "leader": "12345cjm a22123454i 4500",
+  "fields": [
+    { "tag": "008", "value": "012345s7890    fi ||||             foo |"},
+    { "tag": "041", "ind1": " ", "ind2": " ", "subfields": [{ "code": "d", "value": "foo"}]}
+  ]
+}

package/test-fixtures/addMissingField041/f04/metadata.json

@@ -0,0 +1,5 @@
+{
+  "description": "Fixer 04: Add field 041$d foo",
+  "only": false,
+  "fix": true
+}

package/test-fixtures/addMissingField041/f04/record.json

@@ -0,0 +1,6 @@
+{
+  "leader": "12345cjm a22123454i 4500",
+  "fields": [
+    { "tag": "008", "value": "012345s7890    fi ||||             foo |"}
+  ]
+}

package/test-fixtures/addMissingField041/v01/expectedResult.json

@@ -0,0 +1,5 @@
+{
+  "message": [
+  ],
+  "valid": true
+}

package/test-fixtures/addMissingField041/v01/metadata.json

@@ -0,0 +1,5 @@
+{
+  "description": "02/Validation: fails, as 041 can be added'",
+  "only": false,
+  "fix": false
+}

package/test-fixtures/addMissingField041/v01/record.json

@@ -0,0 +1,6 @@
+{
+  "leader": "12345cam a22123454i 4500",
+  "fields": [
+    { "tag": "008", "value": "012345s7890    fi ||||             mul |"}
+  ]
+}

package/test-fixtures/addMissingField041/v02/expectedResult.json

@@ -0,0 +1,6 @@
+{
+  "message": [
+    "Add missing 041 field based on 008/35-37: '041 ## ‡a fin'"
+  ],
+  "valid": false
+}

package/test-fixtures/addMissingField041/v02/metadata.json

@@ -0,0 +1,6 @@
+{
+  "description": "Validation: Add field 041",
+  "comment": "Create a 336",
+  "only": false,
+  "fix": false
+}

package/test-fixtures/addMissingField041/v02/record.json

@@ -0,0 +1,6 @@
+{
+  "leader": "12345cam a22123454i 4500",
+  "fields": [
+    { "tag": "008", "value": "012345s7890    fi ||||             fin |"}
+  ]
+}

package/test-fixtures/addMissingField336/01/metadata.json

@@ -1,7 +1,6 @@
 {
   "description": "Validation: Add field 336",
   "comment": "Create a 336",
-  "enabled": true,
   "only": false,
   "fix": false
 }

package/test-fixtures/addMissingField336/02/metadata.json

@@ -1,7 +1,6 @@
 {
   "description": "Validation: Don't add field 336",
   "comment": "Don't create a 336 as there's already one",
-  "enabled": true,
   "only": false,
   "fix": false
 }

package/test-fixtures/addMissingField336/cod/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (b=cod). Decision based on CF's 008/26=a)",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/cop/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (b=cop). Decision based on CF's 008/26=b)",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/cop2/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (b=cop). Decision based on field 516)",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/crf/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (crf) based on LDR/06=g + 008/25=d/globe",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/cri/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (cri) based on LDR/06=e",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/crt/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (crt) based on LDR/06=e, form of item=f (braille) and 007/00-01=aq",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/f01/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (b=txt), cat kanguage (040$b) is 'swe'",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/f02/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (b=txt)",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/mixed2txt/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (b=txt). Default value for LDR/06=p is overridden by form of item 008/23=r",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/mixed2xxx/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (b=xxx). Default value for LDR/06=p",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/ntm/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (notated music), based on LDR/06=d (does not have exceptions)",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/snd/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (b=snd). Decision based on CF's 008/26=h)",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/spw/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (spw) based on LDR/06=i",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/spw2/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (b=spw). Decision based on field 516",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/spw_txt/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add two fields 336 (b=spw and txt). Decision based on field 516)",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/sti/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (tactile image), based on LDR/06=k",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/sti_projected/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (sti) based on LDR/06=g + 007/00=g",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/tci/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (still image) , based on LDR/06=k and form of item",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/tcm/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (tactile notated music), based on LDR/06=d  with 008/23 form-of-item-'f' triggered exception",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/tdf/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (3D form), based solely on LDR/06=r",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/tdi/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (tdi) based on LDR/06=g + 007/00=m",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/txt/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (b=txt). LDR/06=m, but not other information avaiable",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField336/zzz/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Fix: Add field 336 (b=txt). Decision based on LDR/06=m and field 516",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField337/c1/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "337: LDR/06=m, thus 337$b=c",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField337/g1/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "337: LDR/06=g and lone 007/00=g, so 337$b=g",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField337/n1/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "337: LDR/06=r and no exception found, so 337$b=n",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField337/no_action_required/metadata.json

@@ -1,7 +1,6 @@
 {
   "description": "337: Nothing to do, as 337 already exists",
   "comment": "337 is a bit illegal (on purpose), so this test might break down eventually",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField337/p1/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "337: LDR/06=m and 008/33=p, thus 337$b=p",
-  "enabled": true,
   "only": false,
   "fix": true
 }

package/test-fixtures/addMissingField337/validate_x1/metadata.json

@@ -1,6 +1,5 @@
 {
   "description": "Validation: Add field 337 based on LDR/06=p",
-  "enabled": true,
   "only": false,
   "fix": false
 }