diff --git a/Documentation/source/configuration.rst b/Documentation/source/configuration.rst index 5cc0c0e4ae4a4a7aeac40d61499feb0eae57c539..cfe6207888b5e0d15ccf94b945e27605202d9c38 100644 --- a/Documentation/source/configuration.rst +++ b/Documentation/source/configuration.rst @@ -143,6 +143,52 @@ Using ``runtime-set``: --runtime-set removedFunctionConfigFiles "/Some/Absolute/Path/*.yaml" +.. _configuration-removedSignalConfigFiles: + +removedSignalConfigFiles +^^^^^^^^^^^^^^^^^^^^^^^^^^ + +Configure where to look for configuration files defining the removed signals. Default +is ``Configuration/Removed/Signals/*.yaml`` inside the standard itself. We already try to deliver +as much as possible. +Globing is used, so placeholders like ``*`` are possible, see +https://secure.php.net/manual/en/function.glob.php + +Using :file:`ruleset.xml`: + +.. code:: xml + + <config name="removedSignalConfigFiles" value="/Some/Absolute/Path/*.yaml"/> + +Using ``runtime-set``: + +.. code:: bash + + --runtime-set removedSignalConfigFiles "/Some/Absolute/Path/*.yaml" + +.. _configuration-removedHookConfigFiles: + +removedHookConfigFiles +^^^^^^^^^^^^^^^^^^^^^^^^^^ + +Configure where to look for configuration files defining the removed hooks. Default +is ``Configuration/Removed/Hooks/*.yaml`` inside the standard itself. We already try to deliver +as much as possible. +Globing is used, so placeholders like ``*`` are possible, see +https://secure.php.net/manual/en/function.glob.php + +Using :file:`ruleset.xml`: + +.. code:: xml + + <config name="removedHookConfigFiles" value="/Some/Absolute/Path/*.yaml"/> + +Using ``runtime-set``: + +.. code:: bash + + --runtime-set removedHookConfigFiles "/Some/Absolute/Path/*.yaml" + .. _configuration-removedConstantConfigFiles: removedConstantConfigFiles diff --git a/Documentation/source/features.rst b/Documentation/source/features.rst index 757633582723048b3a028c35cd5dd27dd0575a89..8b4c5e19aa190aa1b22e7c8b936de0f616731728 100644 --- a/Documentation/source/features.rst +++ b/Documentation/source/features.rst @@ -106,6 +106,12 @@ functions. For configuration options see :ref:`configuration-removedConstantConf Check for usage of *removed PHP classes*. The classes are configured in same way as removed functions. For configuration options see :ref:`configuration-removedClassConfigFiles`. +Check for usage of *removed signals*. The signals are configured in same way as removed +functions. For configuration options see :ref:`configuration-removedSignalConfigFiles`. + +Check for usage of *removed hooks*. The hooks are configured in same way as removed +functions. For configuration options see :ref:`configuration-removedHookConfigFiles`. + Check for usage of *removed TypoScript*. The TypoScript objects are configured in same way as removed functions. For configuration options see :ref:`configuration-removedTypoScriptConfigFiles`. This will check whether you are using already removed TypoScript parts, supported are: diff --git a/src/Standards/Typo3Update/Configuration/Removed/Hooks/7.3.yaml b/src/Standards/Typo3Update/Configuration/Removed/Hooks/7.3.yaml new file mode 100644 index 0000000000000000000000000000000000000000..eb05d063afee4a15de70358493c818450be15cb5 --- /dev/null +++ b/src/Standards/Typo3Update/Configuration/Removed/Hooks/7.3.yaml @@ -0,0 +1,11 @@ +# Breaking changes in 7.3: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.3/Index.html#breaking-changes +'7.3': + typo3/index.php->loginScriptHook: + replacement: 'Use the new backend login form API' + docsUrl: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.3/Breaking-66669-BackendLoginControllerRefactored.html + typo3/index.php->loginFormHook: + replacement: 'Use the new backend login form API' + docsUrl: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.3/Breaking-66669-BackendLoginControllerRefactored.html + t3lib/class.t3lib_tceforms.php->getSingleFieldClass: + replacement: null + docsUrl: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.3/Breaking-63846-FormEngineRefactoring.html diff --git a/src/Standards/Typo3Update/Configuration/Removed/Hooks/7.5.yaml b/src/Standards/Typo3Update/Configuration/Removed/Hooks/7.5.yaml new file mode 100644 index 0000000000000000000000000000000000000000..889bd3c64800671bdc87685f3b480f5c53974e40 --- /dev/null +++ b/src/Standards/Typo3Update/Configuration/Removed/Hooks/7.5.yaml @@ -0,0 +1,5 @@ +# Breaking changes in 7.5: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.5/Index.html#breaking-changes +'7.5': + t3lib/class.t3lib_tceforms.php->getMainFieldsClass: + replacement: 'Use FormDataProvider to change data given to the render engine of FormEngine from now on' + docsUrl: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.5/Breaking-69568-FormEngine.html diff --git a/src/Standards/Typo3Update/Configuration/Removed/Signals/7.2.yaml b/src/Standards/Typo3Update/Configuration/Removed/Signals/7.2.yaml new file mode 100644 index 0000000000000000000000000000000000000000..8276a377d2d7873a37dc587c0d432a7086df92f6 --- /dev/null +++ b/src/Standards/Typo3Update/Configuration/Removed/Signals/7.2.yaml @@ -0,0 +1,17 @@ +# Breaking changes in 7.2: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.2/Index.html#breaking-changes +'7.2': + \TYPO3\CMS\Backend\Controller\LoginController::makeLoginBoxImage: + replacement: 'Use the introduced Fluid view to adapt the login screen to your demands' + docsUrl: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.2/Breaking-65939-BackendLoginRefactoring.html + \TYPO3\CMS\Backend\Controller\LoginController::wrapLoginForm: + replacement: 'Use the introduced Fluid view to adapt the login screen to your demands' + docsUrl: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.2/Breaking-65939-BackendLoginRefactoring.html + \TYPO3\CMS\Backend\Controller\LoginController::makeLoginNews: + replacement: 'Use the introduced Fluid view to adapt the login screen to your demands' + docsUrl: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.2/Breaking-65939-BackendLoginRefactoring.html + \TYPO3\CMS\Backend\Controller\LoginController::makeLoginForm: + replacement: 'Use the introduced Fluid view to adapt the login screen to your demands' + docsUrl: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.2/Breaking-65939-BackendLoginRefactoring.html + \TYPO3\CMS\Backend\Controller\LoginController::makeLogoutForm: + replacement: 'Use the introduced Fluid view to adapt the login screen to your demands' + docsUrl: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.2/Breaking-65939-BackendLoginRefactoring.html diff --git a/src/Standards/Typo3Update/Configuration/Removed/Signals/7.3.yaml b/src/Standards/Typo3Update/Configuration/Removed/Signals/7.3.yaml new file mode 100644 index 0000000000000000000000000000000000000000..e8684eb2b045fdb3100fa4323046ebe5d63f547f --- /dev/null +++ b/src/Standards/Typo3Update/Configuration/Removed/Signals/7.3.yaml @@ -0,0 +1,5 @@ +# Breaking changes in 7.3: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.3/Index.html#breaking-changes +'7.3': + \TYPO3\CMS\Lang\Service\UpdateTranslationService::postProcessMirrorUrl: + replacement: 'Change the slot to use the \TYPO3\CMS\Lang\Service\TranslationService::postProcessMirrorUrl signal' + docsUrl: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.3/Breaking-62983-PostProcessMirrorUrlSignalHasMoved.html diff --git a/src/Standards/Typo3Update/Configuration/Removed/Signals/7.4.yaml b/src/Standards/Typo3Update/Configuration/Removed/Signals/7.4.yaml new file mode 100644 index 0000000000000000000000000000000000000000..8f81ad15169fc81ff0196dcce99b9b6ef4eb7cc9 --- /dev/null +++ b/src/Standards/Typo3Update/Configuration/Removed/Signals/7.4.yaml @@ -0,0 +1,5 @@ +# Breaking changes in 7.4: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.4/Index.html#breaking-changes +'7.4': + \TYPO3\CMS\Core\Resource\ResourceStorage::preFileAdd: + replacement: 'The signal will now receive an empty string in $sourceFilePath' + docsUrl: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.4/Breaking-67545-PreFileAddSignalBehaviourChanged.html diff --git a/src/Standards/Typo3Update/Options.php b/src/Standards/Typo3Update/Options.php index f5a47778128f812c1c01b2134f7178c37145d4d2..d5097fc45a85db2f86f62cd6c901b7d34034e4d5 100644 --- a/src/Standards/Typo3Update/Options.php +++ b/src/Standards/Typo3Update/Options.php @@ -69,6 +69,32 @@ class Options ); } + /** + * Returns an array of absolute file names containing removed function configurations. + * + * @return array<string> + */ + public static function getRemovedSignalConfigFiles() + { + return static::getOptionFileNames( + 'removedSignalConfigFiles', + __DIR__ . '/Configuration/Removed/Signals/*.yaml' + ); + } + + /** + * Returns an array of absolute file names containing removed function configurations. + * + * @return array<string> + */ + public static function getRemovedHookConfigFiles() + { + return static::getOptionFileNames( + 'removedHookConfigFiles', + __DIR__ . '/Configuration/Removed/Hooks/*.yaml' + ); + } + /** * Returns an array of absolute file names containing removed constant configurations. * diff --git a/src/Standards/Typo3Update/Sniffs/ExtendedPhpCsSupportTrait.php b/src/Standards/Typo3Update/Sniffs/ExtendedPhpCsSupportTrait.php index 2368d4d1b4df860ca713058dc57e2a3431cf9c09..01898b00926d5c5201d23ba84dd0c6d038e7f7d4 100644 --- a/src/Standards/Typo3Update/Sniffs/ExtendedPhpCsSupportTrait.php +++ b/src/Standards/Typo3Update/Sniffs/ExtendedPhpCsSupportTrait.php @@ -66,4 +66,35 @@ trait ExtendedPhpCsSupportTrait return true; } + + /** + * Returns all parameters for function call as values. + * Quotes are removed from strings. + * + * @param PhpCsFile $phpcsFile + * @param int $stackPtr + * + * @return array<string> + */ + protected function getFunctionCallParameters(PhpCsFile $phpcsFile, $stackPtr) + { + $start = $phpcsFile->findNext(T_OPEN_PARENTHESIS, $stackPtr) + 1; + $parameters = explode(',', $phpcsFile->getTokensAsString( + $start, + $phpcsFile->findNext(T_CLOSE_PARENTHESIS, $stackPtr) - $start + )); + + return array_map([$this, 'getStringContent'], $parameters); + } + + /** + * Remove special chars like quotes from string. + * + * @param string $string + * @return string + */ + public function getStringContent($string) + { + return trim($string, " \t\n\r\0\x0B'\""); + } } diff --git a/src/Standards/Typo3Update/Sniffs/Removed/GenericHookSniff.php b/src/Standards/Typo3Update/Sniffs/Removed/GenericHookSniff.php new file mode 100644 index 0000000000000000000000000000000000000000..75a1ecf10d72a251de2acae25349ea90f50e43a0 --- /dev/null +++ b/src/Standards/Typo3Update/Sniffs/Removed/GenericHookSniff.php @@ -0,0 +1,69 @@ +<?php + +/* + * Copyright (C) 2017 Daniel Siepmann <coding@daniel-siepmann.de> + * + * 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. + */ + +use PHP_CodeSniffer_File as PhpCsFile; +use PHP_CodeSniffer_Tokens as PhpCsTokens; +use Typo3Update\Options; +use Typo3Update\Sniffs\ExtendedPhpCsSupportTrait; +use Typo3Update\Sniffs\Removed\AbstractGenericPhpUsage; + +class Typo3Update_Sniffs_Removed_GenericHookSniff extends AbstractGenericPhpUsage +{ + use ExtendedPhpCsSupportTrait; + + public function register() + { + return PhpCsTokens::$stringTokens; + } + + protected function getRemovedConfigFiles() + { + return Options::getRemovedHookConfigFiles(); + } + + protected function findRemoved(PhpCsFile $phpcsFile, $stackPtr) + { + $tokens = $phpcsFile->getTokens(); + $firstPart = $this->getStringContent($tokens[$stackPtr]['content']); + $secondPart = $this->getStringContent($tokens[$phpcsFile->findNext(PhpCsTokens::$stringTokens, $stackPtr + 1)]['content']); + + $lookup = $firstPart . '->' . $secondPart; + + if ($this->configured->isRemoved($lookup) === false) { + return []; + } + + return [$this->configured->getRemoved($lookup)]; + } + + protected function getIdentifier(array $config) + { + $search = ['/', '.']; + $replace = ['-']; + + return str_replace($search, $replace, $config['fqcn']) . $config['name']; + } + + protected function getOldUsage(array $config) + { + return '["' . $config['fqcn'] . '"]["' . $config['name'] . '"] = ...'; + } +} diff --git a/src/Standards/Typo3Update/Sniffs/Removed/GenericSignalSniff.php b/src/Standards/Typo3Update/Sniffs/Removed/GenericSignalSniff.php new file mode 100644 index 0000000000000000000000000000000000000000..d72601c39cdba9dbc133e778baa525fa167ac5b2 --- /dev/null +++ b/src/Standards/Typo3Update/Sniffs/Removed/GenericSignalSniff.php @@ -0,0 +1,86 @@ +<?php + +/* + * Copyright (C) 2017 Daniel Siepmann <coding@daniel-siepmann.de> + * + * 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. + */ + +use PHP_CodeSniffer_File as PhpCsFile; +use Typo3Update\Options; +use Typo3Update\Sniffs\ExtendedPhpCsSupportTrait; +use Typo3Update\Sniffs\Removed\AbstractGenericPhpUsage; + +class Typo3Update_Sniffs_Removed_GenericSignalSniff extends AbstractGenericPhpUsage +{ + use ExtendedPhpCsSupportTrait; + + protected function getRemovedConfigFiles() + { + return Options::getRemovedSignalConfigFiles(); + } + + protected function findRemoved(PhpCsFile $phpcsFile, $stackPtr) + { + if (!$this->isFunctionCall($phpcsFile, $stackPtr) || $phpcsFile->getTokens()[$stackPtr]['content'] !== 'connect') { + return []; + } + + $parameters = $this->getFunctionCallParameters($phpcsFile, $stackPtr); + if (count($parameters) < 4) { + return []; + } + + $lookup = $this->getClass($parameters[0]) . '::' . $parameters[1]; + + if ($this->configured->isRemoved($lookup) === false) { + return []; + } + + return [$this->configured->getRemoved($lookup)]; + } + + /** + * Returns same formatted class representation for incoming strings. + * + * @param string $string + * @return string + */ + protected function getClass($string) + { + $search = [ + '::class', + '\\\\', + ]; + $replace = [ + '', + '\\', + ]; + + $string = str_replace($search, $replace, $string); + + if ($string[0] !== '\\') { + $string = '\\' . $string; + } + + return $string; + } + + protected function getOldUsage(array $config) + { + return $config['fqcn'] . '::' . $config['name']; + } +} diff --git a/tests/Fixtures/Standards/Typo3Update/Sniffs/Removed/GenericHookSniff/Expected.json b/tests/Fixtures/Standards/Typo3Update/Sniffs/Removed/GenericHookSniff/Expected.json new file mode 100644 index 0000000000000000000000000000000000000000..6b3c4a10a5132556f3c440b533272c4b519698c6 --- /dev/null +++ b/tests/Fixtures/Standards/Typo3Update/Sniffs/Removed/GenericHookSniff/Expected.json @@ -0,0 +1,42 @@ +{ + "files": { + "InputFileForIssues.php": { + "errors": 0, + "messages": [ + { + "column": 43, + "fixable": false, + "line": 22, + "message": "Calls to removed code are not allowed; found [\"typo3/index.php\"][\"loginScriptHook\"] = .... Removed in 7.3. Use the new backend login form API. See: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.3/Breaking-66669-BackendLoginControllerRefactored.html", + "severity": 5, + "source": "Typo3Update.Removed.GenericHook.typo3-indexphploginScriptHook", + "type": "WARNING" + }, + { + "column": 43, + "fixable": false, + "line": 23, + "message": "Calls to removed code are not allowed; found [\"typo3/index.php\"][\"loginFormHook\"] = .... Removed in 7.3. Use the new backend login form API. See: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.3/Breaking-66669-BackendLoginControllerRefactored.html", + "severity": 5, + "source": "Typo3Update.Removed.GenericHook.typo3-indexphploginFormHook", + "type": "WARNING" + }, + { + "column": 9, + "fixable": false, + "line": 28, + "message": "Calls to removed code are not allowed; found [\"typo3/index.php\"][\"loginFormHook\"] = .... Removed in 7.3. Use the new backend login form API. See: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.3/Breaking-66669-BackendLoginControllerRefactored.html", + "severity": 5, + "source": "Typo3Update.Removed.GenericHook.typo3-indexphploginFormHook", + "type": "WARNING" + } + ], + "warnings": 3 + } + }, + "totals": { + "errors": 0, + "fixable": 0, + "warnings": 3 + } +} diff --git a/tests/Fixtures/Standards/Typo3Update/Sniffs/Removed/GenericHookSniff/InputFileForIssues.php b/tests/Fixtures/Standards/Typo3Update/Sniffs/Removed/GenericHookSniff/InputFileForIssues.php new file mode 100644 index 0000000000000000000000000000000000000000..1bbb54659728dcfd3c9fc75fc3d0d081bd7867df --- /dev/null +++ b/tests/Fixtures/Standards/Typo3Update/Sniffs/Removed/GenericHookSniff/InputFileForIssues.php @@ -0,0 +1,33 @@ +<?php + +/* + * Copyright (C) 2017 Daniel Siepmann <coding@daniel-siepmann.de> + * + * 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. + */ + +$GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['typo3/index.php']['loginScriptHook'] = 'Configuration'; +$GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']["typo3/index.php"]["loginFormHook"] = 'Configuration'; + +array_merge( + $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS'], + [ + 'typo3/index.php' => [ + 'loginFormHook' => 'Configuration', + ], + ] +); + diff --git a/tests/Fixtures/Standards/Typo3Update/Sniffs/Removed/GenericSignalSniff/Expected.json b/tests/Fixtures/Standards/Typo3Update/Sniffs/Removed/GenericSignalSniff/Expected.json new file mode 100644 index 0000000000000000000000000000000000000000..8a194da3209ebc8b062e46fb59d0ac7119444a8c --- /dev/null +++ b/tests/Fixtures/Standards/Typo3Update/Sniffs/Removed/GenericSignalSniff/Expected.json @@ -0,0 +1,60 @@ +{ + "files": { + "InputFileForIssues.php": { + "errors": 0, + "messages": [ + { + "column": 28, + "fixable": false, + "line": 25, + "message": "Calls to removed code are not allowed; found \\TYPO3\\CMS\\Backend\\Controller\\LoginController::makeLoginNews. Removed in 7.2. Use the introduced Fluid view to adapt the login screen to your demands. See: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.2/Breaking-65939-BackendLoginRefactoring.html", + "severity": 5, + "source": "Typo3Update.Removed.GenericSignal.LoginController.makeLoginNews", + "type": "WARNING" + }, + { + "column": 28, + "fixable": false, + "line": 32, + "message": "Calls to removed code are not allowed; found \\TYPO3\\CMS\\Backend\\Controller\\LoginController::makeLoginNews. Removed in 7.2. Use the introduced Fluid view to adapt the login screen to your demands. See: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.2/Breaking-65939-BackendLoginRefactoring.html", + "severity": 5, + "source": "Typo3Update.Removed.GenericSignal.LoginController.makeLoginNews", + "type": "WARNING" + }, + { + "column": 28, + "fixable": false, + "line": 39, + "message": "Calls to removed code are not allowed; found \\TYPO3\\CMS\\Backend\\Controller\\LoginController::makeLoginNews. Removed in 7.2. Use the introduced Fluid view to adapt the login screen to your demands. See: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.2/Breaking-65939-BackendLoginRefactoring.html", + "severity": 5, + "source": "Typo3Update.Removed.GenericSignal.LoginController.makeLoginNews", + "type": "WARNING" + }, + { + "column": 28, + "fixable": false, + "line": 46, + "message": "Calls to removed code are not allowed; found \\TYPO3\\CMS\\Backend\\Controller\\LoginController::makeLoginNews. Removed in 7.2. Use the introduced Fluid view to adapt the login screen to your demands. See: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.2/Breaking-65939-BackendLoginRefactoring.html", + "severity": 5, + "source": "Typo3Update.Removed.GenericSignal.LoginController.makeLoginNews", + "type": "WARNING" + }, + { + "column": 28, + "fixable": false, + "line": 53, + "message": "Calls to removed code are not allowed; found \\TYPO3\\CMS\\Backend\\Controller\\LoginController::makeLoginNews. Removed in 7.2. Use the introduced Fluid view to adapt the login screen to your demands. See: https://docs.typo3.org/typo3cms/extensions/core/7.6/Changelog/7.2/Breaking-65939-BackendLoginRefactoring.html", + "severity": 5, + "source": "Typo3Update.Removed.GenericSignal.LoginController.makeLoginNews", + "type": "WARNING" + } + ], + "warnings": 5 + } + }, + "totals": { + "errors": 0, + "fixable": 0, + "warnings": 5 + } +} diff --git a/tests/Fixtures/Standards/Typo3Update/Sniffs/Removed/GenericSignalSniff/InputFileForIssues.php b/tests/Fixtures/Standards/Typo3Update/Sniffs/Removed/GenericSignalSniff/InputFileForIssues.php new file mode 100644 index 0000000000000000000000000000000000000000..608d8b9ad50562741a87cc6a3cfaf3e400deca4f --- /dev/null +++ b/tests/Fixtures/Standards/Typo3Update/Sniffs/Removed/GenericSignalSniff/InputFileForIssues.php @@ -0,0 +1,59 @@ +<?php + +/* + * Copyright (C) 2017 Daniel Siepmann <coding@daniel-siepmann.de> + * + * 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. + */ + +call_user_func(function () { + $signalSlotDispatcher = GeneralUtility::makeInstance(Dispatcher::class); + + $signalSlotDispatcher->connect( + \TYPO3\CMS\Backend\Controller\LoginController::class, + 'makeLoginNews', + 'OwnClass', + 'ownMethod' + ); + + $signalSlotDispatcher->connect( + TYPO3\CMS\Backend\Controller\LoginController::class, + 'makeLoginNews', + 'OwnClass', + 'ownMethod' + ); + + $signalSlotDispatcher->connect( + '\TYPO3\CMS\Backend\Controller\LoginController', + 'makeLoginNews', + 'OwnClass', + 'ownMethod' + ); + + $signalSlotDispatcher->connect( + 'TYPO3\CMS\Backend\Controller\LoginController', + 'makeLoginNews', + 'OwnClass', + 'ownMethod' + ); + + $signalSlotDispatcher->connect( + "TYPO3\\CMS\\Backend\\Controller\\LoginController", + 'makeLoginNews', + 'OwnClass', + 'ownMethod' + ); +}); diff --git a/tests/Sniffs/Removed/GenericHookSniffTest.php b/tests/Sniffs/Removed/GenericHookSniffTest.php new file mode 100644 index 0000000000000000000000000000000000000000..15ed1e69a944f998a318c2b389aa95ce2fd3ee76 --- /dev/null +++ b/tests/Sniffs/Removed/GenericHookSniffTest.php @@ -0,0 +1,28 @@ +<?php + +namespace Typo3Update\Tests\Sniffs\Removed; + +/* + * Copyright (C) 2017 Daniel Siepmann <coding@daniel-siepmann.de> + * + * 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. + */ + +use Typo3Update\Tests\SniffsTest; + +class GenericHookSniffTest extends SniffsTest +{ +} diff --git a/tests/Sniffs/Removed/GenericSignalSniffTest.php b/tests/Sniffs/Removed/GenericSignalSniffTest.php new file mode 100644 index 0000000000000000000000000000000000000000..5a0d4c052cd98eb9ff38a84c6a87e23f8560bb41 --- /dev/null +++ b/tests/Sniffs/Removed/GenericSignalSniffTest.php @@ -0,0 +1,28 @@ +<?php + +namespace Typo3Update\Tests\Sniffs\Removed; + +/* + * Copyright (C) 2017 Daniel Siepmann <coding@daniel-siepmann.de> + * + * 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. + */ + +use Typo3Update\Tests\SniffsTest; + +class GenericSignalSniffTest extends SniffsTest +{ +}