[TASK] Code cleanup 66/51166/3
authorStanislas Rolland <typo3@sjbr.ca>
Fri, 6 Jan 2017 19:05:55 +0000 (14:05 -0500)
committerStanislas Rolland <typo3@sjbr.ca>
Fri, 6 Jan 2017 19:15:02 +0000 (20:15 +0100)
Change-Id: I9c8add128437bfa4cb3b0642601e551d5b957447
Reviewed-on: https://review.typo3.org/51166
Reviewed-by: Stanislas Rolland <typo3@sjbr.ca>
Tested-by: Stanislas Rolland <typo3@sjbr.ca>
Classes/Controller/MenuController.php
Classes/Domain/Model/Page.php
Classes/Domain/Model/PageLanguageOverlay.php
Classes/Domain/Model/SystemLanguage.php
Classes/Domain/Repository/PageLanguageOverlayRepository.php
Classes/Domain/Repository/PageRepository.php
Classes/Domain/Repository/SystemLanguageRepository.php
Classes/ViewHelpers/Widget/MenuViewHelper.php

index b38cdcd..12d15bf 100644 (file)
@@ -4,7 +4,7 @@ namespace SJBR\SrLanguageMenu\Controller;
 /***************************************************************
  *  Copyright notice
  *
- *  (c) 2013-2015 Stanislas Rolland <typo3(arobas)sjbr.ca>
+ *  (c) 2013-2017 Stanislas Rolland <typo3(arobas)sjbr.ca>
  *  All rights reserved
  *
  *  This script is part of the TYPO3 project. The TYPO3 project is
@@ -33,9 +33,11 @@ use TYPO3\CMS\Core\Utility\GeneralUtility;
 use TYPO3\CMS\Extbase\Configuration\ConfigurationManagerInterface;
 use TYPO3\CMS\Extbase\Mvc\Controller\ActionController;
 use TYPO3\CMS\Extbase\Mvc\Exception\StopActionException;
+use TYPO3\CMS\Extbase\Mvc\Request;
 use TYPO3\CMS\Extbase\Mvc\View\ViewInterface;
 use TYPO3\CMS\Extbase\Utility\ArrayUtility;
 use TYPO3\CMS\Fluid\Core\Widget\WidgetRequest;
+use SJBR\SrLanguageMenu\Domain\Model\Page;
 use SJBR\SrLanguageMenu\Domain\Model\PageLanguageOverlay;
 use SJBR\SrLanguageMenu\Domain\Model\SystemLanguage;
 use SJBR\SrLanguageMenu\Utility\LocalizationUtility;
@@ -43,14 +45,14 @@ use SJBR\SrLanguageMenu\Utility\LocalizationUtility;
 /**
  * Controls the rendering of the language menu as a normal content element or as a Fluid widget
  */
-class MenuController extends \TYPO3\CMS\Fluid\Core\Widget\AbstractWidgetController {
-
+class MenuController extends \TYPO3\CMS\Fluid\Core\Widget\AbstractWidgetController
+{
        /**
         * @var array
         */
        protected $supportedRequestTypes = array(
-               'TYPO3\\CMS\\Extbase\\Mvc\\Request',
-               'TYPO3\\CMS\\Fluid\\Core\\Widget\\WidgetRequest'
+               Request::class,
+               WidgetRequest::class
        );
 
        /**
@@ -138,7 +140,7 @@ class MenuController extends \TYPO3\CMS\Fluid\Core\Widget\AbstractWidgetControll
                        }
                }
 
-               $defaultSystemLanguage = $this->objectManager->get('SJBR\\SrLanguageMenu\\Domain\\Model\\SystemLanguage');
+               $defaultSystemLanguage = $this->objectManager->get(SystemLanguage::class);
                $defaultSystemLanguage->setIsoLanguage($defaultIsoLanguage);
                if (trim($this->settings['defaultLanguageTitle'])) {
                        $defaultLanguageTitle = trim($this->settings['defaultLanguageTitle']);
@@ -153,7 +155,7 @@ class MenuController extends \TYPO3\CMS\Fluid\Core\Widget\AbstractWidgetControll
 
                // Beware of inaccessible page
                $page = $this->pageRepository->findByUid($this->getFrontendObject()->id);
-               if ($page instanceof \SJBR\SrLanguageMenu\Domain\Model\Page) {
+               if ($page instanceof Page) {
                        // If "Hide default translation of page" is not set on the page...
                        if (!($page->getL18nCfg()&1)) {
                                // Add default language
index 4576ed9..cb8bfe5 100644 (file)
@@ -1,10 +1,11 @@
 <?php
 namespace SJBR\SrLanguageMenu\Domain\Model;
+
 /***************************************************************
  *  Copyright notice
  *
- *  (c) 2013-2014 Stanislas Rolland <typo3(arobas)sjbr.ca>
- *  
+ *  (c) 2013-2017 Stanislas Rolland <typo3(arobas)sjbr.ca>
+ *
  *  All rights reserved
  *
  *  This script is part of the TYPO3 project. The TYPO3 project is
@@ -23,11 +24,14 @@ namespace SJBR\SrLanguageMenu\Domain\Model;
  *
  *  This copyright notice MUST APPEAR in all copies of the script!
  ***************************************************************/
+
+use TYPO3\CMS\Extbase\DomainObject\AbstractEntity;
+
 /**
  * The Page model
  */
-class Page extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
-
+class Page extends AbstractEntity
+{
        /**
         * l18n_cfg ("Hide default translation of page" and "Hide page if no translation for current language exists")
         *
@@ -40,7 +44,8 @@ class Page extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
         *
         * @return integer
         */
-       public function getL18nCfg() {
+       public function getL18nCfg()
+       {
                return $this->l18nCfg;
        }
 }
\ No newline at end of file
index 2aac760..16ae3f6 100644 (file)
@@ -1,10 +1,11 @@
 <?php
 namespace SJBR\SrLanguageMenu\Domain\Model;
+
 /***************************************************************
  *  Copyright notice
  *
- *  (c) 2013-2014 Stanislas Rolland <typo3(arobas)sjbr.ca>
- *  
+ *  (c) 2013-2017 Stanislas Rolland <typo3(arobas)sjbr.ca>
+ *
  *  All rights reserved
  *
  *  This script is part of the TYPO3 project. The TYPO3 project is
@@ -23,11 +24,16 @@ namespace SJBR\SrLanguageMenu\Domain\Model;
  *
  *  This copyright notice MUST APPEAR in all copies of the script!
  ***************************************************************/
+
+use SJBR\SrLanguageMenu\Domain\Model\Page;
+use SJBR\SrLanguageMenu\Domain\Model\SystemLanguage;
+use TYPO3\CMS\Extbase\DomainObject\AbstractEntity;
+
 /**
  * The Page Language Overlay model
  */
-class PageLanguageOverlay extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
-       
+class PageLanguageOverlay extends AbstractEntity
+{
        /**
         * Page
         *
@@ -45,29 +51,32 @@ class PageLanguageOverlay extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity
        /**
         * Sets the page
         *
-        * @param \SJBR\SrLanguageMenu\Domain\Model\Page $page
+        * @param Page $page
         * @return void
         */
-       public function setPage(\SJBR\SrLanguageMenu\Domain\Model\Page $page) {
+       public function setPage(Page $page)
+       {
                $this->page = $page;
-       }       
+       }
 
        /**
         * Returns the page
         *
-        * @return \SJBR\SrLanguageMenu\Domain\Model\Page
+        * @return Page
         */
-       public function getPage() {
+       public function getPage()
+       {
                return $this->page;
        }
 
        /**
         * Sets the language
         *
-        * @param \SJBR\SrLanguageMenu\Domain\Model\SystemLanguage $language
+        * @param SystemLanguage $language
         * @return void
         */
-       public function setLanguage(\SJBR\SrLanguageMenu\Domain\Model\SystemLanguage $language) {
+       public function setLanguage(SystemLanguage $language)
+       {
                $this->language = $language;
        }
 
@@ -76,8 +85,8 @@ class PageLanguageOverlay extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity
         *
         * @return SystemLanguage
         */
-       public function getLanguage() {
+       public function getLanguage()
+       {
                return $this->language;
        }
-}
-?>
\ No newline at end of file
+}
\ No newline at end of file
index 321d7ad..7a1760b 100644 (file)
@@ -1,33 +1,38 @@
 <?php
 namespace SJBR\SrLanguageMenu\Domain\Model;
+
 /***************************************************************
-*  Copyright notice
-*
-*  (c) 2013-2014 Stanislas Rolland <typo3(arobas)sjbr.ca>
-*
-*  All rights reserved
-*
-*  This script is part of the TYPO3 project. The TYPO3 project 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.
-*
-*  The GNU General Public License can be found at
-*  http://www.gnu.org/copyleft/gpl.html.
-*
-*  This script 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.
-*
-*  This copyright notice MUST APPEAR in all copies of the script!
-***************************************************************/
+ *  Copyright notice
+ *
+ *  (c) 2013-2017 Stanislas Rolland <typo3(arobas)sjbr.ca>
+ *
+ *  All rights reserved
+ *
+ *  This script is part of the TYPO3 project. The TYPO3 project 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 3 of the License, or
+ *  (at your option) any later version.
+ *
+ *  The GNU General Public License can be found at
+ *  http://www.gnu.org/copyleft/gpl.html.
+ *
+ *  This script 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.
+ *
+ *  This copyright notice MUST APPEAR in all copies of the script!
+ ***************************************************************/
+
+use SJBR\StaticInfoTables\Domain\Model\Language;
+use TYPO3\CMS\Extbase\DomainObject\AbstractEntity;
 
 /**
  * The System Language model
  */
-class SystemLanguage extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
+class SystemLanguage extends AbstractEntity
+{
        /**
         * @var string System language name
         */
@@ -36,7 +41,7 @@ class SystemLanguage extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
        /**
         * @var \SJBR\StaticInfoTables\Domain\Model\Language
         */
-       protected $isoLanguage = NULL;
+       protected $isoLanguage = null;
 
        /**
         * Sets the language name
@@ -44,7 +49,8 @@ class SystemLanguage extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
         * @param string $title
         * @return void
         */
-       public function setTitle($title) {
+       public function setTitle($title)
+       {
                $this->title = $title;
        }
 
@@ -53,27 +59,30 @@ class SystemLanguage extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
         *
         * @return string
         */
-       public function getTitle() {
+       public function getTitle()
+       {
                return $this->title;
        }
 
        /**
         * Sets the ISO language
         *
-        * @param \SJBR\StaticInfoTables\Domain\Model\Language $isoLanguage
+        * @param Language $isoLanguage
         * @return void
         */
-       public function setIsoLanguage(\SJBR\StaticInfoTables\Domain\Model\Language $isoLanguage) {
+       public function setIsoLanguage(Language $isoLanguage)
+       {
                $this->isoLanguage = $isoLanguage;
        }
 
        /**
         * Gets the ISO language
         *
-        * @return \SJBR\StaticInfoTables\Domain\Model\Language
+        * @return Language
         */
-       public function getIsoLanguage() {
-               if ($this->isoLanguage !== NULL) {
+       public function getIsoLanguage()
+       {
+               if ($this->isoLanguage !== null) {
                        return clone $this->isoLanguage;
                } else {
                        return $this->isoLanguage;
index 9018ee0..821c8d9 100644 (file)
@@ -1,9 +1,10 @@
 <?php
 namespace SJBR\SrLanguageMenu\Domain\Repository;
+
 /***************************************************************
  *  Copyright notice
  *
- *  (c) 2013 Stanislas Rolland <typo3@sjbr.ca>
+ *  (c) 2013-2017 Stanislas Rolland <typo3@sjbr.ca>
  *  All rights reserved
  *
  *  This script is part of the TYPO3 project. The TYPO3 project is
@@ -25,36 +26,45 @@ namespace SJBR\SrLanguageMenu\Domain\Repository;
  *
  *  This copyright notice MUST APPEAR in all copies of the script!
  ***************************************************************/
+
+use SJBR\SrLanguageMenu\Domain\Model\Page;
+use TYPO3\CMS\Extbase\Object\ObjectManagerInterface;
+use TYPO3\CMS\Extbase\Persistence\Generic\Typo3QuerySettings;
+use TYPO3\CMS\Extbase\Persistence\QueryResultInterface;
+use TYPO3\CMS\Extbase\Persistence\Repository;
+
 /**
  * The page language overlay repository
  */
-class PageLanguageOverlayRepository extends \TYPO3\CMS\Extbase\Persistence\Repository {
+class PageLanguageOverlayRepository extends Repository
+{
        /**
         * Initialize the repository for unrestricted access to page language overlays
         *
-        * @param \TYPO3\CMS\Extbase\Object\ObjectManagerInterface $objectManager
+        * @param ObjectManagerInterface $objectManager
         * @return void
         */
-       public function __construct(\TYPO3\CMS\Extbase\Object\ObjectManagerInterface $objectManager) {
+       public function __construct(ObjectManagerInterface $objectManager)
+       {
                parent::__construct($objectManager);
-               $querySettings = $this->objectManager->get('TYPO3\\CMS\\Extbase\\Persistence\\Generic\\Typo3QuerySettings');
-               $querySettings->setRespectStoragePage(FALSE);
-               $querySettings->setRespectSysLanguage(FALSE);
+               $querySettings = $this->objectManager->get(Typo3QuerySettings::class);
+               $querySettings->setRespectStoragePage(false);
+               $querySettings->setRespectSysLanguage(false);
                $this->setDefaultQuerySettings($querySettings);
        }
 
        /**
-        * 
         *
-        * @param \SJBR\SrLanguageMenu\Domain\Model\Page $page
-        * @return \TYPO3\CMS\Extbase\Persistence\QueryResultInterface|array
-        */     
-       public function findByPage(\SJBR\SrLanguageMenu\Domain\Model\Page $page) {
+        *
+        * @param Page $page
+        * @return QueryResultInterface|array
+        */
+       public function findByPage(Page $page)
+       {
                $query = $this->createQuery();
                $query->matching(
                        $query->equals('pid', $page->getUid())
                );
                return $query->execute();
        }
-}
-?>
\ No newline at end of file
+}
\ No newline at end of file
index 434bbe4..8145885 100644 (file)
@@ -1,9 +1,10 @@
 <?php
 namespace SJBR\SrLanguageMenu\Domain\Repository;
+
 /***************************************************************
  *  Copyright notice
  *
- *  (c) 2013 Stanislas Rolland <typo3@sjbr.ca>
+ *  (c) 2013-2017 Stanislas Rolland <typo3@sjbr.ca>
  *  All rights reserved
  *
  *  This script is part of the TYPO3 project. The TYPO3 project is
@@ -25,9 +26,10 @@ namespace SJBR\SrLanguageMenu\Domain\Repository;
  *
  *  This copyright notice MUST APPEAR in all copies of the script!
  ***************************************************************/
+
+use TYPO3\CMS\Extbase\Persistence\Repository;
+
 /**
  * The page repository
  */
-class PageRepository extends \TYPO3\CMS\Extbase\Persistence\Repository {
-}
-?>
\ No newline at end of file
+class PageRepository extends Repository {}
\ No newline at end of file
index 878c096..a7fe740 100644 (file)
@@ -1,9 +1,10 @@
 <?php
 namespace SJBR\SrLanguageMenu\Domain\Repository;
+
 /***************************************************************
  *  Copyright notice
  *
- *  (c) 2013 Stanislas Rolland <typo3@sjbr.ca>
+ *  (c) 2013-2017 Stanislas Rolland <typo3@sjbr.ca>
  *  All rights reserved
  *
  *  This script is part of the TYPO3 project. The TYPO3 project is
@@ -25,26 +26,32 @@ namespace SJBR\SrLanguageMenu\Domain\Repository;
  *
  *  This copyright notice MUST APPEAR in all copies of the script!
  ***************************************************************/
+
+use TYPO3\CMS\Core\Utility\GeneralUtility;
+use TYPO3\CMS\Extbase\Persistence\QueryResultInterface;
+use TYPO3\CMS\Extbase\Persistence\Repository;
+
 /**
  * System language repository
  */
-class SystemLanguageRepository extends \TYPO3\CMS\Extbase\Persistence\Repository {
+class SystemLanguageRepository extends Repository
+{
        /**
         * Find all system language objects with uid in list
         * If no list is provided, find all system language objects
         *
         * @param string $list: list of uid's
-        * @return \TYPO3\CMS\Extbase\Persistence\QueryResultInterface|array all entries
+        * @return QueryResultInterface|array all entries
         */
-       public function findAllByUidInList($list = '') {
+       public function findAllByUidInList($list = '')
+       {
                if (empty($list)) {
                        return $this->findAll();
                } else {
                        $query = $this->createQuery();
-                       $list = \TYPO3\CMS\Core\Utility\GeneralUtility::trimExplode(',', $list, TRUE);
+                       $list = GeneralUtility::trimExplode(',', $list, true);
                        $query->matching($query->in('uid', $list));
                        return $query->execute();
                }
        }
-}
-?>
\ No newline at end of file
+}
\ No newline at end of file
index cf5d55a..923e371 100644 (file)
@@ -1,30 +1,35 @@
 <?php
 namespace SJBR\SrLanguageMenu\ViewHelpers\Widget;
+
 /***************************************************************
-*  Copyright notice
-*
-*  (c) 2013-2016 Stanislas Rolland <typo3(arobas)sjbr.ca>
-*  All rights reserved
-*
-*  This script is part of the TYPO3 project. The TYPO3 project 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.
-*
-*  The GNU General Public License can be found at
-*  http://www.gnu.org/copyleft/gpl.html.
-*  A copy is found in the textfile GPL.txt and important notices to the license
-*  from the author is found in LICENSE.txt distributed with these scripts.
-*
-*
-*  This script 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.
-*
-*  This copyright notice MUST APPEAR in all copies of the script!
-***************************************************************/
+ *  Copyright notice
+ *
+ *  (c) 2013-2017 Stanislas Rolland <typo3@sjbr.ca>
+ *  All rights reserved
+ *
+ *  This script is part of the TYPO3 project. The TYPO3 project 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.
+ *
+ *  The GNU General Public License can be found at
+ *  http://www.gnu.org/copyleft/gpl.html.
+ *  A copy is found in the textfile GPL.txt and important notices to the license
+ *  from the author is found in LICENSE.txt distributed with these scripts.
+ *
+ *
+ *  This script 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.
+ *
+ *  This copyright notice MUST APPEAR in all copies of the script!
+ ***************************************************************/
+
+use SJBR\SrLanguageMenu\Controller\MenuController;
+use TYPO3\CMS\Fluid\Core\Widget\AbstractWidgetViewHelper;
+
 /**
  * This ViewHelper renders a language menu.
  *
@@ -46,10 +51,10 @@ namespace SJBR\SrLanguageMenu\ViewHelpers\Widget;
  *             2 (the name of the language as set in the system language record in the TYPO3 backend),
  *             3 (the ISO language and, possibly, country codes of the language)
  */
-class MenuViewHelper extends \TYPO3\CMS\Fluid\Core\Widget\AbstractWidgetViewHelper {
-
+class MenuViewHelper extends AbstractWidgetViewHelper
+{
        /**
-        * @var \SJBR\SrLanguageMenu\Controller\MenuController
+        * @var MenuController
         */
        protected $controller;
 
@@ -59,14 +64,16 @@ class MenuViewHelper extends \TYPO3\CMS\Fluid\Core\Widget\AbstractWidgetViewHelp
        protected $backupExtensionName;
 
        /**
-        * @param \SJBR\SrLanguageMenu\Controller\MenuController $controller
+        * @param MenuController $controller
         * @return void
         */
-       public function injectController(\SJBR\SrLanguageMenu\Controller\MenuController $controller) {
+       public function injectController(MenuController $controller)
+       {
                $this->controller = $controller;
        }
 
-       public function initialize() {
+       public function initialize()
+       {
                $this->backupExtensionName = $this->controllerContext->getRequest()->getControllerExtensionName();
                $this->controllerContext->getRequest()->setControllerExtensionName('SrLanguageMenu');
        }
@@ -77,7 +84,8 @@ class MenuViewHelper extends \TYPO3\CMS\Fluid\Core\Widget\AbstractWidgetViewHelp
         * @param integer $languageTitle
         * @return string
         */
-       public function render($languages = NULL, $layout = NULL, $languageTitle = NULL) {
+       public function render($languages = null, $layout = null, $languageTitle = null)
+       {
                $result = $this->initiateSubRequest();
                $this->controllerContext->getRequest()->setControllerExtensionName($this->backupExtensionName);
                return $result;