Avoid using dynamic properties of Icinga\Web\View as arguments

application/controllers/ConfigController.php

@@ -1,4 +1,4 @@
 <?php
 /* Icinga Web 2 | (c) 2013 Icinga Development Team | GPLv2+ */
 
 namespace Icinga\Controllers;
@@ -129,13 +129,14 @@
                 'url'   => 'config/modules'
             ))
             ->activate('modules');
-        $this->view->modules = Icinga::app()->getModuleManager()->select()
+        $modules = Icinga::app()->getModuleManager()->select()
             ->from('modules')
             ->order('enabled', 'desc')
             ->order('installed', 'asc')
             ->order('name');
         $this->setupLimitControl();
-        $this->setupPaginationControl($this->view->modules);
+        $this->setupPaginationControl($modules);
+        $this->view->modules = $modules;
         $this->view->title = $this->translate('Modules');
     }
 

application/controllers/ListController.php

@@ -1,4 +1,4 @@
 <?php
 /* Icinga Web 2 | (c) 2014 Icinga Development Team | GPLv2+ */
 
 namespace Icinga\Controllers;
@@ -50,10 +50,11 @@
                 . 'T[0-9]{2}(?::[0-9]{2}){2}(?:[\+\-][0-9]{2}:[0-9]{2})?)'  // time
                 . ' - (?<loglevel>[A-Za-z]+) - (?<message>.*)(?!.)/msS'     // loglevel, message
         )));
-        $this->view->logData = $resource->select()->order('DESC');
+        $logData = $resource->select()->order('DESC');
 
         $this->setupLimitControl();
-        $this->setupPaginationControl($this->view->logData);
+        $this->setupPaginationControl($logData);
+        $this->view->logData = $logData;
         $this->view->title = $this->translate('Application Log');
     }
 }

application/controllers/RoleController.php

@@ -1,4 +1,4 @@
 <?php
 /* Icinga Web 2 | (c) 2014 Icinga Development Team | GPLv2+ */
 
 namespace Icinga\Controllers;
@@ -59,7 +59,7 @@
     public function listAction()
     {
         $this->createListTabs()->activate('role/list');
-        $this->view->roles = (new RolesConfig())
+        $roles = (new RolesConfig())
             ->select();
 
         $sortAndFilterColumns = [
@@ -69,10 +69,11 @@
             'permissions' => $this->translate('Permissions')
         ];
 
-        $this->setupFilterControl($this->view->roles, $sortAndFilterColumns, ['name']);
+        $this->setupFilterControl($roles, $sortAndFilterColumns, ['name']);
         $this->setupLimitControl();
-        $this->setupPaginationControl($this->view->roles);
-        $this->setupSortControl($sortAndFilterColumns, $this->view->roles, ['name']);
+        $this->setupPaginationControl($roles);
+        $this->setupSortControl($sortAndFilterColumns, $roles, ['name']);
+        $this->view->roles = $roles;
     }
 
     /**

modules/monitoring/application/controllers/ShowController.php

@@ -91,7 +91,7 @@ public function contactAction()
             $this->applyRestriction('monitoring/filter/objects', $notifications);
             $this->view->notifications = $notifications;
             $this->setupLimitControl();
-            $this->setupPaginationControl($this->view->notifications);
+            $this->setupPaginationControl($notifications);
             $this->view->title = $contact->contact_name;
         }
 

modules/monitoring/library/Monitoring/Web/Controller/MonitoredObjectController.php

@@ -117,11 +117,12 @@ public function showAction()
     public function historyAction()
     {
         $this->getTabs()->activate('history');
-        $this->view->history = $this->object->fetchEventhistory()->eventhistory;
-        $this->applyRestriction('monitoring/filter/objects', $this->view->history);
+        $history = $this->object->fetchEventhistory()->eventhistory;
+        $this->applyRestriction('monitoring/filter/objects', $history);
 
         $this->setupLimitControl(50);
-        $this->setupPaginationControl($this->view->history, 50);
+        $this->setupPaginationControl($history, 50);
+        $this->view->history = $history;
         $this->view->object = $this->object;
         $this->render('object/detail-history', null, true);
     }