From cb92d32c79930a79b06cf9615d6a056ca0d0d4f8 Mon Sep 17 00:00:00 2001 From: Dennis Eichhorn Date: Thu, 12 Sep 2019 19:48:00 +0200 Subject: [PATCH] fix phpcs findings --- Admin/Install/Navigation.php | 20 ++++++------- Admin/Installer.php | 18 +++++------ Admin/Status.php | 18 +++++------ Admin/Uninstaller.php | 18 +++++------ Admin/Updater.php | 18 +++++------ Controller/BackendController.php | 30 +++++++++++-------- Controller/Controller.php | 30 +++++++++---------- Models/Event.php | 30 +++++++++---------- Models/EventMapper.php | 30 +++++++++---------- Models/EventType.php | 18 +++++------ Models/PermissionState.php | 18 +++++------ Models/ProgressType.php | 18 +++++------ Theme/Backend/Lang/Navigation.en.lang.php | 10 +++---- Theme/Backend/Lang/api.en.lang.php | 10 +++---- Theme/Backend/Lang/en.lang.php | 10 +++---- Theme/Backend/eventmanagement-create.tpl.php | 10 +++---- Theme/Backend/eventmanagement-list.tpl.php | 10 +++---- Theme/Backend/eventmanagement-profile.tpl.php | 10 +++---- 18 files changed, 166 insertions(+), 160 deletions(-) diff --git a/Admin/Install/Navigation.php b/Admin/Install/Navigation.php index 7beab65..ce8eda5 100644 --- a/Admin/Install/Navigation.php +++ b/Admin/Install/Navigation.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package Modules\EventManagement\Admin\Install - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package Modules\EventManagement\Admin\Install + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ declare(strict_types=1); @@ -19,10 +19,10 @@ use phpOMS\DataStorage\Database\DatabasePool; /** * Navigation class. * - * @package Modules\EventManagement\Admin\Install - * @license OMS License 1.0 - * @link https://orange-management.org - * @since 1.0.0 + * @package Modules\EventManagement\Admin\Install + * @license OMS License 1.0 + * @link https://orange-management.org + * @since 1.0.0 */ class Navigation { @@ -34,7 +34,7 @@ class Navigation * * @return void * - * @since 1.0.0 + * @since 1.0.0 */ public static function install(string $path, DatabasePool $dbPool) : void { diff --git a/Admin/Installer.php b/Admin/Installer.php index bc7abf3..68bac1c 100644 --- a/Admin/Installer.php +++ b/Admin/Installer.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package Modules\EventManagement\Admin - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package Modules\EventManagement\Admin + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ declare(strict_types=1); @@ -19,10 +19,10 @@ use phpOMS\Module\InstallerAbstract; /** * Installer class. * - * @package Modules\EventManagement\Admin - * @license OMS License 1.0 - * @link https://orange-management.org - * @since 1.0.0 + * @package Modules\EventManagement\Admin + * @license OMS License 1.0 + * @link https://orange-management.org + * @since 1.0.0 */ class Installer extends InstallerAbstract { diff --git a/Admin/Status.php b/Admin/Status.php index 254c926..2b6ce08 100644 --- a/Admin/Status.php +++ b/Admin/Status.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package Modules\EventManagement\Admin - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package Modules\EventManagement\Admin + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ declare(strict_types=1); @@ -19,10 +19,10 @@ use phpOMS\Module\StatusAbstract; /** * Navigation class. * - * @package Modules\EventManagement\Admin - * @license OMS License 1.0 - * @link https://orange-management.org - * @since 1.0.0 + * @package Modules\EventManagement\Admin + * @license OMS License 1.0 + * @link https://orange-management.org + * @since 1.0.0 */ class Status extends StatusAbstract { diff --git a/Admin/Uninstaller.php b/Admin/Uninstaller.php index 6d344b4..7aada84 100644 --- a/Admin/Uninstaller.php +++ b/Admin/Uninstaller.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package Modules\EventManagement\Admin - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package Modules\EventManagement\Admin + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ declare(strict_types=1); @@ -19,10 +19,10 @@ use phpOMS\Module\UninstallerAbstract; /** * Uninstaller class. * - * @package Modules\EventManagement\Admin - * @license OMS License 1.0 - * @link https://orange-management.org - * @since 1.0.0 + * @package Modules\EventManagement\Admin + * @license OMS License 1.0 + * @link https://orange-management.org + * @since 1.0.0 */ class Uninstaller extends UninstallerAbstract { diff --git a/Admin/Updater.php b/Admin/Updater.php index 9de02e1..0686e95 100644 --- a/Admin/Updater.php +++ b/Admin/Updater.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package Modules\EventManagement\Admin - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package Modules\EventManagement\Admin + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ declare(strict_types=1); @@ -19,10 +19,10 @@ use phpOMS\Module\UpdaterAbstract; /** * Updater class. * - * @package Modules\EventManagement\Admin - * @license OMS License 1.0 - * @link https://orange-management.org - * @since 1.0.0 + * @package Modules\EventManagement\Admin + * @license OMS License 1.0 + * @link https://orange-management.org + * @since 1.0.0 */ class Updater extends UpdaterAbstract { diff --git a/Controller/BackendController.php b/Controller/BackendController.php index 996e1f8..a9001ae 100644 --- a/Controller/BackendController.php +++ b/Controller/BackendController.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package Modules\EventManagement - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package Modules\EventManagement + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ declare(strict_types=1); @@ -26,22 +26,24 @@ use phpOMS\Views\View; /** * Event Management controller class. * - * @package Modules\EventManagement - * @license OMS License 1.0 - * @link https://orange-management.org - * @since 1.0.0 + * @package Modules\EventManagement + * @license OMS License 1.0 + * @link https://orange-management.org + * @since 1.0.0 */ final class BackendController extends Controller { /** + * Routing end-point for application behaviour. + * * @param RequestAbstract $request Request * @param ResponseAbstract $response Response * @param mixed $data Generic data * * @return RenderableInterface * - * @since 1.0.0 + * @since 1.0.0 * @codeCoverageIgnore */ public function viewEventManagementList(RequestAbstract $request, ResponseAbstract $response, $data = null) : RenderableInterface @@ -57,13 +59,15 @@ final class BackendController extends Controller } /** + * Routing end-point for application behaviour. + * * @param RequestAbstract $request Request * @param ResponseAbstract $response Response * @param mixed $data Generic data * * @return RenderableInterface * - * @since 1.0.0 + * @since 1.0.0 * @codeCoverageIgnore */ public function viewEventManagementCreate(RequestAbstract $request, ResponseAbstract $response, $data = null) : RenderableInterface @@ -76,13 +80,15 @@ final class BackendController extends Controller } /** + * Routing end-point for application behaviour. + * * @param RequestAbstract $request Request * @param ResponseAbstract $response Response * @param mixed $data Generic data * * @return RenderableInterface * - * @since 1.0.0 + * @since 1.0.0 * @codeCoverageIgnore */ public function viewEventManagementProfile(RequestAbstract $request, ResponseAbstract $response, $data = null) : RenderableInterface diff --git a/Controller/Controller.php b/Controller/Controller.php index e9048a8..98b14bd 100644 --- a/Controller/Controller.php +++ b/Controller/Controller.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package Modules\EventManagement - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package Modules\EventManagement + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ declare(strict_types=1); @@ -21,10 +21,10 @@ use phpOMS\Module\WebInterface; /** * Event Management controller class. * - * @package Modules\EventManagement - * @license OMS License 1.0 - * @link https://orange-management.org - * @since 1.0.0 + * @package Modules\EventManagement + * @license OMS License 1.0 + * @link https://orange-management.org + * @since 1.0.0 */ class Controller extends ModuleAbstract implements WebInterface { @@ -32,7 +32,7 @@ class Controller extends ModuleAbstract implements WebInterface /** * Module path. * - * @var string + * @var string * @since 1.0.0 */ public const MODULE_PATH = __DIR__ . '/../'; @@ -40,7 +40,7 @@ class Controller extends ModuleAbstract implements WebInterface /** * Module version. * - * @var string + * @var string * @since 1.0.0 */ public const MODULE_VERSION = '1.0.0'; @@ -48,7 +48,7 @@ class Controller extends ModuleAbstract implements WebInterface /** * Module name. * - * @var string + * @var string * @since 1.0.0 */ public const MODULE_NAME = 'EventManagement'; @@ -56,7 +56,7 @@ class Controller extends ModuleAbstract implements WebInterface /** * Module id. * - * @var int + * @var int * @since 1.0.0 */ public const MODULE_ID = 1004200000; @@ -64,7 +64,7 @@ class Controller extends ModuleAbstract implements WebInterface /** * Providing. * - * @var string[] + * @var string[] * @since 1.0.0 */ protected static array $providing = []; @@ -72,7 +72,7 @@ class Controller extends ModuleAbstract implements WebInterface /** * Dependencies. * - * @var string[] + * @var string[] * @since 1.0.0 */ protected static array $dependencies = []; diff --git a/Models/Event.php b/Models/Event.php index 2c1604f..9243012 100644 --- a/Models/Event.php +++ b/Models/Event.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package Modules\EventManagement\Models - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package Modules\EventManagement\Models + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ declare(strict_types=1); @@ -22,10 +22,10 @@ use phpOMS\Stdlib\Base\Exception\InvalidEnumValue; /** * Event class. * - * @package Modules\EventManagement\Models - * @license OMS License 1.0 - * @link https://orange-management.org - * @since 1.0.0 + * @package Modules\EventManagement\Models + * @license OMS License 1.0 + * @link https://orange-management.org + * @since 1.0.0 */ class Event { @@ -33,7 +33,7 @@ class Event /** * ID. * - * @var int + * @var int * @since 1.0.0 */ private int $id = 0; @@ -67,7 +67,7 @@ class Event /** * Created. * - * @var null|\DateTime + * @var null|\DateTime * @since 1.0.0 */ private ?\DateTime $createdAt = null; @@ -75,7 +75,7 @@ class Event /** * Creator. * - * @var int + * @var int * @since 1.0.0 */ private $createdBy = 0; @@ -255,7 +255,7 @@ class Event /** * @return \DateTime * - * @since 1.0.0 + * @since 1.0.0 */ public function getCreatedAt() : \DateTime { @@ -265,7 +265,7 @@ class Event /** * @return int * - * @since 1.0.0 + * @since 1.0.0 */ public function getCreatedBy() : int { @@ -275,7 +275,7 @@ class Event /** * @param $createdBy Creator * - * @since 1.0.0 + * @since 1.0.0 */ public function setCreatedBy(int $createdBy) : void { diff --git a/Models/EventMapper.php b/Models/EventMapper.php index a030aac..040ebeb 100644 --- a/Models/EventMapper.php +++ b/Models/EventMapper.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package Modules\EventManagement\Models - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package Modules\EventManagement\Models + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ declare(strict_types=1); @@ -21,10 +21,10 @@ use phpOMS\DataStorage\Database\DataMapperAbstract; /** * Mapper class. * - * @package Modules\EventManagement\Models - * @license OMS License 1.0 - * @link https://orange-management.org - * @since 1.0.0 + * @package Modules\EventManagement\Models + * @license OMS License 1.0 + * @link https://orange-management.org + * @since 1.0.0 */ final class EventMapper extends DataMapperAbstract { @@ -32,7 +32,7 @@ final class EventMapper extends DataMapperAbstract /** * Columns. * - * @var array> + * @var array> * @since 1.0.0 */ protected static array $columns = [ @@ -55,7 +55,7 @@ final class EventMapper extends DataMapperAbstract /** * Has many relation. * - * @var array> + * @var array> * @since 1.0.0 */ protected static array $hasMany = [ @@ -76,7 +76,7 @@ final class EventMapper extends DataMapperAbstract /** * Has one relation. * - * @var array> + * @var array> * @since 1.0.0 */ protected static array $ownsOne = [ @@ -89,7 +89,7 @@ final class EventMapper extends DataMapperAbstract /** * Primary table. * - * @var string + * @var string * @since 1.0.0 */ protected static string $table = 'eventmanagement_event'; @@ -97,7 +97,7 @@ final class EventMapper extends DataMapperAbstract /** * Created at. * - * @var string + * @var string * @since 1.0.0 */ protected static string $createdAt = 'eventmanagement_event_created_at'; @@ -105,7 +105,7 @@ final class EventMapper extends DataMapperAbstract /** * Primary field name. * - * @var string + * @var string * @since 1.0.0 */ protected static string $primaryField = 'eventmanagement_event_id'; diff --git a/Models/EventType.php b/Models/EventType.php index 667503d..3d07711 100644 --- a/Models/EventType.php +++ b/Models/EventType.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package Modules\EventManagement\Models - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package Modules\EventManagement\Models + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ declare(strict_types=1); @@ -19,10 +19,10 @@ use phpOMS\Stdlib\Base\Enum; /** * Event type enum. * - * @package Modules\EventManagement\Models - * @license OMS License 1.0 - * @link https://orange-management.org - * @since 1.0.0 + * @package Modules\EventManagement\Models + * @license OMS License 1.0 + * @link https://orange-management.org + * @since 1.0.0 */ abstract class EventType extends Enum { diff --git a/Models/PermissionState.php b/Models/PermissionState.php index 07e5dc5..40e111e 100644 --- a/Models/PermissionState.php +++ b/Models/PermissionState.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package Modules\EventManagement - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package Modules\EventManagement + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ declare(strict_types=1); @@ -19,10 +19,10 @@ use phpOMS\Stdlib\Base\Enum; /** * Permision state enum. * - * @package Modules\EventManagement - * @license OMS License 1.0 - * @link https://orange-management.org - * @since 1.0.0 + * @package Modules\EventManagement + * @license OMS License 1.0 + * @link https://orange-management.org + * @since 1.0.0 */ abstract class PermissionState extends Enum { diff --git a/Models/ProgressType.php b/Models/ProgressType.php index 7222ca4..2131ccc 100644 --- a/Models/ProgressType.php +++ b/Models/ProgressType.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package Modules\EventManagement\Models - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package Modules\EventManagement\Models + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ declare(strict_types=1); @@ -19,10 +19,10 @@ use phpOMS\Stdlib\Base\Enum; /** * Task type enum. * - * @package Modules\EventManagement\Models - * @license OMS License 1.0 - * @link https://orange-management.org - * @since 1.0.0 + * @package Modules\EventManagement\Models + * @license OMS License 1.0 + * @link https://orange-management.org + * @since 1.0.0 */ abstract class ProgressType extends Enum { diff --git a/Theme/Backend/Lang/Navigation.en.lang.php b/Theme/Backend/Lang/Navigation.en.lang.php index 7e68ee7..c1e74a3 100644 --- a/Theme/Backend/Lang/Navigation.en.lang.php +++ b/Theme/Backend/Lang/Navigation.en.lang.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package TBD - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package TBD + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ return ['Navigation' => [ 'EventManagement' => 'Event Management', diff --git a/Theme/Backend/Lang/api.en.lang.php b/Theme/Backend/Lang/api.en.lang.php index 1afe44f..07bb1f6 100644 --- a/Theme/Backend/Lang/api.en.lang.php +++ b/Theme/Backend/Lang/api.en.lang.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package TBD - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package TBD + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ $MODLANG[1] = [ ]; diff --git a/Theme/Backend/Lang/en.lang.php b/Theme/Backend/Lang/en.lang.php index 11a68f2..ba274d6 100644 --- a/Theme/Backend/Lang/en.lang.php +++ b/Theme/Backend/Lang/en.lang.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package TBD - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package TBD + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ return ['EventManagement' => [ 'Account' => 'Account', diff --git a/Theme/Backend/eventmanagement-create.tpl.php b/Theme/Backend/eventmanagement-create.tpl.php index 41de0e3..6c303c4 100644 --- a/Theme/Backend/eventmanagement-create.tpl.php +++ b/Theme/Backend/eventmanagement-create.tpl.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package TBD - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package TBD + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ echo $this->getData('nav')->render(); ?> diff --git a/Theme/Backend/eventmanagement-list.tpl.php b/Theme/Backend/eventmanagement-list.tpl.php index 6f67db4..d8ebf53 100644 --- a/Theme/Backend/eventmanagement-list.tpl.php +++ b/Theme/Backend/eventmanagement-list.tpl.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package TBD - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package TBD + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ $footerView = new \phpOMS\Views\PaginationView($this->app, $this->request, $this->response); diff --git a/Theme/Backend/eventmanagement-profile.tpl.php b/Theme/Backend/eventmanagement-profile.tpl.php index 0773f3f..5a79bcd 100644 --- a/Theme/Backend/eventmanagement-profile.tpl.php +++ b/Theme/Backend/eventmanagement-profile.tpl.php @@ -4,11 +4,11 @@ * * PHP Version 7.4 * - * @package TBD - * @copyright Dennis Eichhorn - * @license OMS License 1.0 - * @version 1.0.0 - * @link https://orange-management.org + * @package TBD + * @copyright Dennis Eichhorn + * @license OMS License 1.0 + * @version 1.0.0 + * @link https://orange-management.org */ $event = $this->getData('event');