From b40d0f6a1edab05b90c0943c0048ff308c72ee03 Mon Sep 17 00:00:00 2001
From: Thomas Gelf <thomas@gelf.net>
Date: Wed, 25 Jun 2014 10:34:09 +0200
Subject: [PATCH] FilterWidget: POST instead of GET

Might have been a JS issue, but instead of messing with the URL just
POSTing is easier here - and works. Filters from the search box are
now added flawlessly to the filter tree.
---
 library/Icinga/Web/Widget/FilterWidget.php             |  2 +-
 .../application/controllers/ListController.php         | 10 ++++++++--
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/library/Icinga/Web/Widget/FilterWidget.php b/library/Icinga/Web/Widget/FilterWidget.php
index c86c550d8..19808a4b4 100644
--- a/library/Icinga/Web/Widget/FilterWidget.php
+++ b/library/Icinga/Web/Widget/FilterWidget.php
@@ -73,7 +73,7 @@ class FilterWidget extends AbstractWidget
     {
         $url = Url::fromRequest();
         $view = $this->view();
-        $html = ' <form method="get" class="inline" action="'
+        $html = ' <form method="post" class="inline" action="'
               . $url
               . '"><input type="text" name="q" style="width: 8em" class="search" value="" placeholder="'
               . t('Add filter...')
diff --git a/modules/monitoring/application/controllers/ListController.php b/modules/monitoring/application/controllers/ListController.php
index 8379aaab3..0491d900e 100644
--- a/modules/monitoring/application/controllers/ListController.php
+++ b/modules/monitoring/application/controllers/ListController.php
@@ -33,7 +33,11 @@ class Monitoring_ListController extends Controller
     protected function hasBetterUrl()
     {
         $url = clone($this->url);
-        $q = $url->shift('q');
+        if ($this->getRequest()->isPost()) {
+            $q = $this->getRequest()->getPost('q');
+        } else {
+            $q = $url->shift('q');
+        }
         if ($q) {
             list($k, $v) = preg_split('/=/', $q);
             $url->addParams(array($k => $v));
@@ -57,6 +61,9 @@ class Monitoring_ListController extends Controller
      */
     public function hostsAction()
     {
+        if ($url = $this->hasBetterUrl()) {
+            return $this->redirectNow($url);
+        }
         $this->addTitleTab('hosts');
         $this->setAutorefreshInterval(10);
         $this->compactView = 'hosts-compact';
@@ -103,7 +110,6 @@ class Monitoring_ListController extends Controller
     public function servicesAction()
     {
         if ($url = $this->hasBetterUrl()) {
-            // TODO: This should NOT render
             return $this->redirectNow($url);
         }
         $this->addTitleTab('services');