geonetwork-ui 2.5.0-dev.da7bc314b → 2.5.0-dev.e060bf76e

package/src/libs/common/domain/src/lib/repository/records-repository.interface.ts

@@ -6,12 +6,15 @@ import {
   SearchParams,
   SearchResults,
 } from '../model/search'
-import { CatalogRecord } from '../model/record'
+import { CatalogRecord, DatasetFeatureCatalog } from '../model/record'
 
 export abstract class RecordsRepositoryInterface {
   abstract search(params: SearchParams): Observable<SearchResults>
   abstract getMatchesCount(filters: FieldFilters): Observable<number>
   abstract getRecord(uniqueIdentifier: string): Observable<CatalogRecord | null>
+  abstract getFeatureCatalog(
+    record: CatalogRecord
+  ): Observable<DatasetFeatureCatalog | null>
   abstract aggregate(params: AggregationsParams): Observable<Aggregations>
   abstract getSimilarRecords(
     similarTo: CatalogRecord

package/src/libs/common/fixtures/src/lib/records.fixtures.ts

@@ -160,6 +160,38 @@ As such, **it is not very interesting at all.**`,
     extras: {
       isPublishedToAll: true,
       edit: true,
+      featureTypes: [
+        {
+          attributeTable: [
+            {
+              code: 'OBJECTID',
+              name: 'OBJECTID',
+              link: '',
+              definition: 'Object identifier',
+              type: 'OID',
+            },
+            {
+              code: 'NOM',
+              name: 'Nom',
+              link: '',
+              definition: 'Nom de la rue',
+              type: 'String (48)',
+            },
+            {
+              code: 'RUE',
+              name: 'Rue',
+              link: '',
+              definition: '',
+              type: 'String (50)',
+            },
+          ],
+          code: '',
+          aliases: '',
+          typeName: "Catalogue d'attributs",
+          definition: '',
+          isAbstract: 'false',
+        },
+      ],
     },
   },
   {
@@ -312,6 +344,49 @@ export const simpleDatasetRecordFixture = (): DatasetRecord => ({
   translations: {},
 })
 
+export const simpleDatasetRecordWithFcatsFixture = (): DatasetRecord => ({
+  uniqueIdentifier: 'my-dataset-with-fcats',
+  extras: {
+    featureCatalogIdentifier: 'feature-catalog-identifier',
+  },
+  kind: 'dataset',
+  otherLanguages: [],
+  defaultLanguage: 'en',
+  recordUpdated: new Date('2022-02-01T14:12:00.000Z'),
+  resourceCreated: new Date('2022-09-01T12:18:19.000Z'),
+  resourceUpdated: new Date('2022-12-04T14:12:00.000Z'),
+  status: 'ongoing',
+  title: 'A very interesting dataset with a related feature catalog',
+  abstract: `This dataset has been established for testing purposes.`,
+  ownerOrganization: { name: 'MyOrganization', translations: {} },
+  contacts: [
+    {
+      email: 'bob@org.net',
+      position: 'developer',
+      organization: { name: 'MyOrganization', translations: {} },
+      role: 'point_of_contact',
+      firstName: 'Bob',
+      lastName: 'TheGreat',
+    },
+  ],
+  contactsForResource: [],
+  keywords: [],
+  topics: ['testData'],
+  licenses: [],
+  legalConstraints: [],
+  securityConstraints: [],
+  otherConstraints: [],
+  lineage:
+    'This record was edited manually to test the feature catalog parsing',
+  spatialRepresentation: 'grid',
+  overviews: [],
+  spatialExtents: [],
+  temporalExtents: [],
+  onlineResources: [],
+  updateFrequency: { per: 'month', updatedTimes: 3 },
+  translations: {},
+})
+
 export const simpleDatasetRecordAsXmlFixture =
   (): string => `<?xml version="1.0" encoding="UTF-8"?>
 <mdb:MD_Metadata xmlns:mdb="http://standards.iso.org/iso/19115/-3/mdb/2.0" xmlns:mcc="http://standards.iso.org/iso/19115/-3/mcc/1.0" xmlns:gco="http://standards.iso.org/iso/19115/-3/gco/1.0" xmlns:cit="http://standards.iso.org/iso/19115/-3/cit/2.0" xmlns:mri="http://standards.iso.org/iso/19115/-3/mri/1.0" xmlns:mco="http://standards.iso.org/iso/19115/-3/mco/1.0" xmlns:gcx="http://standards.iso.org/iso/19115/-3/gcx/1.0" xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:mmi="http://standards.iso.org/iso/19115/-3/mmi/1.0" xmlns:mrd="http://standards.iso.org/iso/19115/-3/mrd/1.0" xmlns:mrl="http://standards.iso.org/iso/19115/-3/mrl/2.0">

package/src/libs/feature/dataviz/src/lib/chart-view/chart-view.component.ts

@@ -2,7 +2,9 @@ import {
   ChangeDetectionStrategy,
   ChangeDetectorRef,
   Component,
+  Inject,
   Input,
+  Optional,
   Output,
 } from '@angular/core'
 import { marker } from '@biesbjerg/ngx-translate-extract-marker'
@@ -65,6 +67,7 @@ marker('chart.aggregation.count')
   standalone: true,
 })
 export class ChartViewComponent {
+  @Input() cacheActive = true
   @Input() set link(value: DatasetOnlineResource) {
     this.currentLink$.next(value)
   }
@@ -134,7 +137,7 @@ export class ChartViewComponent {
     switchMap((link) => {
       this.error = null
       this.loading = true
-      return this.dataService.getDataset(link).pipe(
+      return this.dataService.getDataset(link, this.cacheActive).pipe(
         catchError((error) => {
           this.handleError(error)
           return EMPTY

package/src/libs/feature/dataviz/src/lib/service/data.service.ts

@@ -242,8 +242,11 @@ export class DataService {
     }))
   }
 
-  readAsGeoJson(link: DatasetOnlineResource): Observable<FeatureCollection> {
-    return this.getDataset(link).pipe(
+  readAsGeoJson(
+    link: DatasetOnlineResource,
+    cacheActive: boolean
+  ): Observable<FeatureCollection> {
+    return this.getDataset(link, cacheActive).pipe(
       switchMap((dataset) => dataset.selectAll().read()),
       map((features) => ({
         type: 'FeatureCollection',
@@ -252,13 +255,21 @@ export class DataService {
     )
   }
 
-  getDataset(link: DatasetOnlineResource): Observable<BaseReader> {
+  getDataset(
+    link: DatasetOnlineResource,
+    cacheActive: boolean
+  ): Observable<BaseReader> {
     if (link.type === 'service' && link.accessServiceProtocol === 'wfs') {
       const wfsUrlEndpoint = this.proxy.getProxiedUrl(link.url.toString())
       return from(
-        openDataset(wfsUrlEndpoint, 'wfs', {
-          wfsFeatureType: link.name,
-        })
+        openDataset(
+          wfsUrlEndpoint,
+          'wfs',
+          {
+            wfsFeatureType: link.name,
+          },
+          cacheActive
+        )
       )
     } else if (link.type === 'download') {
       const linkProxifiedUrl = this.proxy.getProxiedUrl(link.url.toString())
@@ -267,7 +278,9 @@ export class DataService {
         SupportedTypes.indexOf(format as any) > -1
           ? (format as SupportedType)
           : undefined
-      return from(openDataset(linkProxifiedUrl, supportedType)).pipe()
+      return from(
+        openDataset(linkProxifiedUrl, supportedType, undefined, cacheActive)
+      ).pipe()
     } else if (
       link.type === 'service' &&
       link.accessServiceProtocol === 'esriRest'
@@ -276,7 +289,7 @@ export class DataService {
         link.url.toString(),
         'geojson'
       )
-      return from(openDataset(url, 'geojson')).pipe()
+      return from(openDataset(url, 'geojson', undefined, cacheActive)).pipe()
     } else if (
       link.type === 'service' &&
       link.accessServiceProtocol === 'ogcFeatures'
@@ -284,7 +297,7 @@ export class DataService {
       return from(this.getDownloadUrlsFromOgcApi(link.url.href)).pipe(
         switchMap((collectionInfo) => {
           const geojsonUrl = collectionInfo.jsonDownloadLink
-          return openDataset(geojsonUrl, 'geojson')
+          return openDataset(geojsonUrl, 'geojson', undefined, cacheActive)
         }),
         tap((url) => {
           if (url === null) {

package/src/libs/feature/dataviz/src/lib/table-view/table-view.component.ts

@@ -33,6 +33,7 @@ import { CommonModule } from '@angular/common'
   standalone: true,
 })
 export class TableViewComponent {
+  @Input() cacheActive = true
   @Input() set link(value: DatasetOnlineResource) {
     this.currentLink$.next(value)
   }
@@ -66,7 +67,7 @@ export class TableViewComponent {
   ) {}
 
   getDatasetReader(link: DatasetOnlineResource): Observable<BaseReader> {
-    return this.dataService.getDataset(link)
+    return this.dataService.getDataset(link, this.cacheActive)
   }
 
   onTableSelect(event) {

package/src/libs/feature/editor/src/lib/components/constraint-card/constraint-card.component.html

@@ -2,7 +2,7 @@
   <gn-ui-form-field-rich
     class="mt-[-8px]"
     [label]="label"
-    [hint]="hint | translate"
+    [placeholder]="hint | translate"
     [value]="constraint_.text ?? ''"
     (valueChange)="handleConstraintTextChange($event)"
   ></gn-ui-form-field-rich>

package/src/libs/feature/editor/src/lib/components/record-form/form-field/form-field-contacts-for-resource/form-field-contacts-for-resource.component.ts

@@ -79,6 +79,7 @@ export class FormFieldContactsForResourceComponent
     'owner',
     'point_of_contact',
     'author',
+    'publisher',
   ]
 
   roleSectionsToDisplay: Role[] = []

package/src/libs/feature/editor/src/lib/components/record-form/form-field/form-field-rich/form-field-rich.component.ts

@@ -28,7 +28,7 @@ import { FormFieldWrapperComponent } from '../../../../../../../../../libs/ui/la
 export class FormFieldRichComponent {
   @Input() label: string
   @Input() hint: string
-  @Input() placeholder = 'Votre texte ici' //TODO: translate
+  @Input() placeholder = ''
   @Input() value: string
 
   @Output() valueChange: EventEmitter<string> = new EventEmitter()

package/src/libs/feature/editor/src/lib/components/wizard-field/wizard-field.component.html

@@ -16,19 +16,6 @@
         placeholder=""
       ></gn-ui-text-input>
     </div>
-    <div
-      class="flex-1 w-11/12"
-      *ngIf="wizardFieldType.CHIPS === wizardFieldConfig.type"
-    >
-      <gn-ui-chips-input
-        #chips
-        [selectedItems]="wizardFieldData"
-        placeholder=""
-        [id]="wizardFieldConfig.id"
-        [url]="wizardFieldConfig.options.url"
-        [loadOnce]="wizardFieldConfig.options.loadOnce"
-      ></gn-ui-chips-input>
-    </div>
     <div
       class="h-32 w-11/12"
       *ngIf="wizardFieldType.TEXT_AREA === wizardFieldConfig.type"

package/src/libs/feature/editor/src/lib/components/wizard-field/wizard-field.component.ts

@@ -23,7 +23,6 @@ import { WizardFieldModel } from '../../models/wizard-field.model'
 import { WizardFieldType } from '../../models/wizard-field.type'
 import { WizardService } from '../../services/wizard.service'
 import {
-  ChipsInputComponent,
   DropdownSelectorComponent,
   TextAreaComponent,
   TextInputComponent,
@@ -74,7 +73,6 @@ export class WizardFieldComponent implements AfterViewInit, OnDestroy {
   @Input() wizardFieldConfig: WizardFieldModel
 
   @ViewChild('searchText') searchText: TextInputComponent
-  @ViewChild('chips') chips: ChipsInputComponent
   @ViewChild('textArea') textArea: TextAreaComponent
   @ViewChild('dropdown') dropdown: DropdownSelectorComponent
 
@@ -96,9 +94,6 @@ export class WizardFieldComponent implements AfterViewInit, OnDestroy {
       case WizardFieldType.TEXT: {
         return data || ''
       }
-      case WizardFieldType.CHIPS: {
-        return data ? JSON.parse(data) : []
-      }
       case WizardFieldType.TEXT_AREA: {
         return data || ''
       }
@@ -131,10 +126,6 @@ export class WizardFieldComponent implements AfterViewInit, OnDestroy {
         this.initializeTextInputListener()
         break
       }
-      case WizardFieldType.CHIPS: {
-        this.initializeChipsListener()
-        break
-      }
       case WizardFieldType.TEXT_AREA: {
         this.initializeTextAreaListener()
         return
@@ -161,17 +152,6 @@ export class WizardFieldComponent implements AfterViewInit, OnDestroy {
     )
   }
 
-  private initializeChipsListener() {
-    this.subs.add(
-      this.chips.itemsChange.subscribe((items) => {
-        this.wizardService.onWizardWizardFieldDataChanged(
-          this.wizardFieldConfig.id,
-          JSON.stringify(items)
-        )
-      })
-    )
-  }
-
   private initializeTextAreaListener() {
     this.subs.add(
       this.textArea.valueChange.subscribe((value) => {

package/src/libs/feature/editor/src/lib/fields.config.ts

@@ -275,8 +275,8 @@ export const DEFAULT_CONFIGURATION: EditorConfig = {
       labelKey: marker('editor.record.form.page.description'),
       sections: [
         TITLE_SECTION,
-        ABOUT_SECTION,
         CLASSIFICATION_SECTION,
+        ABOUT_SECTION,
         GEOGRAPHICAL_COVERAGE_SECTION,
       ],
     },

package/src/libs/feature/editor/src/lib/models/wizard-field.type.ts

@@ -1,7 +1,6 @@
 export enum WizardFieldType {
   TEXT,
   TEXT_AREA,
-  CHIPS,
   DATA_PICKER,
   DROPDOWN,
 }

package/src/libs/feature/record/src/lib/data-view/data-view.component.html

@@ -8,15 +8,28 @@
     [choices]="choices"
     (selectValue)="selectLink($event)"
   ></gn-ui-dropdown-selector>
-  <div class="relative h-[460px]">
-    <gn-ui-table-view
-      *ngIf="mode === 'table'"
-      [link]="selectedLink$ | async"
-    ></gn-ui-table-view>
-    <gn-ui-chart-view
-      *ngIf="mode === 'chart'"
-      (chartConfig$)="setChartConfig($event)"
-      [link]="selectedLink$ | async"
-    ></gn-ui-chart-view>
-  </div>
+  <ng-container *ngIf="hidePreview; else dataView">
+    <gn-ui-popup-alert
+      type="warning"
+      icon="matErrorOutlineOutline"
+      class="block h-12 p-1"
+    >
+      <span translate>record.feature.limit</span>
+    </gn-ui-popup-alert>
+  </ng-container>
+  <ng-template #dataView>
+    <div class="relative h-[460px]">
+      <gn-ui-table-view
+        *ngIf="mode === 'table'"
+        [cacheActive]="cacheActive$ | async"
+        [link]="selectedLink$ | async"
+      ></gn-ui-table-view>
+      <gn-ui-chart-view
+        *ngIf="mode === 'chart'"
+        (chartConfig$)="setChartConfig($event)"
+        [cacheActive]="cacheActive$ | async"
+        [link]="selectedLink$ | async"
+      ></gn-ui-chart-view>
+    </div>
+  </ng-template>
 </div>

package/src/libs/feature/record/src/lib/data-view/data-view.component.ts

@@ -5,8 +5,8 @@ import {
   Output,
 } from '@angular/core'
 import { getLinkLabel, getLinkPriority } from '../../../../../../libs/util/shared/src'
-import { BehaviorSubject, combineLatest } from 'rxjs'
-import { map, tap } from 'rxjs/operators'
+import { BehaviorSubject, combineLatest, of } from 'rxjs'
+import { catchError, finalize, map, switchMap, tap } from 'rxjs/operators'
 import { MdViewFacade } from '../state'
 import { DatavizConfigurationModel } from '../../../../../../libs/common/domain/src/lib/model/dataviz/dataviz-configuration.model'
 import { DatasetOnlineResource } from '../../../../../../libs/common/domain/src/lib/model/record'
@@ -17,6 +17,7 @@ import {
 } from '../../../../../../libs/feature/dataviz/src'
 import { CommonModule } from '@angular/common'
 import { TranslateModule } from '@ngx-translate/core'
+import { PopupAlertComponent } from '../../../../../../libs/ui/widgets/src'
 
 @Component({
   selector: 'gn-ui-data-view',
@@ -30,12 +31,21 @@ import { TranslateModule } from '@ngx-translate/core'
     TableViewComponent,
     TranslateModule,
     ChartViewComponent,
+    PopupAlertComponent,
   ],
 })
 export class DataViewComponent {
   @Input() mode: 'table' | 'chart'
   @Input() displaySource = true
+  @Input() set exceedsLimit(value: boolean) {
+    this.excludeWfs$.next(value)
+  }
   @Output() chartConfig$ = new BehaviorSubject<DatavizConfigurationModel>(null)
+  cacheActive$ = this.mdViewFacade.isHighUpdateFrequency$.pipe(
+    map((highF) => !highF)
+  )
+  hidePreview = false
+  excludeWfs$ = new BehaviorSubject(false)
   compatibleDataLinks$ = combineLatest([
     this.mdViewFacade.dataLinks$,
     this.mdViewFacade.geoDataLinks$,
@@ -49,7 +59,7 @@ export class DataViewComponent {
   dropdownChoices$ = this.compatibleDataLinks$.pipe(
     tap((links) => {
       if (links.indexOf(this.selectedLink$.value) === -1) {
-        this.selectedLink$.next(links[0])
+        this.selectLink(JSON.stringify(links[0]))
       }
     }),
     map((links) =>
@@ -70,6 +80,18 @@ export class DataViewComponent {
   selectLink(linkAsString: string) {
     const link: DatasetOnlineResource = JSON.parse(linkAsString)
     link.url = new URL(link.url)
-    this.selectedLink$.next(link)
+    this.excludeWfs$
+      .pipe(
+        tap((excludeWfs) => {
+          this.hidePreview =
+            link['accessServiceProtocol'] === 'wfs' &&
+            excludeWfs &&
+            this.mode === 'chart'
+              ? true
+              : false
+          this.selectedLink$.next(link)
+        })
+      )
+      .subscribe()
   }
 }

package/src/libs/feature/record/src/lib/map-view/map-view.component.html

@@ -14,80 +14,91 @@
     >
     </gn-ui-external-viewer-button>
   </div>
-  <div
-    class="relative w-full h-full bg-white border border-gray-300 rounded-lg overflow-hidden"
-  >
-    <gn-ui-map-container
-      #mapContainer
-      [context]="mapContext$ | async"
-      (featuresClick)="onMapFeatureSelect($event)"
-    ></gn-ui-map-container>
-    <div
-      class="top-[1em] right-[1em] p-3 bg-white absolute overflow-y-auto overflow-x-hidden max-h-72 w-56"
-      [class.hidden]="!selection"
+  <ng-container *ngIf="hidePreview; else mapView">
+    <gn-ui-popup-alert
+      type="warning"
+      icon="matErrorOutlineOutline"
+      class="block h-12 p-1"
     >
-      <gn-ui-button
-        type="light"
-        (buttonClick)="resetSelection()"
-        style="
-          --gn-ui-button-padding: 0px;
-          --gn-ui-button-width: 24px;
-          --gn-ui-button-height: 24px;
-        "
-        extraClass="absolute right-[0.5em] ml-[8px] mr-[10px]"
-      >
-        <ng-icon name="matClose" class="align-middle text-sm"></ng-icon>
-      </gn-ui-button>
-      <gn-ui-feature-detail [feature]="selection"></gn-ui-feature-detail>
-    </div>
-
+      <span translate>record.feature.limit</span>
+    </gn-ui-popup-alert>
+  </ng-container>
+  <ng-template #mapView>
     <div
-      class="top-[1em] p-3 bg-white absolute overflow-y-auto overflow-x-hidden max-h-72 w-56"
-      [ngClass]="{ 'right-[1em]': !selection, 'right-[16em]': selection }"
-      [hidden]="!showLegend || !legendExists"
+      class="relative w-full h-full bg-white border border-gray-300 rounded-lg overflow-hidden"
     >
-      <div class="flex justify-between items-center mb-2">
-        <div class="text-primary font-bold">Legend</div>
+      <gn-ui-map-container
+        #mapContainer
+        [context]="mapContext$ | async"
+        (featuresClick)="onMapFeatureSelect($event)"
+      ></gn-ui-map-container>
+      <div
+        class="top-[1em] right-[1em] p-3 bg-white absolute overflow-y-auto overflow-x-hidden max-h-72 w-56"
+        [class.hidden]="!selection"
+      >
         <gn-ui-button
           type="light"
-          (buttonClick)="toggleLegend()"
+          (buttonClick)="resetSelection()"
           style="
             --gn-ui-button-padding: 0px;
             --gn-ui-button-width: 24px;
             --gn-ui-button-height: 24px;
           "
-          extraClass="ml-[8px] mr-[10px]"
+          extraClass="absolute right-[0.5em] ml-[8px] mr-[10px]"
         >
           <ng-icon name="matClose" class="align-middle text-sm"></ng-icon>
         </gn-ui-button>
+        <gn-ui-feature-detail [feature]="selection"></gn-ui-feature-detail>
       </div>
-      <gn-ui-map-legend
-        [context]="mapContext$ | async"
-        (legendStatusChange)="onLegendStatusChange($event)"
-      ></gn-ui-map-legend>
-    </div>
 
-    <gn-ui-button
-      *ngIf="!showLegend && legendExists && !selection"
-      type="outline"
-      (buttonClick)="toggleLegend()"
-      extraClass="absolute top-[1em] right-[1em] rounded p-1 text-xs bg-white"
-    >
-      Legend
-    </gn-ui-button>
+      <div
+        class="top-[1em] p-3 bg-white absolute overflow-y-auto overflow-x-hidden max-h-72 w-56"
+        [ngClass]="{ 'right-[1em]': !selection, 'right-[16em]': selection }"
+        [hidden]="!showLegend || !legendExists"
+      >
+        <div class="flex justify-between items-center mb-2">
+          <div class="text-primary font-bold">Legend</div>
+          <gn-ui-button
+            type="light"
+            (buttonClick)="toggleLegend()"
+            style="
+              --gn-ui-button-padding: 0px;
+              --gn-ui-button-width: 24px;
+              --gn-ui-button-height: 24px;
+            "
+            extraClass="ml-[8px] mr-[10px]"
+          >
+            <ng-icon name="matClose" class="align-middle text-sm"></ng-icon>
+          </gn-ui-button>
+        </div>
+        <gn-ui-map-legend
+          [context]="mapContext$ | async"
+          (legendStatusChange)="onLegendStatusChange($event)"
+        ></gn-ui-map-legend>
+      </div>
 
-    <gn-ui-loading-mask
-      *ngIf="loading"
-      class="absolute inset-0"
-      [message]="'map.loading.data' | translate"
-    ></gn-ui-loading-mask>
-    <gn-ui-popup-alert
-      *ngIf="error"
-      type="warning"
-      icon="matErrorOutlineOutline"
-      class="absolute m-2 inset-0"
-    >
-      <span translate>{{ error }}</span>
-    </gn-ui-popup-alert>
-  </div>
+      <gn-ui-button
+        *ngIf="!showLegend && legendExists && !selection"
+        type="outline"
+        (buttonClick)="toggleLegend()"
+        extraClass="absolute top-[1em] right-[1em] rounded p-1 text-xs bg-white"
+      >
+        Legend
+      </gn-ui-button>
+
+      <gn-ui-loading-mask
+        *ngIf="loading"
+        class="absolute inset-0"
+        [message]="'map.loading.data' | translate"
+      ></gn-ui-loading-mask>
+      <gn-ui-popup-alert
+        *ngIf="error"
+        type="warning"
+        icon="matErrorOutlineOutline"
+        class="absolute m-2 inset-0"
+      >
+        <span translate>{{ error }}</span>
+      </gn-ui-popup-alert>
+    </div>
+  </ng-template>
 </div>

package/src/libs/feature/record/src/lib/map-view/map-view.component.ts

@@ -82,13 +82,14 @@ marker('wfs.feature.limit')
   viewProviders: [provideIcons({ matClose })],
 })
 export class MapViewComponent implements AfterViewInit {
-  @Input() set excludeWfs(value: boolean) {
+  @Input() set exceedsLimit(value: boolean) {
     this.excludeWfs$.next(value)
   }
   @Input() displaySource = true
   @ViewChild('mapContainer') mapContainer: MapContainerComponent
 
   excludeWfs$ = new BehaviorSubject(false)
+  hidePreview = false
   selection: Feature
   showLegend = true
   legendExists = false
@@ -139,9 +140,10 @@ export class MapViewComponent implements AfterViewInit {
         return of([])
       }
       if (excludeWfs && link.accessServiceProtocol === 'wfs') {
-        this.error = 'wfs.feature.limit'
+        this.hidePreview = true
         return of([])
       }
+      this.hidePreview = false
       this.loading = true
       this.error = null
       return this.getLayerFromLink(link).pipe(
@@ -239,7 +241,8 @@ export class MapViewComponent implements AfterViewInit {
           link.accessServiceProtocol === 'ogcFeatures')) ||
       link.type === 'download'
     ) {
-      return this.dataService.readAsGeoJson(link).pipe(
+      const cacheActive = true // TODO implement whether should be true or false
+      return this.dataService.readAsGeoJson(link, cacheActive).pipe(
         map((data) => ({
           type: 'geojson',
           data,