Evaluate, Test and Move the items und library/Icinga/Web to the source tree

Fix PSR compliance.

refs #4256

library/Icinga/Web/Form.php

@@ -1,4 +1,6 @@
 <?php
+// {{{ICINGA_LICENSE_HEADER}}}
+// {{{ICINGA_LICENSE_HEADER}}}
 
 namespace Icinga\Web;
 
@@ -6,13 +8,17 @@ use Zend_Form;
 use Zend_Controller_Front as Front; // TODO: Get from App
 use Zend_Controller_Action_HelperBroker as ZfActionHelper;
 
+/**
+ * Class Form
+ * @package Icinga\Web
+ */
 class Form extends Zend_Form
 {
     protected $request;
 
     /**
-     * @param Zend_Controller_Request_Abstract $request
      * @param array $options[optional]
+     * @internal param \Icinga\Web\Zend_Controller_Request_Abstract $request
      */
     public function __construct($options = null)
     {

library/Icinga/Web/Hook/Grapher.php

@@ -1,8 +1,7 @@
 <?php
+// {{{ICINGA_LICENSE_HEADER}}}
+// {{{ICINGA_LICENSE_HEADER}}}
 
-/**
- * Icinga Web Grapher Hook
- */
 namespace Icinga\Web\Hook;
 
 /**

library/Icinga/Web/Hook/Toptray.php

@@ -1,30 +1,62 @@
 <?php
+// {{{ICINGA_LICENSE_HEADER}}}
+// {{{ICINGA_LICENSE_HEADER}}}
 
 namespace Icinga\Web\Hook;
+
 use Icinga\Application\Logger as Logger;
 
-abstract class Toptray {
+/**
+ * Class Toptray
+ * @package Icinga\Web\Hook
+ */
+abstract class Toptray
+{
+    /**
+     *
+     */
     const ALIGN_LEFT = "pull-left";
+
+    /**
+     *
+     */
     const ALIGN_NONE = "";
-    const ALIGN_RIGHT =  "pull-right";
+
+    /**
+     *
+     */
+    const ALIGN_RIGHT = "pull-right";
+
+    /**
+     * @var string
+     */
     protected $align = self::ALIGN_NONE;
 
+    /**
+     * @param $align
+     */
     public function setAlignment($align)
     {
         $this->align = $align;
     }
 
+    /**
+     * @return string
+     */
     final public function getWidgetDOM()
     {
         try {
-            return '<ul class="nav '.$this->align.'" >'.$this->buildDOM().'</ul>';
+            return '<ul class="nav ' . $this->align . '" >' . $this->buildDOM() . '</ul>';
         } catch (\Exception $e) {
-            Logger::error("Could not create tray widget : %s",$e->getMessage());
+            Logger::error("Could not create tray widget : %s", $e->getMessage());
             return '';
         }
 
 
     }
 
+    /**
+     * @return mixed
+     */
     abstract protected function buildDOM();
 }

library/Icinga/Web/ModuleActionController.php

@@ -1,8 +1,7 @@
 <?php
+// {{{ICINGA_LICENSE_HEADER}}}
+// {{{ICINGA_LICENSE_HEADER}}}
 
-/**
- * Module action controller
- */
 namespace Icinga\Web;
 
 use Icinga\Application\Config;

library/Icinga/Web/Paginator/Adapter/QueryAdapter.php

@@ -1,6 +1,9 @@
 <?php
+// {{{ICINGA_LICENSE_HEADER}}}
+// {{{ICINGA_LICENSE_HEADER}}}
 
 namespace Icinga\Web\Paginator\Adapter;
+
 /**
  * @see Zend_Paginator_Adapter_Interface
  */

library/Icinga/Web/Paginator/ScrollingStyle/SlidingWithBorder.php

@@ -1,4 +1,6 @@
 <?php
+// @codingStandardsIgnoreStart
+
 // {{{ICINGA_LICENSE_HEADER}}}
 // {{{ICINGA_LICENSE_HEADER}}}
 
@@ -65,3 +67,5 @@ class Icinga_Web_Paginator_ScrollingStyle_SlidingWithBorder implements Zend_Pagi
         return $range;
     }
 }
+
+// @codingStandardsIgnoreEnd

library/Icinga/Web/Widget/AbstractWidget.php

@@ -1,8 +1,7 @@
 <?php
+// {{{ICINGA_LICENSE_HEADER}}}
+// {{{ICINGA_LICENSE_HEADER}}}
 
-/**
- * Web Widget abstract class
- */
 namespace Icinga\Web\Widget;
 
 use Icinga\Exception\ProgrammingError;

library/Icinga/Web/Widget/Form.php

@@ -1,8 +1,7 @@
 <?php
+// {{{ICINGA_LICENSE_HEADER}}}
+// {{{ICINGA_LICENSE_HEADER}}}
 
-/**
- * Form
- */
 namespace Icinga\Web\Widget;
 
 /**

library/Icinga/Web/Widget/Tab.php

@@ -1,8 +1,7 @@
 <?php
+// {{{ICINGA_LICENSE_HEADER}}}
+// {{{ICINGA_LICENSE_HEADER}}}
 
-/**
- * Single tab
- */
 namespace Icinga\Web\Widget;
 
 use Icinga\Exception\ProgrammingError;
@@ -40,18 +39,16 @@ class Tab extends AbstractWidget
      * @var array
      */
     protected $properties = array(
-        'name'      => null,
-        'title'     => '',
-        'url'       => null,
+        'name' => null,
+        'title' => '',
+        'url' => null,
         'urlParams' => array(),
-        'icon'      => null,
+        'icon' => null,
     );
 
     /**
      * Health check at initialization time
-     *
-     * @throws Icinga\Exception\ProgrammingError if tab name is missing
-     *
+     * @throws \Icinga\Exception\ProgrammingError if tab name is missing
      * @return void
      */
     protected function init()
@@ -75,7 +72,7 @@ class Tab extends AbstractWidget
      */
     public function setActive($active = true)
     {
-        $this->active = (bool) $active;
+        $this->active = (bool)$active;
         return $this;
     }
 
@@ -100,10 +97,15 @@ class Tab extends AbstractWidget
         $class = $this->isActive() ? ' class="active"' : '';
         $caption = $this->title;
         if ($this->icon !== null) {
-            $caption = $view->img($this->icon, array(
-                'width'  => 16,
-                'height' => 16
-            )) . ' ' . $caption;
+            $caption = $view->img(
+                $this->icon,
+                array(
+                    'width' => 16,
+                    'height' => 16
+                )
+            )
+            . ' '
+            . $caption;
         }
         if ($this->url !== null) {
             $tab = $view->qlink(

library/Icinga/Web/Widget/Tabs.php

@@ -1,8 +1,7 @@
 <?php
+// {{{ICINGA_LICENSE_HEADER}}}
+// {{{ICINGA_LICENSE_HEADER}}}
 
-/**
- * Navigation tabs
- */
 namespace Icinga\Web\Widget;
 
 use Icinga\Exception\ProgrammingError;

test/php/library/Icinga/Web/Paginator/Adapter/QueryAdapterTest.php

@@ -56,15 +56,24 @@ class QueryAdapterTest extends \PHPUnit_Framework_TestCase
         $backend = new Statusdat($this->config);
         $query = $backend->select()->from('hostlist');
 
-        $paginator = new QueryAdapter($query);
+        $adapter = new QueryAdapter($query);
 
-        $this->assertEquals(30, $paginator->count());
+        $this->assertEquals(30, $adapter->count());
 
-        $data = $paginator->getItems(0, 10);
+        $data = $adapter->getItems(0, 10);
 
         $this->assertCount(10, $data);
 
-        $data = $paginator->getItems(10, 20);
+        $data = $adapter->getItems(10, 20);
         $this->assertCount(10, $data);
     }
+
+    public function testLimit2()
+    {
+        $backend = new Statusdat($this->config);
+        $query = $backend->select()->from('hostlist');
+
+        $adapter = new QueryAdapter($query);
+        $this->assertEquals(30, $adapter->count());
+    }
 }

test/php/library/Icinga/Web/Paginator/ScrollingStyle/SlidingWithBorderTest.php

@@ -27,6 +27,8 @@ require_once '../../library/Icinga/Protocol/AbstractQuery.php';
 require_once '../../library/Icinga/Protocol/Statusdat/IReader.php';
 require_once '../../library/Icinga/Protocol/Statusdat/Reader.php';
 require_once '../../library/Icinga/Protocol/Statusdat/Query.php';
+require_once '../../library/Icinga/Protocol/Statusdat/Parser.php';
+require_once '../../library/Icinga/Protocol/Statusdat/RuntimeStateContainer.php';
 
 require_once '../../library/Icinga/Web/Paginator/ScrollingStyle/SlidingWithBorder.php';