fix: keep the user in the service catalog

Signed-off-by: Thierry Bugier <tbugier@teclib.com>
pluginsGLPI · Apr 21, 2020 · 7f9451a · 7f9451a
1 parent 13ee703
commit 7f9451a
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 14 deletions.
diff --git a/inc/wizard.class.php b/inc/wizard.class.php
@@ -105,18 +105,11 @@ public static function header($title) {
       echo '</a></li>';
 
       if (Session::haveRight("reservation", ReservationItem::RESERVEANITEM)) {
-         $found_available_res = $DB->request([
-            'COUNT' => 'cpt',
-            'FROM'  => ReservationItem::getTable(),
-            'WHERE' => getEntitiesRestrictCriteria(ReservationItem::getTable(), 'entities_id'),
-         ])->next();
-         if ($found_available_res['cpt'] > 0) {
-            echo '<li class="' . ($activeMenuItem == self::MENU_RESERVATIONS ? 'plugin_formcreator_selectedMenuItem' : '') . '">';
-            echo '<a href="' . $CFG_GLPI["root_doc"].'/plugins/formcreator/front/reservationitem.php' . '">';
-            echo '<span class="fa fa-calendar-check fa-calendar-check-o fc_list_icon" title="'.__('Book an asset', 'formcreator').'"></span>';
-            echo '<span class="label">'.__('Book an asset', 'formcreator').'</span>';
-            echo '</a></li>';
-         }
+         echo '<li class="' . ($activeMenuItem == self::MENU_RESERVATIONS ? 'plugin_formcreator_selectedMenuItem' : '') . '">';
+         echo '<a href="' . $CFG_GLPI["root_doc"].'/plugins/formcreator/front/reservationitem.php' . '">';
+         echo '<span class="fa fa-calendar-check fa-calendar-check-o fc_list_icon" title="'.__('Book an asset', 'formcreator').'"></span>';
+         echo '<span class="label">'.__('Book an asset', 'formcreator').'</span>';
+         echo '</a></li>';
       }
 
       if (RSSFeed::canView()) {

diff --git a/setup.php b/setup.php
@@ -171,8 +171,8 @@ function plugin_init_formcreator() {
                }
             }
          }
-         if (strpos($_SERVER['REQUEST_URI'], "front/ticket.form.php") !== false) {
-            if (plugin_formcreator_replaceHelpdesk()) {
+         if (plugin_formcreator_replaceHelpdesk()) {
+            if (strpos($_SERVER['REQUEST_URI'], "front/ticket.form.php") !== false) {
                if (!isset($_POST['update'])) {
                   $decodedUrl = [];
                   $forceTab = '';
@@ -183,6 +183,18 @@ function plugin_init_formcreator() {
                   Html::redirect($CFG_GLPI["root_doc"] . '/plugins/formcreator/front/issue.form.php?id=' . $_GET['id'] . '&sub_itemtype=Ticket' . $forceTab);
                }
             }
+
+            $pages = [
+               'front/reservationitem.php' => 'plugins/formcreator/front/reservationitem.php',
+               'front/helpdesk.faq.php' => 'plugins/formcreator/front/wizard.php',
+               'front/ticket.php' => 'plugins/formcreator/front/issue.php',
+            ];
+            foreach ($pages as $srcPage => $dstPage) {
+               if (strpos($_SERVER['REQUEST_URI'], $srcPage) !== false && strpos($_SERVER['REQUEST_URI'], $dstPage) === false) {
+                  Html::redirect($CFG_GLPI["root_doc"] . '/' . $dstPage);
+                  break;
+               }
+            }
          }
 
          // Massive Action definition