[CLEANUP] Fix length of comments to meet CGL 76/29376/3
authorRoland Waldner <roland.waldner@in2code.de>
Thu, 10 Apr 2014 14:34:23 +0000 (16:34 +0200)
committerMarkus Klein <klein.t3@mfc-linz.at>
Sat, 12 Apr 2014 14:11:28 +0000 (16:11 +0200)
Resolves: #57819
Releases: 6.2
Change-Id: I24a860dda49504e8745fe195f74c519a73a22991
Reviewed-on: https://review.typo3.org/29376
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Markus Klein
Tested-by: Markus Klein
typo3/sysext/cms/tslib/index_ts.php
typo3/sysext/core/Classes/Core/Bootstrap.php

index 6a5b44b..e443ae7 100644 (file)
  *
  *  This copyright notice MUST APPEAR in all copies of the script!
  ***************************************************************/
+
 /**
- * This is the MAIN DOCUMENT of the TypoScript driven standard front-end (from the "cms" extension)
- * Basically put this is the "index.php" script which all requests for TYPO3 delivered pages goes to in the frontend (the website)
- * The script configures constants, includes libraries and does a little logic here and there in order to instantiate the right classes to create the webpage.
- * All the real data processing goes on in the "tslib/" classes which this script will include and use as needed.
+ * This is the MAIN DOCUMENT of the TypoScript driven standard front-end (from
+ * the "cms" extension)
+ *
+ * Basically put this is the "index.php" script which all requests for TYPO3
+ * delivered pages goes to in the frontend (the website) The script configures
+ * constants, includes libraries and does a little logic here and there in order
+ * to instantiate the right classes to create the webpage.
+ *
+ * All the real data processing goes on in the "tslib/" classes which this script
+ * will include and use as needed.
  *
  * @author Kasper Skårhøj <kasperYYYY@typo3.com>
  */
@@ -119,8 +126,9 @@ $TT->pull();
 /** @var $BE_USER \TYPO3\CMS\Backend\FrontendBackendUserAuthentication */
 $BE_USER = $TSFE->initializeBackendUser();
 
-// Process the ID, type and other parameters
-// After this point we have an array, $page in TSFE, which is the page-record of the current page, $id
+// Process the ID, type and other parameters.
+// After this point we have an array, $page in TSFE, which is the page-record
+// of the current page, $id.
 $TT->push('Process ID', '');
 // Initialize admin panel since simulation settings are required here:
 if ($TSFE->isBackendUserLoggedIn()) {
@@ -132,10 +140,14 @@ if ($TSFE->isBackendUserLoggedIn()) {
 $TSFE->checkAlternativeIdMethods();
 $TSFE->clear_preview();
 $TSFE->determineId();
-// Now, if there is a backend user logged in and he has NO access to this page, then re-evaluate the id shown!
-if ($TSFE->isBackendUserLoggedIn() && (!$BE_USER->extPageReadAccess($TSFE->page) || \TYPO3\CMS\Core\Utility\GeneralUtility::_GP('ADMCMD_noBeUser'))) {
-       // \TYPO3\CMS\Core\Utility\GeneralUtility::_GP('ADMCMD_noBeUser') is placed here because
-       // \TYPO3\CMS\Version\Hook\PreviewHook might need to know if a backend user is logged in!
+
+// Now, if there is a backend user logged in and he has NO access to this page,
+// then re-evaluate the id shown! _GP('ADMCMD_noBeUser') is placed here because
+// \TYPO3\CMS\Version\Hook\PreviewHook might need to know if a backend user is logged in.
+if (
+       $TSFE->isBackendUserLoggedIn()
+       && (!$BE_USER->extPageReadAccess($TSFE->page) || \TYPO3\CMS\Core\Utility\GeneralUtility::_GP('ADMCMD_noBeUser'))
+) {
        // Remove user
        unset($BE_USER);
        $TSFE->beUserLogin = 0;
@@ -144,6 +156,7 @@ if ($TSFE->isBackendUserLoggedIn() && (!$BE_USER->extPageReadAccess($TSFE->page)
        $TSFE->clear_preview();
        $TSFE->determineId();
 }
+
 $TSFE->makeCacheHash();
 $TT->pull();
 
@@ -176,8 +189,10 @@ $TT->push('Setting language and locale', '');
 $TSFE->settingLanguage();
 $TSFE->settingLocale();
 $TT->pull();
+
 // Convert POST data to internal "renderCharset" if different from the metaCharset
 $TSFE->convPOSTCharset();
+
 // Check JumpUrl
 $TSFE->setExternalJumpUrl();
 $TSFE->checkJumpUrlReferer();
index 3938098..056eac6 100644 (file)
@@ -145,12 +145,12 @@ class Bootstrap {
        }
 
        /**
-        * Run the base setup that checks server environment,
-        * determines pathes, populates base files and sets common configuration.
+        * Run the base setup that checks server environment, determines pathes,
+        * populates base files and sets common configuration.
         *
         * Script execution will be aborted if something fails here.
         *
-        * @param string $relativePathPart Relative path of the entry script back to document root
+        * @param string $relativePathPart Relative path of entry script back to document root
         * @return Bootstrap
         * @internal This is not a public API method, do not use in own extensions
         */
@@ -163,7 +163,7 @@ class Bootstrap {
        /**
         * Redirect to install tool if LocalConfiguration.php is missing.
         *
-        * @param string $pathUpToDocumentRoot Can contain eg. '../' if called from a sub directory
+        * @param string $pathUpToDocumentRoot Can contain '../' if called from a sub directory
         * @return Bootstrap
         * @internal This is not a public API method, do not use in own extensions
         */