Fix codestyle, change wrong 'dashboard' parameter to pane causing pane switching to fail

- Fixed concatenation
- Added license header where missing
- Fixed the index controller fetching the wrong parameter ('dashboard' instead of 'pane')

 refs #4192
This commit is contained in:
Jannis Moßhammer 2013-08-08 17:42:34 +02:00
parent 237f7f1e60
commit 886da7d0cb
11 changed files with 217 additions and 27 deletions

View File

@ -1,4 +1,31 @@
<?php <?php
// @codingStandardsIgnoreStart
// {{{ICINGA_LICENSE_HEADER}}}
/**
* This file is part of Icinga 2 Web.
*
* Icinga 2 Web - Head for multiple monitoring backends.
* Copyright (C) 2013 Icinga Development Team
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version 2
* of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
*
* @copyright 2013 Icinga Development Team <info@icinga.org>
* @license http://www.gnu.org/licenses/gpl-2.0.txt GPL, version 2
* @author Icinga Development Team <info@icinga.org>
*/
// {{{ICINGA_LICENSE_HEADER}}}
use Icinga\Web\ActionController; use Icinga\Web\ActionController;
use Icinga\Web\Url; use Icinga\Web\Url;
@ -94,9 +121,9 @@ class DashboardController extends ActionController
{ {
$dashboard = $this->getDashboard(); $dashboard = $this->getDashboard();
if ($this->_getParam('dashboard')) { if ($this->_getParam('pane')) {
$dashboardName = $this->_getParam('dashboard'); $pane = $this->_getParam('pane');
$dashboard->activate($dashboardName); $dashboard->activate($pane);
} }
$this->view->tabs = $dashboard->getTabs(); $this->view->tabs = $dashboard->getTabs();
$this->view->tabs->add( $this->view->tabs->add(
@ -110,3 +137,5 @@ class DashboardController extends ActionController
$this->view->dashboard = $dashboard; $this->view->dashboard = $dashboard;
} }
} }
// @codingStandardsIgnoreEnd

View File

@ -1,2 +1,3 @@
<h3>Add dashboard URL</h3> <h3>Add dashboard URL</h3>
<?= $this->form->render($this) ?>
<?= $this->form->render($this); ?>

View File

@ -1,3 +1,3 @@
<?= $this->tabs->render($this) ?> <?= $this->tabs->render($this); ?>
<?= $this->dashboard->render($this) ?> <?= $this->dashboard->render($this); ?>

View File

@ -3,7 +3,7 @@
<h4><i class="icon-warning-sign"> </i>Saving dashboard failed</h4> <h4><i class="icon-warning-sign"> </i>Saving dashboard failed</h4>
<br/> <br/>
<p> <p>
Your dashboard couldn't be stored (error: "<?= $this->exceptionMessage ?>"). This could have one or more Your dashboard couldn't be stored (error: "<?= $this->exceptionMessage; ?>"). This could have one or more
of the following reasons: of the following reasons:
</p> </p>
<ul> <ul>
@ -23,7 +23,7 @@
<p> <p>
<pre> <pre>
<code> <code>
<?= $this->escape($this->iniConfigurationString) ?> <?= $this->escape($this->iniConfigurationString); ?>
</code> </code>
</pre> </pre>
</p> </p>

View File

@ -1,4 +1,30 @@
<?php <?php
// {{{ICINGA_LICENSE_HEADER}}}
/**
* This file is part of Icinga 2 Web.
*
* Icinga 2 Web - Head for multiple monitoring backends.
* Copyright (C) 2013 Icinga Development Team
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version 2
* of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
*
* @copyright 2013 Icinga Development Team <info@icinga.org>
* @license http://www.gnu.org/licenses/gpl-2.0.txt GPL, version 2
* @author Icinga Development Team <info@icinga.org>
*/
// {{{ICINGA_LICENSE_HEADER}}}
namespace Icinga\Web\Widget; namespace Icinga\Web\Widget;
@ -112,7 +138,7 @@ class Dashboard implements Widget
// make sure empty dashboards don't cause errors // make sure empty dashboards don't cause errors
$iniString = trim($this->toIni()); $iniString = trim($this->toIni());
if (!$iniString) { if (!$iniString) {
$iniString = " "; $iniString = ' ';
} }
if (!@file_put_contents($file, $iniString)) { if (!@file_put_contents($file, $iniString)) {
$error = error_get_last(); $error = error_get_last();
@ -146,13 +172,17 @@ class Dashboard implements Widget
/** /**
* Creates a new empty pane with the given title * Creates a new empty pane with the given title
* *
* @param $title * @param string $title
*
* @return Dashboard $this
*/ */
public function createPane($title) public function createPane($title)
{ {
$pane = new Pane($title); $pane = new Pane($title);
$pane->setTitle($title); $pane->setTitle($title);
$this->addPane($pane); $this->addPane($pane);
return $this;
} }
/** /**

View File

@ -1,5 +1,30 @@
<?php <?php
// {{{ICINGA_LICENSE_HEADER}}}
/**
* This file is part of Icinga 2 Web.
*
* Icinga 2 Web - Head for multiple monitoring backends.
* Copyright (C) 2013 Icinga Development Team
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version 2
* of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
*
* @copyright 2013 Icinga Development Team <info@icinga.org>
* @license http://www.gnu.org/licenses/gpl-2.0.txt GPL, version 2
* @author Icinga Development Team <info@icinga.org>
*/
// {{{ICINGA_LICENSE_HEADER}}}
namespace Icinga\Web\Widget\Dashboard; namespace Icinga\Web\Widget\Dashboard;
use Icinga\Util\Dimension; use Icinga\Util\Dimension;

View File

@ -1,8 +1,34 @@
<?php <?php
// {{{ICINGA_LICENSE_HEADER}}}
/**
* This file is part of Icinga 2 Web.
*
* Icinga 2 Web - Head for multiple monitoring backends.
* Copyright (C) 2013 Icinga Development Team
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version 2
* of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
*
* @copyright 2013 Icinga Development Team <info@icinga.org>
* @license http://www.gnu.org/licenses/gpl-2.0.txt GPL, version 2
* @author Icinga Development Team <info@icinga.org>
*/
// {{{ICINGA_LICENSE_HEADER}}}
namespace Icinga\Web\Widget\Dashboard; namespace Icinga\Web\Widget\Dashboard;
use Icinga\Exception\ConfigurationError; use Icinga\Exception\ConfigurationError;
use Icinga\Exception\ProgrammingError;
use Icinga\Web\Widget\Widget; use Icinga\Web\Widget\Widget;
use Zend_Config; use Zend_Config;
use Zend_View_Abstract; use Zend_View_Abstract;
@ -70,6 +96,7 @@ class Pane implements Widget
* Overwrite the title of this pane * Overwrite the title of this pane
* *
* @param string $title The new title to use for this pane * @param string $title The new title to use for this pane
*
* @return Pane $this * @return Pane $this
*/ */
public function setTitle($title) public function setTitle($title)

View File

@ -1,4 +1,30 @@
<?php <?php
// {{{ICINGA_LICENSE_HEADER}}}
/**
* This file is part of Icinga 2 Web.
*
* Icinga 2 Web - Head for multiple monitoring backends.
* Copyright (C) 2013 Icinga Development Team
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version 2
* of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
*
* @copyright 2013 Icinga Development Team <info@icinga.org>
* @license http://www.gnu.org/licenses/gpl-2.0.txt GPL, version 2
* @author Icinga Development Team <info@icinga.org>
*/
// {{{ICINGA_LICENSE_HEADER}}}
namespace Icinga\Web\Widget; namespace Icinga\Web\Widget;

View File

@ -1,4 +1,30 @@
<?php <?php
// {{{ICINGA_LICENSE_HEADER}}}
/**
* This file is part of Icinga 2 Web.
*
* Icinga 2 Web - Head for multiple monitoring backends.
* Copyright (C) 2013 Icinga Development Team
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version 2
* of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
*
* @copyright 2013 Icinga Development Team <info@icinga.org>
* @license http://www.gnu.org/licenses/gpl-2.0.txt GPL, version 2
* @author Icinga Development Team <info@icinga.org>
*/
// {{{ICINGA_LICENSE_HEADER}}}
namespace Icinga\Web\Widget; namespace Icinga\Web\Widget;

View File

@ -1,4 +1,30 @@
<?php <?php
// {{{ICINGA_LICENSE_HEADER}}}
/**
* This file is part of Icinga 2 Web.
*
* Icinga 2 Web - Head for multiple monitoring backends.
* Copyright (C) 2013 Icinga Development Team
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version 2
* of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
*
* @copyright 2013 Icinga Development Team <info@icinga.org>
* @license http://www.gnu.org/licenses/gpl-2.0.txt GPL, version 2
* @author Icinga Development Team <info@icinga.org>
*/
// {{{ICINGA_LICENSE_HEADER}}}
namespace Icinga\Web\Widget; namespace Icinga\Web\Widget;