[TASK] Add rule "return_type_declaration" to php-cs-fixer 53/56553/2
authorGeorg Ringer <georg.ringer@gmail.com>
Wed, 21 Mar 2018 19:45:15 +0000 (20:45 +0100)
committerGeorg Ringer <georg.ringer@gmail.com>
Wed, 4 Apr 2018 20:38:13 +0000 (22:38 +0200)
Unify the space before the colon of return type.

Resolves: #84515
Releases: master, 8.7
Change-Id: Ie107e574965f834c8fcaa8c2aa4f44f7cc41d7c5
Reviewed-on: https://review.typo3.org/56553
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
25 files changed:
Build/.php_cs
typo3/sysext/backend/Classes/Backend/ToolbarItems/UserToolbarItem.php
typo3/sysext/backend/Classes/Controller/ContentElement/ElementInformationController.php
typo3/sysext/backend/Classes/Controller/ContentElement/NewContentElementController.php
typo3/sysext/backend/Classes/Form/FormDataProvider/AbstractItemProvider.php
typo3/sysext/backend/Classes/Template/ModuleTemplate.php
typo3/sysext/core/Classes/Authentication/AbstractUserAuthentication.php
typo3/sysext/core/Classes/Configuration/Richtext.php
typo3/sysext/core/Classes/Database/ReferenceIndex.php
typo3/sysext/core/Classes/Messaging/AbstractMessage.php
typo3/sysext/core/Classes/Messaging/FlashMessageRendererResolver.php
typo3/sysext/core/Classes/Messaging/FlashMessageService.php
typo3/sysext/core/Classes/Messaging/Renderer/BootstrapRenderer.php
typo3/sysext/core/Classes/Messaging/Renderer/FlashMessageRendererInterface.php
typo3/sysext/core/Classes/Messaging/Renderer/ListRenderer.php
typo3/sysext/core/Classes/Messaging/Renderer/PlaintextRenderer.php
typo3/sysext/core/Classes/Session/Backend/SessionBackendInterface.php
typo3/sysext/core/Classes/Session/SessionManager.php
typo3/sysext/core/Classes/Utility/StringUtility.php
typo3/sysext/core/Classes/ViewHelpers/IconForRecordViewHelper.php
typo3/sysext/core/Tests/Unit/Utility/StringUtilityTest.php
typo3/sysext/frontend/Classes/ContentObject/TypolinkModifyLinkConfigForPageLinksHookInterface.php
typo3/sysext/frontend/Tests/Unit/Typolink/AbstractTypolinkBuilderTest.php
typo3/sysext/install/Classes/Controller/Action/Tool/AllConfiguration.php
typo3/sysext/rte_ckeditor/Classes/Form/Element/RichTextElement.php

index 812f396..3573867 100644 (file)
@@ -81,5 +81,6 @@ return PhpCsFixer\Config::create()
         'no_useless_else' => true,
         'phpdoc_types' => true,
         'phpdoc_types_order' => ['null_adjustment' => 'always_last', 'sort_algorithm' => 'none'],
+        'return_type_declaration' => ['space_before' => 'none'],
     ])
     ->setFinder($finder);
index 4783beb..12d575f 100644 (file)
@@ -122,7 +122,7 @@ class UserToolbarItem implements ToolbarItemInterface
      *
      * @return StandaloneView
      */
-    protected function getFluidTemplateObject(string $filename):StandaloneView
+    protected function getFluidTemplateObject(string $filename): StandaloneView
     {
         $view = GeneralUtility::makeInstance(StandaloneView::class);
         $view->setLayoutRootPaths(['EXT:backend/Resources/Private/Layouts']);
index 45d3572..627e67e 100644 (file)
@@ -248,7 +248,7 @@ class ElementInformationController
      *
      * @return array
      */
-    protected function getPageTitle() : array
+    protected function getPageTitle(): array
     {
         $pageTitle = [
             'title' => BackendUtility::getRecordTitle($this->table, $this->row, false)
@@ -272,7 +272,7 @@ class ElementInformationController
      *
      * @return array
      */
-    protected function getPreview() : array
+    protected function getPreview(): array
     {
         $preview = [];
         // Perhaps @todo in future: Also display preview for records - without fileObject
@@ -318,7 +318,7 @@ class ElementInformationController
      *
      * @return array
      */
-    protected function getPropertiesForTable() : array
+    protected function getPropertiesForTable(): array
     {
         $propertiesForTable = [];
         $lang = $this->getLanguageService();
@@ -426,7 +426,7 @@ class ElementInformationController
      *
      * @return array
      */
-    protected function getReferences() : array
+    protected function getReferences(): array
     {
         $references = [];
         switch ($this->type) {
@@ -451,7 +451,7 @@ class ElementInformationController
      *
      * @return array
      */
-    protected function getBackButton() : array
+    protected function getBackButton(): array
     {
         return ['returnUrl' => GeneralUtility::sanitizeLocalUrl(GeneralUtility::_GET('returnUrl'))];
     }
@@ -611,7 +611,7 @@ class ElementInformationController
      * @param string $ref Filename or uid
      * @return array
      */
-    protected function makeRefFrom($table, $ref) : array
+    protected function makeRefFrom($table, $ref): array
     {
         $refFromLines = [];
         $lang = $this->getLanguageService();
index 40c3f3b..6d82a81 100644 (file)
@@ -568,7 +568,7 @@ class NewContentElementController extends AbstractModule
      * @param string $filename
      * @return StandaloneView
      */
-    protected function getFluidTemplateObject(string $filename = 'Main.html'):StandaloneView
+    protected function getFluidTemplateObject(string $filename = 'Main.html'): StandaloneView
     {
         /** @var StandaloneView $view */
         $view = GeneralUtility::makeInstance(StandaloneView::class);
index 4016b9d..1052ed0 100644 (file)
@@ -953,7 +953,7 @@ abstract class AbstractItemProvider
      * @param string $localFieldName Current handle field name
      * @return QueryBuilder
      */
-    protected function buildForeignTableQueryBuilder(array $result, string $localFieldName):  QueryBuilder
+    protected function buildForeignTableQueryBuilder(array $result, string $localFieldName): QueryBuilder
     {
         $backendUser = $this->getBackendUser();
 
index c316d1e..11542cb 100644 (file)
@@ -740,7 +740,7 @@ class ModuleTemplate
     /**
      * @return bool
      */
-    public function isUiBlock() : bool
+    public function isUiBlock(): bool
     {
         return $this->uiBlock;
     }
index e59910f..53e4ee6 100644 (file)
@@ -1609,7 +1609,7 @@ abstract class AbstractUserAuthentication
      * @internal
      * @return string
      */
-    public function getSessionId() : string
+    public function getSessionId(): string
     {
         return $this->id;
     }
@@ -1618,7 +1618,7 @@ abstract class AbstractUserAuthentication
      * @internal
      * @return string
      */
-    public function getLoginType() : string
+    public function getLoginType(): string
     {
         return $this->loginType;
     }
index 49aeae7..1e8d636 100644 (file)
@@ -141,7 +141,7 @@ class Richtext
     /**
      * @return BackendUserAuthentication
      */
-    protected function getBackendUser() : BackendUserAuthentication
+    protected function getBackendUser(): BackendUserAuthentication
     {
         return $GLOBALS['BE_USER'];
     }
@@ -179,7 +179,7 @@ class Richtext
      * @param string $recordType Record type value
      * @return array
      */
-    protected function getPageTsConfiguration(string $table, string $field, int $pid, string $recordType) : array
+    protected function getPageTsConfiguration(string $table, string $field, int $pid, string $recordType): array
     {
         // Load PageTSconfig configuration
         $fullPageTsConfig = $this->getRtePageTsConfigOfPid($pid);
index 7627168..9a118d0 100644 (file)
@@ -340,7 +340,7 @@ class ReferenceIndex
      * @param array $record Record from $tableName
      * @return array
      */
-    protected function generateDataUsingRecord(string $tableName, array $record) : array
+    protected function generateDataUsingRecord(string $tableName, array $record): array
     {
         $this->relations = [];
         $deleteField = $GLOBALS['TCA'][$tableName]['ctrl']['delete'];
index 403fd1e..c4dccb6 100644 (file)
@@ -54,7 +54,7 @@ abstract class AbstractMessage
      *
      * @return string The message's title.
      */
-    public function getTitle() : string
+    public function getTitle(): string
     {
         return $this->title;
     }
@@ -74,7 +74,7 @@ abstract class AbstractMessage
      *
      * @return string The message.
      */
-    public function getMessage() : string
+    public function getMessage(): string
     {
         return $this->message;
     }
@@ -94,7 +94,7 @@ abstract class AbstractMessage
      *
      * @return int The message' severity, must be one of AbstractMessage::INFO or similar contstants
      */
-    public function getSeverity() : int
+    public function getSeverity(): int
     {
         return $this->severity;
     }
index c19d399..50487c9 100644 (file)
@@ -40,7 +40,7 @@ class FlashMessageRendererResolver
      *
      * @return FlashMessageRendererInterface
      */
-    public function resolve() : FlashMessageRendererInterface
+    public function resolve(): FlashMessageRendererInterface
     {
         $rendererClass = $this->resolveFlashMessageRenderClass();
         $renderer = GeneralUtility::makeInstance($rendererClass);
@@ -56,7 +56,7 @@ class FlashMessageRendererResolver
      *
      * @return string
      */
-    protected function resolveFlashMessageRenderClass() : string
+    protected function resolveFlashMessageRenderClass(): string
     {
         $context = $this->resolveContext();
         $renderClass = $this->renderer['_default'];
@@ -77,7 +77,7 @@ class FlashMessageRendererResolver
      *
      * @return string
      */
-    protected function resolveContext() : string
+    protected function resolveContext(): string
     {
         $context = '';
         if (TYPO3_REQUESTTYPE & TYPO3_REQUESTTYPE_CLI) {
index 2bccb30..aea35e1 100644 (file)
@@ -38,7 +38,7 @@ class FlashMessageService implements SingletonInterface
      * @return FlashMessageQueue
      * @api
      */
-    public function getMessageQueueByIdentifier($identifier = 'core.template.flashMessages') : FlashMessageQueue
+    public function getMessageQueueByIdentifier($identifier = 'core.template.flashMessages'): FlashMessageQueue
     {
         if (!isset($this->flashMessageQueues[$identifier])) {
             $this->flashMessageQueues[$identifier] = GeneralUtility::makeInstance(
index 3d74501..377612c 100644 (file)
@@ -54,7 +54,7 @@ class BootstrapRenderer implements FlashMessageRendererInterface
      * @param FlashMessage[] $flashMessages
      * @return string Representation of the flash message
      */
-    public function render(array $flashMessages) : string
+    public function render(array $flashMessages): string
     {
         return $this->getMessageAsMarkup($flashMessages);
     }
@@ -66,7 +66,7 @@ class BootstrapRenderer implements FlashMessageRendererInterface
      *
      * @return string The message severity class name
      */
-    protected function getClass(FlashMessage $flashMessage) : string
+    protected function getClass(FlashMessage $flashMessage): string
     {
         return 'alert-' . self::$classes[$flashMessage->getSeverity()];
     }
@@ -78,7 +78,7 @@ class BootstrapRenderer implements FlashMessageRendererInterface
      *
      * @return string The message severity icon name
      */
-    protected function getIconName(FlashMessage $flashMessage) : string
+    protected function getIconName(FlashMessage $flashMessage): string
     {
         return self::$icons[$flashMessage->getSeverity()];
     }
@@ -89,7 +89,7 @@ class BootstrapRenderer implements FlashMessageRendererInterface
      * @param FlashMessage[] $flashMessages
      * @return string
      */
-    protected function getMessageAsMarkup(array $flashMessages) : string
+    protected function getMessageAsMarkup(array $flashMessages): string
     {
         $markup = [];
         $markup[] = '<div class="typo3-messages">';
index 639d7e9..43db6bc 100644 (file)
@@ -28,5 +28,5 @@ interface FlashMessageRendererInterface
      * @param FlashMessage[] $flashMessages
      * @return string Representation of the flash message
      */
-    public function render(array $flashMessages) : string;
+    public function render(array $flashMessages): string;
 }
index fbc1108..1664114 100644 (file)
@@ -55,7 +55,7 @@ class ListRenderer implements FlashMessageRendererInterface
      * @param FlashMessage[] $flashMessages
      * @return string Representation of the flash message
      */
-    public function render(array $flashMessages) : string
+    public function render(array $flashMessages): string
     {
         return $this->getMessageAsMarkup($flashMessages);
     }
@@ -67,7 +67,7 @@ class ListRenderer implements FlashMessageRendererInterface
      *
      * @return string The message severity class name
      */
-    protected function getClass(FlashMessage $flashMessage) : string
+    protected function getClass(FlashMessage $flashMessage): string
     {
         return 'alert-' . self::$classes[$flashMessage->getSeverity()];
     }
@@ -79,7 +79,7 @@ class ListRenderer implements FlashMessageRendererInterface
      *
      * @return string The message severity icon name
      */
-    protected function getIconName(FlashMessage $flashMessage) : string
+    protected function getIconName(FlashMessage $flashMessage): string
     {
         return self::$icons[$flashMessage->getSeverity()];
     }
@@ -90,7 +90,7 @@ class ListRenderer implements FlashMessageRendererInterface
      * @param FlashMessage[] $flashMessages
      * @return string
      */
-    protected function getMessageAsMarkup(array $flashMessages) : string
+    protected function getMessageAsMarkup(array $flashMessages): string
     {
         $markup = [];
         $markup[] = '<ul class="typo3-messages">';
index 52f58e3..fa1f443 100644 (file)
@@ -56,7 +56,7 @@ class PlaintextRenderer implements FlashMessageRendererInterface
      * @param FlashMessage[] $flashMessages
      * @return string Representation of the flash message as plain text
      */
-    public function render(array $flashMessages) : string
+    public function render(array $flashMessages): string
     {
         $messages = [];
 
index 2dad5aa..e7b88f7 100644 (file)
@@ -46,7 +46,7 @@ interface SessionBackendInterface
      *
      * @return array Return a list of all user sessions. The list may be empty.
      */
-    public function getAll() : array;
+    public function getAll(): array;
 
     /**
      * Read session data
@@ -55,7 +55,7 @@ interface SessionBackendInterface
      * @return array Returns the session data
      * @throws SessionNotFoundException
      */
-    public function get(string $sessionId) : array;
+    public function get(string $sessionId): array;
 
     /**
      * Delete a session record
@@ -63,7 +63,7 @@ interface SessionBackendInterface
      * @param string $sessionId
      * @return bool true if the session was deleted, false it session could not be found
      */
-    public function remove(string $sessionId) : bool;
+    public function remove(string $sessionId): bool;
 
     /**
      * Write session data. This method prevents overriding existing session data.
@@ -75,7 +75,7 @@ interface SessionBackendInterface
      * @return array The newly created session record.
      * @throws SessionNotCreatedException
      */
-    public function set(string $sessionId, array $sessionData) : array;
+    public function set(string $sessionId, array $sessionData): array;
 
     /**
      * Updates the session data.
@@ -87,7 +87,7 @@ interface SessionBackendInterface
      * @return array $sessionData The newly updated session record.
      * @throws SessionNotUpdatedException
      */
-    public function update(string $sessionId, array $sessionData) : array;
+    public function update(string $sessionId, array $sessionData): array;
 
     /**
      * Garbage Collection
index 1d2f712..dd27f32 100644 (file)
@@ -45,7 +45,7 @@ class SessionManager implements SingletonInterface
      * @return SessionBackendInterface
      * @throws \InvalidArgumentException
      */
-    public function getSessionBackend(string $identifier) : SessionBackendInterface
+    public function getSessionBackend(string $identifier): SessionBackendInterface
     {
         if (!isset($this->sessionBackends[$identifier])) {
             if (!isset($GLOBALS['TYPO3_CONF_VARS']['SYS']['session'][$identifier]) || !is_array($GLOBALS['TYPO3_CONF_VARS']['SYS']['session'][$identifier])) {
@@ -70,7 +70,7 @@ class SessionManager implements SingletonInterface
      * @return SessionBackendInterface
      * @throws \InvalidArgumentException
      */
-    protected function createSessionBackendFromConfiguration(string $identifier, array $configuration) : SessionBackendInterface
+    protected function createSessionBackendFromConfiguration(string $identifier, array $configuration): SessionBackendInterface
     {
         $className = $configuration['backend'];
 
index 5678fc0..30ff5cc 100644 (file)
@@ -103,7 +103,7 @@ class StringUtility
      * @param string $selector
      * @return string
      */
-    public static function escapeCssSelector(string $selector) : string
+    public static function escapeCssSelector(string $selector): string
     {
         return preg_replace('/([#:.\\[\\],=@])/', '\\\\$1', $selector);
     }
index 47f5e1f..af3b68c 100644 (file)
@@ -47,7 +47,7 @@ class IconForRecordViewHelper extends AbstractViewHelper
      *
      * @return string
      */
-    public function render() : string
+    public function render(): string
     {
         $table = $this->arguments['table'];
         $size = $this->arguments['size'];
index 8fe5b96..af34f1b 100644 (file)
@@ -232,7 +232,7 @@ class StringUtilityTest extends \TYPO3\TestingFramework\Core\Unit\UnitTestCase
     /**
      * @return array
      */
-    public function escapeCssSelectorDataProvider() : array
+    public function escapeCssSelectorDataProvider(): array
     {
         return [
             ['data.field', 'data\\.field'],
index eb8e928..588f856 100644 (file)
@@ -29,5 +29,5 @@ interface TypolinkModifyLinkConfigForPageLinksHookInterface
      *
      * @return array The modified $linkConfiguration
      */
-    public function modifyPageLinkConfiguration(array $linkConfiguration, array $linkDetails, array $pageRow) : array;
+    public function modifyPageLinkConfiguration(array $linkConfiguration, array $linkDetails, array $pageRow): array;
 }
index 003034c..892aeed 100644 (file)
@@ -247,7 +247,7 @@ class AbstractTypolinkBuilderTest extends UnitTestCase
      *
      * @return array [[$expected, $conf, $name, $respectFrameSetOption, $fallbackTarget],]
      */
-    public function resolveTargetAttributeDataProvider() : array
+    public function resolveTargetAttributeDataProvider(): array
     {
         $targetName = $this->getUniqueId('name_');
         $target = $this->getUniqueId('target_');
index eccd742..ff9016b 100644 (file)
@@ -202,7 +202,7 @@ class AllConfiguration extends Action\AbstractAction
     /**
      * @return array
      */
-    protected function getDefaultConfigArrayComments() : array
+    protected function getDefaultConfigArrayComments(): array
     {
         /** @var \TYPO3\CMS\Core\Configuration\ConfigurationManager $configurationManager */
         $configurationManager = GeneralUtility::makeInstance(\TYPO3\CMS\Core\Configuration\ConfigurationManager::class);
index fc4690c..954fc64 100644 (file)
@@ -64,7 +64,7 @@ class RichTextElement extends AbstractFormElement
      * @return array
      * @throws \InvalidArgumentException
      */
-    public function render() : array
+    public function render(): array
     {
         $resultArray = $this->initializeResultArray();
         $parameterArray = $this->data['parameterArray'];
@@ -166,7 +166,7 @@ class RichTextElement extends AbstractFormElement
      * @param string $fieldId
      * @return string
      */
-    protected function getCkEditorRequireJsModuleCode(string $fieldId) : string
+    protected function getCkEditorRequireJsModuleCode(string $fieldId): string
     {
         $configuration = $this->prepareConfigurationForEditor();