[TASK] Move login-related constants to EXT:felogin 92/43392/4
authorBenjamin Mack <benni@typo3.org>
Thu, 17 Sep 2015 10:38:19 +0000 (12:38 +0200)
committerBenni Mack <benni@typo3.org>
Thu, 17 Sep 2015 13:32:39 +0000 (15:32 +0200)
Due to historic reasons EXT:css_styled_content has defined
constants that were used before EXT:felogin was there.

The constants were set correctly, but haven't been handed over
to EXT:felogin, thus they were useless since the old loginform was
removed.

The patch adds the constants to EXT:felogin so they are also added
when using e.g. EXT:fluid_styled_content as the
"defaultContentRendering" feature is used.

Resolves: #69914
Releases: master
Change-Id: Ib13372158352d25112caef77ec6d1c9e80a6027e
Reviewed-on: http://review.typo3.org/43392
Reviewed-by: Oliver Thiele <mail@oliver-thiele.de>
Tested-by: Oliver Thiele <mail@oliver-thiele.de>
Reviewed-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Tested-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
typo3/sysext/css_styled_content/static/constants.txt
typo3/sysext/css_styled_content/static/v6.2/constants.txt
typo3/sysext/felogin/ext_localconf.php
typo3/sysext/felogin/ext_typoscript_setup.txt

index b9591f8..2711a50 100644 (file)
@@ -95,16 +95,6 @@ styles.content.mailform {
   badMess =
 }
 
-styles.content.loginform {
-  target = _top
-    # cat=content/cLogin; type=int+; label= PID of user archive: Enter the page-uid number (PID) of the folder where you keep your fe_users that are supposed to login on this site. This setting is necessary, if login is going to work!
-  pid =
-    # cat=content/cLogin; type=; label= Message, Login: This is the message (if any) that is popped-up when a user logs in as a front-end user
-  goodMess =
-    # cat=content/cLogin; type=; label= Login template: Enter the path for the HTML template to be used
-  templateFile = EXT:felogin/template.html
-}
-
 styles.content.table {
     # cat=content/cTable/k2; type=color; label= Table background color, Color 1: Background color for table when "Backgr. Color" has been set to "Color 1"
   backgroundColor.1 = #EDEBF1
index dae6ce4..bd00dc5 100644 (file)
@@ -95,16 +95,6 @@ styles.content.mailform {
   badMess =
 }
 
-styles.content.loginform {
-  target = _top
-    # cat=content/cLogin; type=int+; label= PID of user archive: Enter the page-uid number (PID) of the folder where you keep your fe_users that are supposed to login on this site. This setting is necessary, if login is going to work!
-  pid =
-    # cat=content/cLogin; type=; label= Message, Login: This is the message (if any) that is popped-up when a user logs in as a front-end user
-  goodMess =
-    # cat=content/cLogin; type=; label= Login template: Enter the path for the HTML template to be used
-  templateFile = EXT:felogin/template.html
-}
-
 styles.content.searchform {
     # cat=content/cSearch; type=; label= Message, Searchform: This is the message (if any) that is popped-up when a user performs a search
   goodMess =
index c9e16c2..3914de6 100644 (file)
@@ -1,14 +1,16 @@
 <?php
 defined('TYPO3_MODE') or die();
 
-// add plugin controller
-\TYPO3\CMS\Core\Utility\ExtensionManagementUtility::addTypoScript('TYPO3.CMS.Felogin', 'setup', '
-# Setting "felogin" plugin TypoScript
-plugin.tx_felogin_pi1 = USER_INT
-plugin.tx_felogin_pi1.userFunc = TYPO3\\CMS\\Felogin\\Controller\\FrontendLoginController->main
-');
-
 // Add a default TypoScript for the CType "login" (also replaces history login functionality)
+\TYPO3\CMS\Core\Utility\ExtensionManagementUtility::addTypoScript('TYPO3.CMS.Felogin', 'constants', '
+styles.content.loginform {
+    # cat=content/cLogin; type=int+; label= PID of user archive: Enter the page-uid number (PID) of the folder where you keep your fe_users that are supposed to login on this site. This setting is necessary, if login is going to work!
+  pid =
+    # cat=content/cLogin; type=; label= Login template: Enter the path for the HTML template to be used
+  templateFile = EXT:felogin/template.html
+}
+', 'defaultContentRendering');
+
 \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::addTypoScript('TYPO3.CMS.Felogin', 'setup', '
 # Setting "felogin" plugin TypoScript
 tt_content.login = COA
index 3fbc090..8ad6c67 100644 (file)
@@ -1,36 +1,39 @@
+# Setting "felogin" plugin TypoScript
+plugin.tx_felogin_pi1 = USER_INT
 plugin.tx_felogin_pi1 {
-               #storagePid - where are the user records? use single value or a comma separated list
+       userFunc = TYPO3\CMS\Felogin\Controller\FrontendLoginController->main
+
+       # storagePid - where are the user records? use single value or a comma separated list
        storagePid = {$styles.content.loginform.pid}
        recursive =
 
-               #Template File
+       # Location of the template file, it is EXT:felogin/template.html by default
        templateFile = {$styles.content.loginform.templateFile}
 
-               #baseURL for the link generation
+       baseURL for the link generation
        feloginBaseURL =
 
-               #wrapContentInBaseClass
+       # should a wrapper class be set for this content element
        wrapContentInBaseClass = 1
 
-
-               #typolink-configuration for links / urls
-               #parameter and additionalParams are set by extension
+       # typolink-configuration for links / urls
+       # parameter and additionalParams are set by extension
        linkConfig {
                target =
                ATagParams = rel="nofollow"
        }
 
-               #preserve GET vars - define "all" or comma separated list of GET-vars that should be included by link generation
+       preserve GET vars - define "all" or comma separated list of GET-vars that should be included by link generation
        preserveGETvars = all
 
-
-               #additional fields
+       # additional fields
        showForgotPasswordLink =
        showPermaLogin =
 
-               # time in hours how long the link for forget password is valid
+       # time in hours how long the link for forget password is valid
        forgotLinkHashValidTime = 12
 
+       # when setting a new password, this is the minimum amount of characters
        newPasswordMinLength = 6
 
        welcomeHeader_stdWrap {
@@ -105,21 +108,20 @@ plugin.tx_felogin_pi1 {
                }
        }
 
-               #redirect
+       # redirect options
        redirectMode =
        redirectFirstMethod =
        redirectPageLogin =
        redirectPageLoginError =
        redirectPageLogout =
 
-       #disable redirect with one switch
+       # disable redirect with one switch
        redirectDisable =
 
        email_from =
        email_fromName =
        replyTo =
 
-
        # Allowed Referrer-Redirect-Domains:
        domains =
 
@@ -130,16 +132,10 @@ plugin.tx_felogin_pi1 {
 
        # Expose the information on whether or not the account for which a new password was requested exists. By default, that information is not disclosed for privacy reasons.
        exposeNonexistentUserInForgotPasswordDialog = 0
-}
 
-plugin.tx_felogin_pi1._CSS_DEFAULT_STYLE (
-       .tx-felogin-pi1 label {
-               display: block;
-       }
-)
-
-plugin.tx_felogin_pi1._LOCAL_LANG.default {
-}
-
-plugin.tx_felogin_pi1._DEFAULT_PI_VARS {
+       _CSS_DEFAULT_STYLE (
+               .tx-felogin-pi1 label {
+                       display: block;
+               }
+       )
 }
\ No newline at end of file