From bb00445ae2870c5da8e8127aea1dec37aee6152a Mon Sep 17 00:00:00 2001
From: Mike Koch <mkoch227@gmail.com>
Date: Sun, 26 Jul 2015 23:45:50 -0400
Subject: [PATCH] #209 Build modals at end to prevent nested forms, provide
 suggestions from language file

---
 admin/manage_statuses.php | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/admin/manage_statuses.php b/admin/manage_statuses.php
index 36a5f168..1337ef2a 100644
--- a/admin/manage_statuses.php
+++ b/admin/manage_statuses.php
@@ -89,6 +89,10 @@ require_once(HESK_PATH . 'inc/show_admin_nav.inc.php');
                 $openStatusesSql = 'SELECT `ID`, `IsNewTicketStatus`, `IsStaffReopenedStatus`, `IsDefaultStaffReplyStatus` FROM
                     `'.hesk_dbEscape($hesk_settings['db_pfix']).'statuses` WHERE `IsClosed` = 0';
                 $statusesRS = hesk_dbQuery($statusesSql);
+                $statuses = array();
+                while ($row = hesk_dbFetchAssoc($statusesRS)) {
+                    array_push($statuses, $row);
+                }
                 ?>
                 <form class="form-horizontal" method="post" action="manage_statuses.php" role="form">
                     <div class="panel panel-default">
@@ -117,7 +121,7 @@ require_once(HESK_PATH . 'inc/show_admin_nav.inc.php');
                             <tbody>
                             <?php
                             $j = 1;
-                            while ($row = hesk_dbFetchAssoc($statusesRS)):
+                            foreach ($statuses as $row):
                             ?>
                                 <tr id="s<?php echo $row['ID']; ?>_row">
                                     <td style="color: <?php echo $row['TextColor']; ?>; font-weight: bold">
@@ -156,10 +160,8 @@ require_once(HESK_PATH . 'inc/show_admin_nav.inc.php');
                                     </td>
                                 </tr>
                             <?php
-                                buildEditModal($row['ID']);
-                                buildConfirmDeleteModal($row['ID']);
                                 $j++;
-                                endwhile; ?>
+                                endforeach; ?>
                             </tbody>
                         </table>
                     </div>
@@ -301,6 +303,10 @@ require_once(HESK_PATH . 'inc/show_admin_nav.inc.php');
 </div>
 
 <?php
+foreach ($statuses as $status) {
+    buildEditModal($status['ID']);
+    buildConfirmDeleteModal($status['ID']);
+}
 buildCreateModal();
 
 require_once(HESK_PATH . 'inc/footer.inc.php');
@@ -477,7 +483,9 @@ function buildEditModal($statusId) {
                                     if (isset($textArray[$language])) {
                                         $text = $textArray[$language];
                                     } else {
+                                        hesk_setLanguage($language);
                                         $text = $hesklang[$status['Key']];
+                                        hesk_resetLanguage();
                                         $warning = 'has-warning';
                                     }
                                 ?>