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

package/src/merge-fields/mergeField.js

@@ -27,8 +27,8 @@ const debugDev = debug.extend('dev');
 function removeEnnakkotieto(field) {
   const tmp = field.subfields.filter(subfield => !isEnnakkotietoSubfieldG(subfield));
   // remove only iff some other subfield remains
-  if (tmp.length > 0) { // eslint-disable-line functional/no-conditional-statements
-    field.subfields = tmp; // eslint-disable-line functional/immutable-data
+  if (tmp.length > 0) {
+    field.subfields = tmp;
   }
 }
 
@@ -44,11 +44,10 @@ function copyrightYearHack(baseRecord, baseField, sourceField) {
     const value = sf.value.replace(/\.$/u, '');
     baseRecord.insertField({'tag': '264', 'ind1': ' ', 'ind2': '4', 'subfields': [{'code': 'c', value}]});
     // Modify original subfield:
-    sf.value = removeCopyright(sf.value); // eslint-disable-line functional/immutable-data
+    sf.value = removeCopyright(sf.value);
   });
 }
 
-// eslint-disable-next-line max-params
 function mergeField2(baseRecord, baseField, sourceField, config, candFieldPairs880 = []) {
   //// Identical fields
   // No need to check every subfield separately.
@@ -60,9 +59,9 @@ function mergeField2(baseRecord, baseField, sourceField, config, candFieldPairs8
   // If a base ennakkotieto is merged with real data, remove ennakkotieto subfield:
   // (If our prepub normalizations are ok, this should not be needed.
   //  However, it's simple and works well enough, so let's keep it here.)
-  if (baseField.subfields?.find(sf => isEnnakkotietoSubfieldG(sf)) && !sourceField.subfields?.find(sf => isEnnakkotietoSubfieldG(sf))) { // eslint-disable-line functional/no-conditional-statements
+  if (baseField.subfields?.find(sf => isEnnakkotietoSubfieldG(sf)) && !sourceField.subfields?.find(sf => isEnnakkotietoSubfieldG(sf))) {
     removeEnnakkotieto(baseField);
-    baseField.merged = 1; // eslint-disable-line functional/immutable-data
+    baseField.merged = 1;
   }
 
   copyrightYearHack(baseRecord, baseField, sourceField);
@@ -91,7 +90,7 @@ function mergeField2(baseRecord, baseField, sourceField, config, candFieldPairs8
 
     mergeOrAddSubfield(baseField, subfieldData, candFieldPairs880); // candSubfield);
     const newValue = fieldToString(baseField);
-    if (originalBaseValue !== newValue) { // eslint-disable-line functional/no-conditional-statements
+    if (originalBaseValue !== newValue) {
       nvdebug(`   SUBFIELD MERGE RESULT: '${newValue}'`, debugDev);
       //debug(`   TODO: sort subfields, handle punctuation...`);
     }
@@ -110,7 +109,7 @@ function skipMergeField(baseRecord, sourceField, config) {
   // Skip duplicate field:
   if (baseRecord.fields.some(baseField => !baseField.mergeCandidate && fieldsAreIdentical(sourceField, baseField))) {
     nvdebug(`skipMergeField(): field '${fieldToString(sourceField)}' already exists! No merge required!`, debugDev);
-    sourceField.deleted = 1; // eslint-disable-line functional/immutable-data
+    sourceField.deleted = 1;
     return true;
   }
 
@@ -145,9 +144,9 @@ function swapDataBetweenFields(field1, field2) {
   return;
 
   function swapNamedData(name) {
-    const data = field1[name]; // eslint-disable-line functional/immutable-data
-    field1[name] = field2[name]; // eslint-disable-line functional/immutable-data
-    field2[name] = data; // eslint-disable-line functional/immutable-data
+    const data = field1[name];
+    field1[name] = field2[name];
+    field2[name] = data;
   }
 
 }
@@ -155,11 +154,11 @@ function swapDataBetweenFields(field1, field2) {
 export function mergeField(baseRecord, sourceRecord, sourceField, config) {
   nvdebug(`SELF: ${fieldToString(sourceField)}`, debugDev);
 
-  sourceField.mergeCandidate = true; // eslint-disable-line functional/immutable-data
+  sourceField.mergeCandidate = true;
   // skip duplicates and special cases:
   if (skipMergeField(baseRecord, sourceField, config)) {
     nvdebug(`mergeField(): don't merge '${fieldToString(sourceField)}'`, debugDev);
-    delete sourceField.mergeCandidate; // eslint-disable-line functional/immutable-data
+    delete sourceField.mergeCandidate;
     return false;
   }
 
@@ -167,7 +166,7 @@ export function mergeField(baseRecord, sourceRecord, sourceField, config) {
   const counterpartField = getCounterpart(baseRecord, sourceRecord, sourceField, config);
 
   if (counterpartField) {
-    if (sourceRecordIsBetter(counterpartField, sourceField)) { // eslint-disable-line functional/no-conditional-statements
+    if (sourceRecordIsBetter(counterpartField, sourceField)) {
       swapDataBetweenFields(counterpartField, sourceField);
     }
 
@@ -175,13 +174,13 @@ export function mergeField(baseRecord, sourceRecord, sourceField, config) {
     nvdebug(`mergeField(): Got counterpart: '${fieldToString(counterpartField)}'. Thus try merge...`, debugDev);
     nvdebug(`PAIR: ${candFieldPairs880 ? fieldsToString(candFieldPairs880) : 'NADA'}`, debugDev);
     mergeField2(baseRecord, counterpartField, sourceField, config, candFieldPairs880);
-    sourceField.deleted = 1; // eslint-disable-line functional/immutable-data
-    delete sourceField.mergeCandidate; // eslint-disable-line functional/immutable-data
+    sourceField.deleted = 1;
+    delete sourceField.mergeCandidate;
     return true;
   }
   // NB! Counterpartless field is inserted to 7XX even if field.tag says 1XX:
   debugDev(`mergeField(): No mergable counterpart found for '${fieldToString(sourceField)}'.`);
-  delete sourceField.mergeCandidate; // eslint-disable-line functional/immutable-data
+  delete sourceField.mergeCandidate;
   return false;
 }
 

package/src/merge-fields/mergeIndicator.js

@@ -109,7 +109,7 @@ export function mergeIndicators(toField, fromField, config) {
 
     // MRA-300: If source contains the (un)holy $9 FENNI<KEEP>, we prefer that value regardless of whatever...
     if (!fieldIsFenniKept(toField) && fieldIsFenniKept(fromField)) {
-      toField.ind1 = fromField.ind1; // eslint-disable-line functional/immutable-data
+      toField.ind1 = fromField.ind1;
       return;
     }
 
@@ -122,7 +122,7 @@ export function mergeIndicators(toField, fromField, config) {
       const preferredValue = getPreferredValue(preferredValues, fromField.ind1, toField.ind1);
       if (typeof preferredValue !== 'undefined') {
         //nvdebug(`${preferredValue} WINS!`, debugDev);
-        toField.ind1 = preferredValue; // eslint-disable-line functional/immutable-data
+        toField.ind1 = preferredValue;
         return;
       }
       //nvdebug(`No winner found indicator 1: '${toField.ind1}' vs '${fromField.ind1}', keep '${toField.ind1}'`, debugDev);
@@ -151,8 +151,8 @@ export function mergeIndicators(toField, fromField, config) {
     nvdebug(`Apply base 260->264 tag swap hack`, debugDev);
     nvdebug(` ${fieldToString(toField)}\n ${fieldToString(fromField)}`, debugDev);
 
-    toField.tag = '264'; // eslint-disable-line functional/immutable-data
-    toField.ind2 = fromField.ind2; // eslint-disable-line functional/immutable-data
+    toField.tag = '264';
+    toField.ind2 = fromField.ind2;
   }
 
   function mergeIndicator2(toField, fromField, config) {
@@ -175,7 +175,7 @@ export function mergeIndicators(toField, fromField, config) {
       //nvdebug(`  Try to merge indicator 2. Got preferred values '${preferredValues}'`, debugDev);
       const preferredValue = getPreferredValue(preferredValues, fromField.ind2, toField.ind2);
       if (typeof preferredValue !== 'undefined') {
-        toField.ind2 = preferredValue; // eslint-disable-line functional/immutable-data
+        toField.ind2 = preferredValue;
         return;
       }
     }

package/src/merge-fields/mergeOrAddPostprocess.js

@@ -12,7 +12,7 @@ function postprocessBaseRecord(base) {
     // NB! Relator terms are now expanded and translated already at preprocess stage!
 
     // remove merge-specific information:
-    if (field.merged) { // eslint-disable-line functional/no-conditional-statements
+    if (field.merged) {
       // Field level ideas about things that could be done here:
       // - Fix indicators?
       // Record level fixes should be implemented as validators/fixers
@@ -23,20 +23,20 @@ function postprocessBaseRecord(base) {
       sortAdjacentRelatorTerms(field); // Sort $e subfields with each other
       fieldFixPunctuation(field);
 
-      delete field.merged; // eslint-disable-line functional/immutable-data
+      delete field.merged;
     }
 
-    if (field.useExternalEndPunctuation) { // eslint-disable-line functional/no-conditional-statements
-      delete field.useExternalEndPunctuation; // eslint-disable-line functional/immutable-data
+    if (field.useExternalEndPunctuation) {
+      delete field.useExternalEndPunctuation;
     }
 
-    if (field.added) { // eslint-disable-line functional/no-conditional-statements
-      delete field.added; // eslint-disable-line functional/immutable-data
+    if (field.added) {
+      delete field.added;
     }
 
     /*
-    if (field.deleted) { // eslint-disable-line functional/no-conditional-statements
-      delete field.deleted; // eslint-disable-line functional/immutable-data
+    if (field.deleted) {
+      delete field.deleted;
     }
 */
 
@@ -46,7 +46,7 @@ function postprocessBaseRecord(base) {
 
 function removeDeletedFields(record) {
   // remove fields that are marked as deleted:
-  record.fields = record.fields.filter(f => !f.deleted); // eslint-disable-line functional/immutable-data
+  record.fields = record.fields.filter(f => !f.deleted);
 }
 
 

package/src/merge-fields/mergeOrAddSubfield.js

@@ -116,9 +116,9 @@ function mergeOrAddSubfieldNotRequired(targetField, candSubfieldData) {
 function addSubfield(targetField, candSubfield) {
   nvdebug(` Added subfield '${subfieldToString(candSubfield)}' to field`, debugDev);
   // Add subfield to the end of all subfields. NB! Implement a separate function that does this + subfield reordering somehow...
-  targetField.subfields.push(candSubfield); // eslint-disable-line functional/immutable-data
+  targetField.subfields.push(candSubfield);
 
-  targetField.merged = 1; // eslint-disable-line functional/immutable-data
+  targetField.merged = 1;
 
   setPunctuationFlag(targetField, candSubfield);
   sortAdjacentSubfields(targetField);
@@ -129,7 +129,7 @@ function setPunctuationFlag(field, addedSubfield) {
   if (isControlSubfieldCode(addedSubfield.code)) { // These are never punctuation related
     return;
   }
-  field.useExternalEndPunctuation = 1; // eslint-disable-line functional/immutable-data
+  field.useExternalEndPunctuation = 1;
 }
 
 
@@ -184,7 +184,7 @@ export function mergeOrAddSubfield(targetField, candSubfieldData, candFieldPairs
       return;
     }
     nvdebug(`    A: Yes. Add repeatable subfield '${subfieldToString(candSubfield)}'`, debugDev);
-    targetField.merged = 1; // eslint-disable-line functional/immutable-data
+    targetField.merged = 1;
     setPunctuationFlag(targetField, candSubfield);
     addSubfield(targetField, candSubfield);
     return;
@@ -196,7 +196,7 @@ export function mergeOrAddSubfield(targetField, candSubfieldData, candFieldPairs
   function mergeSubfieldPostprocessor() {
     if (original !== fieldToString(targetField)) {
       nvdebug(`    A: Merge. Subfield '${candSubfieldAsString}' replaces the original subfield.`, debugDev);
-      targetField.merged = 1; // eslint-disable-line functional/immutable-data
+      targetField.merged = 1;
       setPunctuationFlag(targetField, candSubfield);
       return;
     }
@@ -207,7 +207,7 @@ export function mergeOrAddSubfield(targetField, candSubfieldData, candFieldPairs
   function addSubfieldWithPreviouslyUnseenSubfieldCode() {
     if (!fieldHasSubfield(targetField, candSubfield.code)) {
       nvdebug(`    A: Yes. Add previously unseen subfield '${subfieldToString(candSubfield)}'`, debugDev);
-      targetField.merged = 1; // eslint-disable-line functional/immutable-data
+      targetField.merged = 1;
       setPunctuationFlag(targetField, candSubfield);
       candFieldPairs880.forEach(pair => resetPaired880(pair, targetField, candSubfield));
       addSubfield(targetField, candSubfield);

package/src/merge-fields/mergeSubfield.js

@@ -45,12 +45,12 @@ function anyYear(str) {
 function replaceEntrysBirthAndDeathYear(targetField, candSubfield, relevantSubfields) {
   if (birthYearAndDeathYear.test(candSubfield.value)) {
     if (onlyBirthYear.test(relevantSubfields[0].value) && parseInt(relevantSubfields[0].value, 10) === parseInt(candSubfield.value, 10)) {
-      relevantSubfields[0].value = candSubfield.value; // eslint-disable-line functional/immutable-data
+      relevantSubfields[0].value = candSubfield.value;
       return true;
     }
 
     if (onlyDeathYear.test(relevantSubfields[0].value) && getDeathYear(relevantSubfields[0].value) === getDeathYear(candSubfield.value)) {
-      relevantSubfields[0].value = candSubfield.value; // eslint-disable-line functional/immutable-data
+      relevantSubfields[0].value = candSubfield.value;
       return true;
     }
   }
@@ -65,7 +65,7 @@ function replaceDatesAssociatedWithName(targetField, candSubfield, relevantSubfi
   }
 
   if (!anyYear(relevantSubfields[0].value) && anyYear(candSubfield.value)) {
-    relevantSubfields[0].value = candSubfield.value; // eslint-disable-line functional/immutable-data
+    relevantSubfields[0].value = candSubfield.value;
     return true;
   }
 
@@ -174,7 +174,7 @@ function preferHyphenatedISBN(field, candSubfield, relevantSubfields) {
   if (!pair) {
     return false;
   }
-  pair.value = candSubfield.value; // eslint-disable-line functional/immutable-data
+  pair.value = candSubfield.value;
   return true;
 }
 
@@ -189,7 +189,7 @@ function preferHttpsOverHttp(candSubfield, relevantSubfields) {
   if (!pair) {
     return false;
   }
-  pair.value = candSubfield.value; // eslint-disable-line functional/immutable-data
+  pair.value = candSubfield.value;
   return true;
 }
 
@@ -209,7 +209,7 @@ function preferQualifierVersion(field, candSubfield, relevantSubfields) {
     return true; // True, but don't prefer the source value
   }
 
-  pair.value = candSubfield.value; // eslint-disable-line functional/immutable-data
+  pair.value = candSubfield.value;
   return true;
 
   function subfieldQualifierCheck(subfield, name, qualifier) {
@@ -232,7 +232,7 @@ function preferSourceCorporateName(field, candSubfield, pair) {
   nvdebug(`CORP base '${pair.value}' vs '${candSubfield.value}'`, debugDev);
   const prefer = actualPrefenceCheck();
   if (prefer) {
-    pair.value = candSubfield.value; // eslint-disable-line functional/immutable-data
+    pair.value = candSubfield.value;
     return true;
   }
   return false;
@@ -290,7 +290,7 @@ export function mergeSubfield(targetField, candSubfield) {
   // 260 $a value "[S.l]" is the main type for this.
   const meaninglessSubfields = relevantSubfields.filter(sf => !valueCarriesMeaning(targetField.tag, sf.code, sf.value));
   if (meaninglessSubfields.length > 0) {
-    meaninglessSubfields[0].value = candSubfield.value; // eslint-disable-line functional/immutable-data
+    meaninglessSubfields[0].value = candSubfield.value;
     return true;
   }
 

package/src/merge-fields/removeDuplicateSubfields.js

@@ -22,7 +22,7 @@ export function fieldRemoveDuplicateSubfields(field) {
   }
 
   const strippedField = cloneAndRemovePunctuation(field); // make punctuation-less version
-  /* eslint-disable */
+
   let seen = {};
 
   field.subfields = field.subfields.filter((sf, i) => notSeenBefore(sf, i));
@@ -45,6 +45,5 @@ export function fieldRemoveDuplicateSubfields(field) {
     seen[subfieldAsString] = subfieldAsString;
     return true;
   }
-  /* eslint-enable */
 
 }

package/src/merge-fields/worldKnowledge.js

@@ -48,7 +48,7 @@ export function normalizeForSamenessCheck(tag, subfieldCode, originalValue) {
 
 function normalizePersonalName(originalValue) {
   // Use more readable "Forename Surname" format in comparisons:
-  return originalValue.replace(/^([^,]+), ([^,]+)$/u, '$2 $1'); // eslint-disable-line prefer-named-capture-group
+  return originalValue.replace(/^([^,]+), ([^,]+)$/u, '$2 $1');
 }
 
 const sallittu506a = ['sallittu kaikenikäisille', 'sallittu', 's']; // downcased, without punctuation

package/src/mergeField500Lisapainokset.js

@@ -133,7 +133,6 @@ export function mergeLisapainokset(record) {
     return;
   }
 
-  /* eslint-disable */
   const collapsedArray = extractAllPrintData(relevantFields);
   if (collapsedArray.length === 0) {
     return;
@@ -149,5 +148,4 @@ export function mergeLisapainokset(record) {
       return;
     }
   });
-  /* eslint-enable */
 }

package/src/mergeRelatorTermFields.js

@@ -34,7 +34,7 @@ export default function () {
     const msg = mergeRelatorTermFields(record, false);
     const res = {message: msg};
 
-    res.valid = !(res.message.length >= 1); // eslint-disable-line functional/immutable-data
+    res.valid = !(res.message.length >= 1);
     return res;
   }
 }
@@ -49,7 +49,7 @@ function createNormalizedClone(field) {
 function createNormalizedCloneWithoutRelatorTerms(field) {
   const clonedField = createNormalizedClone(field);
   // Remove relator terms $e subfi:
-  clonedField.subfields = clonedField.subfields.filter(sf => sf.code !== 'e'); // eslint-disable-line functional/immutable-data
+  clonedField.subfields = clonedField.subfields.filter(sf => sf.code !== 'e');
   return clonedField;
 }
 
@@ -74,7 +74,7 @@ function copyRelatorSubfields(fromField, toField) {
   const relatorTermSubfieldCode = tagToRelatorTermSubfieldCode(fromField.tag);
   const newRelatorTerms = extractAddableRelatorTerms(fromField, toField);
 
-  newRelatorTerms.forEach(term => toField.subfields.push({code: relatorTermSubfieldCode, value: term})); // eslint-disable-line functional/immutable-data
+  newRelatorTerms.forEach(term => toField.subfields.push({code: relatorTermSubfieldCode, value: term})); // eslint-disable-line array-callback-return
 
 }
 

package/src/modernize-502.js

@@ -19,7 +19,7 @@ export default function () {
       normalizeField502(field);
     });
 
-    // message.valid = !(message.message.length >= 1); // eslint-disable-line functional/immutable-data
+    // message.valid = !(message.message.length >= 1);
     return res;
   }
 
@@ -30,7 +30,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 = normalizeField502(clone(field));
     const mod = fieldToString(normalizedField);
     if (orig !== mod) { // Fail as the input is "broken"/"crap"/sumthing
-      res.message.push(`Fix '${orig}' => '${mod}'`); // eslint-disable-line functional/immutable-data
+      res.message.push(`Fix '${orig}' => '${mod}'`);
       return;
     }
     return;
@@ -63,7 +63,7 @@ export function normalizeField502(field) {
   }
 
   // "a = acd[0]"" is way more readable than "[a] = acd"...
-  const a = acd[0]; // eslint-disable-line prefer-destructuring
+  const a = acd[0];
   const c = acd[1].code === 'c' ? acd[1] : null;
   const d = acd[acd.length - 1].code === 'd' ? acd[acd.length - 1] : null;
 
@@ -74,8 +74,8 @@ export function normalizeField502(field) {
   }
 
   const newValue = `${extractA()}--${extractC()}${extractD()}`;
-  a.value = newValue; // eslint-disable-line functional/immutable-data
-  field.subfields = field.subfields.filter(sf => !['c', 'd'].includes(sf.code)); // eslint-disable-line functional/immutable-data
+  a.value = newValue;
+  field.subfields = field.subfields.filter(sf => !['c', 'd'].includes(sf.code));
   return field;
 
   function extractA() {

package/src/multiple-subfield-0.js

@@ -16,16 +16,16 @@ export default function () {
   function fix(record) {
     function fixField(field) {
       const removableSubfields = fieldGetDeletableSubfields(field);
-      removableSubfields.forEach(sf => record.removeSubfield(sf, field));
+      removableSubfields.forEach(sf => record.removeSubfield(sf, field)); // eslint-disable-line array-callback-return
     }
 
     const res = {message: [], fix: [], valid: true};
 
     const relevantFields = getRelevantFields(record);
 
-    relevantFields.forEach(field => fixField(field));
+    relevantFields.forEach(field => fixField(field)); // eslint-disable-line array-callback-return
 
-    // message.valid = !(message.message.length >= 1); // eslint-disable-line functional/immutable-data
+    // message.valid = !(message.message.length >= 1);
     return res;
   }
 
@@ -40,7 +40,7 @@ export default function () {
     const relevantFields = getRelevantFields(record);
     const messages = relevantFields.map(field => validateField(field));
     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/non-breaking-space.js

@@ -26,7 +26,6 @@ export default function () {
     }
   }
 
-  /* eslint-disable functional/immutable-data,functional/no-conditional-statements */
   function fix(record) {
     record.fields.forEach((field) => {
       if (field.value) {
@@ -41,7 +40,6 @@ export default function () {
       });
     });
   }
-  /* eslint-enable functional/immutable-data,functional/no-conditional-statements */
 
   function valueContainsNonBreakingSpace({value}) {
     return (/\u00A0/u).test(value);

package/src/normalize-dashes.js

@@ -28,7 +28,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;
   }
 
@@ -41,7 +41,7 @@ export default function () {
     if (orig === mod) { // Fail as the input is "broken"/"crap"/sumthing
       return;
     }
-    res.message.push(`'TODO: ${orig}' => '${mod}'`); // eslint-disable-line functional/immutable-data
+    res.message.push(`'TODO: ${orig}' => '${mod}'`);
     return;
   }
 }
@@ -54,7 +54,7 @@ function fixDashes(field) {
 
   nvdebug(`Dashing ${fieldToString(field)}`);
 
-  field.subfields.forEach(sf => subfieldFixDashes(sf));
+  field.subfields.forEach(sf => subfieldFixDashes(sf)); // eslint-disable-line array-callback-return
 
   return field;
 
@@ -63,7 +63,7 @@ function fixDashes(field) {
       return;
     }
     // Normalize dashes U+2010 ... U+2015 to '-':
-    subfield.value = subfield.value.replace(/[\u2010\u2011\u2012\u2013\u2014\u2015]/ug, '-'); // eslint-disable-line functional/immutable-data
+    subfield.value = subfield.value.replace(/[\u2010\u2011\u2012\u2013\u2014\u2015]/ug, '-');
   }
 
 }

package/src/normalize-identifiers.js

@@ -6,7 +6,7 @@ import {fieldToString} from './utils';
 
 /*
 function nvdebug(message, func) {
-  if (func) { // eslint-disable-line functional/no-conditional-statements
+  if (func) {
     func(message);
   }
   console.info(message); // eslint-disable-line no-console
@@ -23,7 +23,7 @@ export default function () {
 
   function fix(record) {
     const res = {message: [], fix: [], valid: true};
-    //message.fix = []; // eslint-disable-line functional/immutable-data
+    //message.fix = [];
 
     // Actual parsing of all fields
     /*
@@ -40,7 +40,7 @@ export default function () {
       //validateField(field, true, message);
     });
 
-    // message.valid = !(message.message.length >= 1); // eslint-disable-line functional/immutable-data
+    // message.valid = !(message.message.length >= 1);
     return res;
   }
 
@@ -59,7 +59,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;
   }
 
@@ -74,7 +74,7 @@ export default function () {
     const orig = fieldToString(field);
     const mod = fieldToString(normalizedField);
     if (orig !== mod) { // Fail as the input is "broken"/"crap"/sumthing
-      res.message.push(`'${orig}' could do with control number identifier normalization`); // eslint-disable-line functional/immutable-data
+      res.message.push(`'${orig}' could do with control number identifier normalization`);
       return;
     }
 
@@ -190,7 +190,7 @@ export function fieldNormalizeControlNumbers(field) {
     const targetFormat = normalizeAs(field.tag, sf.code);
     if (targetFormat !== undefined) {
       //nvdebug(`NORMALIZE SUBFIELD $${sf.code} IN FIELD: '${fieldToString(field)}' TO ${targetFormat}`);
-      sf.value = normalizeControlSubfieldValue(sf.value, targetFormat); // eslint-disable-line functional/immutable-data
+      sf.value = normalizeControlSubfieldValue(sf.value, targetFormat);
       return;
     }
   });

package/src/normalize-qualifying-information.js

@@ -27,7 +27,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;
   }
 
@@ -38,7 +38,7 @@ export default function () {
     normalizeQualifyingInformationField(normalizedField);
     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,14 +51,14 @@ function normalizeQualifyingInformationField(field) {
     return;
   }
 
-  field.subfields.forEach(sf => fixSubfield(sf));
+  field.subfields.forEach(sf => fixSubfield(sf)); // eslint-disable-line array-callback-return
   return;
 
   function fixSubfield(sf) {
     if (sf.code !== 'q') {
       return;
     }
-    sf.value = normalizeValue(sf.value); // eslint-disable-line functional/immutable-data
+    sf.value = normalizeValue(sf.value);
   }
 
   function normalizeValue(val) {

package/src/normalize-utf8-diacritics.js

@@ -23,7 +23,7 @@ export default function () {
 
   function fix(record) {
     const res = {message: [], fix: [], valid: true};
-    //message.fix = []; // eslint-disable-line functional/immutable-data
+    //message.fix = [];
 
     // Actual parsing of all fields
     /*
@@ -37,7 +37,7 @@ export default function () {
       //validateField(field, true, message);
     });
 
-    // message.valid = !(message.message.length >= 1); // eslint-disable-line functional/immutable-data
+    // message.valid = !(message.message.length >= 1);
     return res;
   }
 
@@ -55,7 +55,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;
   }
 
@@ -68,7 +68,7 @@ export default function () {
     const normalizedField = fieldFixComposition(clone(field));
     const mod = fieldToString(normalizedField);
     if (orig !== mod) { // Fail as the input is "broken"/"crap"/sumthing
-      res.message.push(`'${orig}' requires normalization`); // eslint-disable-line functional/immutable-data
+      res.message.push(`'${orig}' requires normalization`);
       return;
     }
     return;
@@ -113,10 +113,10 @@ export function fieldFixComposition(field) {
   //const originalValue = fieldToString(field);
   //nvdebug(`fFC: '${originalValue}'`, debug);
   field.subfields.forEach((subfield, index) => {
-    field.subfields[index].value = fixComposition(subfield.value); // eslint-disable-line functional/immutable-data
+    field.subfields[index].value = fixComposition(subfield.value);
   });
   //const newValue = fieldToString(field);
-  //if (originalValue !== newValue) { // eslint-disable-line functional/no-conditional-statements
+  //if (originalValue !== newValue) {
   //  debug(`FIXCOMP: '${originalValue}' => '${newValue}'`);
   //}
   return field;
@@ -127,7 +127,7 @@ export function fieldRemoveDecomposedDiacritics(field) {
   // Raison d'être/motivation: "Sirén" and diacriticless "Siren" might refer to a same surname, so this normalization
   // allows us to compare authors and avoid duplicate fields.
   field.subfields.forEach((sf) => {
-    sf.value = removeDecomposedDiacritics(sf.value); // eslint-disable-line functional/immutable-data
+    sf.value = removeDecomposedDiacritics(sf.value);
   });
 
   function removeDecomposedDiacritics(value = '') {