From d483c41832d122636ed7ab289c7498916bab3749 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=80=9CVeLiTi=E2=80=9D?= <“info@veliti.nl”>
Date: Thu, 4 Apr 2024 11:08:50 +0200
Subject: [PATCH] CIM37 - Improved error handling geolocation

---
 equipment.php        | 46 +++++++++++++++++++++++---------------------
 equipment_manage.php |  8 ++++++--
 2 files changed, 30 insertions(+), 24 deletions(-)

diff --git a/equipment.php b/equipment.php
index 470c60c..69801fb 100644
--- a/equipment.php
+++ b/equipment.php
@@ -221,31 +221,33 @@ $view .= '<div class="content-block">
 if (!empty($responses->geolocation) || $responses->geolocation != ''){
 
     $geolocation = json_decode($responses->geolocation,true);
-    $view .= '<div class="content-block">
-                <div id="map" style="height:300px;z-index: 0;">
-                </div>
-                <script>
-                // initialize Leaflet
-                var map = L.map(\'map\').setView({lon: '.$geolocation[1].', lat: '.$geolocation[0].'}, 10);
+    if ($geolocation[1] != ''){
+        $view .= '<div class="content-block">
+                    <div id="map" style="height:300px;z-index: 0;">
+                    </div>
+                    <script>
+                    // initialize Leaflet
+                    var map = L.map(\'map\').setView({lon: '.$geolocation[1].', lat: '.$geolocation[0].'}, 10);
 
-                //Add TSS Avatar to MAP
-                var TSSemp = L.icon({
-                    iconUrl: \'./assets/images/EP.png\',
-                    iconSize:     [50, 50], // size of the icon
-                });
-                L.marker(['.$geolocation[0].','.$geolocation[1].'], {icon: TSSemp}).addTo(map);
+                    //Add TSS Avatar to MAP
+                    var TSSemp = L.icon({
+                        iconUrl: \'./assets/images/EP.png\',
+                        iconSize:     [50, 50], // size of the icon
+                    });
+                    L.marker(['.$geolocation[0].','.$geolocation[1].'], {icon: TSSemp}).addTo(map);
 
-                // add the OpenStreetMap tiles
-                L.tileLayer(\'https://tile.openstreetmap.org/{z}/{x}/{y}.png\', {
-                    maxZoom: 19,
-                    attribution: \'&copy; <a href="https://openstreetmap.org/copyright">OpenStreetMap contributors</a>\'
-                }).addTo(map);
+                    // add the OpenStreetMap tiles
+                    L.tileLayer(\'https://tile.openstreetmap.org/{z}/{x}/{y}.png\', {
+                        maxZoom: 19,
+                        attribution: \'&copy; <a href="https://openstreetmap.org/copyright">OpenStreetMap contributors</a>\'
+                    }).addTo(map);
 
-                // show the scale bar on the lower left corner
-                L.control.scale({imperial: true, metric: true}).addTo(map);
-                </script>
-    </div>
-    ';
+                    // show the scale bar on the lower left corner
+                    L.control.scale({imperial: true, metric: true}).addTo(map);
+                    </script>
+        </div>
+        ';
+    }
 }
 
 //Get all related service events
diff --git a/equipment_manage.php b/equipment_manage.php
index 6576273..6fedefd 100644
--- a/equipment_manage.php
+++ b/equipment_manage.php
@@ -30,7 +30,7 @@ $equipment = [
     'service_date' => $date,
     'warranty_date' => $date,
     'order_ref' => '',
-    'geolocation' => ''
+    'geolocation' => []
 ];
 
 $equipment_ID = $_GET['equipmentID'] ?? '';
@@ -65,6 +65,8 @@ if (isset($_GET['equipmentID'])) {
     
     if ($update_allowed === 1 || $equipment_owner === 1 || $update_allowed_special === 1){
         if (isset($_POST['submit'])) {
+            //GEOLOCATION ARRAY convert to JSON
+            $_POST['geolocation'] = json_encode($_POST['geolocation'],JSON_UNESCAPED_UNICODE);
             //GET ALL POST DATA
             $data = json_encode($_POST, JSON_UNESCAPED_UNICODE);
             //Secure data
@@ -101,6 +103,8 @@ if (isset($_GET['equipmentID'])) {
 } else {
     // Create a new equipment
     if (isset($_POST['submit']) && $create_allowed === 1) {
+        //GEOLOCATION ARRAY convert to JSON
+        $_POST['geolocation'] = json_encode($_POST['geolocation'],JSON_UNESCAPED_UNICODE);
         //GET ALL POST DATA
         $data = json_encode($_POST, JSON_UNESCAPED_UNICODE);
         //Secure data
@@ -217,7 +221,7 @@ $view .= '
 $changelog = showlog('equipment',$equipment_ID)?? '';
 
 //GEOLOCATION
-$geo_details = $equipment['geolocation'] ?? '';
+$geo_details = json_decode($equipment['geolocation']) ?? '';
 if (is_array($geo_details)) {                      
     $geodetails_lat = $geo_details[0];
     $geodetails_lon = $geo_details[1];