fix: ids in GeoJSON

M-Media-Group · Dec 29, 2023 · 65c03ed · 65c03ed
1 parent 035f282
commit 65c03ed
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/app/Http/Resources/MarkerGeoJsonResource.php b/app/Http/Resources/MarkerGeoJsonResource.php
@@ -26,7 +26,8 @@ public function toArray($request)
         $skipProps = [
             'x',
             'y',
-            'location'
+            'location',
+            'id'
         ];
 
         // All the rest of the properties available on the model, except the ones we want to skip, should be in an array. For this, we need to get the object as Laravel would provide it, which would hide all the properties we want to skip, as well as all those in $hidden.
@@ -44,6 +45,7 @@ public function toArray($request)
                     $this->y,
                 ],
             ],
+            'id' => $this->id,
             'properties' => [
                 ...$properties
             ],

diff --git a/tests/Unit/MarkerTest.php b/tests/Unit/MarkerTest.php
@@ -97,8 +97,8 @@ public function testSeeAllMapMarkersInGeoJsonFormat()
                         'type',
                         'coordinates',
                     ],
+                    'id',
                     'properties' => [
-                        'id',
                         'description',
                         'link',
                         'elevation',