From d19b555594f9f72999297ec69eaafe646ae4e401 Mon Sep 17 00:00:00 2001
From: Ronit Jadhav <ronitjadhav17@gmail.com>
Date: Tue, 23 Jan 2024 16:15:41 +0100
Subject: [PATCH] resolved PR comments

---
 .../add-layer-from-wfs.component.ts                  |  1 -
 .../map/src/lib/map-context/map-context.model.ts     |  1 -
 .../map/src/lib/map-context/map-context.service.ts   | 12 ++++++------
 3 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/libs/feature/map/src/lib/add-layer-from-wfs/add-layer-from-wfs.component.ts b/libs/feature/map/src/lib/add-layer-from-wfs/add-layer-from-wfs.component.ts
index 4967a68f8b..a3ec798089 100644
--- a/libs/feature/map/src/lib/add-layer-from-wfs/add-layer-from-wfs.component.ts
+++ b/libs/feature/map/src/lib/add-layer-from-wfs/add-layer-from-wfs.component.ts
@@ -58,7 +58,6 @@ export class AddLayerFromWfsComponent implements OnInit {
       name: layer.name,
       url: this.wfsUrl.toString(),
       type: MapContextLayerTypeEnum.WFS,
-      featureCount: 10000,
     }
     this.mapFacade.addLayer({ ...layerToAdd, title: layer.title })
   }
diff --git a/libs/feature/map/src/lib/map-context/map-context.model.ts b/libs/feature/map/src/lib/map-context/map-context.model.ts
index 70c2f3189b..e005842a20 100644
--- a/libs/feature/map/src/lib/map-context/map-context.model.ts
+++ b/libs/feature/map/src/lib/map-context/map-context.model.ts
@@ -32,7 +32,6 @@ interface MapContextLayerWfsModel {
   type: 'wfs'
   url: string
   name: string
-  featureCount?: number
 }
 
 interface LayerXyzModel {
diff --git a/libs/feature/map/src/lib/map-context/map-context.service.ts b/libs/feature/map/src/lib/map-context/map-context.service.ts
index c428921a54..8f8c316c31 100644
--- a/libs/feature/map/src/lib/map-context/map-context.service.ts
+++ b/libs/feature/map/src/lib/map-context/map-context.service.ts
@@ -40,6 +40,8 @@ export const DEFAULT_VIEW: MapContextViewModel = {
   zoom: 2,
 }
 
+export const WFS_MAX_FEATURES = 10000
+
 @Injectable({
   providedIn: 'root',
 })
@@ -111,12 +113,10 @@ export class MapContextService {
               urlObj.searchParams.set('typename', layerModel.name)
               urlObj.searchParams.set('srsname', 'EPSG:3857')
               urlObj.searchParams.set('bbox', `${extent.join(',')},EPSG:3857`)
-              if ('featureCount' in layerModel && layerModel.featureCount) {
-                urlObj.searchParams.set(
-                  'maxFeatures',
-                  layerModel.featureCount.toString()
-                )
-              }
+              urlObj.searchParams.set(
+                'maxFeatures',
+                WFS_MAX_FEATURES.toString()
+              )
               return urlObj.toString()
             },
             strategy: bboxStrategy,