@natlibfi/marc-record-validators-melinda 11.6.6 → 11.6.7

package/src/cyrillux.js

@@ -24,10 +24,10 @@ export default function (config = {}) {
   };
 
   function preprocessConfig() {
-    config.retainCyrillic = typeof config.retainCyrillic === 'undefined' ? true : config.retainCyrillic; // eslint-disable-line functional/immutable-data
-    config.doISO9Transliteration = typeof config.doISO9Transliteration === 'undefined' ? true : config.doISO9Transliteration; // eslint-disable-line functional/immutable-data
-    config.doSFS4900Transliteration = typeof config.doSFS4900Transliteration === 'undefined' ? true : config.doSFS4900Transliteration; // eslint-disable-line functional/immutable-data
-    config.preferSFS4900 = setPreference(); // eslint-disable-line functional/immutable-data
+    config.retainCyrillic = typeof config.retainCyrillic === 'undefined' ? true : config.retainCyrillic;
+    config.doISO9Transliteration = typeof config.doISO9Transliteration === 'undefined' ? true : config.doISO9Transliteration;
+    config.doSFS4900Transliteration = typeof config.doSFS4900Transliteration === 'undefined' ? true : config.doSFS4900Transliteration;
+    config.preferSFS4900 = setPreference();
 
     function setPreference() {
       if (!config.doSFS4900Transliteration) {
@@ -52,9 +52,9 @@ export default function (config = {}) {
 
     const nBefore = record.fields.length;
 
-    record.fields = processFields(record.fields); // eslint-disable-line functional/immutable-data
+    record.fields = processFields(record.fields);
 
-    if (nBefore < record.fields.length) { // eslint-disable-line functional/no-conditional-statements
+    if (nBefore < record.fields.length) {
       reindexSubfield6OccurenceNumbers().fix(record);
       sortFields().fix(record);
     }
@@ -84,7 +84,7 @@ export default function (config = {}) {
       validateField(field, res, record);
     });
 
-    res.valid = !(res.message.length >= 1); // eslint-disable-line functional/immutable-data
+    res.valid = !(res.message.length >= 1);
     return res;
   }
 
@@ -92,16 +92,16 @@ export default function (config = {}) {
     const orig = fieldToString(field);
 
     const normalizedFields = processField(clone(field), record);
-    const mod = fieldsToString(normalizedFields).replace(/\t__SEPARATOR__\t/ug, ', ').replace(/ (‡6 [0-9][0-9][0-9])-[0-9][0-9]+/gu, ' $1-NN'); // eslint-disable-line prefer-named-capture-group
+    const mod = fieldsToString(normalizedFields).replace(/\t__SEPARATOR__\t/ug, ', ').replace(/ (‡6 [0-9][0-9][0-9])-[0-9][0-9]+/gu, ' $1-NN');
     if (orig !== mod) { // Fail as the input is "broken"/"crap"/sumthing
-      res.message.push(`CHANGE: ${orig} => ${mod}`); // eslint-disable-line functional/immutable-data
+      res.message.push(`CHANGE: ${orig} => ${mod}`);
       return;
     }
     return;
   }
 
   function isCyrillicCharacter(char) {
-    return XRegExp('[\\p{Cyrillic}]').test(char); // eslint-disable-line new-cap
+    return XRegExp('[\\p{Cyrillic}]').test(char);
   }
 
   function containsCyrillicCharacters(str) { // from melinda-ui-cyrillux
@@ -192,7 +192,7 @@ export default function (config = {}) {
     // Translitetation goes to field 880:
 
     //const subfield6 = newField.subfields.find(sf => sf.code === '6');
-    newField.tag = '880'; // eslint-disable-line functional/immutable-data
+    newField.tag = '880';
     resetSubfield6Tag(subfield6, field.tag);
     return newField;
 
@@ -326,8 +326,8 @@ export default function (config = {}) {
 
   function createFieldForSfs4900Comparison(field, tag) {
     const clonedField = clone(field);
-    clonedField.tag = tag; // eslint-disable-line functional/immutable-data
-    clonedField.subfields = clonedField.subfields.filter(sf => sf.code !== '9' || sf.value !== sfs4900Trans); // eslint-disable-line functional/immutable-data
+    clonedField.tag = tag;
+    clonedField.subfields = clonedField.subfields.filter(sf => sf.code !== '9' || sf.value !== sfs4900Trans);
     return fieldStripPunctuation(clonedField);
   }
 
@@ -348,7 +348,7 @@ export default function (config = {}) {
     const newSFS4900Field = config.doSFS4900Transliteration ? mapFieldToSfs4900(field, occurrenceNumberAsString, languageCode) : undefined; // SFS-4900
 
     // Trigger the drop of original counterpart $6 :
-    pairedField.cyrilluxSkip = 1; // eslint-disable-line functional/immutable-data
+    pairedField.cyrilluxSkip = 1;
 
     return [newMainField, newCyrillicField, newSFS4900Field].filter(f => f);
   }

package/src/disambiguateSeriesStatements.js

@@ -83,14 +83,14 @@ export default function () {
 
     // fixer:
     if (reallyFix) {
-      currField.subfields = currField.subfields.filter(sf => !deletableStrings.includes(subfieldToString(sf))); // eslint-disable-line functional/immutable-data
+      currField.subfields = currField.subfields.filter(sf => !deletableStrings.includes(subfieldToString(sf)));
       fieldFixPunctuation(currField);
       return fix490x(recordType, remainingFields, reallyFix, message);
     }
     // validators:
     const clonedField = clone(currField);
     const originalString = fieldToString(clonedField);
-    clonedField.subfields = clonedField.subfields.filter(sf => !deletableStrings.includes(subfieldToString(sf))); // eslint-disable-line functional/immutable-data
+    clonedField.subfields = clonedField.subfields.filter(sf => !deletableStrings.includes(subfieldToString(sf)));
 
     const newMessage = `Replace '${originalString}' with '${fieldToString(clonedField)}'`;
 
@@ -198,7 +198,7 @@ export function search(sruClient, query, one = false) {
 
     sruClient.searchRetrieve(query)
       .on('record', xmlString => {
-        promises.push(MARCXML.from(xmlString, noValidation)); // eslint-disable-line functional/immutable-data
+        promises.push(MARCXML.from(xmlString, noValidation));
       })
       .on('end', async () => {
         try {

package/src/double-commas.js

@@ -6,8 +6,8 @@ export default function () {
         .get(/^700$/u)
         .some(f => f.subfields.every(sf => sf.code === 'e' && (/,,/u).test(sf.value)))
     }),
-    fix: record => record.get(/^700$/u).forEach(f => f.subfields.filter(sf => sf.code === 'e').forEach(sf => {
-      sf.value = sf.value.replace(/,,/u, ','); // eslint-disable-line functional/immutable-data
+    fix: record => record.get(/^700$/u).forEach(f => f.subfields.filter(sf => sf.code === 'e').forEach(sf => { // eslint-disable-line array-callback-return
+      sf.value = sf.value.replace(/,,/u, ',');
     }))
   };
 }

package/src/duplicates-ind1.js

@@ -20,7 +20,7 @@ export default function (tagPattern) {
     iterateFields(record)
       .filter(item => item.validation === false)
       .map(({validation, ...item}) => item) // eslint-disable-line no-unused-vars
-      .forEach(({obj}) => record.removeField(obj));
+      .forEach(({obj}) => record.removeField(obj)); // eslint-disable-line array-callback-return
   }
 
   function iterateFields(record) {

package/src/empty-fields.js

@@ -33,15 +33,15 @@ export default function () {
   function fix(record) {
     const errors = findErrors(record.fields);
     errors.forEach(error => {
-      if (error.code === ERRORS.MISSING_SUBFIELD_VALUES) { // eslint-disable-line functional/no-conditional-statements
-        if (error.emptySubfields.length === error.field.subfields.length) { // eslint-disable-line functional/no-conditional-statements
+      if (error.code === ERRORS.MISSING_SUBFIELD_VALUES) {
+        if (error.emptySubfields.length === error.field.subfields.length) {
           record.removeField(error.field);
-        } else { // eslint-disable-line functional/no-conditional-statements
+        } else {
           error.emptySubfields.forEach(sf => {
             record.removeSubfield(sf, error.field);
           });
         }
-      } else { // eslint-disable-line functional/no-conditional-statements
+      } else {
         record.removeField(error.field);
       }
     });

package/src/ending-punctuation.js

@@ -26,9 +26,6 @@
 *
 */
 
-/* eslint-disable complexity, max-params */
-
-
 // Import {validPuncMarks, finnishTerms, confSpec} from './ending-punctuation-conf.js';
 import {validPuncMarks, finnishTerms, confSpec} from './ending-punctuation-conf';
 import createDebugLogger from 'debug';
@@ -48,8 +45,8 @@ export default function () {
   function validatePunc(record, fix) {
     const message = {message: []};
 
-    if (fix) { // eslint-disable-line functional/no-conditional-statements
-      message.fix = []; // eslint-disable-line functional/immutable-data
+    if (fix) {
+      message.fix = [];
     }
 
     // Actual parsing of all fields
@@ -61,7 +58,7 @@ export default function () {
       validateField(field, false, fix, message);
     });
 
-    message.valid = !(message.message.length >= 1); // eslint-disable-line functional/immutable-data
+    message.valid = !(message.message.length >= 1);
     return message;
   }
 }
@@ -83,52 +80,52 @@ function validateField(field, linkedTag, fix, message) {
     // Last char should be punc, but its not one of marks nor dot
     if (punc && !(lastPuncMark || lastPuncDot)) {
       // Console.log("1. Invalid punctuation - missing")
-      message.message.push(`Field ${tag} has invalid ending punctuation`); // eslint-disable-line functional/immutable-data
-      if (fix) { // eslint-disable-line functional/no-conditional-statements
-        subfield.value = subfield.value.concat('.'); // eslint-disable-line functional/immutable-data
-        message.fix.push(`Field ${tag} - Added punctuation to $${subfield.code}`); // eslint-disable-line functional/immutable-data
+      message.message.push(`Field ${tag} has invalid ending punctuation`);
+      if (fix) {
+        subfield.value = subfield.value.concat('.');
+        message.fix.push(`Field ${tag} - Added punctuation to $${subfield.code}`);
       }
 
       // Last char is dot, but previous char is one of punc marks, like 'Question?.'
     } else if (lastPuncDot && subfield.value.length > 1 && puncMarks.includes(subfield.value.charAt(subfield.value.length - 2))) {
       // Console.log("2. Invalid punctuation - duplicate, like '?.'")
-      message.message.push(`Field ${tag} has invalid ending punctuation`); // eslint-disable-line functional/immutable-data
-      if (fix) { // eslint-disable-line functional/no-conditional-statements
-        subfield.value = subfield.value.slice(0, -1); // eslint-disable-line functional/immutable-data
-        message.fix.push(`Field ${tag} - Removed double punctuation from $${subfield.code}`); // eslint-disable-line functional/immutable-data
+      message.message.push(`Field ${tag} has invalid ending punctuation`);
+      if (fix) {
+        subfield.value = subfield.value.slice(0, -1);
+        message.fix.push(`Field ${tag} - Removed double punctuation from $${subfield.code}`);
       }
 
       // Last char shouldn't be dot !! This is behind checkEnd boolean, because of dots at end of abbreviations, so this is checked only in special cases !!//
     } else if (checkEnd && (!punc && lastPuncDot)) {
       // Console.log("3. Invalid punctuation - Shouldn't be dot, is")
-      message.message.push(`Field ${tag} has invalid ending punctuation`); // eslint-disable-line functional/immutable-data
-      if (fix) { // eslint-disable-line functional/no-conditional-statements
-        subfield.value = subfield.value.slice(0, -1); // eslint-disable-line functional/immutable-data
-        message.fix.push(`Field ${tag} - Removed punctuation from $${subfield.code}`); // eslint-disable-line functional/immutable-data
+      message.message.push(`Field ${tag} has invalid ending punctuation`);
+      if (fix) {
+        subfield.value = subfield.value.slice(0, -1);
+        message.fix.push(`Field ${tag} - Removed punctuation from $${subfield.code}`);
       }
     }
   }
 
   // Special cases from here on
   function specialCases(res, field, tag) {
-    let lastSubField = null; // eslint-disable-line functional/no-let
+    let lastSubField = null;
     // Punctuation should be only after specific field
     if (res.special.afterOnly) {
       lastSubField = findLastSubfield(field);
 
       if (lastSubField) {
-        if (typeof res.special.strict === 'undefined') { // eslint-disable-line functional/no-conditional-statements
-          res.special.strict = true; // eslint-disable-line functional/immutable-data
+        if (typeof res.special.strict === 'undefined') {
+          res.special.strict = true;
         }
 
-        if (lastSubField.code === res.special.afterOnly) { // eslint-disable-line functional/no-conditional-statements
+        if (lastSubField.code === res.special.afterOnly) {
           normalPuncRules(lastSubField, res.punc, tag, res.special.strict, false);
-        } else { // eslint-disable-line functional/no-conditional-statements
+        } else {
           normalPuncRules(lastSubField, !res.punc, tag, true, false);
         }
       }
     } else if (res.special.noPuncIfField) {
-      if (field.subfields.some(subField => subField.code === res.special.noPuncIfField) === false) { // eslint-disable-line functional/no-conditional-statements
+      if (field.subfields.some(subField => subField.code === res.special.noPuncIfField) === false) {
         lastSubField = findLastSubfield(field);
         normalPuncRules(lastSubField, res.punc, tag, true, false);
       }
@@ -136,25 +133,25 @@ function validateField(field, linkedTag, fix, message) {
       lastSubField = findLastSubfield(field);
       if (lastSubField && lastSubField.code === res.special.puncSubField) {
         // Ind2 match, check second if at punc rules with special punc char override
-        if (res.special.ifInd2 && res.special.ifInd2.includes(field.ind2)) { // eslint-disable-line functional/no-conditional-statements
+        if (res.special.ifInd2 && res.special.ifInd2.includes(field.ind2)) {
           normalPuncRules(lastSubField, res.special.ifBoth, tag, true, res.special.ifLastCharNot);
 
           // Matches execption to special rule, noPuncIfInd2 (likely with value 4, that indicates copyright mark)
-        } else if (res.special.noPuncIfInd2 && field.ind2 && res.special.noPuncIfInd2.includes(field.ind2)) { // eslint-disable-line functional/no-conditional-statements
+        } else if (res.special.noPuncIfInd2 && field.ind2 && res.special.noPuncIfInd2.includes(field.ind2)) {
           normalPuncRules(lastSubField, !res.special.ifBoth, tag, true, res.special.ifLastCharNot);
 
           // Does not match rules -> shouldn't have punc
-        } else { // eslint-disable-line functional/no-conditional-statements
+        } else {
           normalPuncRules(lastSubField, !res.special.ifBoth, tag, true, res.special.ifLastCharNot);
         }
       }
-    } else if (res.special.secondLastIfLast) { // eslint-disable-line functional/no-conditional-statements
+    } else if (res.special.secondLastIfLast) {
       field.subfields.forEach(subField => {
-        if (isNaN(subField.code) && res.special.secondLastIfLast !== subField.code) { // eslint-disable-line functional/no-conditional-statements
+        if (isNaN(subField.code) && res.special.secondLastIfLast !== subField.code) {
           lastSubField = subField;
         } // Not control field
 
-        if (typeof res.special.last !== 'undefined' && res.special.secondLastIfLast === subField.code) { // eslint-disable-line functional/no-conditional-statements
+        if (typeof res.special.last !== 'undefined' && res.special.secondLastIfLast === subField.code) {
           normalPuncRules(subField, res.special.last, tag, true, false);
         } // Strict because this field should not be abbreviation
       });
@@ -166,10 +163,10 @@ function validateField(field, linkedTag, fix, message) {
 
       if (lastSubField) {
         const languageField = field.subfields.find(({code}) => code === res.special.termField);
-        if (languageField && languageField.value && finnishTerms.some(p => p.test(languageField.value))) { // eslint-disable-line functional/no-conditional-statements
+        if (languageField && languageField.value && finnishTerms.some(p => p.test(languageField.value))) {
           // If (languageField && languageField.value && finnishTerms.indexOf(languageField.value) > -1) {
           normalPuncRules(lastSubField, res.punc, tag, true, false);
-        } else { // eslint-disable-line functional/no-conditional-statements
+        } else {
           normalPuncRules(lastSubField, res.special.else, tag, false, false); // Strict because of years etc (648, 650)
         }
       }
@@ -177,22 +174,22 @@ function validateField(field, linkedTag, fix, message) {
     } else if (res.special.lastOf) {
       lastSubField = null;
       field.subfields.filter(subField => 'value' in subField).forEach(subField => {
-        if (isNaN(subField.code) && res.special.lastOf.indexOf(subField.code) > -1) { // eslint-disable-line functional/no-conditional-statements
+        if (isNaN(subField.code) && res.special.lastOf.indexOf(subField.code) > -1) {
           lastSubField = subField;
         } // Not control field
 
-        if (res.special.mandatory && res.special.mandatory.indexOf(subField.code) > -1) { // eslint-disable-line functional/no-conditional-statements
+        if (res.special.mandatory && res.special.mandatory.indexOf(subField.code) > -1) {
           normalPuncRules(subField, res.punc, tag, true, false);
         } // Strict because of mandatory
       });
 
-      if (lastSubField) { // eslint-disable-line functional/no-conditional-statements
+      if (lastSubField) {
         normalPuncRules(lastSubField, res.punc, tag, false, false);
       }
 
       // If field has linked rules (tag: 880), find rules and re-validate
-    } else if (res.special.linked) { // eslint-disable-line functional/no-conditional-statements
-      let linkedTag = null; // eslint-disable-line functional/no-let
+    } else if (res.special.linked) {
+      let linkedTag = null;
       try {
         linkedTag = parseInt(field.subfields.find(({code}) => code === res.special.linked).value.substr(0, 3), 10); // Substring feels stupid, but is used in MarcRecord to extract tag.
       } catch (err) {
@@ -204,13 +201,13 @@ function validateField(field, linkedTag, fix, message) {
     }
   }
 
-  let res = null; // eslint-disable-line functional/no-let
-  let tag = null; // eslint-disable-line functional/no-let
-  let lastSubField = null; // eslint-disable-line functional/no-let
+  let res = null;
+  let tag = null;
+  let lastSubField = null;
 
-  if (linkedTag) { // eslint-disable-line functional/no-conditional-statements
+  if (linkedTag) {
     tag = linkedTag;
-  } else { // eslint-disable-line functional/no-conditional-statements
+  } else {
     try {
       tag = parseInt(field.tag, 10);
     } catch (err) {
@@ -231,7 +228,7 @@ function validateField(field, linkedTag, fix, message) {
 
   // Field does not have subfields; invalid
   if (typeof field.subfields === 'undefined' || field.subfields === null) {
-    message.message.push(`Field ${field.tag} has invalid ending punctuation`); // eslint-disable-line functional/immutable-data
+    message.message.push(`Field ${field.tag} has invalid ending punctuation`);
     return;
   }
 
@@ -239,10 +236,10 @@ function validateField(field, linkedTag, fix, message) {
   if (typeof res.special === 'undefined' || res.special === null) {
     lastSubField = findLastSubfield(field);
 
-    if (lastSubField) { // eslint-disable-line functional/no-conditional-statements
+    if (lastSubField) {
       normalPuncRules(lastSubField, res.punc, field.tag, false, false, fix, message);
     }
-  } else { // eslint-disable-line functional/no-conditional-statements
+  } else {
     try {
       specialCases(res, field, field.tag);
     } catch (e) {
@@ -253,13 +250,13 @@ function validateField(field, linkedTag, fix, message) {
 
 export function validateSingleField(field, linkedTag, fix) {
   const message = {};
-  message.message = []; // eslint-disable-line functional/immutable-data
-  if (fix) { // eslint-disable-line functional/no-conditional-statements
-    message.fix = []; // eslint-disable-line functional/immutable-data
+  message.message = [];
+  if (fix) {
+    message.fix = [];
   }
 
   validateField(field, linkedTag, fix, message);
-  message.valid = !(message.message.length >= 1); // eslint-disable-line functional/immutable-data
+  message.valid = !(message.message.length >= 1);
   return message;
 }
 

package/src/ending-whitespace.js

@@ -18,7 +18,6 @@ export default function () {
     return valid ? {valid, messages: []} : {valid, messages};
   }
 
-  /* eslint-disable functional/immutable-data,functional/no-conditional-statements */
   function fix(record) {
     record.fields.forEach(({subfields}) => {
       if (subfields === undefined) {
@@ -32,7 +31,6 @@ export default function () {
       });
     });
   }
-  /* eslint-enable functional/immutable-data,functional/no-conditional-statements */
 
   function valueEndsWithWhitespace({value}) {
     return (/\s$/u).test(value);

package/src/field-008-18-34-character-groups.js

@@ -30,7 +30,7 @@ export default function () {
       validateField(field, res, typeOfMaterial);
     });
 
-    res.valid = !(res.message.length >= 1); // eslint-disable-line functional/immutable-data
+    res.valid = !(res.message.length >= 1);
     return res;
   }
 
@@ -43,7 +43,7 @@ export default function () {
     const normalizedField = justifyAndSortField008CharacterGroups(clone(field), typeOfMaterial);
     const mod = fieldToString(normalizedField);
     if (orig !== mod) { // Fail as the input is "broken"/"crap"/sumthing
-      res.message.push(`TODO: '${orig}' => '${mod}'`); // eslint-disable-line functional/immutable-data
+      res.message.push(`TODO: '${orig}' => '${mod}'`);
       return;
     }
     return;
@@ -69,7 +69,7 @@ function processCharacterGroup(field, group) {
   //console.info(`008/${group.start}-${group.end}: '${originalContent}'`); // eslint-disable-line no-console
   const charArray = content.split('');
 
-  charArray.sort(function(a, b) { // eslint-disable-line functional/immutable-data, prefer-arrow-callback
+  charArray.sort(function(a, b) {
     return scoreChar(a) - scoreChar(b);
   });
 
@@ -80,7 +80,7 @@ function processCharacterGroup(field, group) {
 
   //console.info(`'${fieldToString(field)}' =>`); // eslint-disable-line no-console
 
-  field.value = `${field.value.substring(0, group.start)}${newContent}${field.value.substring(group.end + 1)}`; // eslint-disable-line functional/immutable-data
+  field.value = `${field.value.substring(0, group.start)}${newContent}${field.value.substring(group.end + 1)}`;
   //console.info(`'${fieldToString(field)}'`); // eslint-disable-line no-console
 
   function fixBlanks(str) {
@@ -120,7 +120,7 @@ export function justifyAndSortField008CharacterGroups(field, typeOfMaterial) {
 
   const relevantCharacterGroups = characterGroups.filter(gr => gr.type === typeOfMaterial);
 
-  relevantCharacterGroups.forEach(group => processCharacterGroup(field, group));
+  relevantCharacterGroups.forEach(group => processCharacterGroup(field, group)); // eslint-disable-line array-callback-return
 
   //justifyField008CharacterGroups(field, typeOfMaterial); // Oops: also sorts...
 

package/src/field-505-separators.js

@@ -26,7 +26,7 @@ export default function () {
       validateField(field, res);
     });
 
-    res.valid = !(res.message.length >= 1); // eslint-disable-line functional/immutable-data
+    res.valid = !(res.message.length >= 1);
     return res;
   }
 
@@ -36,7 +36,7 @@ export default function () {
     const normalizedField = fix505(clone(field));
     const mod = fieldToString(normalizedField);
     if (orig !== mod) { // Fail as the input is "broken"/"crap"/sumthing
-      res.message.push(`'TODO: ${orig}' => '${mod}'`); // eslint-disable-line functional/immutable-data
+      res.message.push(`'TODO: ${orig}' => '${mod}'`);
       return;
     }
     return;
@@ -47,10 +47,10 @@ export default function () {
 function field505FixSubfieldA(field) {
   const a = field.subfields.filter(sf => sf.code === 'a');
 
-  a.forEach(sf => fixSubfieldA(sf));
+  a.forEach(sf => fixSubfieldA(sf)); // eslint-disable-line array-callback-return
 
   function fixSubfieldA(a) {
-    a.value = a.value.replace(/ ; /ug, ' -- '); // eslint-disable-line functional/immutable-data
+    a.value = a.value.replace(/ ; /ug, ' -- ');
   }
 }
 
@@ -58,13 +58,13 @@ function field505FixSubfieldTRG(field) {
   // Modify subfield if next subfield is $t:
   const subfieldsThatWillBeModified = field.subfields.filter((sf, i) => i + 1 < field.subfields.length && field.subfields[i + 1].code === 't');
 
-  subfieldsThatWillBeModified.forEach(sf => fixSubfieldThatPrecedesT(sf));
+  subfieldsThatWillBeModified.forEach(sf => fixSubfieldThatPrecedesT(sf)); // eslint-disable-line array-callback-return
 
   function fixSubfieldThatPrecedesT(sf) {
     if (!sf.value) {
       return;
     }
-    sf.value = sf.value.replace(/ ;$/u, ' --'); // eslint-disable-line functional/immutable-data
+    sf.value = sf.value.replace(/ ;$/u, ' --');
   }
 }
 

package/src/field-521-fix.js

@@ -33,7 +33,7 @@ export default function () {
       validateField(field, res);
     });
 
-    res.valid = !(res.message.length >= 1); // eslint-disable-line functional/immutable-data
+    res.valid = !(res.message.length >= 1);
     return res;
   }
 
@@ -43,7 +43,7 @@ export default function () {
     const normalizedField = fix521(clone(field));
     const mod = fieldToString(normalizedField);
     if (orig !== mod) { // Fail as the input is "broken"/"crap"/sumthing
-      res.message.push(`'TODO: ${orig}' => '${mod}'`); // eslint-disable-line functional/immutable-data
+      res.message.push(`'TODO: ${orig}' => '${mod}'`);
       return;
     }
     return;
@@ -51,15 +51,15 @@ export default function () {
 }
 
 function fixSubfieldA(a) {
-  a.value = a.value. // eslint-disable-line functional/immutable-data
+  a.value = a.value.
     // MET-332:
-    replace(/^(Ikäsuositus) ([0-9])/u, '$1: $2'); // eslint-disable-line prefer-named-capture-group, functional/immutable-data
+    replace(/^(Ikäsuositus) ([0-9])/u, '$1: $2');
 }
 
 function fixSubfieldAInternalPunctuation(field) {
   const a = field.subfields.filter(sf => sf.code === 'a');
 
-  a.forEach(sf => fixSubfieldA(sf));
+  a.forEach(sf => fixSubfieldA(sf)); // eslint-disable-line array-callback-return
 }
 
 function getIndicator1(field) {
@@ -75,7 +75,7 @@ function getIndicator1(field) {
 function fixIndicator1(field) {
   const value = getIndicator1(field);
   if (value) {
-    field.ind1 = value; // eslint-disable-line functional/immutable-data
+    field.ind1 = value;
     return;
   }
 

package/src/field-exclusion.js

@@ -69,8 +69,8 @@ export default function (config) {
       forEach(obj, ([key, val]) => {
         configMatchesSpec(val, key, confSpec);
         // Concat all excluded elements to array
-        if (confSpec[key].excl) { // eslint-disable-line functional/no-conditional-statements
-          excluded.push(...confSpec[key].excl); // eslint-disable-line functional/immutable-data
+        if (confSpec[key].excl) {
+          excluded.push(...confSpec[key].excl);
         }
       });
 
@@ -97,10 +97,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]) => {
@@ -112,7 +112,7 @@ export default function (config) {
       });
     }
 
-    if (key === 'dependencies') { // eslint-disable-line functional/no-conditional-statements
+    if (key === 'dependencies') {
       forEach(data, ([, subObj]) => {
         if (!(typeof subObj === 'object' && !Array.isArray(subObj) && Object.keys(subObj).length === 1 && isRegExp(subObj.leader))) {
           throw new TypeError('Configuration not valid - Invalid dependencies config');
@@ -177,21 +177,21 @@ export default function (config) {
           return false;
         })) {
           // All configuration fields match, element should be excluded.
-          if (fix) { // eslint-disable-line functional/no-conditional-statements
-            excluded.push(element); // eslint-disable-line functional/immutable-data
-          } else { // eslint-disable-line functional/no-conditional-statements
-            res.message.push(`Field $${element.tag} should be excluded`); // eslint-disable-line functional/immutable-data
+          if (fix) {
+            excluded.push(element);
+          } else {
+            res.message.push(`Field $${element.tag} should be excluded`);
           }
         }
       });
 
-      excluded.forEach(field => record.removeField(field));
+      excluded.forEach(field => record.removeField(field)); // eslint-disable-line array-callback-return
     });
 
     // 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/field-exclusion.spec.js

@@ -318,7 +318,7 @@ describe('field-exclusion', () => {
   describe('#validate: Simple multi tag configuration (spec)', () => {
     const config = [
       {
-        tag: /^(648|650|651|655)$/u // eslint-disable-line prefer-named-capture-group
+        tag: /^(648|650|651|655)$/u
       }
     ];
 
@@ -427,7 +427,7 @@ describe('field-exclusion', () => {
 
   // Simple multi tag configuration
   describe('#validate: Simple multi tag configuration - No object (spec)', () => {
-    const config = [/^(648|650|651|655)$/u]; // eslint-disable-line prefer-named-capture-group
+    const config = [/^(648|650|651|655)$/u];
 
     const recordValid = new MarcRecord({
       leader: 'foo',
@@ -617,8 +617,8 @@ describe('field-exclusion', () => {
   describe('#validate: Complex multi tag configuration (spec)', () => {
     const config = [
       {
-        tag: /^(648|650|651|655)$/u, // eslint-disable-line prefer-named-capture-group
-        subfields: [{code: /^2$/u, value: /^(ysa|musa|allars|cilla)$/u}] // eslint-disable-line prefer-named-capture-group
+        tag: /^(648|650|651|655)$/u,
+        subfields: [{code: /^2$/u, value: /^(ysa|musa|allars|cilla)$/u}]
       }
     ];