@natlibfi/marc-record-validators-melinda 11.3.8 → 11.4.0

package/src/translate-terms.js

@@ -0,0 +1,274 @@
+import clone from 'clone';
+import createDebugLogger from 'debug';
+import {fieldHasSubfield, fieldToString, nvdebug} from './utils';
+
+
+const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda:sortSubfields');
+const defaultTags = ['648', '650', '651', '655'];
+
+const swapLanguageCode = {'fin': 'swe', 'fi': 'sv', 'sv': 'fi', 'swe': 'fin'};
+const changeAbbrHash = {'fi': 'fin', 'fin': 'fi', 'sv': 'swe', 'swe': 'sv'};
+
+//const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/translate-term');
+const termCache = {};
+
+// Author(s): Nicholas Volk
+export default function () {
+
+
+  return {
+    description: 'Translate yso (648, 650, 651) and slm (655) terms (FIN <=> SWE)',
+    validate, fix
+  };
+
+  async function fix(record) {
+    const newFields = await getFields(record, defaultTags, []);
+
+    newFields.forEach(nf => nvdebug(`Add new field '${fieldToString(nf)}'`, debug));
+
+    newFields.forEach(nf => record.insertField(nf));
+
+    const newFieldsAsStrings = newFields.map(f => fieldToString(f));
+
+
+    return {message: [], fix: newFieldsAsStrings, valid: true};
+  }
+
+  async function validate(record) {
+    const newFields = await getFields(record, defaultTags, []);
+    if (newFields.length === 0) {
+      return {'message': [], 'valid': true};
+    }
+    const messages = newFields.map(f => fieldToString(f));
+
+    return {'message': messages, 'valid': false};
+  }
+
+  async function getFields(record, tags, fieldsToAdd) {
+    const [currTag, ...remainingTags] = tags;
+    if (!currTag) {
+      return fieldsToAdd;
+    }
+    const missingFields = await deriveMissingFields(record, currTag);
+
+    const tmp = await getFields(record, remainingTags, [...fieldsToAdd, ...missingFields]);
+    return tmp;
+  }
+
+  function getPairlessFinnishAndSwedishFields(record, tag) {
+    const expectedLex = mapTagToLex(tag);
+    if (!expectedLex) {
+      return [];
+    }
+    const fields = record.get(tag);
+    const finnishFields = fields.filter(f => isRelevantField(f, 'fin'));
+    const swedishFields = fields.filter(f => isRelevantField(f, 'swe'));
+    const finnishOnly = getMisses(finnishFields, swedishFields);
+    const swedishOnly = getMisses(swedishFields, finnishFields);
+
+    //console.log(` Looking at ${finnishOnly.length} + ${swedishOnly.length} fields`); // eslint-disable-line no-console
+    return [...finnishOnly, ...swedishOnly].filter(f => tagAndFieldAgree(f));
+
+    function tagAndFieldAgree(field) {
+      // Check that tag and $2 value are pairable:
+      const lexData = getLexiconAndLanguage(field); // $2 data
+      return expectedLex === lexData.lex;
+    }
+  }
+
+  async function deriveMissingFields(record, tag) {
+    const pairlessFields = getPairlessFinnishAndSwedishFields(record, tag);
+
+    /* eslint-disable */
+    // Dunno how to handle loop+promise combo in our normal coding style. Spent half a day trying... (I reckon it takes like 2 minuts to do this properly...)
+    let prefLabels = [];
+    for (let i=0; i < pairlessFields.length; i += 1) {
+      prefLabels[i] = await getPrefLabel(pairlessFields[i]);
+    }
+    /* eslint-enable */
+
+    const missingFields = pairlessFields.map((f, i) => pairField(f, prefLabels[i]));
+    return missingFields.filter(f => f);
+  }
+
+  function pairField(field, prefLabels) {
+    if (!prefLabels) {
+      return undefined;
+    }
+    //console.log(`pairField() WP 1: ${fieldToString(field)}`); // eslint-disable-line no-console
+    const lexAndLang = getLexiconAndLanguage(field);
+    //console.log(`pairField() WP 2: ${JSON.stringify(lexAndLang)}`); // eslint-disable-line no-console
+    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
+    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
+    const newField = {tag: field.tag, ind1: field.ind1, ind2: field.ind2, subfields: [sfA, sf2, sf0]};
+    return newField;
+  }
+
+  function getLexiconAndLanguage(field) {
+    const subfield2 = field.subfields.find(sf => sf.code === '2');
+    if (subfield2.value === 'slm/fin') {
+      return {'lex': 'slm', 'lang': 'fin'};
+    }
+    if (subfield2.value === 'slm/swe') {
+      return {'lex': 'slm', 'lang': 'swe'};
+    }
+    if (subfield2.value === 'yso/fin') {
+      return {'lex': 'yso', 'lang': 'fin'};
+    }
+    if (subfield2.value === 'yso/swe') {
+      return {'lex': 'yso', 'lang': 'swe'};
+    }
+    return {};
+  }
+
+  async function getPrefLabel(field) {
+    // Tag vs $2 correlation has already been checked!
+    const uri = fieldToUri(field);
+    if (!uri) { // $0 is invalid or sumthing
+      return undefined;
+    }
+    const prefLabels = await getTermData(uri);
+    if (!prefLabels) { // Sanity check. Miss caused by illegal id etc.
+      nvdebug(`No labels found for ${uri}`, debug);
+      return undefined;
+    }
+    const lexData = getLexiconAndLanguage(field); // $2 data
+    const lang = changeAbbr(lexData.lang);
+
+
+    const subfieldA = field.subfields.find(sf => sf.code === 'a');
+
+    const prefLabel = prefLabels.find(pl => pl.lang === lang);
+    //console.info(`Compare prefLabel '${prefLabel.value}' AND $a '${subfieldA.value}'`); // eslint-disable-line no-console
+    if (prefLabel.value === subfieldA.value) {
+      nvdebug(`'${fieldToString(field)}' requires translating`, debug);
+      return prefLabels;
+    }
+    return undefined;
+  }
+
+  function swapLanguageCodeBetweenLanguages(code) {
+    if (swapLanguageCode[code]) {
+      return swapLanguageCode[code];
+    }
+    return code;
+  }
+
+  function changeAbbr(abbr) {
+    if (changeAbbrHash[abbr]) {
+      return changeAbbrHash[abbr];
+    }
+    return abbr;
+  }
+
+  function swaggerQuery(uri) {
+    // This would work for only yso, not yso-paikat etc `https://api.finto.fi/rest/v1/yso/data?format=application%2Fjson&uri=${uri}`;
+    return `https://api.finto.fi/rest/v1/data?uri=${uri}&format=application%2Fjson`; // This is simpler, but contains more irrelevant data
+  }
+
+  async function getTermData(uri) {
+    //console.log(`getTermData(${uri})`); // eslint-disable-line no-console
+    if (termCache[uri]) { // Don't think current implementation uses the cache any more.
+      //console.log(`CACHED ${uri}`); // eslint-disable-line no-console
+      return termCache[uri];
+    }
+    const tmp = await getTermDataFromFinto(uri);
+    termCache[uri] = tmp; // eslint-disable-line functional/immutable-data, require-atomic-updates
+    return tmp;
+  }
+
+  async function getTermDataFromFinto(uri) {
+    const headers = {'Accept': 'application/json'};
+    const uri2 = swaggerQuery(uri);
+
+    const response = await fetch(uri2, {method: 'GET', headers});
+    if (!response.ok) {
+      return undefined;
+    }
+    const json = await response.json();
+
+    if (!json.graph) {
+      return undefined;
+    }
+    const arr = json.graph;
+    const [hit] = arr.filter(row => row.uri === uri);
+    //console.log(`NEW JSON: ${JSON.stringify(hit.prefLabel)}`); // eslint-disable-line no-console
+    return hit.prefLabel;
+  }
+
+
+  function fieldToUri(field) {
+    const lex = mapTagToLex(field.tag);
+
+    const subfield0 = field.subfields.find(sf => sf.code === '0');
+    const id = subfield0.value.replace(/^[^0-9]+/u, '');
+    if (lex === 'yso') {
+      //return `http%3A%2F%2Fwww.yso.fi%2Fonto%2Fyso%2Fp${id}`;
+      return `http://www.yso.fi/onto/yso/p${id}`;
+    }
+    if (lex === 'slm') {
+      return `http://urn.fi/URN:NBN:fi:au:slm:s${id}`;
+    }
+    return undefined;
+  }
+
+  function isRelevantField(field, lang) {
+    const fieldAsString = fieldToString(field);
+
+    // We should probably allow an optional $8 as the first subfield.
+    if (!fieldAsString.match(/^... #7 ‡a [^‡]+ ‡2 [^‡]+ ‡0 [^‡]+(?: ‡9 [A-Z]+<(?:KEEP|DROP)>)*$/u)) {
+      return false;
+    }
+    const lex = mapTagToLex(field.tag);
+    const lexLang = `${lex}/${lang}`;
+    if (!fieldHasSubfield(field, '2', lexLang)) {
+      return false;
+    }
+    return fieldHasValidSubfield0(field);
+  }
+
+  function fieldHasValidSubfield0(field) {
+    const lex = mapTagToLex(field.tag);
+    const subfield0 = field.subfields.find(sf => sf.code === '0');
+    if (lex === 'yso' && subfield0.value.match(/^http:\/\/www\.yso\.fi\/onto\/yso\/p[0-9]+$/u)) {
+      return true;
+    }
+    if (lex === 'slm' && subfield0.value.match(/^http:\/\/urn\.fi\/URN:NBN:fi:au:slm:s[0-9]+$/u)) {
+      return true;
+    }
+    return false;
+  }
+
+  function getMisses(fieldList1, fieldList2) {
+    return fieldList1.filter(f => !hasSubfield0Match(f, fieldList2));
+  }
+
+  function hasSubfield0Match(field, pairFields) {
+    const subfield0 = field.subfields.find(sf => sf.code === '0');
+    return pairFields.some(f => f.subfields.some(sf => sf.code === '0' && sf.value === subfield0.value));
+  }
+
+
+  function mapTagToLex(tag) {
+    if (tag === '655') {
+      return 'slm';
+    }
+    return 'yso';
+  }
+
+
+  /*
+  function getValidIdentifiers(record, tag, lang) {
+    const lex = mapTagToLex(tag);
+    const subfield2Value = `${lex}/${lang}`;
+    const candFields = record.get(tag).filter(f => f.subfields.some(sf => sf.code === '2' && sf.value === subfield2Value)); // TODO: filter
+    return [];
+  }
+    */
+
+}
+

package/src/translate-terms.spec.js

@@ -0,0 +1,52 @@
+import {expect} from 'chai';
+import {MarcRecord} from '@natlibfi/marc-record';
+import validatorFactory from './translate-terms';
+import {READERS} from '@natlibfi/fixura';
+import generateTests from '@natlibfi/fixugen';
+import createDebugLogger from 'debug';
+
+generateTests({
+  callback,
+  path: [__dirname, '..', 'test-fixtures', 'translate-terms'],
+  useMetadataFile: true,
+  recurse: false,
+  fixura: {
+    reader: READERS.JSON
+  },
+  mocha: {
+    before: () => testValidatorFactory()
+  }
+});
+const debug = createDebugLogger('@natlibfi/marc-record-validators-melinda/translate-terms:test');
+
+async function testValidatorFactory() {
+  const validator = await validatorFactory();
+
+  expect(validator)
+    .to.be.an('object')
+    .that.has.any.keys('description', 'validate');
+
+  expect(validator.description).to.be.a('string');
+  expect(validator.validate).to.be.a('function');
+}
+
+async function callback({getFixture, enabled = true, fix = false}) {
+  if (enabled === false) {
+    debug('TEST SKIPPED!');
+    return;
+  }
+
+  const validator = await validatorFactory();
+  const record = new MarcRecord(getFixture('record.json'));
+  const expectedResult = getFixture('expectedResult.json');
+  // console.log(expectedResult); // eslint-disable-line
+
+  if (!fix) {
+    const result = await validator.validate(record);
+    expect(result).to.eql(expectedResult);
+    return;
+  }
+
+  await validator.fix(record);
+  expect(record).to.eql(expectedResult);
+}

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

@@ -1,5 +1,5 @@
 {
-  "description": "Fix: field 100 required translitteration",
+  "description": "01 Fix: field 100 required translitteration",
   "only": false,
   "fix": true
 }

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

@@ -1,10 +1,11 @@
 {
-  "description": "Fix: just convert field 100 to ISO-9 latinitsa",
+  "description": "03 Fix: just convert field 100 to ISO-9 latinitsa, don't retain the original nor do sfs4900",
   "comment": "Tests usage of config and omitting of $6 and $9",
   "only": false,
   "fix": true,
   "config": {
-    "doISO9Transliteration": false,
-    "doSFS4900Transliteration": false
+    "doISO9Transliteration": true,
+    "doSFS4900Transliteration": false,
+    "retainCyrillic": false
   }
 }

package/test-fixtures/cyrillux/f03b/expectedResult.json

@@ -0,0 +1,21 @@
+{
+  "_validationOptions": {},
+  "leader": "12345cam  22123454i 4500",
+  "fields": [
+    { "tag": "100", "ind1": "1", "ind2": " ", "subfields": [
+      { "code": "a", "value": "Modin, Ûrij Ivanovič." }
+    ]},
+    { "tag": "245", "ind1": "1", "ind2": " ", "subfields": [
+      { "code": "a", "value": "Sudʹby razvedčikov." }
+    ]},
+    {"tag": "500", "ind1": " ", "ind2": " ", "subfields": [
+      { "code": "6", "value": "880-01"},
+      { "code": "a", "value": "FOO."}
+    ]},
+    {"tag": "880", "ind1": " ", "ind2": " ", "subfields": [
+      { "code": "6", "value": "500-01"},
+      { "code": "a", "value": "BAR."}
+    ]}
+  ]
+}
+

package/test-fixtures/cyrillux/f03b/metadata.json

@@ -0,0 +1,11 @@
+{
+  "description": "03b Fix: just convert field 100 to SFS4900 latinitsa, don't retain the original nor do ISO-9",
+  "comment": "Tests usage of config and omitting of $6 and $9",
+  "only": false,
+  "fix": true,
+  "config": {
+    "doISO9Transliteration": false,
+    "doSFS4900Transliteration": true,
+    "retainCyrillic": false
+  }
+}

package/test-fixtures/cyrillux/f03b/record.json

@@ -0,0 +1,19 @@
+{
+  "leader": "12345cam  22123454i 4500",
+  "fields": [
+    { "tag": "100", "ind1": "1", "ind2": " ", "subfields": [
+      { "code": "a", "value": "Модин, Юрий Иванович." }
+    ]},
+    {"tag": "245", "ind1": "1", "ind2": " ", "subfields": [
+      { "code": "a", "value": "Судьбы разведчиков."}
+    ]},
+    {"tag": "500", "ind1": " ", "ind2": " ", "subfields": [
+      { "code": "6", "value": "880-01"},
+      { "code": "a", "value": "FOO."}
+    ]},
+    {"tag": "880", "ind1": " ", "ind2": " ", "subfields": [
+      { "code": "6", "value": "500-01"},
+      { "code": "a", "value": "BAR."}
+    ]}
+  ]
+}

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

@@ -1,10 +1,11 @@
 {
-  "description": "Fix: just convert field 100 to ISO-9 latinitsa",
+  "description": "04 Fix: just convert field 100 to ISO-9 latinitsa",
   "comment": "Tests usage of config and omitting of $6 and $9",
   "only": false,
   "fix": true,
   "config": {
-    "doISO9Transliteration": false,
-    "doSFS4900Transliteration": false
+    "doISO9Transliteration": true,
+    "doSFS4900Transliteration": false,
+    "retainCyrillic": false
   }
 }

package/test-fixtures/cyrillux/f08/metadata.json

@@ -1,5 +1,5 @@
 {
-  "description": "Fix MELINDA-10330: fields 100  and 245 are already in SFS-4900 and their only 880 pairs are Cyrillic",
+  "description": "08 Fix MELINDA-10330: fields 100  and 245 are already in SFS-4900 and their only 880 pairs are Cyrillic",
   "comment": "Use 880 to create 100/245 in ISO-9. Move original SFS-4900 to 880, add $9s",
   "comment #2": "Note that the original punctuation in 100/245 is overwritten with punctuation from paired 880",
   "only": false,

package/test-fixtures/cyrillux/f09/metadata.json

@@ -1,5 +1,5 @@
 {
-  "description": "Fix MELINDA-10330, round 3: big file that had issues",
+  "description": "09 Fix MELINDA-10330, round 3: big file that had issues",
   "comment": "Some sorting issues: SIDs are sorted by $c (bug) and LOW precedes SID...",
   "only": false,
   "fix": true,

package/test-fixtures/cyrillux/f11/expectedResult.json

@@ -0,0 +1,18 @@
+{
+  "_validationOptions": {},
+  "leader": "12345cam  22123454i 4500",
+  "fields": [
+    { "tag": "008", "value": "01234567890123456789012345678901234ukr89"},
+    { "tag": "100", "ind1": "1", "ind2": " ", "subfields": [
+        { "code": "6", "value": "880-01" },
+        { "code": "a", "value": "Gogolʹ." },
+        { "code": "9", "value": "ISO9 <TRANS>" }
+    ]},
+    { "tag": "880", "ind1": "1", "ind2": " ", "subfields": [
+      { "code": "6", "value": "100-01" },
+      { "code": "a", "value": "Гоголь." },
+      { "code": "9", "value": "CYRILLIC <TRANS>" }
+    ]}
+  ]
+}
+

package/test-fixtures/cyrillux/f11/metadata.json

@@ -0,0 +1,9 @@
+{
+  "description": "03: transliterate (ISO-9 only)",
+  "only": false,
+  "fix": true,
+  "config": {
+    "doISO9Transliteration": true,
+    "doSFS4900Transliteration": false
+  }
+}

package/test-fixtures/cyrillux/f11/record.json

@@ -0,0 +1,9 @@
+{
+  "leader": "12345cam  22123454i 4500",
+  "fields": [
+    { "tag": "008", "value": "01234567890123456789012345678901234ukr89"},
+    { "tag": "100", "ind1": "1", "ind2": " ", "subfields": [
+        { "code": "a", "value": "Гоголь." }
+    ]}
+  ]
+}

package/test-fixtures/cyrillux/f11b/expectedResult.json

@@ -0,0 +1,18 @@
+{
+  "_validationOptions": {},
+  "leader": "12345cam  22123454i 4500",
+  "fields": [
+    { "tag": "008", "value": "01234567890123456789012345678901234ukr89"},
+    { "tag": "100", "ind1": "1", "ind2": " ", "subfields": [
+        { "code": "6", "value": "880-01" },
+        { "code": "a", "value": "Gogolʹ." },
+        { "code": "9", "value": "ISO9 <TRANS>" }
+    ]},
+    { "tag": "880", "ind1": "1", "ind2": " ", "subfields": [
+      { "code": "6", "value": "100-01" },
+      { "code": "a", "value": "Гоголь." },
+      { "code": "9", "value": "CYRILLIC <TRANS>" }
+    ]}
+  ]
+}
+

package/test-fixtures/cyrillux/f11b/metadata.json

@@ -0,0 +1,9 @@
+{
+  "description": "f11b: use output from the f11",
+  "only": false,
+  "fix": true,
+  "config": {
+    "doISO9Transliteration": true,
+    "doSFS4900Transliteration": false
+  }
+}

package/test-fixtures/cyrillux/f11b/record.json

@@ -0,0 +1,16 @@
+{
+  "leader": "12345cam  22123454i 4500",
+  "fields": [
+    { "tag": "008", "value": "01234567890123456789012345678901234ukr89"},
+    { "tag": "100", "ind1": "1", "ind2": " ", "subfields": [
+        { "code": "6", "value": "880-01" },
+        { "code": "a", "value": "Gogolʹ." },
+        { "code": "9", "value": "ISO9 <TRANS>" }
+    ]},
+    { "tag": "880", "ind1": "1", "ind2": " ", "subfields": [
+      { "code": "6", "value": "100-01" },
+      { "code": "a", "value": "Гоголь." },
+      { "code": "9", "value": "CYRILLIC <TRANS>" }
+    ]}
+  ]
+}

package/test-fixtures/translate-terms/01/expectedResult.json

@@ -0,0 +1,37 @@
+{
+  "_validationOptions": {},
+  "fields": [
+    { "tag": "650", "ind1": " ", "ind2": "7", "subfields": [
+      { "code": "a", "value": "laiturit" },
+      { "code": "2", "value": "yso/fin" },
+      { "code": "0", "value": "http://www.yso.fi/onto/yso/p13299" }
+    ]},
+    { "tag": "650", "ind1": " ", "ind2": "7", "subfields": [
+      { "code": "a", "value": "marxismi" },
+      { "code": "2", "value": "yso/fin" },
+      { "code": "0", "value": "http://www.yso.fi/onto/yso/p444" }
+    ]},
+    { "tag": "650", "ind1": " ", "ind2": "7", "subfields": [
+      { "code": "a", "value": "marxism" },
+      { "code": "2", "value": "yso/swe" },
+      { "code": "0", "value": "http://www.yso.fi/onto/yso/p444" }
+    ]},
+    { "tag": "650", "ind1": " ", "ind2": "7", "subfields": [
+      { "code": "a", "value": "bryggor" },
+      { "code": "2", "value": "yso/swe" },
+      { "code": "0", "value": "http://www.yso.fi/onto/yso/p13299" }
+    ]},
+    { "tag": "651", "ind1": " ", "ind2": "7", "subfields": [
+      { "code": "a", "value": "Ivalo (Enare)" },
+      { "code": "2", "value": "yso/swe" },
+      { "code": "0", "value": "http://www.yso.fi/onto/yso/p111739" }
+    ]},
+    { "tag": "651", "ind1": " ", "ind2": "7", "subfields": [
+      { "code": "a", "value": "Ivalo (Inari)" },
+      { "code": "2", "value": "yso/fin" },
+      { "code": "0", "value": "http://www.yso.fi/onto/yso/p111739" }
+    ]}
+
+  ],
+  "leader": ""
+}

package/test-fixtures/translate-terms/01/metadata.json

@@ -0,0 +1,6 @@
+{
+  "description": "01: insert one fin->swe and one swe->fin translation",
+  "comment": "Fields are inserted in tag order. Inserted fields are not sorted.",
+  "fix": true,
+  "only": false
+}

package/test-fixtures/translate-terms/01/record.json

@@ -0,0 +1,27 @@
+{
+  "_validationOptions": {},
+  "leader": "",
+  "fields": [
+    { "tag": "650", "ind1": " ", "ind2": "7", "subfields": [
+        { "code": "a", "value": "laiturit" },
+        { "code": "2", "value": "yso/fin" },
+        { "code": "0", "value": "http://www.yso.fi/onto/yso/p13299" }
+      ]
+    },
+    { "tag": "650", "ind1": " ", "ind2": "7", "subfields": [
+      { "code": "a", "value": "marxismi" },
+      { "code": "2", "value": "yso/fin" },
+      { "code": "0", "value": "http://www.yso.fi/onto/yso/p444" }
+    ]},
+    { "tag": "650", "ind1": " ", "ind2": "7", "subfields": [
+      { "code": "a", "value": "marxism" },
+      { "code": "2", "value": "yso/swe" },
+      { "code": "0", "value": "http://www.yso.fi/onto/yso/p444" }
+    ]},
+    { "tag": "651", "ind1": " ", "ind2": "7", "subfields": [
+      { "code": "a", "value": "Ivalo (Enare)" },
+      { "code": "2", "value": "yso/swe" },
+      { "code": "0", "value": "http://www.yso.fi/onto/yso/p111739" }
+    ]}
+  ]
+}

package/test-fixtures/translate-terms/02/expectedResult.json

@@ -0,0 +1,37 @@
+{
+  "_validationOptions": {},
+    "fields": [
+    { "tag": "648", "ind1": " ", "ind2": "7", "subfields": [
+        { "code": "a", "value": "1960-luku" },
+        { "code": "2", "value": "yso/fin" },
+        { "code": "0", "value": "http://www.yso.fi/onto/yso/p6196061969" }
+      ]
+    },
+    { "tag": "648", "ind1": " ", "ind2": "7", "subfields": [
+      { "code": "a", "value": "1960-talet" },
+      { "code": "2", "value": "yso/swe" },
+      { "code": "0", "value": "http://www.yso.fi/onto/yso/p6196061969" }
+    ]},
+    { "tag": "655", "ind1": " ", "ind2": "7", "subfields": [
+      { "code": "a", "value": "damtidningar" },
+      { "code": "2", "value": "slm/swe" },
+      { "code": "0", "value": "http://urn.fi/URN:NBN:fi:au:slm:s161" }
+    ]},
+    { "tag": "655", "ind1": " ", "ind2": "7", "subfields": [
+      { "code": "a", "value": "nekrologit" },
+      { "code": "2", "value": "slm/fin" },
+      { "code": "0", "value": "http://urn.fi/URN:NBN:fi:au:slm:s1324" }
+    ]},
+    { "tag": "655", "ind1": " ", "ind2": "7", "subfields": [
+      { "code": "a", "value": "nekrologer" },
+      { "code": "2", "value": "slm/swe" },
+      { "code": "0", "value": "http://urn.fi/URN:NBN:fi:au:slm:s1324" }
+    ]},
+    { "tag": "655", "ind1": " ", "ind2": "7", "subfields": [
+      { "code": "a", "value": "naistenlehdet" },
+      { "code": "2", "value": "slm/fin" },
+      { "code": "0", "value": "http://urn.fi/URN:NBN:fi:au:slm:s161" }
+    ]}
+  ],
+  "leader": ""
+}

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

@@ -0,0 +1,6 @@
+{
+  "description": "02: insert one fin->swe (yso-aika) and one swe->fin (slm) translation",
+  "comment": "Fields are inserted in tag order. Inserted fields are not sorted.",
+  "fix": true,
+  "only": false
+}