[TASK] Add rule "return_type_declaration" to php-cs-fixer 14/56414/5
authorGeorg Ringer <georg.ringer@gmail.com>
Wed, 21 Mar 2018 19:45:15 +0000 (20:45 +0100)
committerWouter Wolters <typo3@wouterwolters.nl>
Thu, 22 Mar 2018 08:49:54 +0000 (09:49 +0100)
Unify the space before the colon of return type.

Resolves: #84515
Releases: master, 8.7
Change-Id: Ie107e574965f834c8fcaa8c2aa4f44f7cc41d7c5
Reviewed-on: https://review.typo3.org/56414
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Mathias Schreiber <mathias.schreiber@typo3.com>
Tested-by: Mathias Schreiber <mathias.schreiber@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
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/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/Tests/Unit/Utility/StringUtilityTest.php
typo3/sysext/frontend/Classes/ContentObject/ScalableVectorGraphicsContentObject.php
typo3/sysext/frontend/Classes/ContentObject/TypolinkModifyLinkConfigForPageLinksHookInterface.php
typo3/sysext/frontend/Tests/Unit/Controller/TypoScriptFrontendControllerTest.php
typo3/sysext/frontend/Tests/Unit/Typolink/AbstractTypolinkBuilderTest.php
typo3/sysext/rte_ckeditor/Classes/Form/Element/RichTextElement.php
typo3/sysext/t3editor/Classes/Controller/TypoScriptReferenceController.php

index 2eae965..79e48f8 100644 (file)
@@ -83,5 +83,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 0eec6d9..5793ecd 100644 (file)
@@ -171,7 +171,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 e7c8a24..84f5b50 100644 (file)
@@ -291,7 +291,7 @@ class ElementInformationController
      *
      * @return array
      */
-    protected function getPageTitle() : array
+    protected function getPageTitle(): array
     {
         $pageTitle = [
             'title' => BackendUtility::getRecordTitle($this->table, $this->row, false)
@@ -315,7 +315,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
@@ -361,7 +361,7 @@ class ElementInformationController
      *
      * @return array
      */
-    protected function getPropertiesForTable() : array
+    protected function getPropertiesForTable(): array
     {
         $propertiesForTable = [];
         $lang = $this->getLanguageService();
@@ -470,7 +470,7 @@ class ElementInformationController
      * @param ServerRequestInterface $request
      * @return array
      */
-    protected function getReferences(ServerRequestInterface $request) : array
+    protected function getReferences(ServerRequestInterface $request): array
     {
         $references = [];
         switch ($this->type) {
index 0b64bb2..8c37b36 100644 (file)
@@ -954,7 +954,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 35994f3..7aed5f9 100644 (file)
@@ -735,7 +735,7 @@ class ModuleTemplate
     /**
      * @return bool
      */
-    public function isUiBlock() : bool
+    public function isUiBlock(): bool
     {
         return $this->uiBlock;
     }
index bb5ed44..bb3edd2 100644 (file)
@@ -1527,7 +1527,7 @@ abstract class AbstractUserAuthentication implements LoggerAwareInterface
      * @internal
      * @return string
      */
-    public function getSessionId() : string
+    public function getSessionId(): string
     {
         return $this->id;
     }
@@ -1536,7 +1536,7 @@ abstract class AbstractUserAuthentication implements LoggerAwareInterface
      * @internal
      * @return string
      */
-    public function getLoginType() : string
+    public function getLoginType(): string
     {
         return $this->loginType;
     }
index 25cbd7c..59085e1 100644 (file)
@@ -138,7 +138,7 @@ class Richtext
     /**
      * @return BackendUserAuthentication
      */
-    protected function getBackendUser() : BackendUserAuthentication
+    protected function getBackendUser(): BackendUserAuthentication
     {
         return $GLOBALS['BE_USER'];
     }
@@ -176,7 +176,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 40ff6c3..f4915cd 100644 (file)
@@ -372,7 +372,7 @@ class ReferenceIndex implements LoggerAwareInterface
      * @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 4b5f34c..d19c88d 100644 (file)
@@ -54,7 +54,7 @@ abstract class AbstractMessage implements \JsonSerializable
      *
      * @return string The message's title.
      */
-    public function getTitle() : string
+    public function getTitle(): string
     {
         return $this->title;
     }
@@ -74,7 +74,7 @@ abstract class AbstractMessage implements \JsonSerializable
      *
      * @return string The message.
      */
-    public function getMessage() : string
+    public function getMessage(): string
     {
         return $this->message;
     }
@@ -94,7 +94,7 @@ abstract class AbstractMessage implements \JsonSerializable
      *
      * @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 1f18ff1..8752cb2 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 d26a9b5..608c168 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 fec6346..d58edb3 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 b84be03..2eb73cb 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 c3e3b6b..afc6650 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 41794ba..f55eb2e 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 25e971c..f2f7591 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 a541d79..6610129 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])) {
             $configuration = $GLOBALS['TYPO3_CONF_VARS']['SYS']['session'][$identifier] ?? false;
@@ -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 5e0b77d..54efb7d 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 3bcb4fc..250a098 100644 (file)
@@ -233,7 +233,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 36e1fea..8d5a0b3 100644 (file)
@@ -28,7 +28,7 @@ class ScalableVectorGraphicsContentObject extends AbstractContentObject
      * @param array $conf Array of TypoScript properties
      * @return string
      */
-    public function render($conf = []) : string
+    public function render($conf = []): string
     {
         $renderMode = isset($conf['renderMode.'])
             ? $this->cObj->stdWrap($conf['renderMode'], $conf['renderMode.'])
@@ -46,7 +46,7 @@ class ScalableVectorGraphicsContentObject extends AbstractContentObject
      *
      * @return string
      */
-    protected function renderInline(array $conf) : string
+    protected function renderInline(array $conf): string
     {
         $src = $this->resolveAbsoluteSourcePath($conf);
         list($width, $height, $isDefaultWidth, $isDefaultHeight) = $this->getDimensions($conf);
@@ -88,7 +88,7 @@ class ScalableVectorGraphicsContentObject extends AbstractContentObject
      *
      * @return string
      */
-    protected function renderObject(array $conf) : string
+    protected function renderObject(array $conf): string
     {
         $src = $this->resolveAbsoluteSourcePath($conf);
         list($width, $height) = $this->getDimensions($conf);
@@ -139,7 +139,7 @@ class ScalableVectorGraphicsContentObject extends AbstractContentObject
      *
      * @return string
      */
-    protected function resolveAbsoluteSourcePath(array $conf) : string
+    protected function resolveAbsoluteSourcePath(array $conf): string
     {
         $src = isset($conf['src.']) ? $this->cObj->stdWrap($conf['src'], $conf['src.']) : $conf['src'];
         return GeneralUtility::getFileAbsFileName($src);
@@ -150,7 +150,7 @@ class ScalableVectorGraphicsContentObject extends AbstractContentObject
      *
      * @return array
      */
-    protected function getDimensions(array $conf) : array
+    protected function getDimensions(array $conf): array
     {
         $isDefaultWidth = false;
         $isDefaultHeight = false;
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 65d2fc0..c07a5f3 100644 (file)
@@ -429,7 +429,7 @@ class TypoScriptFrontendControllerTest extends \TYPO3\TestingFramework\Core\Unit
         $this->assertEquals($expected, $this->subject->linkVars);
     }
 
-    public function calculateLinkVarsDataProvider() : array
+    public function calculateLinkVarsDataProvider(): array
     {
         return [
             'simple variable' => [
index 3d5b589..25e8fbe 100644 (file)
@@ -252,7 +252,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 708baa9..f594507 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'];
@@ -162,7 +162,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();
 
index 5cd9cea..74837b7 100644 (file)
@@ -70,7 +70,7 @@ class TypoScriptReferenceController
      *
      * @return array
      */
-    protected function getTypes() : array
+    protected function getTypes(): array
     {
         $types = $this->xmlDoc->getElementsByTagName('type');
         $typeArr = [];