Merge pull request #409 from geonetwork/intersect-geom-boost

Datahub: Boost results with intersecting geom
geonetwork · Jan 16, 2023 · 93744ce · 93744ce
2 parents 885fc96 + 2c8ce9b
commit 93744ce
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 25 deletions.
diff --git a/libs/util/shared/src/lib/elasticsearch/elasticsearch.service.spec.ts b/libs/util/shared/src/lib/elasticsearch/elasticsearch.service.spec.ts
@@ -211,7 +211,7 @@ describe('ElasticsearchService', () => {
           type: 'Polygon',
         }
       })
-      it('adds a criteria for intersecting with it and boosting on geoms within', () => {
+      it('adds boosting of 7 for intersecting with it and boosting of 10 on geoms within', () => {
         const query = service['buildPayloadQuery'](
           {
             Org: {
@@ -225,16 +225,7 @@ describe('ElasticsearchService', () => {
         )
         expect(query).toEqual({
           bool: {
-            filter: [
-              {
-                geo_shape: {
-                  geom: {
-                    shape: geojsonPolygon,
-                    relation: 'intersects',
-                  },
-                },
-              },
-            ],
+            filter: [],
             must: [
               {
                 terms: {
@@ -281,6 +272,15 @@ describe('ElasticsearchService', () => {
                   boost: 10.0,
                 },
               },
+              {
+                geo_shape: {
+                  geom: {
+                    shape: geojsonPolygon,
+                    relation: 'intersects',
+                  },
+                  boost: 7.0,
+                },
+              },
             ],
           },
         })

diff --git a/libs/util/shared/src/lib/elasticsearch/elasticsearch.service.ts b/libs/util/shared/src/lib/elasticsearch/elasticsearch.service.ts
@@ -165,23 +165,26 @@ export class ElasticsearchService {
       })
     }
     if (geometry) {
-      should.push({
-        geo_shape: {
-          geom: {
-            shape: geometry,
-            relation: 'within',
+      should.push(
+        {
+          geo_shape: {
+            geom: {
+              shape: geometry,
+              relation: 'within',
+            },
+            boost: 10.0,
           },
-          boost: 10.0,
         },
-      })
-      filter.push({
-        geo_shape: {
-          geom: {
-            shape: geometry,
-            relation: 'intersects',
+        {
+          geo_shape: {
+            geom: {
+              shape: geometry,
+              relation: 'intersects',
+            },
+            boost: 7.0,
           },
-        },
-      })
+        }
+      )
     }
 
     return {