geonetwork-ui 2.3.2 → 2.4.0-alpha.1.2ade9018

package/src/libs/api/metadata-converter/src/lib/iso19139/iso19139.converter.ts

@@ -2,6 +2,8 @@ import {
   CatalogRecord,
   CatalogRecordKeys,
   DatasetRecord,
+  ModelTranslations,
+  RecordTranslations,
   ServiceRecord,
 } from '../../../../../../libs/common/domain/src/lib/model/record'
 import { XmlElement } from '@rgrove/parse-xml'
@@ -18,7 +20,7 @@ import {
   readAbstract,
   readContacts,
   readContactsForResource,
-  readDistributions,
+  readDefaultLanguage,
   readIsoTopics,
   readKeywords,
   readKind,
@@ -27,6 +29,7 @@ import {
   readLineage,
   readOnlineResources,
   readOtherConstraints,
+  readOtherLanguages,
   readOverviews,
   readOwnerOrganization,
   readRecordUpdated,
@@ -34,6 +37,7 @@ import {
   readResourcePublished,
   readResourceUpdated,
   readSecurityConstraints,
+  readSpatialExtents,
   readSpatialRepresentation,
   readStatus,
   readTemporalExtents,
@@ -45,21 +49,22 @@ import {
   writeAbstract,
   writeContacts,
   writeContactsForResource,
-  writeDistributions,
+  writeDefaultLanguage,
   writeGraphicOverviews,
   writeKeywords,
   writeKind,
+  writeLanguages,
   writeLegalConstraints,
   writeLicenses,
   writeLineage,
   writeOnlineResources,
   writeOtherConstraints,
-  writeOwnerOrganization,
   writeRecordUpdated,
   writeResourceCreated,
   writeResourcePublished,
   writeResourceUpdated,
   writeSecurityConstraints,
+  writeSpatialExtents,
   writeSpatialRepresentation,
   writeStatus,
   writeTemporalExtents,
@@ -72,7 +77,7 @@ import {
 export class Iso19139Converter extends BaseConverter<string> {
   protected readers: Record<
     CatalogRecordKeys,
-    (rootEl: XmlElement) => unknown
+    (rootEl: XmlElement, translations: RecordTranslations) => unknown
   > = {
     uniqueIdentifier: readUniqueIdentifier,
     kind: readKind,
@@ -98,14 +103,15 @@ export class Iso19139Converter extends BaseConverter<string> {
     spatialRepresentation: readSpatialRepresentation,
     overviews: readOverviews,
     lineage: readLineage,
-    distributions: readDistributions,
     onlineResources: readOnlineResources,
     temporalExtents: readTemporalExtents,
+    spatialExtents: readSpatialExtents,
+    otherLanguages: readOtherLanguages,
+    defaultLanguage: readDefaultLanguage,
     // TODO
-    spatialExtents: () => [],
     extras: () => undefined,
     landingPage: () => undefined,
-    languages: () => [],
+    translations: () => undefined,
   }
 
   protected writers: Record<
@@ -114,7 +120,7 @@ export class Iso19139Converter extends BaseConverter<string> {
   > = {
     uniqueIdentifier: writeUniqueIdentifier,
     kind: writeKind,
-    ownerOrganization: writeOwnerOrganization,
+    ownerOrganization: () => undefined, // fixme: find a way to store this value properly
     recordUpdated: writeRecordUpdated,
     recordCreated: () => undefined, // not supported in ISO19139
     recordPublished: () => undefined, // not supported in ISO19139
@@ -136,60 +142,113 @@ export class Iso19139Converter extends BaseConverter<string> {
     spatialRepresentation: writeSpatialRepresentation,
     overviews: writeGraphicOverviews,
     lineage: writeLineage,
-    distributions: writeDistributions,
     onlineResources: writeOnlineResources,
     temporalExtents: writeTemporalExtents,
+    spatialExtents: writeSpatialExtents,
+    otherLanguages: writeLanguages,
+    defaultLanguage: writeDefaultLanguage,
     // TODO
-    spatialExtents: () => undefined,
     extras: () => undefined,
     landingPage: () => undefined,
-    languages: () => undefined,
+    translations: () => undefined,
   }
 
   protected beforeDocumentCreation(rootElement: XmlElement) {
     // to override
   }
 
+  protected afterRecordRead(record: CatalogRecord): CatalogRecord {
+    function fixLanguages(target: { translations?: ModelTranslations }) {
+      if (!('translations' in target)) return
+      for (const fieldName in target.translations) {
+        const fieldTranslations = target.translations[fieldName] ?? {}
+        // remove redundant translations
+        if (record.defaultLanguage in fieldTranslations) {
+          delete fieldTranslations[record.defaultLanguage]
+        }
+        // remove translated values if they have no translations
+        if (Object.keys(fieldTranslations).length === 0) {
+          delete target.translations[fieldName]
+          continue
+        }
+        // add languages that have translations to the root language list
+        for (const lang in fieldTranslations) {
+          if (!record.otherLanguages.includes(lang)) {
+            record.otherLanguages.push(lang)
+          }
+        }
+      }
+    }
+    fixLanguages(record)
+    record.keywords.map(fixLanguages)
+    record.onlineResources.map(fixLanguages)
+    record.licenses.map(fixLanguages)
+    record.legalConstraints.map(fixLanguages)
+    record.securityConstraints.map(fixLanguages)
+    record.otherConstraints.map(fixLanguages)
+    record.contacts.map((c) => fixLanguages(c.organization))
+    record.contactsForResource.map((c) => fixLanguages(c.organization))
+    fixLanguages(record.ownerOrganization)
+    if (record.kind === 'dataset') {
+      record.spatialExtents.map(fixLanguages)
+    }
+
+    // remove default language from otherLanguages list
+    if (record.otherLanguages.includes(record.defaultLanguage)) {
+      record.otherLanguages = record.otherLanguages.filter(
+        (lang) => lang !== record.defaultLanguage
+      )
+    }
+
+    return record
+  }
+
   async readRecord(document: string): Promise<CatalogRecord> {
     const doc = parseXmlString(document)
     const rootEl = getRootElement(doc)
 
-    const uniqueIdentifier = this.readers['uniqueIdentifier'](rootEl)
-    const kind = this.readers['kind'](rootEl)
-    const ownerOrganization = this.readers['ownerOrganization'](rootEl)
-    const title = this.readers['title'](rootEl)
-    const abstract = this.readers['abstract'](rootEl)
-    const contacts = this.readers['contacts'](rootEl)
-    const contactsForResource = this.readers['contactsForResource'](rootEl)
-    const recordUpdated = this.readers['recordUpdated'](rootEl)
-    const recordCreated = this.readers['recordCreated'](rootEl)
-    const recordPublished = this.readers['recordPublished'](rootEl)
-    const resourceCreated = this.readers['resourceCreated'](rootEl)
-    const resourceUpdated = this.readers['resourceUpdated'](rootEl)
-    const resourcePublished = this.readers['resourcePublished'](rootEl)
-    const keywords = this.readers['keywords'](rootEl)
-    const topics = this.readers['topics'](rootEl)
-    const legalConstraints = this.readers['legalConstraints'](rootEl)
-    const otherConstraints = this.readers['otherConstraints'](rootEl)
-    const securityConstraints = this.readers['securityConstraints'](rootEl)
-    const licenses = this.readers['licenses'](rootEl)
-    const overviews = this.readers['overviews'](rootEl)
-    const landingPage = this.readers['landingPage'](rootEl)
+    const tr: RecordTranslations = {}
+    const uniqueIdentifier = this.readers['uniqueIdentifier'](rootEl, tr)
+    const kind = this.readers['kind'](rootEl, tr)
+    const ownerOrganization = this.readers['ownerOrganization'](rootEl, tr)
+    const title = this.readers['title'](rootEl, tr)
+    const abstract = this.readers['abstract'](rootEl, tr)
+    const contacts = this.readers['contacts'](rootEl, tr)
+    const contactsForResource = this.readers['contactsForResource'](rootEl, tr)
+    const recordUpdated = this.readers['recordUpdated'](rootEl, tr)
+    const recordCreated = this.readers['recordCreated'](rootEl, tr)
+    const recordPublished = this.readers['recordPublished'](rootEl, tr)
+    const resourceCreated = this.readers['resourceCreated'](rootEl, tr)
+    const resourceUpdated = this.readers['resourceUpdated'](rootEl, tr)
+    const resourcePublished = this.readers['resourcePublished'](rootEl, tr)
+    const keywords = this.readers['keywords'](rootEl, tr)
+    const topics = this.readers['topics'](rootEl, tr)
+    const legalConstraints = this.readers['legalConstraints'](rootEl, tr)
+    const otherConstraints = this.readers['otherConstraints'](rootEl, tr)
+    const securityConstraints = this.readers['securityConstraints'](rootEl, tr)
+    const licenses = this.readers['licenses'](rootEl, tr)
+    const overviews = this.readers['overviews'](rootEl, tr)
+    const landingPage = this.readers['landingPage'](rootEl, tr)
+    const onlineResources = this.readers['onlineResources'](rootEl, tr)
+    const otherLanguages = this.readers['otherLanguages'](rootEl, tr)
+    const defaultLanguage = this.readers['defaultLanguage'](rootEl, tr)
 
     if (kind === 'dataset') {
-      const status = this.readers['status'](rootEl)
-      const spatialRepresentation =
-        this.readers['spatialRepresentation'](rootEl)
-      const spatialExtents = this.readers['spatialExtents'](rootEl)
-      const temporalExtents = this.readers['temporalExtents'](rootEl)
-      const lineage = this.readers['lineage'](rootEl)
-      const distributions = this.readers['distributions'](rootEl)
-      const updateFrequency = this.readers['updateFrequency'](rootEl)
+      const status = this.readers['status'](rootEl, tr)
+      const spatialRepresentation = this.readers['spatialRepresentation'](
+        rootEl,
+        tr
+      )
+      const spatialExtents = this.readers['spatialExtents'](rootEl, tr)
+      const temporalExtents = this.readers['temporalExtents'](rootEl, tr)
+      const lineage = this.readers['lineage'](rootEl, tr)
+      const updateFrequency = this.readers['updateFrequency'](rootEl, tr)
 
-      return {
+      return this.afterRecordRead({
         uniqueIdentifier,
         kind,
-        languages: [],
+        otherLanguages,
+        defaultLanguage,
         ...(recordCreated && { recordCreated }),
         ...(recordPublished && { recordPublished }),
         recordUpdated,
@@ -213,16 +272,17 @@ export class Iso19139Converter extends BaseConverter<string> {
         overviews,
         spatialExtents,
         temporalExtents,
-        distributions,
+        onlineResources,
         updateFrequency,
+        translations: tr,
         ...(landingPage && { landingPage }),
-      } as DatasetRecord
+      } as DatasetRecord)
     } else {
-      const onlineResources = this.readers['onlineResources'](rootEl)
-      return {
+      return this.afterRecordRead({
         uniqueIdentifier,
         kind,
-        languages: [],
+        otherLanguages,
+        defaultLanguage,
         ...(recordCreated && { recordCreated }),
         ...(recordPublished && { recordPublished }),
         recordUpdated,
@@ -242,8 +302,9 @@ export class Iso19139Converter extends BaseConverter<string> {
         otherConstraints,
         overviews,
         onlineResources,
+        translations: tr,
         ...(landingPage && { landingPage }),
-      } as ServiceRecord
+      } as ServiceRecord)
     }
   }
 
@@ -271,6 +332,8 @@ export class Iso19139Converter extends BaseConverter<string> {
     fieldChanged('uniqueIdentifier') &&
       this.writers['uniqueIdentifier'](record, rootEl)
     fieldChanged('kind') && this.writers['kind'](record, rootEl)
+    fieldChanged('defaultLanguage') &&
+      this.writers['defaultLanguage'](record, rootEl)
 
     fieldChanged('contacts') && this.writers['contacts'](record, rootEl)
     fieldChanged('ownerOrganization') &&
@@ -305,6 +368,8 @@ export class Iso19139Converter extends BaseConverter<string> {
     fieldChanged('licenses') && this.writers['licenses'](record, rootEl)
     fieldChanged('otherConstraints') &&
       this.writers['otherConstraints'](record, rootEl)
+    fieldChanged('onlineResources') &&
+      this.writers['onlineResources'](record, rootEl)
 
     if (record.kind === 'dataset') {
       fieldChanged('status') && this.writers['status'](record, rootEl)
@@ -315,14 +380,14 @@ export class Iso19139Converter extends BaseConverter<string> {
       fieldChanged('overviews') && this.writers['overviews'](record, rootEl)
       fieldChanged('temporalExtents') &&
         this.writers['temporalExtents'](record, rootEl)
-      fieldChanged('distributions') &&
-        this.writers['distributions'](record, rootEl)
+      fieldChanged('spatialExtents') &&
+        this.writers['spatialExtents'](record, rootEl)
       fieldChanged('lineage') && this.writers['lineage'](record, rootEl)
-    } else {
-      fieldChanged('onlineResources') &&
-        this.writers['onlineResources'](record, rootEl)
     }
 
+    fieldChanged('otherLanguages') &&
+      this.writers['otherLanguages'](record, rootEl)
+
     this.beforeDocumentCreation(rootEl)
 
     const newDocument = createDocument(rootEl)