From 41f1ac4a907d05f18b29fc1e383e13dc4fd91a17 Mon Sep 17 00:00:00 2001
From: Yonas Habteab <yonas.habteab@icinga.com>
Date: Wed, 6 Apr 2022 19:47:22 +0200
Subject: [PATCH] Use Menu class instead of HomeMenu

---
 application/controllers/LayoutController.php  |  4 +-
 .../forms/Dashboard/BaseDashboardForm.php     |  6 ++-
 application/layouts/scripts/layout.phtml      |  2 +-
 .../layouts/scripts/parts/navigation.phtml    |  4 +-
 application/views/scripts/layout/menu.phtml   |  3 +-
 library/Icinga/Web/Menu.php                   | 46 +++++++++++++++++++
 6 files changed, 58 insertions(+), 7 deletions(-)

diff --git a/application/controllers/LayoutController.php b/application/controllers/LayoutController.php
index 217c275d7..237681cfa 100644
--- a/application/controllers/LayoutController.php
+++ b/application/controllers/LayoutController.php
@@ -4,7 +4,7 @@
 namespace Icinga\Controllers;
 
 use Icinga\Web\Controller\ActionController;
-use Icinga\Web\HomeMenu;
+use Icinga\Web\Menu;
 
 /**
  * Create complex layout parts
@@ -18,7 +18,7 @@ class LayoutController extends ActionController
     {
         $this->setAutorefreshInterval(15);
         $this->_helper->layout()->disableLayout();
-        $this->view->menuRenderer = (new HomeMenu())->getRenderer();
+        $this->view->menuRenderer = (new Menu())->getRenderer();
     }
 
     public function announcementsAction()
diff --git a/application/forms/Dashboard/BaseDashboardForm.php b/application/forms/Dashboard/BaseDashboardForm.php
index 5490caade..2fd48ccbc 100644
--- a/application/forms/Dashboard/BaseDashboardForm.php
+++ b/application/forms/Dashboard/BaseDashboardForm.php
@@ -76,7 +76,11 @@ abstract class BaseDashboardForm extends CompatForm
      */
     protected function createCancelButton()
     {
-        return $this->createElement('submitButton', 'btn_cancel', ['class' => 'modal-cancel', 'label' => t('Cancel')]);
+        return $this->createElement('submitButton', 'btn_cancel', [
+            'class'                    => 'modal-cancel',
+            'label'                    => t('Cancel'),
+            'data-icinga-modal-cancel' => true
+        ]);
     }
 
     /**
diff --git a/application/layouts/scripts/layout.phtml b/application/layouts/scripts/layout.phtml
index 1800f2c15..c4417c6c9 100644
--- a/application/layouts/scripts/layout.phtml
+++ b/application/layouts/scripts/layout.phtml
@@ -87,7 +87,7 @@ $innerLayoutScript = $this->layout()->innerLayout . '.phtml';
         <section class="modal-window">
             <div class="modal-header">
                 <h1></h1>
-                <button type="button"><?= $this->icon('cancel') ?></button>
+                <button type="button" data-icinga-modal-cancel><?= $this->icon('cancel') ?></button>
             </div>
             <div class="modal-area">
                 <div id="modal-content" data-base-target="modal-content" tabindex data-no-icinga-ajax></div>
diff --git a/application/layouts/scripts/parts/navigation.phtml b/application/layouts/scripts/parts/navigation.phtml
index fef541838..dd973f5c5 100644
--- a/application/layouts/scripts/parts/navigation.phtml
+++ b/application/layouts/scripts/parts/navigation.phtml
@@ -1,6 +1,6 @@
 <?php
 
-use Icinga\Web\HomeMenu;
+use Icinga\Web\Menu;
 
 // Don't render a menu for unauthenticated users unless menu is auth aware
 if (! $this->auth()->isAuthenticated()) {
@@ -25,7 +25,7 @@ if (! $this->auth()->isAuthenticated()) {
         'layout/menu.phtml',
         'default',
         array(
-            'menuRenderer' => (new HomeMenu())->getRenderer()->setUseStandardItemRenderer()
+            'menuRenderer' => (new Menu())->getRenderer()->setUseStandardItemRenderer()
         )
     ) ?>
 </div>
diff --git a/application/views/scripts/layout/menu.phtml b/application/views/scripts/layout/menu.phtml
index c50a6d2e5..20188fdc4 100644
--- a/application/views/scripts/layout/menu.phtml
+++ b/application/views/scripts/layout/menu.phtml
@@ -5,7 +5,8 @@ use Icinga\Web\Widget\SearchDashboard;
 $searchDashboard = new SearchDashboard();
 $searchDashboard->setUser($this->Auth()->getUser());
 
-if ($searchDashboard->search('dummy')->getActiveHome()->getEntry('search')->hasEntries()): ?>
+$searchPane = $searchDashboard->search('dummy')->getActiveHome()->getEntry(SearchDashboard::SEARCH_PANE);
+if ($searchPane->hasEntries()): ?>
     <form action="<?= $this->href('search') ?>" method="get" role="search" class="search-control">
         <input type="text" name="q" id="search" class="search search-input" required
                placeholder="<?= $this->translate('Search') ?> &hellip;"
diff --git a/library/Icinga/Web/Menu.php b/library/Icinga/Web/Menu.php
index 5d62ef28b..353bd92fa 100644
--- a/library/Icinga/Web/Menu.php
+++ b/library/Icinga/Web/Menu.php
@@ -5,8 +5,12 @@ namespace Icinga\Web;
 
 use Icinga\Application\Logger;
 use Icinga\Authentication\Auth;
+use Icinga\Model\Home;
+use Icinga\Web\Dashboard\DashboardHome;
+use Icinga\Web\Navigation\DashboardHomeItem;
 use Icinga\Web\Navigation\Navigation;
 use Icinga\Web\Dashboard\Dashboard;
+use ipl\Stdlib\Filter;
 
 /**
  * Main menu for Icinga Web 2
@@ -19,6 +23,8 @@ class Menu extends Navigation
     public function __construct()
     {
         $this->init();
+        $this->initHome();
+
         $this->load('menu-item');
     }
 
@@ -148,4 +154,44 @@ class Menu extends Navigation
             ]));
         }
     }
+
+    public function initHome()
+    {
+        $user = Dashboard::getUser();
+        $dashboardItem = $this->getItem('dashboard');
+
+        $homes = Home::on(Dashboard::getConn());
+        $homes->filter(Filter::equal('username', $user->getUsername()));
+
+        foreach ($homes as $home) {
+            $dashboardHome = new DashboardHomeItem($home->name, [
+                'uuid'     => $home->id,
+                'label'    => t($home->label),
+                'priority' => $home->priority,
+                'type'     => $home->type,
+            ]);
+
+            $dashboardItem->addChild($dashboardHome);
+        }
+    }
+
+    /**
+     * Load dashboard homes form the navigation menu
+     *
+     * @return DashboardHome[]
+     */
+    public function loadHomes()
+    {
+        $homes = [];
+        foreach ($this->getItem('dashboard')->getChildren() as $child) {
+            if (! $child instanceof DashboardHomeItem) {
+                continue;
+            }
+
+            $homes[$child->getName()] = DashboardHome::create($child);
+        }
+
+        return $homes;
+    }
+
 }