@natlibfi/marc-record-validators-melinda 12.0.7 → 12.0.8

package/src/subfield6Utils.js

@@ -1,12 +1,10 @@
-//import createDebugLogger from 'debug';
-// const debug = createDebugLogger('@natlibfi/marc-record-validator-melinda/subfield6Utils');
-
+import createDebugLogger from 'debug';
 import {add8s, fieldsGetAllSubfield8LinkingNumbers, getSubfield8LinkingNumber, isValidSubfield8} from './subfield8Utils.js';
 import {fieldHasSubfield, fieldToString, /* fieldsToString, */ nvdebug, subfieldToString} from './utils.js';
 
-//const debug = createDebugLogger('@natlibfi/melinda-marc-record-merge-reducers:subfield6Utils');
+const debug = createDebugLogger('@natlibfi/melinda-marc-record-merge-reducers:subfield6Utils');
 //const debugData = debug.extend('data');
-//const debugDev = debug.extend('dev');
+const debugDev = debug.extend('dev');
 
 // NB! Subfield 6 is non-repeatable and it should always comes first!
 // NB! Index size should always be 2 (preceding 0 required for 01..09) However, support for 100+ was added on 2023-02-27.
@@ -41,7 +39,7 @@ export function subfield6GetOccurrenceNumberAsInteger(subfield) {
     return 0;
   }
   const result = parseInt(index, 10);
-  //nvdebug(`SF6: ${subfield.value} => ${index} => ${result}`, debug);
+  //nvdebug(`SF6: ${subfield.value} => ${index} => ${result}`, debugDev, debugDev);
   return result;
 }
 
@@ -52,7 +50,7 @@ export function subfield6ResetOccurrenceNumber(subfield, occurrenceNumber) {
   const occurrenceNumberAsString = typeof occurrenceNumber === 'number' ? intToOccurrenceNumberString(occurrenceNumber) : occurrenceNumber;
 
   const newValue = subfield.value.substring(0, 4) + occurrenceNumberAsString + subfield6GetTail(subfield);
-  //nvdebug(`Set subfield $6 value from ${subfieldToString(subfield)} to ${newValue}`);
+  //nvdebug(`Set subfield $6 value from ${subfieldToString(subfield)} to ${newValue}`, debugDev);
   subfield.value = newValue;
 }
 
@@ -71,7 +69,7 @@ export function subfield6HasWantedTagAndOccurrenceNumber(subfield, tagAndOccurre
   }
   // We could also use generic code and go getTag()+'-'+getIndex() instead of regexp...
   const key = subfield.value.replace(/^([0-9][0-9][0-9]-[0-9][0-9]+).*$/u, '$1');
-  //nvdebug(` Compare '${key}' vs '${tagAndOccurrenceNumber}'`);
+  //nvdebug(` Compare '${key}' vs '${tagAndOccurrenceNumber}'`, debugDev);
   return key === tagAndOccurrenceNumber;
 }
 
@@ -94,7 +92,7 @@ export function fieldGetUnambiguousOccurrenceNumber(field) {
 }
 
 export function fieldHasOccurrenceNumber(field, occurrenceNumber) {
-  //nvdebug(`${occurrenceNumber} vs ${fieldToString(field)}`);
+  //nvdebug(`${occurrenceNumber} vs ${fieldToString(field)}`, debugDev);
   return field.subfields && field.subfields.some(sf => subfield6GetOccurrenceNumber(sf) === occurrenceNumber);
 }
 
@@ -119,13 +117,13 @@ export function intToOccurrenceNumberString(i) {
 
 export function fieldGetMaxSubfield6OccurrenceNumberAsInteger(field) {
   // used by reducer!
-  //nvdebug(`Checking subfields $6 from ${JSON.stringify(field)}`);
+  //nvdebug(`Checking subfields $6 from ${JSON.stringify(field)}`, debugDev);
   const sf6s = field.subfields ? field.subfields.filter(subfield => isValidSubfield6(subfield)) : [];
   if (sf6s.length === 0) {
     return 0;
   }
   // There should always be one, but here we check every subfield.
-  //nvdebug(`Got ${field.subfields} $6-subfield(s) from ${JSON.stringify(field)}`, debug);
+  //nvdebug(`Got ${field.subfields} $6-subfield(s) from ${JSON.stringify(field)}`, debugDev);
   const vals = sf6s.map(sf => subfield6GetOccurrenceNumberAsInteger(sf));
   return Math.max(...vals);
 }
@@ -155,20 +153,20 @@ export function fieldHasValidSubfield6(field) {
 
 export function isSubfield6Pair(field, otherField) {
   // No need to log this:
-  //nvdebug(`LOOK for $6-pair:\n ${fieldToString(field)}\n ${fieldToString(otherField)}`);
+  //nvdebug(`LOOK for $6-pair:\n ${fieldToString(field)}\n ${fieldToString(otherField)}`, debugDev);
   if (!fieldHasValidSubfield6(field) || !fieldHasValidSubfield6(otherField)) {
     return false;
   }
 
   if (!tagsArePairable6(field.tag, otherField.tag)) {
-    //nvdebug(` FAILED. REASON: TAGS NOT PAIRABLE!`);
+    //nvdebug(` FAILED. REASON: TAGS NOT PAIRABLE!`, debugDev);
     return false;
   }
 
 
   const fieldIndex = fieldGetUnambiguousOccurrenceNumber(field);
   if (fieldIndex === undefined || fieldIndex === '00') {
-    //nvdebug(` FAILED. REASON: NO INDEX FOUND`);
+    //nvdebug(` FAILED. REASON: NO INDEX FOUND`, debugDev);
     return false;
   }
 
@@ -176,12 +174,12 @@ export function isSubfield6Pair(field, otherField) {
 
 
   if (fieldIndex !== otherFieldIndex) {
-    //nvdebug(` FAILURE: INDEXES: ${fieldIndex} vs ${otherFieldIndex}`);
+    //nvdebug(` FAILURE: INDEXES: ${fieldIndex} vs ${otherFieldIndex}`, debugDev);
     return false;
   }
 
   if (fieldGetUnambiguousTag(field) !== otherField.tag || field.tag !== fieldGetUnambiguousTag(otherField)) {
-    //nvdebug(` FAILURE: TAG vs $6 TAG`);
+    //nvdebug(` FAILURE: TAG vs $6 TAG`, debugDev);
     return false;
   }
   return true;
@@ -219,11 +217,11 @@ export function fieldGetOccurrenceNumberPairs(field, candFields) {
   //nvdebug(`  Trying to finds pair for ${fieldToString(field)} in ${candFields.length} fields`);
   const pairs = candFields.filter(otherField => isSubfield6Pair(field, otherField));
   if (pairs.length === 0) {
-    //nvdebug(`NO PAIRS FOUND FOR '${fieldToString(field)}'`);
+    //nvdebug(`NO PAIRS FOUND FOR '${fieldToString(field)}'`, debugDev);
     return pairs;
   }
-  //nvdebug(`${pairs.length} PAIR(S) FOUND FOR '${fieldToString(field)}'`);
-  pairs.forEach(pairedField => nvdebug(`  '${fieldToString(pairedField)}'`));
+  //nvdebug(`${pairs.length} PAIR(S) FOUND FOR '${fieldToString(field)}'`, debugDev);
+  pairs.forEach(pairedField => nvdebug(`  '${fieldToString(pairedField)}'`, debugDev));
   return pairs;
 }
 
@@ -268,7 +266,7 @@ export function fieldGetSubfield6Pair(field, record) {
     return undefined;
   }
   // NB! It is theoretically possible to have multiple pairable 880 fields (one for each encoding)
-  nvdebug(`fieldGetSubfield6Pair(): ${fieldToString(field)} => ${fieldToString(pairedFields[0])}`);
+  nvdebug(`fieldGetSubfield6Pair(): ${fieldToString(field)} => ${fieldToString(pairedFields[0])}`, debugDev);
   return pairedFields[0];
 }
 */
@@ -357,7 +355,7 @@ function guessTargetLinkingNumber(fields, defaultTargetLinkingNumber) {
 export function fieldsToNormalizedString(fields, defaultTargetLinkingNumber = 0, normalizeOccurrenceNumber = false, normalizeEntryTag = false) {
   const targetLinkingNumber = guessTargetLinkingNumber(fields, defaultTargetLinkingNumber);
 
-  //nvdebug(`fieldsToNormalizedString: OCC: ${normalizeOccurrenceNumber}`);
+  //nvdebug(`fieldsToNormalizedString: OCC: ${normalizeOccurrenceNumber}`, debugDev);
   const strings = fields.map(field => fieldToNormalizedString(field, targetLinkingNumber, normalizeOccurrenceNumber, normalizeEntryTag));
   strings.sort();
   return strings.join('\t__SEPARATOR__\t');
@@ -369,32 +367,32 @@ export function fieldsToNormalizedString(fields, defaultTargetLinkingNumber = 0,
 export function removeField6IfNeeded(field, record, fieldsAsString) {
   const pairField = fieldGetSubfield6Pair(field, record);
   const asString = pairField ? fieldsToNormalizedString([field, pairField]) : fieldToNormalizedString(field);
-  nvdebug(`SOURCE: ${asString} -- REALITY: ${fieldToString(field)}`);
+  nvdebug(`SOURCE: ${asString} -- REALITY: ${fieldToString(field)}`, debugDev);
   const tmp = pairField ? fieldToString(pairField) : 'HUTI';
-  nvdebug(`PAIR: ${tmp}`);
-  nvdebug(`BASE:\n ${fieldsAsString.join('\n ')}`);
+  nvdebug(`PAIR: ${tmp}`, debugDev);
+  nvdebug(`BASE:\n ${fieldsAsString.join('\n ')}`, debugDev);
   if (!fieldsAsString.includes(asString)) {
     return;
   }
-  nvdebug(`Duplicate $6 removal: ${fieldToString(field)}`);
+  nvdebug(`Duplicate $6 removal: ${fieldToString(field)}`, debugDev);
   record.removeField(field);
 
   if (pairField === undefined) {
     return;
   }
-  nvdebug(`Duplicate $6 removal (pair): ${fieldToString(pairField)}`);
+  nvdebug(`Duplicate $6 removal (pair): ${fieldToString(pairField)}`, debugDev);
   record.removeField(pairField);
 }
 */
 
 function getFirstField(record, fields) {
   const fieldsAsStrings = fields.map(field => fieldToString(field));
-  //record.fields.forEach((field, i) => nvdebug(`${i}:\t${fieldToString(field)}`));
-  //nvdebug(`getFirstField: ${fieldsAsStrings.join('\t')}`);
+  //record.fields.forEach((field, i) => nvdebug(`${i}:\t${fieldToString(field)}`, debugDev));
+  //nvdebug(`getFirstField: ${fieldsAsStrings.join('\t')}`, debugDev);
   const i = record.fields.findIndex(field => fieldsAsStrings.includes(fieldToString(field)));
   if (i > -1) {
     const field = record.fields[i];
-    //nvdebug(`1st F: ${i + 1}/${record.fields.length} ${fieldToString(field)}`);
+    //nvdebug(`1st F: ${i + 1}/${record.fields.length} ${fieldToString(field)}`, debugDev);
     return field;
   }
   return undefined;
@@ -445,7 +443,7 @@ export function isFirstLinkedSubfield6Field(field, record) {
   }
   const chain = getAllLinkedSubfield6Fields(field, record);
   if (!isRelevantSubfield6Chain(chain)) {
-    //nvdebug(`Rejected 6: ${fieldsToString(chain)}`);
+    //nvdebug(`Rejected 6: ${fieldsToString(chain)}`, debugDev);
     return false;
   }
 
@@ -472,13 +470,13 @@ export function get6s(field, candidateFields) { // NB! Convert field to fields!!
   if (sixes.length === 0) {
     return [field];
   }
-  //nvdebug(`SIXES: ${sixes.length}`);
+  //nvdebug(`SIXES: ${sixes.length}`, debugDev);
   const occurrenceNumbers = sixes.map(sf => subfield6GetOccurrenceNumber(sf)).filter(value => value !== undefined && value !== '00');
-  //nvdebug(occurrenceNumbers.join(' -- '));
+  //nvdebug(occurrenceNumbers.join(' -- '), debugDev);
 
   const relevantFields = candidateFields.filter(f => occurrenceNumbers.some(o => fieldHasOccurrenceNumber(f, o)));
-  //nvdebug(`${fieldToString(field)}: $6-RELFIELDS FOUND: ${relevantFields.length}...`);
-  //relevantFields.forEach(f => nvdebug(fieldToString(f)));
+  //nvdebug(`${fieldToString(field)}: $6-RELFIELDS FOUND: ${relevantFields.length}...`, debugDev);
+  //relevantFields.forEach(f => nvdebug(fieldToString(f), debugDev));
   return relevantFields;
 }
 
@@ -488,6 +486,6 @@ export function resetSubfield6Tag(subfield, tag) {
   }
   // NB! mainly for 1XX<->7XX transfers
   const newValue = `${tag}-${subfield.value.substring(4)}`;
-  //nvdebug(`Set subfield $6 value from ${subfieldToString(subfield)} to ${newValue}`, debugDev);
+  //nvdebug(`Set subfield $6 value from ${subfieldToString(subfield)} to ${newValue}`, debugDevDev);
   subfield.value = newValue;
 }

package/src/subfield8Utils.js

@@ -1,7 +1,10 @@
 // import createDebugLogger from 'debug';
+// import {fieldToString, nvdebug} from './utils.js';
+
 // const debug = createDebugLogger('@natlibfi/marc-record-validator-melinda/subfield8Utils');
+//const debugData = debug.extend('data');
+//const debugDev = debug.extend('dev');
 
-// import {fieldToString, nvdebug} from './utils.js';
 
 const sf8Regexp = /^([1-9][0-9]*)(?:\.[0-9]+)?(?:\\[acprux])?$/u;
 
@@ -10,9 +13,9 @@ export function isValidSubfield8(subfield) {
     return false;
   }
 
-  //nvdebug(`   IS VALID $8? '${subfieldToString(subfield)}'`);
+  //nvdebug(`   IS VALID $8? '${subfieldToString(subfield)}'`, debugDev);
   const match = subfield.value.match(sf8Regexp);
-  //nvdebug(`   IS VALID $8? '${subfieldToString(subfield)}' vs ${match.length}}`);
+  //nvdebug(`   IS VALID $8? '${subfieldToString(subfield)}' vs ${match.length}}`, debugDev);
   return match && match.length > 0;
 }
 
@@ -56,7 +59,7 @@ export function fieldsGetAllSubfield8LinkingNumbers(fields) {
     field.subfields.forEach(sf => {
       const linkingNumber = getSubfield8LinkingNumber(sf);
       if (linkingNumber > 0 && !subfield8LinkingNumbers.includes(linkingNumber)) {
-        //nvdebug(` LINK8: Add subfield \$8 ${linkingNumber} to seen values list`);
+        //nvdebug(` LINK8: Add subfield \$8 ${linkingNumber} to seen values list`, debugDev);
         subfield8LinkingNumbers.push(linkingNumber);
       }
     });
@@ -76,10 +79,10 @@ export function add8s(fields, record) {
     return fields;
   }
 
-  //nvdebug(`Linking number(s): ${linkingNumbers.join(', ')}`);
+  //nvdebug(`Linking number(s): ${linkingNumbers.join(', ')}`, debugDev);
   linkingNumbers.forEach(number => collectLinkingNumberFields(number));
 
-  //fields.forEach(f => nvdebug(`AFTER ADDING 8s: '${fieldToString(f)}'`));
+  //fields.forEach(f => nvdebug(`AFTER ADDING 8s: '${fieldToString(f)}'`, debugDev));
 
   return fields;
 
@@ -88,7 +91,7 @@ export function add8s(fields, record) {
     fields = fields.filter(f => !fieldHasLinkingNumber(f, linkingNumber));
     // Add them and their "sisters" back:
     const addableFields = record.fields.filter(f => fieldHasLinkingNumber(f, linkingNumber));
-    //addableFields.forEach(f => nvdebug(`(RE-?)ADD ${fieldToString(f)}`));
+    //addableFields.forEach(f => nvdebug(`(RE-?)ADD ${fieldToString(f)}`, debugDev));
     fields = fields.concat(addableFields);
 
   }

package/src/translate-terms.js

@@ -4,6 +4,9 @@ import {fieldHasSubfield, fieldToString, nvdebug} from './utils.js';
 
 
 const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda:translate-terms');
+//const debugData = debug.extend('data');
+const debugDev = debug.extend('dev');
+
 const defaultTags = ['648', '650', '651', '655'];
 
 const swapLanguageCode = {'fin': 'swe', 'fi': 'sv', 'sv': 'fi', 'swe': 'fin'};
@@ -12,6 +15,7 @@ const changeAbbrHash = {'fi': 'fin', 'fin': 'fi', 'sv': 'swe', 'swe': 'sv'};
 const termCache = {};
 
 // Author(s): Nicholas Volk
+// eslint-disable-next-line max-lines-per-function
 export default function () {
 
 
@@ -23,7 +27,7 @@ export default function () {
   async function fix(record) {
     const newFields = await getFields(record, defaultTags, []);
 
-    newFields.forEach(nf => nvdebug(`Add new field '${fieldToString(nf)}'`, debug));
+    newFields.forEach(nf => nvdebug(`Add new field '${fieldToString(nf)}'`, debugDev));
 
     newFields.forEach(nf => record.insertField(nf));
 
@@ -65,7 +69,7 @@ export default function () {
     const finnishOnly = getMisses(finnishFields, swedishFields);
     const swedishOnly = getMisses(swedishFields, finnishFields);
 
-    //console.log(` Looking at ${finnishOnly.length} + ${swedishOnly.length} fields`); // eslint-disable-line no-console
+    //nvdebug(` Looking at ${finnishOnly.length} + ${swedishOnly.length} fields`, debugDev);
     return [...finnishOnly, ...swedishOnly].filter(f => tagAndFieldAgree(f));
 
     function tagAndFieldAgree(field) {
@@ -93,12 +97,12 @@ export default function () {
     if (!prefLabels) {
       return undefined;
     }
-    //console.log(`pairField() WP 1: ${fieldToString(field)}`); // eslint-disable-line no-console
+    //nvdebug(`pairField() WP 1: ${fieldToString(field)}`, debugDev);
     const lexAndLang = getLexiconAndLanguage(field);
-    //console.log(`pairField() WP 2: ${JSON.stringify(lexAndLang)}`); // eslint-disable-line no-console
+    //nvdebug(`pairField() WP 2: ${JSON.stringify(lexAndLang)}`, debugDev);
     const twoLetterOtherLang = swapLanguageCodeBetweenLanguages(changeAbbr(lexAndLang.lang));
     const prefLabel = prefLabels.find(l => l.lang === twoLetterOtherLang);
-    //console.log(`pairField() WP 4: ${JSON.stringify(prefLabel)}`); // eslint-disable-line no-console
+    //nvdebug(`pairField() WP 4: ${JSON.stringify(prefLabel)}`, debugDev);
     const sfA = {'code': 'a', 'value': prefLabel.value}; // field.subfields.field(sf => sf.code === 'a');
     const sf0 = clone(field.subfields.find(sf => sf.code === '0'));
     const sf2 = {'code': '2', 'value': `${lexAndLang.lex}/${lexAndLang.lang === 'fin' ? 'swe' : 'fin'}`}; // swap fin <=> swe
@@ -118,7 +122,7 @@ export default function () {
     const data = await getTermData(uri);
 
     if (!data) { // Sanity check. Miss caused by illegal id etc.
-      nvdebug(`No labels found for ${uri}`, debug);
+      nvdebug(`No labels found for ${uri}`, debugDev);
       return undefined;
     }
 
@@ -223,9 +227,9 @@ function mapTagToLex(tag) {
 }
 
 export async function getTermData(uri) {
-  nvdebug(`getTermData(${uri})`);
+  nvdebug(`getTermData(${uri})`, debugDev);
   if (termCache[uri]) { // Don't think current implementation uses the cache any more.
-    //console.log(`CACHED ${uri}`); // eslint-disable-line no-console
+    //nvdebug(`CACHED ${uri}`, debugDev);
     return termCache[uri];
   }
   const tmp = await getTermDataFromFinto(uri);
@@ -252,7 +256,7 @@ async function getTermDataFromFinto(uri) {
       prefLabel: processLabel(hit?.prefLabel || undefined),
       altLabel: processLabel(hit?.altLabel || undefined)
     };
-    //console.log(`NEW JSON: ${JSON.stringify(hit)}`); // eslint-disable-line no-console
+    //nvdebug(`NEW JSON: ${JSON.stringify(hit)}`, debugDev);
 
     return subset;
 

package/src/typeOfDate-008.js

@@ -1,6 +1,9 @@
 import createDebugLogger from 'debug';
+import { nvdebug } from './utils.js';
 
 const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/typeOfDate-008');
+//const debugData = debug.extend('data');
+const debugDev = debug.extend('dev');
 
 export default function () {
   return {
@@ -15,7 +18,7 @@ export default function () {
     const c1114 = f008.value.substring(11, 15);
     // if 008 06 = s, and 11-14 = #### (not year/digits)
     if (c06 === 't' && !(/[0-9u]{4}/u).test(c1114)) {
-      debug('is t and not valid 1114');
+      nvdebug('is t and not valid 1114', debugDev);
       return {valid: false, message: 'Invalid 008 06'};
     }
 

package/src/update-field-540.js

@@ -1,7 +1,9 @@
 //import createDebugLogger from 'debug';
-import {fieldToString} from './utils.js';
+import {fieldToString /*, nvdebug*/} from './utils.js';
 
 //const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/update-field-540');
+//const debugData = debug.extend('data');
+//const debugDev = debug.extend('dev');
 
 // Author(s): Nicholas Volk
 export default function () {
@@ -54,7 +56,7 @@ function validLicenseInSubfieldC(subfieldC, license) {
   if (subfieldC.code !== 'c') {
     return false;
   }
-  //nvdebug(`Compare ${subfieldC.value} vs ${license.license}`);
+  //nvdebug(`Compare ${subfieldC.value} vs ${license.license}`, debugDev);
   return license.license === subfieldC.value;
 }
 
@@ -62,7 +64,7 @@ function validUrlInSubfieldU(subfieldU, license) {
   if (subfieldU.code !== 'u') {
     return false;
   }
-  //nvdebug(`Compare ${subfieldU.value} vs ${license.url}`);
+  //nvdebug(`Compare ${subfieldU.value} vs ${license.url}`, debugDev);
   return license.url === subfieldU.value;
 }
 
@@ -78,12 +80,12 @@ function fieldHasOldCcLicense(field, fix) {
   if (field.tag !== '540') {
     return false;
   }
-  //nvdebug(`NORM 540: ${fieldToString(field)}`);
+  //nvdebug(`NORM 540: ${fieldToString(field)}`, debugDev);
   const validLicense = licences.find(license => field.subfields.some(sf => validLicenseInSubfieldC(sf, license)) && field.subfields.some(sf => validUrlInSubfieldU(sf, license)));
   if (!validLicense) {
     return false;
   }
-  //nvdebug(` Found valid license`);
+  //nvdebug(` Found valid license`, debugDev);
   if (fix) {
     const subfieldsC = field.subfields.filter(sf => validLicenseInSubfieldC(sf, validLicense));
     subfieldsC.forEach(c => fixC(field, c));

package/src/urn.js

@@ -1,14 +1,17 @@
 //import fetch from 'node-fetch';
-import {isElectronicMaterial} from './utils.js';
+import {isElectronicMaterial, nvdebug} from './utils.js';
 import createDebugLogger from 'debug';
 
 const URN_GENERATOR_URL = 'https://generator.urn.fi/cgi-bin/urn_generator.cgi?type=nbn';
 
+// eslint-disable-next-line max-lines-per-function
 export default function (isLegalDeposit = false, useMelindaTemp = true) {
   const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda:urn');
   const debugData = debug.extend('data');
+  const debugDev = debug.extend('dev');
 
-  //console.log(`IS LEGAL DEPOSIT? ${isLegalDeposit ? 'YES' : 'NO'}`); // eslint-disable-line no-console
+
+  //nvdebug(`IS LEGAL DEPOSIT? ${isLegalDeposit ? 'YES' : 'NO'}`, debugDev); // eslint-disable-line no-console
 
   // We should check that the f856 with URN has second indicator '0' (Resource), ' ' (No information provided) or '8' (No display constant generated)
   // - if second indicator is '1' (Version of resource) or '2' (Related resource) the URN in f856 is not correct for the resource described in the record
@@ -35,12 +38,13 @@ export default function (isLegalDeposit = false, useMelindaTemp = true) {
     fix
   };
 
+  // eslint-disable-next-line max-lines-per-function
   async function fix(record) {
     const f856sUrn = record.fields.filter(hasLegalDepositURN);
-    debugData(`f856sUrn: ${JSON.stringify(f856sUrn)}`);
+    nvdebug(`f856sUrn: ${JSON.stringify(f856sUrn)}`, debugDev);
 
     const ldSubfields = isLegalDeposit ? createLDSubfields() : [];
-    debugData(`IsLegalDeposit: ${isLegalDeposit}, LegalDepositSubfields: ${JSON.stringify(ldSubfields)}`);
+    nvdebug(`IsLegalDeposit: ${isLegalDeposit}, LegalDepositSubfields: ${JSON.stringify(ldSubfields)}`, debugDev);
 
     // We add the URN even if we're not getting the legalDeposit - where does this URN resolve?
     // We probably should not do these additions
@@ -111,7 +115,7 @@ export default function (isLegalDeposit = false, useMelindaTemp = true) {
         return acc;
       }, undefined);
 
-      debugData(`isbns: ${isbn}`);
+      nvdebug(`isbns: ${isbn}`, debugDev);
 
       const {generated, value} = await createURN(isbn);
       return {code: 'u', value, generated};
@@ -161,12 +165,12 @@ export default function (isLegalDeposit = false, useMelindaTemp = true) {
   }
 
   function validateLD(f856sUrn) {
-    debug(`Validating the existence of legal deposit subfields`);
+    nvdebug(`Validating the existence of legal deposit subfields`, debugDev);
     const ldSubfields = createLDSubfields();
     const f856sUrnWithLdSubfields = f856sUrn.filter(field => fieldHasLDSubfields(field, ldSubfields));
     if (f856sUrnWithLdSubfields.length > 0) {
-      debug(`Record has ${f856sUrnWithLdSubfields.length} URN fields with all necessary legal deposit subfields`);
-      debugData(`f856sUrnWithLdSubfields: ${JSON.stringify(f856sUrnWithLdSubfields)}`);
+      nvdebug(`Record has ${f856sUrnWithLdSubfields.length} URN fields with all necessary legal deposit subfields`, debugDev);
+      nvdebug(`f856sUrnWithLdSubfields: ${JSON.stringify(f856sUrnWithLdSubfields)}`, debugData);
       return true;
     }
     return false;
@@ -175,22 +179,22 @@ export default function (isLegalDeposit = false, useMelindaTemp = true) {
   function validate(record) {
     // if not electronic skip this validator
     if (!isElectronicMaterial(record)) {
-      debug(`Record is not electronic - no need to validate legal deposit URNs`);
+      nvdebug(`Record is not electronic - no need to validate legal deposit URNs`, debugDev);
       return {valid: true};
     }
 
     const f856sUrn = record.fields.filter(hasLegalDepositURN);
 
     if (f856sUrn.length > 0) {
-      debug(`Record has ${f856sUrn.length} URN fields`);
-      debugData(`f856sUrn: ${JSON.stringify(f856sUrn)}`);
+      nvdebug(`Record has ${f856sUrn.length} URN fields`, debugDev);
+      nvdebug(`f856sUrn: ${JSON.stringify(f856sUrn)}`, debugData);
 
       if (!isLegalDeposit || validateLD(f856sUrn)) {
-        debug(`Record is valid`);
+        nvdebug(`Record is valid`, debugDev);
         return {valid: true};
       }
     }
-    debug(`No (valid) URN fields - Record is not valid`);
+    nvdebug(`No (valid) URN fields - Record is not valid`, debugDev);
     return {valid: false};
   }
 }

package/test-fixtures/drop-terms/02/metadata.json

@@ -2,5 +2,5 @@
   "description": "02: Drop some bad terms (validation)",
   "fix": false,
   "skip": false,
-  "only": true
+  "only": false
 }

package/test-fixtures/drop-terms/03/metadata.json

@@ -2,7 +2,7 @@
   "description": "03: Drop incorrect $0 from a bad term 'saiturit' (using non-default config, otherwise the whole field would be removed)",
   "fix": false,
   "skip": false,
-  "only": true,
+  "only": false,
 
   "config": {
     "constraints": [

package/test-fixtures/drop-terms/04/metadata.json

@@ -3,7 +3,7 @@
   "comment": "Uses a very unlikely config, used mainly for tests",
   "fix": false,
   "skip": false,
-  "only": true,
+  "only": false,
 
   "config": {
     "constraints": [