From 8013474e75edc92a8181ff690fe0def9e0b69938 Mon Sep 17 00:00:00 2001
From: stevenA <steve.akerman@aboukir.com>
Date: Sun, 9 Jan 2022 13:05:35 +0100
Subject: [PATCH] Fix issue where selecting new device in rnav caused exception

Correct window.location redirect syntax
---
 module.xml              | 3 ++-
 views/hardware.rnav.php | 8 ++++++--
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/module.xml b/module.xml
index a223b5f..5b42a62 100644
--- a/module.xml
+++ b/module.xml
@@ -1,7 +1,7 @@
 <module>
 	<rawname>sccp_manager</rawname>
 	<name>SCCP Manager</name>
-	<version>14.3.0.27</version>
+	<version>14.3.0.28</version>
 	<type>setup</type>
 	<category>SCCP Connectivity</category>
 	<publisher>Steve Lad, Alex GP</publisher>
@@ -33,6 +33,7 @@
 		 * Version 14.3.0.25 * - Fix Issue 55
 		 * Version 14.3.0.26 * - Fix Issue 54
 		 * Version 14.3.0.27 * - Standardise treatment of SIP and SCCP hardware.
+		 * Version 14.3.0.28 * - Fix issue with RNav which caused exception with new device
 	</changelog>
 	<location>https://github.com/chan-sccp/sccp_manager</location>
 	<supported>
diff --git a/views/hardware.rnav.php b/views/hardware.rnav.php
index 89914b8..d30776f 100644
--- a/views/hardware.rnav.php
+++ b/views/hardware.rnav.php
@@ -17,13 +17,17 @@
     class="table">
     <thead>
         <tr>
-              <th data-sortable="true" data-field="mac"><?php echo _('SEP ID')?></th>
+              <th data-sortable="true" data-field="name"><?php echo _('SEP ID')?></th>
               <th data-sortable="true" data-field="description"><?php echo _('Descriptions')?></th>
         </tr>
     </thead>
 </table>
 <script type="text/javascript">
     $("#sccpnavgrid").on('click-row.bs.table',function(e,row,elem){
-        window.location = '?display=sccp_phone&tech_hardware=cisco&id='+row['mac'];
+        if (row['new_hw'] == 'Y' ) {
+            window.location = '?display=sccp_phone&tech_hardware=cisco&new_id=' +row['name'] +'&' +row['type'];
+        } else {
+            window.location = '?display=sccp_phone&tech_hardware=cisco&id='+row['name'];
+        };
     })
 </script>