[BUGFIX] felogin plugin still uses old class name in TypoScript 90/31290/6
authorBenjamin Mack <benni@typo3.org>
Wed, 2 Jul 2014 14:10:39 +0000 (16:10 +0200)
committerHelmut Hummel <helmut.hummel@typo3.org>
Fri, 25 Jul 2014 19:49:56 +0000 (21:49 +0200)
FE Login still uses TypoScript with a manual
inclusion of the original file and then the
original class name.

plugin.tx_felogin_pi1 = USER_INT
plugin.tx_felogin_pi1 {
includeLibs = EXT:felogin/pi1/class.tx_felogin_pi1.php
userFunc = tx_felogin_pi1->main
}

This way it is impossible to XCLASS the new
namespaced class "FrontendLoginController"
without also changing the USER_INT userFunc
path.

In order to remove the deprecation, we need to
change the class name - no-brainer.

Resolves: #60063
Releases: 6.3
Change-Id: If09e5a3fdc3317623d5ea1de9af04cb04e606656
Reviewed-on: http://review.typo3.org/31290
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Helmut Hummel <helmut.hummel@typo3.org>
Tested-by: Helmut Hummel <helmut.hummel@typo3.org>
typo3/sysext/felogin/Classes/Controller/FrontendLoginController.php
typo3/sysext/felogin/ext_localconf.php
typo3/sysext/felogin/pi1/class.tx_felogin_pi1.php [deleted file]

index cf922d0..5c67976 100644 (file)
@@ -34,6 +34,7 @@ class FrontendLoginController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
         * Path to this script relative to the extension dir.
         *
         * @todo Define visibility
+        * @TODO This is still set to the "old" class location since the locallang.xlf file in the same dir is loaded by pi_loadLL
         */
        public $scriptRelPath = 'pi1/class.tx_felogin_pi1.php';
 
index eccb058..d1d4968 100644 (file)
@@ -5,8 +5,7 @@ defined('TYPO3_MODE') or die();
 $pluginContent = trim('
 plugin.tx_felogin_pi1 = USER_INT
 plugin.tx_felogin_pi1 {
-  includeLibs = EXT:felogin/pi1/class.tx_felogin_pi1.php
-  userFunc = tx_felogin_pi1->main
+       userFunc = TYPO3\\CMS\\Felogin\\Controller\\FrontendLoginController->main
 }
 ');
 \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::addTypoScript($_EXTKEY, 'setup', '
diff --git a/typo3/sysext/felogin/pi1/class.tx_felogin_pi1.php b/typo3/sysext/felogin/pi1/class.tx_felogin_pi1.php
deleted file mode 100644 (file)
index ceb8174..0000000
+++ /dev/null
@@ -1,7 +0,0 @@
-<?php
-/*
- * @deprecated since 6.0, the classname tx_felogin_pi1 and this file is obsolete
- * and will be removed with 6.2. The class was renamed and is now located at:
- * typo3/sysext/felogin/Classes/Controller/FrontendLoginController.php
- */
-require_once \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::extPath('felogin') . 'Classes/Controller/FrontendLoginController.php';