geonetwork-ui 2.7.0-dev.21bdf506c → 2.7.0-dev.2d4634844

package/src/libs/ui/widgets/src/lib/popup-alert/popup-alert.component.ts

@@ -22,7 +22,12 @@ import { matErrorOutlineOutline } from '@ng-icons/material-icons/outline'
   changeDetection: ChangeDetectionStrategy.OnPush,
   standalone: true,
   imports: [CommonModule, NgIconComponent],
-  providers: [provideIcons({ matErrorOutlineOutline })],
+  providers: [
+    provideIcons({ matErrorOutlineOutline }),
+    provideNgIconsConfig({
+      size: '1.5em',
+    }),
+  ],
 })
 export class PopupAlertComponent implements OnInit {
   @Input() icon: string

package/src/libs/util/app-config/src/lib/app-config.ts

@@ -231,6 +231,7 @@ export function loadAppConfig() {
           'filter_geometry_url',
           'search_preset',
           'advanced_filters',
+          'limit',
         ],
         warnings,
         errors
@@ -257,6 +258,7 @@ export function loadAppConfig() {
                 filters: param.filters,
               })),
               ADVANCED_FILTERS: parsedSearchSection.advanced_filters,
+              LIMIT: parsedSearchSection.limit,
             } as SearchConfig)
 
       const parsedMetadataQualitySection = parseConfigSection(

package/src/libs/util/app-config/src/lib/fixtures.ts

@@ -124,7 +124,7 @@ export const wrongLanguageCodeConfigFixture = () => `
 [global]
 geonetwork4_api_url = "/geonetwork/srv/api"
 proxy_path = "/proxy/?url="
-metadata_language = "fra"
+metadata_language = "abc"
 
 [map]
 

package/src/libs/util/app-config/src/lib/model.ts

@@ -60,6 +60,7 @@ export interface SearchConfig {
   FILTER_GEOMETRY_DATA?: string
   SEARCH_PRESET?: SearchPreset[]
   ADVANCED_FILTERS?: []
+  LIMIT?: number
 }
 
 export interface MetadataQualityConfig {

package/src/libs/util/app-config/src/lib/parse-utils.ts

@@ -1,4 +1,4 @@
-import { LANGUAGE_CODES_ISO_3 } from './constants'
+import { toLang2 } from '../../../../../libs/util/i18n/src'
 
 const flatten = (
   base: string,
@@ -129,13 +129,10 @@ export function checkMetadataLanguage(
   parsedConfigSection: any,
   outWarnings: string[]
 ) {
-  if (
-    LANGUAGE_CODES_ISO_3.indexOf(
-      parsedConfigSection.metadata_language.toLowerCase()
-    ) === -1
-  ) {
+  const lang2 = toLang2(parsedConfigSection.metadata_language.toLowerCase())
+  if (lang2?.length !== 2) {
     outWarnings.push(
-      `In the [global] section: metadata_language = "${parsedConfigSection.metadata_language}" is not in ISO 639-2/B format`
+      `In the [global] section: metadata_language = "${parsedConfigSection.metadata_language}" is not a recognized ISO 639 language code`
     )
   }
   return parsedConfigSection

package/src/libs/util/i18n/src/lib/language-codes.ts

@@ -1,20 +1,32 @@
 import { marker } from '@biesbjerg/ngx-translate-extract-marker'
 
+/**
+ * Taken from https://en.wikipedia.org/wiki/List_of_ISO_639_language_codes
+ * Note: some languages have multiple 3-char codes, like 'fre' and 'fra'; in that case, the one to be used
+ * in priority is the one defined last
+ */
 const LANG_3_TO_2_MAPPER = {
   eng: 'en',
+  nld: 'nl', // duplicate for "dut"
   dut: 'nl',
+  fra: 'fr', // duplicate for "fre"
   fre: 'fr',
+  deu: 'de', // duplicate for "ger"
   ger: 'de',
   kor: 'ko',
   spa: 'es',
+  ces: 'cs', // duplicate for "cze"
   cze: 'cs',
   cat: 'ca',
   fin: 'fi',
+  isl: 'is', // duplicate for "ice"
   ice: 'is',
   ita: 'it',
   por: 'pt',
   rus: 'ru',
+  zho: 'zh', // duplicate for "chi"
   chi: 'zh',
+  slk: 'sk', // duplicate for "slo"
   slo: 'sk',
   roh: 'rm',
   ara: 'ar',
@@ -25,10 +37,12 @@ const LANG_3_TO_2_MAPPER = {
   tur: 'tr',
   arm: 'hy',
   aze: 'az',
+  kat: 'ka', // duplicate for "geo"
   geo: 'ka',
   ukr: 'uk',
+  cym: 'cy', // duplicate for "wel"
   wel: 'cy',
-}
+} as const
 
 export const LANGUAGE_NAMES = {
   en: 'English',
@@ -58,26 +72,63 @@ export const LANGUAGE_NAMES = {
   ka: 'ქართული',
   uk: 'українська',
   wel: 'Cymraeg',
-}
+} as const
+
+export type LanguageCode3 = keyof typeof LANG_3_TO_2_MAPPER
+export type LanguageCode2 = (typeof LANG_3_TO_2_MAPPER)[LanguageCode3]
 
 export const LANG_2_TO_3_MAPPER = Object.entries(LANG_3_TO_2_MAPPER).reduce(
   (mapperObject, langEntry) => {
     return { ...mapperObject, [langEntry[1]]: langEntry[0] }
   },
   {}
-)
+) as Record<LanguageCode2, LanguageCode3>
 
-export function getLang3FromLang2(lang2: string): string {
-  return LANG_2_TO_3_MAPPER[lang2] ?? lang2
-}
+/**
+ * This can be:
+ * - an ISO 639-2 language code in 3 characters (e.g. 'eng', 'fre', 'ger')
+ * - an ISO 639-1 language code in 2 characters (e.g. 'en', 'fr', 'de')
+ * - a 2-character language code with locale (e.g. 'fr_FR', 'fr_CA')
+ */
+export type LanguageCodeLike = LanguageCode2 | LanguageCode3 | string
 
-export function getLang2FromLang3(lang3: string): string {
-  return LANG_3_TO_2_MAPPER[lang3] ?? lang3
+/**
+ * Converts a language code in any format to the ISO 639-2 format (3 characters)
+ * Returns the given string if the corresponding language code could not be recognized
+ */
+export function toLang3(lang: LanguageCodeLike): LanguageCode3 | string {
+  if (!lang) {
+    // also handle falsy values just in case
+    return lang
+  }
+  if (lang.length === 3) {
+    return LANG_2_TO_3_MAPPER[LANG_3_TO_2_MAPPER[lang.toLowerCase()]] ?? lang
+  }
+  const lang2 = lang.toLowerCase().substring(0, 2)
+  return LANG_2_TO_3_MAPPER[lang2] ?? lang
 }
 
-export function getLocalizedIndexKey(lang2: string): string {
-  const lang3 = getLang3FromLang2(lang2)
-  return lang3 ? `lang${lang3}` : null
+/**
+ * Converts a language code in any format to the ISO 639-1 format (2 characters)
+ * Returns the given string if the corresponding language code could not be recognized
+ */
+export function toLang2(lang: LanguageCodeLike): LanguageCode2 | string {
+  if (!lang) {
+    // also handle falsy values just in case
+    return lang
+  }
+  if (lang.length === 3) {
+    return LANG_3_TO_2_MAPPER[lang.toLowerCase()] ?? lang
+  }
+  const lang2 = lang.toLowerCase().substring(0, 2)
+  if (lang2 in LANG_2_TO_3_MAPPER) {
+    return lang2
+  }
+  if (lang.match(/[a-z]{2}_[A-Z]{2}/)) {
+    // remove locale code even if the language code is not known
+    return lang2
+  }
+  return lang
 }
 
 marker('language.en')

package/src/libs/util/shared/src/index.ts

@@ -3,4 +3,4 @@ export * from './lib/utils'
 export * from './lib/links'
 export * from './lib/image-fallback.directive'
 export * from './lib/gn-ui-version'
-export * from './lib/record.util'
+export * from './lib/record'

package/src/libs/util/shared/src/lib/links/link-utils.ts

@@ -23,8 +23,8 @@ export const FORMATS = {
   excel: {
     extensions: [
       'excel',
-      'xls',
       'xlsx',
+      'xls',
       'ms-excel',
       'openxmlformats-officedocument',
     ],
@@ -75,10 +75,17 @@ export const FORMATS = {
     mimeTypes: ['application/geopackage+sqlite3'],
   },
   zip: {
-    extensions: ['zip', 'tar.gz'],
+    extensions: ['zip', 'tar.gz', 'gz', '7z', '7zip'],
     priority: 8,
     color: '#B0CB52',
-    mimeTypes: ['application/zip', 'application/x-zip'],
+    mimeTypes: [
+      'application/zip',
+      'application/x-zip',
+      'application/x-compressed',
+      'application/x-7z-compressed',
+      'application/gzip',
+      'application/x-gzip',
+    ],
   },
   pdf: {
     extensions: ['pdf'],
@@ -92,39 +99,57 @@ export const FORMATS = {
     color: '#C4A98F',
     mimeTypes: ['image/jpg'],
   },
+  png: {
+    extensions: ['png'],
+    priority: 10,
+    color: '#8B5A3C',
+    mimeTypes: ['image/png'],
+  },
+  tiff: {
+    extensions: ['tiff', 'tif', 'geotiff', 'geotif'],
+    priority: 11,
+    color: '#6B4423',
+    mimeTypes: ['image/tiff'],
+  },
   svg: {
     extensions: ['svg'],
-    priority: 10,
+    priority: 12,
     color: '#EB6D82',
     mimeTypes: ['image/svg+xml'],
   },
   dxf: {
     extensions: ['dxf'],
-    priority: 11,
+    priority: 13,
     color: '#DCCD00',
     mimeTypes: ['application/x-dxf', 'image/x-dxf'],
   },
   html: {
     extensions: ['html', 'htm'],
-    priority: 12,
+    priority: 14,
     color: '#C0C9B6',
     mimeTypes: ['text/html'],
   },
   fgb: {
     extensions: ['fgb', 'flatgeobuf'],
-    priority: 13,
+    priority: 15,
     color: '#A8111C',
     mimeTypes: ['application/flatgeobuf'],
   },
   jsonfg: {
     extensions: ['jsonfg', 'jsonfgc'],
-    priority: 14,
+    priority: 16,
     color: '#009EE0',
     mimeTypes: [
       'application/vnd.ogc.fg+json',
       'application/vnd.ogc.fg+json;compatibility=geojson',
     ],
   },
+  webp: {
+    extensions: ['webp'],
+    priority: 17,
+    color: '#5A9E6F',
+    mimeTypes: ['image/webp'],
+  },
 } as const
 
 export type FileFormat = keyof typeof FORMATS
@@ -190,6 +215,7 @@ export function isFormatInQueryParam(
   alias: string
 ): boolean {
   const queryParams = link.url.searchParams
+  if (!queryParams) return false
   for (const [key, value] of queryParams.entries()) {
     if (key === 'format' || key === 'f') {
       return value === alias

package/src/libs/util/shared/src/lib/record/index.ts

@@ -0,0 +1,2 @@
+export * from './quality-score.util'
+export * from './record.util'

package/src/libs/util/shared/src/lib/record/quality-score.util.ts

@@ -0,0 +1,72 @@
+import {
+  CatalogRecord,
+  RecordKind,
+} from '../../../../../../libs/common/domain/src/lib/model/record'
+
+type TValidatorMapper = {
+  [key: string]: (metadata: Partial<CatalogRecord>) => boolean
+}
+
+const ValidatorMapper: TValidatorMapper = {
+  title: (record) => !!record?.title,
+  abstract: (record) => !!record?.abstract,
+  keywords: (record) => (record?.keywords?.length ?? 0) > 0,
+  legalConstraints: (record) =>
+    !!(
+      record?.legalConstraints?.length &&
+      record.legalConstraints.some((c) => c?.text?.trim().length > 0)
+    ),
+  contacts: (record) =>
+    !!record?.contacts?.[0]?.email &&
+    record.contacts[0].email !== 'missing@missing.com',
+  updateFrequency: (record) =>
+    !!record?.updateFrequency && record.updateFrequency !== 'unknown',
+  topics: (record) => (record?.topics?.length ?? 0) > 0,
+  organisation: (record) => !!record?.contacts?.[0]?.organization?.name,
+  source: (record) => !!record?.extras?.sourcesIdentifiers,
+} as const
+
+export type ValidatorMapperKeys = keyof typeof ValidatorMapper & string
+
+export function getAllKeysValidator() {
+  return Object.keys(ValidatorMapper)
+}
+
+function getMappersFromKind(kind: RecordKind) {
+  let kindKeys = <ValidatorMapperKeys[]>[]
+  const commonsKeys = <ValidatorMapperKeys[]>[
+    'title',
+    'abstract',
+    'keywords',
+    'legalConstraints',
+    'contacts',
+  ]
+
+  switch (kind) {
+    case 'reuse':
+      kindKeys = ['topics', 'organisation', 'source']
+      break
+    case 'service':
+      kindKeys = []
+      break
+    case 'dataset':
+    default:
+      kindKeys = ['updateFrequency', 'topics', 'organisation']
+  }
+
+  return [...commonsKeys, ...kindKeys]
+}
+
+export function getQualityValidators(
+  record: Partial<CatalogRecord>,
+  propsToValidate: ValidatorMapperKeys[]
+) {
+  const filteredProps = propsToValidate.filter((prop) =>
+    getMappersFromKind(record.kind).includes(prop)
+  )
+
+  return filteredProps.map((name) => ({
+    name,
+    validator: () => ValidatorMapper[name](record),
+  }))
+}

package/src/libs/util/shared/src/lib/{record.util.ts → record/record.util.ts}

@@ -10,7 +10,7 @@ import {
   OnlineLinkResource,
   Organization,
   ServiceEndpoint,
-} from '../../../../../libs/common/domain/src/lib/model/record'
+} from '../../../../../../libs/common/domain/src/lib/model/record'
 
 type TranslatedRecordObjects =
   | BaseRecord

package/src/libs/util/shared/src/lib/utils/geojson.ts

@@ -1,8 +1,11 @@
-import { Feature, FeatureCollection, Geometry } from 'geojson'
+import type { Feature, FeatureCollection, Geometry, Position } from 'geojson'
 
+/**
+ * @returns The geometry if available, otherwise null.
+ */
 export function getGeometryFromGeoJSON(
   data: FeatureCollection | Feature | Geometry
-): Geometry {
+): Geometry | null {
   if (data.type === 'FeatureCollection') {
     return data?.features?.[0]?.geometry || null
   }
@@ -24,3 +27,70 @@ export function getGeometryFromGeoJSON(
   }
   return null
 }
+
+// FIXME: this type should be more generic across the project
+export type BoundingBox = [number, number, number, number]
+
+export function getGeometryBoundingBox(geometry: Geometry): BoundingBox {
+  // use the bounding box if specified in the GeoJSON object
+  if (geometry.bbox) {
+    return geometry.bbox.length > 4
+      ? [geometry.bbox[0], geometry.bbox[1], geometry.bbox[3], geometry.bbox[4]]
+      : (geometry.bbox as BoundingBox)
+  }
+
+  const coordinatesReducer = (prev: BoundingBox, coords: Position) =>
+    [
+      Math.min(prev[0], coords[0]),
+      Math.min(prev[1], coords[1]),
+      Math.max(prev[2], coords[0]),
+      Math.max(prev[3], coords[1]),
+    ] as BoundingBox
+  const coordinatesArrayReducer = (
+    prev: BoundingBox,
+    coordsArray: Position[]
+  ) => {
+    const bbox = coordsArray.reduce(coordinatesReducer, emptyExtent)
+    return [
+      Math.min(prev[0], bbox[0]),
+      Math.min(prev[1], bbox[1]),
+      Math.max(prev[2], bbox[2]),
+      Math.max(prev[3], bbox[3]),
+    ] as BoundingBox
+  }
+  const emptyExtent = [Infinity, Infinity, -Infinity, -Infinity] as BoundingBox
+
+  switch (geometry.type) {
+    case 'MultiPolygon':
+      return geometry.coordinates.reduce((prev, polygonCoords) => {
+        const bbox = polygonCoords.reduce(coordinatesArrayReducer, emptyExtent)
+        return [
+          Math.min(prev[0], bbox[0]),
+          Math.min(prev[1], bbox[1]),
+          Math.max(prev[2], bbox[2]),
+          Math.max(prev[3], bbox[3]),
+        ] as BoundingBox
+      }, emptyExtent)
+    case 'GeometryCollection':
+      return geometry.geometries.reduce((prev, geom) => {
+        const bbox = getGeometryBoundingBox(geom)
+        return [
+          Math.min(prev[0], bbox[0]),
+          Math.min(prev[1], bbox[1]),
+          Math.max(prev[2], bbox[2]),
+          Math.max(prev[3], bbox[3]),
+        ] as BoundingBox
+      }, emptyExtent)
+    case 'Polygon':
+    case 'MultiLineString':
+      return geometry.coordinates.reduce(coordinatesArrayReducer, emptyExtent)
+    case 'LineString':
+    case 'MultiPoint':
+      return geometry.coordinates.reduce<BoundingBox>(
+        coordinatesReducer,
+        emptyExtent
+      )
+    case 'Point':
+      return coordinatesReducer(emptyExtent, geometry.coordinates)
+  }
+}

package/src/libs/util/shared/src/lib/utils/index.ts

@@ -4,9 +4,11 @@ export * from './format-fields'
 export * from './fuzzy-filter'
 export * from './geojson'
 export * from './image-resize'
+export * from './mobile-screen'
 export * from './no-duplicate-file-name'
 export * from './parse'
 export * from './remove-whitespace'
+export * from './scroll'
 export * from './sort-by'
 export * from './strip-html'
 export * from './temporal-extent-union'

package/src/libs/util/shared/src/lib/utils/mobile-screen.ts

@@ -0,0 +1,14 @@
+import { fromEvent, startWith, map, shareReplay } from 'rxjs'
+
+/**
+ * This returns true when the screen size is under 768px, which is the mobile threshold.
+ */
+const MOBILE_MAX_WIDTH = 768
+
+export function getIsMobile() {
+  return fromEvent(window, 'resize').pipe(
+    startWith(window.innerWidth),
+    map(() => window.innerWidth < MOBILE_MAX_WIDTH),
+    shareReplay({ bufferSize: 1, refCount: true })
+  )
+}

package/src/libs/util/shared/src/lib/utils/scroll.ts

@@ -0,0 +1,48 @@
+import { ViewportScroller } from '@angular/common'
+import { Event, NavigationEnd, Router, Scroll } from '@angular/router'
+import { filter, pairwise, tap } from 'rxjs'
+
+/**
+ * When route is changed, Angular interprets a simple query params change as "forward navigation" too.
+ * Using the pairwise function allows us to have both the previous and current router events, which we can
+ * use to effectively compare the two navigation events and see if they actually change route, or only
+ * the route parameters (i.e. search params stored in query params).
+ *
+ * Related to: https://github.com/angular/angular/issues/26744
+ */
+export function handleScrollOnNavigation(
+  router: Router,
+  viewportScroller: ViewportScroller
+): void {
+  router.events
+    .pipe(
+      filter((e: Event): e is Scroll => e instanceof Scroll),
+      pairwise()
+    )
+    .subscribe(([previous, current]: [Scroll, Scroll]) => {
+      if (current.position) {
+        // Backward navigation
+        viewportScroller.scrollToPosition(current.position)
+      } else if (current.anchor) {
+        // Anchor navigation
+        viewportScroller.scrollToAnchor(current.anchor)
+      } else {
+        // Check if routes match, or if it is only a query param change
+        if (
+          previous.routerEvent instanceof NavigationEnd &&
+          current.routerEvent instanceof NavigationEnd &&
+          getBaseRoute(previous.routerEvent.urlAfterRedirects) !==
+            getBaseRoute(current.routerEvent.urlAfterRedirects)
+        ) {
+          // Routes don't match, this is actual forward navigation
+          // Default behavior: scroll to top
+          viewportScroller.scrollToPosition([0, 0])
+        }
+      }
+    })
+}
+
+function getBaseRoute(url: string): string {
+  // return url without query params
+  return url.split('?')[0]
+}