@natlibfi/marc-record-validators-melinda 11.6.6 → 11.6.7-alpha.1

package/src/removeInferiorDataFields.js

@@ -36,7 +36,7 @@ export default function () {
 
     const res = {message: duplicates};
 
-    res.valid = res.message.length < 1; // eslint-disable-line functional/immutable-data
+    res.valid = res.message.length < 1;
     return res;
   }
 }
@@ -46,7 +46,7 @@ function deriveInferiorChains(fields, record) {
   //nvdebug(`======= GOT ${fields.length} FIELDS TO CHAINIFY`);
   const hash = {};
 
-  fields.forEach(f => fieldToChainToDeletables(f));
+  fields.forEach(f => fieldToChainToDeletables(f)); // eslint-disable-line array-callback-return
 
   return hash;
 
@@ -65,8 +65,8 @@ function deriveInferiorChains(fields, record) {
     const arr = deriveChainDeletables([chainAsString]);
     //nvdebug(`GOT ${arr.length} DELETABLES FOR ${chainAsString}`);
     arr.forEach(val => {
-      if (!(val in hash)) { // eslint-disable-line functional/no-conditional-statements
-        hash[val] = field; // eslint-disable-line functional/immutable-data
+      if (!(val in hash)) {
+        hash[val] = field;
       }
     });
   }
@@ -78,7 +78,7 @@ function deriveInferiorChains(fields, record) {
     }
 
     // Fix MRA-476 (part 1): one $6 value can be worse than the other
-    const withoutScriptIdentificationCode = chainAsString.replace(/( ‡6 [0-9X][0-9][0-9]-(?:XX|[0-9]+))\/[^ ]+/u, '$1'); // eslint-disable-line prefer-named-capture-group
+    const withoutScriptIdentificationCode = chainAsString.replace(/( ‡6 [0-9X][0-9][0-9]-(?:XX|[0-9]+))\/[^ ]+/u, '$1');
 
     // Remove keepless versions:
     const keepless = chainAsString.replace(/ ‡9 [A-Z]+<KEEP>/u, '');
@@ -150,15 +150,15 @@ export function removeInferiorChains(record, fix = true) {
     const triggeringChain = fieldToChain(triggeringField, record);
 
     // If the inferior (deletable) chain is 1XX-based, convert the triggering better chain from 7XX to 1XX:
-    if (chainContains1XX(chain)) { // eslint-disable-line functional/no-conditional-statements
-      triggeringChain.forEach(f => sevenToOne(f, triggeringChain));
+    if (chainContains1XX(chain)) {
+      triggeringChain.forEach(f => sevenToOne(f, triggeringChain)); // eslint-disable-line array-callback-return
     }
     //nvdebug(`iRIS6C: ${chainAsString}`);
     const deletedString = fieldsToString(chain);
     const message = `DEL: '${deletedString}'  REASON: '${fieldsToString(triggeringChain)}'`;
-    if (fix) { // eslint-disable-line functional/no-conditional-statements
+    if (fix) {
       //nvdebug(`INFERIOR $6 CHAIN REMOVAL: ${message}}`, debug);
-      chain.forEach(field => record.removeField(field));
+      chain.forEach(field => record.removeField(field)); // eslint-disable-line array-callback-return
     }
     return innerRemoveInferiorChains(remainingFields, [...deletedStringsArray, message]);
   }
@@ -174,9 +174,9 @@ export function removeInferiorChains(record, fix = true) {
     }
     // Retag field 7XX as 1XX and fix corresponding occurrence numbers as well:
     const pairs = fieldGetOccurrenceNumberPairs(field, chain);
-    field.tag = `1${field.tag.substring(1)}`; // eslint-disable-line functional/immutable-data
+    field.tag = `1${field.tag.substring(1)}`;
     // There should always be one pair, but I'm not sanity-checking this
-    pairs.forEach(pairedField => fieldSevenToOneOccurrenceNumber(pairedField));
+    pairs.forEach(pairedField => fieldSevenToOneOccurrenceNumber(pairedField)); // eslint-disable-line array-callback-return
   }
 
 }
@@ -186,7 +186,7 @@ function getIdentifierlessAndKeeplessSubsets(fieldAsString) {
   // The rules below are not perfect (in complex cases they don't catch all permutations), but good enough:
   // Remove identifier(s) (MELKEHITYS-2383-ish):
 
-  const identifierlessString = fieldAsString.replace(/ ‡[01] [^‡]+($| ‡)/u, '$1'); // eslint-disable-line prefer-named-capture-group
+  const identifierlessString = fieldAsString.replace(/ ‡[01] [^‡]+($| ‡)/u, '$1');
   const keeplessString = fieldAsString.replace(/ ‡9 [A-Z]+<KEEP>/u, '');
 
   return [identifierlessString, keeplessString].filter(val => val !== fieldAsString);
@@ -208,7 +208,7 @@ function deriveIndividualDeletables490(todoList, deletables = []) {
   // Without final $v or $x:
   const withoutFinalVOrX = fieldAsString.replace(/ *[;,] ‡[vx] [^‡]+$/u, '');
   // Add intermediate $x-less version
-  const xless = fieldAsString.replace(/, ‡x [^‡]+(, ‡x| ; ‡v)/u, '$1'); // eslint-disable-line prefer-named-capture-group
+  const xless = fieldAsString.replace(/, ‡x [^‡]+(, ‡x| ; ‡v)/u, '$1');
 
   // Add $xv-less version (handled by recursion?)
   const xvless = fieldAsString.replace(/, ‡x [^‡]+ ‡v [^‡]+$/u, '');
@@ -219,7 +219,7 @@ function deriveIndividualDeletables490(todoList, deletables = []) {
   const arr = [sixless, withoutFinalVOrX, xless, xvless, modifiedInd2].filter(val => val !== fieldAsString);
 
   /*
-  if (arr.length) { // eslint-disable-line functional/no-conditional-statements
+  if (arr.length) {
     nvdebug(`${arr.length} derivation(s) for ${fieldAsString}`);
     nvdebug(arr.join('\n'));
   }
@@ -262,7 +262,7 @@ function deriveIndividualDeletables(record) {
 
     const accentless = getAccentlessVersion(currString);
     const d490 = deriveIndividualDeletables490([currString]);
-    const subsets = getIdentifierlessAndKeeplessSubsets(currString); // eslint-disable-line no-param-reassign
+    const subsets = getIdentifierlessAndKeeplessSubsets(currString);
     const moreToDo = [...accentless, ...d490, ...subsets];
 
 
@@ -301,7 +301,7 @@ function deriveIndividualDeletables(record) {
         replace(/ ‡[rg] /gu, ' '). // remove $r and $g subfields
         replace(/ ‡t /u, ' ‡a '). // change first $t to $a
         // ind2: '1' => '#':
-        replace(/^505 (.)0/u, '505 $1#'); // eslint-disable-line prefer-named-capture-group
+        replace(/^505 (.)0/u, '505 $1#');
       if (tmp !== currString) {
         return processTodoList([tmp, ...stillToDo, ...moreToDo], [...deletables, tmp]);
       }
@@ -318,11 +318,11 @@ function deriveIndividualDeletables(record) {
 
     // MET-381: remove occurence number TAG-00, if TAG-NN existists
     if (currString.match(/^880.* ‡6 [0-9][0-9][0-9]-(?:[1-9][0-9]|0[1-9])/u)) {
-      const tmp = currString.replace(/( ‡6 [0-9][0-9][0-9])-[0-9]+/u, '$1-00'); // eslint-disable-line prefer-named-capture-group
+      const tmp = currString.replace(/( ‡6 [0-9][0-9][0-9])-[0-9]+/u, '$1-00');
       //nvdebug(`MET-381: ADD TO DELETABLES: '${tmp}'`);
       //deletableStringsArray.push(tmp);
       if (tmp.match(/ ‡6 [0-9][0-9][0-9]-00\/[^ ]+ /u)) {
-        const tmp2 = tmp.replace(/( ‡6 [0-9][0-9][0-9]-00)[^ ]+/u, '$1'); // eslint-disable-line prefer-named-capture-group
+        const tmp2 = tmp.replace(/( ‡6 [0-9][0-9][0-9]-00)[^ ]+/u, '$1');
         //nvdebug(`MET-381: ADD TO DELETABLES: '${tmp2}'`);
         return processTodoList([...stillToDo, ...moreToDo], [...deletables, tmp, tmp2]);
       }
@@ -411,7 +411,7 @@ export function removeIndividualInferiorDatafields(record, fix = true) { // No $
 
   const deletedFieldsAsStrings = hits.map(f => fieldToString(f));
 
-  if (fix) { // eslint-disable-line functional/no-conditional-statements
+  if (fix) {
     hits.forEach(field => {
       //nvdebug(`Remove inferior field: ${fieldToString(field)}`, debug);
       record.removeField(field);

package/src/resolvable-ext-references-melinda.js

@@ -22,8 +22,8 @@ export default function ({endpoint, prefixPattern, fields}) {
     // Filter matching field keys from record.fields
     const subfields = record.fields.reduce((prev, current) => {
       Object.keys(fields).forEach(key => {
-        if (key === current.tag) { // eslint-disable-line functional/no-conditional-statements
-          prev.push(current); // eslint-disable-line functional/immutable-data
+        if (key === current.tag) {
+          prev.push(current);
         }
       });
       return prev;
@@ -32,10 +32,10 @@ export default function ({endpoint, prefixPattern, fields}) {
     // Filter matching objects from subfields
     const matchingTags = [...subfields].reduce((prev, current) => {
       Object.keys(fields).forEach(key => {
-        if (key === current.tag) { // eslint-disable-line functional/no-conditional-statements
+        if (key === current.tag) {
           current.subfields.filter(item => {
-            if (Object.values(fields[key]).filter(value => value === item.code)[0]) { // eslint-disable-line functional/no-conditional-statements
-              prev.push({tag: current.tag, code: item.code, value: item.value}); // eslint-disable-line functional/immutable-data
+            if (Object.values(fields[key]).filter(value => value === item.code)[0]) {
+              prev.push({tag: current.tag, code: item.code, value: item.value});
             }
 
             return prev;
@@ -47,9 +47,9 @@ export default function ({endpoint, prefixPattern, fields}) {
 
     // Matching prefixPattern is removed from object value field.
     matchingTags.forEach(obj => {
-      if (prefixPattern.test(obj.value)) { // eslint-disable-line functional/no-conditional-statements
-        obj.value = obj.value.replace(prefixPattern, ''); // eslint-disable-line functional/immutable-data
-        removedPrefixes.push(obj); // eslint-disable-line functional/immutable-data
+      if (prefixPattern.test(obj.value)) {
+        obj.value = obj.value.replace(prefixPattern, '');
+        removedPrefixes.push(obj);
       }
     });
     return resolveValidation(removedPrefixes);

package/src/resolveOrphanedSubfield6s.js

@@ -15,12 +15,12 @@ export default function () {
   function fix(record) {
     nvdebug('Fix SF6 orphaned occurrence numbers');
     const res = {message: [], fix: [], valid: true};
-    //message.fix = []; // eslint-disable-line functional/immutable-data
+    //message.fix = [];
 
     // This can not really fail...
     recordFixSubfield6OccurrenceNumbers(record);
 
-    //message.valid = !(message.message.length >= 1); // eslint-disable-line functional/immutable-data
+    //message.valid = !(message.message.length >= 1);
     return res;
   }
 
@@ -33,10 +33,10 @@ export default function () {
 
     const res = {message: []};
 
-    if (orphanedFields.length > 0) { // eslint-disable-line functional/no-conditional-statements
-      res.message = [`${orphanedFields.length} orphaned occurrence number field(s) detected`]; // eslint-disable-line functional/immutable-data
+    if (orphanedFields.length > 0) {
+      res.message = [`${orphanedFields.length} orphaned occurrence number field(s) detected`];
     }
-    res.valid = res.message.length < 1; // eslint-disable-line functional/immutable-data
+    res.valid = res.message.length < 1;
     return res;
   }
 }
@@ -45,12 +45,12 @@ export function recordFixSubfield6OccurrenceNumbers(record) {
   const fieldsContainingSubfield6 = record.fields.filter(field => fieldHasSubfield(field, '6'));
   const orphanedFields = getOrphanedFields(fieldsContainingSubfield6);
 
-  orphanedFields.forEach(field => fieldFixOrphanedSubfields(field));
+  orphanedFields.forEach(field => fieldFixOrphanedSubfields(field)); // eslint-disable-line array-callback-return
 
   function fieldFixOrphanedSubfields(field) {
     // Field 880: orphaned $6 subfields: occurrence number is changed to '00':
     if (field.tag === '880') {
-      field.subfields.forEach(sf => field880FixOrphanedSubfield(sf));
+      field.subfields.forEach(sf => field880FixOrphanedSubfield(sf)); // eslint-disable-line array-callback-return
       return;
     }
     // Non-880 fields get their orphaned $6s removed:
@@ -59,7 +59,7 @@ export function recordFixSubfield6OccurrenceNumbers(record) {
       record.removeField(field);
       return;
     }
-    field.subfields = remainingSubfields; // eslint-disable-line functional/immutable-data
+    field.subfields = remainingSubfields;
   }
 
   function field880FixOrphanedSubfield(subfield) {
@@ -78,7 +78,7 @@ function findPairForSubfield6OccurrenceNumber(subfield6, myTag, candPairFields)
     return undefined;
   }
   nvdebug(`LOOKING FOR PAIR: ${myTag} ${subfieldToString(subfield6)}`);
-  candPairFields.forEach(field => fieldToString(field));
+  candPairFields.forEach(field => fieldToString(field)); // eslint-disable-line array-callback-return
 
   // Only valid $6 value that fails to map to another field is iffy...
   const referredTag = subfield6.value.substring(0, 3);

package/src/sanitize-vocabulary-source-codes.js

@@ -15,7 +15,7 @@ export default function () {
   function fix(record) {
     const res = {message: [], fix: [], valid: true};
     const relevantFields = getRelevantFields(record);
-    relevantFields.forEach(f => fieldSanitizeVocabularySourceCode(f));
+    relevantFields.forEach(f => fieldSanitizeVocabularySourceCode(f)); // eslint-disable-line array-callback-return
     return res;
   }
 
@@ -23,9 +23,9 @@ export default function () {
     const res = {message: []};
     const relevantFields = getRelevantFields(record);
 
-    relevantFields.forEach(field => validateField(field, res));
+    relevantFields.forEach(field => validateField(field, res)); // eslint-disable-line array-callback-return
 
-    res.valid = !(res.message.length >= 1); // eslint-disable-line functional/immutable-data
+    res.valid = !(res.message.length >= 1);
     return res;
   }
 
@@ -39,12 +39,12 @@ export default function () {
     const normalizedField = fieldSanitizeVocabularySourceCode(clone(field));
     const mod = fieldToString(normalizedField);
     if (orig !== mod) { // Fail as the input is "broken"/"crap"/sumthing
-      res.message.push(`FIXABLE: '${orig}' => '${mod}'`); // eslint-disable-line functional/immutable-data
+      res.message.push(`FIXABLE: '${orig}' => '${mod}'`);
       return;
     }
     // Handle illegal values here
     if (fieldHasUnfixableVocabularySourceCode(field)) {
-      res.message.push(`CAN'T BE FIXED AUTOMATICALLY: '${orig}'`); // eslint-disable-line functional/immutable-data
+      res.message.push(`CAN'T BE FIXED AUTOMATICALLY: '${orig}'`);
       return;
     }
     return;
@@ -52,7 +52,7 @@ export default function () {
 
 
   function fieldSanitizeVocabularySourceCode(field) {
-    field.subfields.forEach(sf => subfieldSanitizeVocabularySourceCode(sf));
+    field.subfields.forEach(sf => subfieldSanitizeVocabularySourceCode(sf)); // eslint-disable-line array-callback-return
     return field;
   }
 
@@ -60,7 +60,7 @@ export default function () {
     if (subfield.code !== '2') {
       return;
     }
-    subfield.value = stringFixVocabularySourceCode(subfield.value); // eslint-disable-line functional/immutable-data
+    subfield.value = stringFixVocabularySourceCode(subfield.value);
   }
 
 }
@@ -72,7 +72,7 @@ function stringFixVocabularySourceCode(value) {
   // Try to remove spaces, change '//' to '/' and remove final '.' and '/':
   const tmp = value.replace(/ /ug, '')
     .replace(/\/+/ug, '/')
-    .replace(/(.)[./]$/gu, '$1') // eslint-disable-line prefer-named-capture-group
+    .replace(/(.)[./]$/gu, '$1')
     .replace(/^yso-(?:aika|paikat)\//u, 'yso/'); // IMP-HELMET crap. Also, they still have a '.' at the end of $a...
 
   // NB! Use the modified value ONLY if the result (tmp variable) is a legal subfield code...

package/src/sort-tags.js

@@ -27,8 +27,8 @@ function sortPatternFields(record, tagPattern) {
   const matchingTags = record.fields.map(field => tagPattern.some(pattern => pattern.test(field.tag)) ? field : null).filter(tag => tag);
   const sortedArray = sortFields(record.fields);
   const fixedArray = sortedArray.filter(field => !tagPattern.some(pattern => pattern.test(field.tag)));
-  fixedArray.splice(index(sortedArray, tagPattern), 0, ...matchingTags); // eslint-disable-line functional/immutable-data
-  record.fields = fixedArray; // eslint-disable-line functional/immutable-data
+  fixedArray.splice(index(sortedArray, tagPattern), 0, ...matchingTags);
+  record.fields = fixedArray;
 }
 
 function sortFields(fields) {

package/src/sortFields.js

@@ -22,7 +22,7 @@ export default function () {
   function fix(record) {
     const res = {message: [], fix: [], valid: true};
 
-    record.fields.sort(fieldOrderComparator); // eslint-disable-line functional/immutable-data
+    record.fields.sort(fieldOrderComparator);
 
     return res;
   }
@@ -31,16 +31,16 @@ export default function () {
     const res = {message: []};
 
     const fields = record.fields.map(f => clone(f));
-    fields.sort(fieldOrderComparator); // eslint-disable-line functional/immutable-data
+    fields.sort(fieldOrderComparator);
 
 
     const relocatedFields = fields.filter((f, i) => fieldToString(f) !== fieldToString(record.fields[i]));
 
-    if (relocatedFields.length > 0) { // eslint-disable-line functional/no-conditional-statements
-      res.message.push(`${relocatedFields.length} field(s) in new places`); // eslint-disable-line functional/immutable-data
+    if (relocatedFields.length > 0) {
+      res.message.push(`${relocatedFields.length} field(s) in new places`);
     }
 
-    res.valid = !(res.message.length >= 1); // eslint-disable-line functional/immutable-data
+    res.valid = !(res.message.length >= 1);
     return res;
   }
 }
@@ -217,7 +217,7 @@ export function fieldOrderComparator(fieldA, fieldB) {
   return globalFieldOrderComparator(fieldA, fieldB, sorterFunctions);
 }
 
-function sortByIndexTerms(fieldA, fieldB) { // eslint-disable-line complexity, max-statements
+function sortByIndexTerms(fieldA, fieldB) {
 
   const indexTermFields = ['600', '610', '611', '630', '648', '650', '651', '652', '653', '654', '655', '656', '657', '658', '659', '662'];
 

package/src/sortRelatorTerms.js

@@ -41,12 +41,12 @@ export default function () {
       sortAdjacentRelatorTerms(clonedField, typeOfMaterial);
       const clonedFieldAsString = fieldToString(clonedField);
       const fieldAsString = fieldToString(field);
-      if (fieldAsString !== clonedFieldAsString) { // eslint-disable-line functional/no-conditional-statements
-        res.message.push(`${fieldAsString} => ${clonedFieldAsString}`); // eslint-disable-line functional/immutable-data
+      if (fieldAsString !== clonedFieldAsString) {
+        res.message.push(`${fieldAsString} => ${clonedFieldAsString}`);
       }
     });
 
-    res.valid = !(res.message.length >= 1); // eslint-disable-line functional/immutable-data
+    res.valid = !(res.message.length >= 1);
     return res;
   }
 }
@@ -103,8 +103,8 @@ function swapRelatorTermSubfields(field, typeOfMaterial = undefined) {
 
     // Swap:
     const tmp = field.subfields[index - 1];
-    field.subfields[index - 1] = field.subfields[index]; // eslint-disable-line functional/immutable-data
-    field.subfields[index] = tmp; // eslint-disable-line functional/immutable-data
+    field.subfields[index - 1] = field.subfields[index];
+    field.subfields[index] = tmp;
     fieldFixPunctuation(field);
     return true;
   }

package/src/sortSubfields.js

@@ -54,12 +54,12 @@ export default function (defaultTagPattern) {
       sortAdjacentSubfields(clonedField);
       const clonedFieldAsString = fieldToString(clonedField);
       const fieldAsString = fieldToString(field);
-      if (fieldAsString !== clonedFieldAsString) { // eslint-disable-line functional/no-conditional-statements
-        res.message.push(clonedFieldAsString); // eslint-disable-line functional/immutable-data
+      if (fieldAsString !== clonedFieldAsString) {
+        res.message.push(clonedFieldAsString);
       }
     });
 
-    res.valid = !(res.message.length >= 1); // eslint-disable-line functional/immutable-data
+    res.valid = !(res.message.length >= 1);
     return res;
   }
 }
@@ -157,8 +157,8 @@ function swapSubfields(field, sortOrder) {
     }
     // Swap:
     const tmp = field.subfields[index - 1];
-    field.subfields[index - 1] = sf; // eslint-disable-line functional/immutable-data
-    field.subfields[index] = tmp; // eslint-disable-line functional/immutable-data
+    field.subfields[index - 1] = sf;
+    field.subfields[index] = tmp;
     return true;
   });
 
@@ -195,7 +195,7 @@ function moveSubfield6ToTheFront(field) {
   // https://www.loc.gov/marc/bibliographic/ecbdcntf.html says "Subfield $6 is always the first subfield in the field."
   const sf6s = field.subfields.filter(sf => sf.code === '6');
   const others = field.subfields.filter(sf => sf.code !== '6');
-  field.subfields = [...sf6s, ...others]; // eslint-disable-line functional/immutable-data
+  field.subfields = [...sf6s, ...others];
 }
 
 export function sortAdjacentSubfields(field, externalSortOrder = []) {

package/src/stripPunctuation.js

@@ -20,7 +20,7 @@ export default function () {
   function fix(record) {
     nvdebug('Strip punctuation to data fields: fixer');
     const res = {message: [], fix: [], valid: true};
-    record.fields.forEach(f => fieldStripPunctuation(f));
+    record.fields.forEach(f => fieldStripPunctuation(f)); // eslint-disable-line array-callback-return
     return res;
   }
 
@@ -36,7 +36,7 @@ export default function () {
 
     const res = {message: messages};
 
-    res.valid = res.message.length < 1; // eslint-disable-line functional/immutable-data
+    res.valid = res.message.length < 1;
     return res;
   }
 }

package/src/subfield-exclusion.js

@@ -77,10 +77,10 @@ export default function (config) {
     }
 
     // Check subfields recursively
-    if (key === 'subfields') { // eslint-disable-line functional/no-conditional-statements
+    if (key === 'subfields') {
       forEach(data, ([, subObj]) => {
         // Console.log("subObj: ", subObj, " type: ", typeof subObj, !(Array.isArray(subObj)))
-        if (typeof subObj === 'object' && !Array.isArray(subObj)) { // eslint-disable-line functional/no-conditional-statements
+        if (typeof subObj === 'object' && !Array.isArray(subObj)) {
           checkMandatory(spec[key], subObj);
 
           forEach(subObj, ([subKey, subVal]) => {
@@ -128,7 +128,7 @@ export default function (config) {
 
           // Configuration object not found from found element
           return false;
-        })) { // eslint-disable-line functional/no-conditional-statements
+        })) {
           // All configuration fields match, check if some subfields should be excluded.
           confObj.subfields.forEach(subField => {
             const excluded = [];
@@ -138,17 +138,17 @@ export default function (config) {
               const subFieldCodeFine = subField.code && elemSub.code && subField.code.test(elemSub.code);
               const subFieldValueFine = subField.value && elemSub.value && subField.value.test(elemSub.value);
               if (subFieldCodeFine && (typeof subField.value === 'undefined' || subFieldValueFine)) {
-                if (fix) { // eslint-disable-line functional/no-conditional-statements
-                  excluded.push(elemSub); // eslint-disable-line functional/immutable-data
-                } else { // eslint-disable-line functional/no-conditional-statements
-                  res.message.push(`Subfield $${element.tag}$$${elemSub.code}should be excluded`); // eslint-disable-line functional/immutable-data
+                if (fix) {
+                  excluded.push(elemSub);
+                } else {
+                  res.message.push(`Subfield $${element.tag}$$${elemSub.code}should be excluded`);
                 }
               }
             });
 
-            excluded.forEach(sf => record.removeSubfield(sf, element));
+            excluded.forEach(sf => record.removeSubfield(sf, element)); // eslint-disable-line array-callback-return
             // If no subfields remains, the whole field will be removed as well:
-            if (element.subfields && element.subfields.length === 0) { // eslint-disable-line functional/no-conditional-statements
+            if (element.subfields && element.subfields.length === 0) {
               record.removeField(element);
             }
           });
@@ -158,8 +158,8 @@ export default function (config) {
 
     // Fix does not send response
     if (!fix) {
-      if (res.message.length > 0) { // eslint-disable-line functional/no-conditional-statements
-        res.valid = false; // eslint-disable-line functional/immutable-data
+      if (res.message.length > 0) {
+        res.valid = false;
       }
 
       return res;

package/src/subfield6Utils.js

@@ -51,9 +51,9 @@ export function subfield6ResetOccurrenceNumber(subfield, occurrenceNumber) {
   }
   const occurrenceNumberAsString = typeof occurrenceNumber === 'number' ? intToOccurrenceNumberString(occurrenceNumber) : occurrenceNumber;
 
-  const newValue = subfield.value.substring(0, 4) + occurrenceNumberAsString + subfield6GetTail(subfield); // eslint-disable-line functional/immutable-data
+  const newValue = subfield.value.substring(0, 4) + occurrenceNumberAsString + subfield6GetTail(subfield);
   //nvdebug(`Set subfield $6 value from ${subfieldToString(subfield)} to ${newValue}`);
-  subfield.value = newValue; // eslint-disable-line functional/immutable-data
+  subfield.value = newValue;
 }
 
 
@@ -70,7 +70,7 @@ export function subfield6HasWantedTagAndOccurrenceNumber(subfield, tagAndOccurre
     return false;
   }
   // 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'); // eslint-disable-line prefer-named-capture-group
+  const key = subfield.value.replace(/^([0-9][0-9][0-9]-[0-9][0-9]+).*$/u, '$1');
   nvdebug(` Compare '${key}' vs '${tagAndOccurrenceNumber}'`);
   return key === tagAndOccurrenceNumber;
 }
@@ -99,7 +99,7 @@ export function fieldHasOccurrenceNumber(field, occurrenceNumber) {
 }
 
 export function fieldResetOccurrenceNumber(field, newOccurrenceNumber, oldOccurrenceNumber = undefined) {
-  field.subfields.forEach(subfield => innerReset(subfield));
+  field.subfields.forEach(subfield => innerReset(subfield)); // eslint-disable-line array-callback-return
 
   function innerReset(subfield) {
     // (Optional) Check that this is really the occurrence number we wan't to reseot
@@ -203,14 +203,14 @@ function subfieldSevenToOneOccurrenceNumber(subfield) {
   if (subfield.code !== '6' || subfield.value.substring(0, 1) !== '7') {
     return;
   }
-  subfield.value = `1${subfield.value.substring(1)}`; // eslint-disable-line functional/immutable-data
+  subfield.value = `1${subfield.value.substring(1)}`;
 }
 
 export function fieldSevenToOneOccurrenceNumber(field) {
   if (field.tag !== '880') {
     return;
   }
-  field.subfields.forEach(sf => subfieldSevenToOneOccurrenceNumber(sf));
+  field.subfields.forEach(sf => subfieldSevenToOneOccurrenceNumber(sf)); // eslint-disable-line array-callback-return
 }
 
 
@@ -223,7 +223,7 @@ export function fieldGetOccurrenceNumberPairs(field, candFields) {
     return pairs;
   }
   nvdebug(`${pairs.length} PAIR(S) FOUND FOR '${fieldToString(field)}'`);
-  pairs.forEach(pairedField => nvdebug(`  '${fieldToString(pairedField)}'`));
+  pairs.forEach(pairedField => nvdebug(`  '${fieldToString(pairedField)}'`)); // eslint-disable-line array-callback-return
   return pairs;
 }
 
@@ -355,7 +355,7 @@ function guessTargetLinkingNumber(fields, defaultTargetLinkingNumber) {
     return defaultTargetLinkingNumber;
   }
   const linkingNumbers = fieldsGetAllSubfield8LinkingNumbers(fields);
-  return linkingNumbers.length === 1 ? linkingNumbers[0] : 0; // eslint-disable-line no-param-reassign
+  return linkingNumbers.length === 1 ? linkingNumbers[0] : 0;
 }
 
 export function fieldsToNormalizedString(fields, defaultTargetLinkingNumber = 0, normalizeOccurrenceNumber = false, normalizeEntryTag = false) {
@@ -363,7 +363,7 @@ export function fieldsToNormalizedString(fields, defaultTargetLinkingNumber = 0,
 
   nvdebug(`fieldsToNormalizedString: OCC: ${normalizeOccurrenceNumber}`);
   const strings = fields.map(field => fieldToNormalizedString(field, targetLinkingNumber, normalizeOccurrenceNumber, normalizeEntryTag));
-  strings.sort(); // eslint-disable-line functional/immutable-data
+  strings.sort();
   return strings.join('\t__SEPARATOR__\t');
 }
 
@@ -482,7 +482,7 @@ export function get6s(field, candidateFields) { // NB! Convert field to fields!!
 
   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)));
+  relevantFields.forEach(f => nvdebug(fieldToString(f))); // eslint-disable-line array-callback-return
   return relevantFields;
 }
 
@@ -493,5 +493,5 @@ 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);
-  subfield.value = newValue; // eslint-disable-line functional/immutable-data
+  subfield.value = newValue;
 }

package/src/subfield8Utils.js

@@ -3,7 +3,7 @@
 
 import {fieldToString, nvdebug} from './utils';
 
-const sf8Regexp = /^([1-9][0-9]*)(?:\.[0-9]+)?(?:\\[acprux])?$/u; // eslint-disable-line prefer-named-capture-group
+const sf8Regexp = /^([1-9][0-9]*)(?:\.[0-9]+)?(?:\\[acprux])?$/u;
 
 export function isValidSubfield8(subfield) {
   if (subfield.code !== '8') {
@@ -48,7 +48,6 @@ export function recordGetFieldsWithSubfield8LinkingNumber(record, linkingNumber)
 
 
 export function fieldsGetAllSubfield8LinkingNumbers(fields) {
-  /* eslint-disable */
   let subfield8LinkingNumbers = [];
   fields.forEach(field => {
     if (!field.subfields) {
@@ -64,8 +63,6 @@ export function fieldsGetAllSubfield8LinkingNumbers(fields) {
   });
 
   return subfield8LinkingNumbers;
-  /* eslint-enable */
-
 }
 
 export function recordGetAllSubfield8LinkingNumbers(record) {
@@ -80,19 +77,19 @@ export function add8s(fields, record) {
   }
 
   nvdebug(`Linking number(s): ${linkingNumbers.join(', ')}`);
-  linkingNumbers.forEach(number => collectLinkingNumberFields(number));
+  linkingNumbers.forEach(number => collectLinkingNumberFields(number)); // eslint-disable-line array-callback-return
 
-  fields.forEach(f => nvdebug(`AFTER ADDING 8s: '${fieldToString(f)}'`));
+  fields.forEach(f => nvdebug(`AFTER ADDING 8s: '${fieldToString(f)}'`)); // eslint-disable-line array-callback-return
 
   return fields;
 
   function collectLinkingNumberFields(linkingNumber) {
     // Remove existing hits (to avoid field repetition):
-    fields = fields.filter(f => !fieldHasLinkingNumber(f, linkingNumber)); // eslint-disable-line functional/immutable-data, no-param-reassign
+    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)}`));
-    fields = fields.concat(addableFields); // eslint-disable-line functional/immutable-data, no-param-reassign
+    addableFields.forEach(f => nvdebug(`(RE-?)ADD ${fieldToString(f)}`));  // eslint-disable-line array-callback-return
+    fields = fields.concat(addableFields);
 
   }
 }