[TASK] Cleanup usage of booleans in EXT:frontend 71/42471/7
authorStefan Neufeind <typo3.neufeind@speedpartner.de>
Fri, 20 Nov 2015 14:41:30 +0000 (15:41 +0100)
committerBenni Mack <benni@typo3.org>
Thu, 11 Feb 2016 21:06:04 +0000 (22:06 +0100)
Change-Id: I65b6b7f14752676c1c90fb6ff0b382de5aaaa53e
Resolves: #68896
Releases: master
Reviewed-on: https://review.typo3.org/42471
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
typo3/sysext/frontend/Classes/Authentication/FrontendUserAuthentication.php
typo3/sysext/frontend/Classes/ContentObject/ContentObjectRenderer.php
typo3/sysext/frontend/Classes/ContentObject/Menu/AbstractMenuContentObject.php
typo3/sysext/frontend/Classes/Controller/TypoScriptFrontendController.php
typo3/sysext/frontend/Classes/Page/PageGenerator.php
typo3/sysext/frontend/Classes/Plugin/AbstractPlugin.php

index ad27722..1757cc0 100644 (file)
@@ -99,7 +99,7 @@ class FrontendUserAuthentication extends AbstractUserAuthentication
     /**
      * @var bool
      */
-    public $is_permanent;
+    public $is_permanent = false;
 
     /**
      * @var int|NULL
@@ -188,7 +188,7 @@ class FrontendUserAuthentication extends AbstractUserAuthentication
     public function getNewSessionRecord($tempuser)
     {
         $insertFields = parent::getNewSessionRecord($tempuser);
-        $insertFields['ses_permanent'] = $this->is_permanent;
+        $insertFields['ses_permanent'] = $this->is_permanent ? 1 : 0;
         return $insertFields;
     }
 
@@ -237,11 +237,11 @@ class FrontendUserAuthentication extends AbstractUserAuthentication
                 // we need to force setting the session cookie here
                 $this->forceSetCookie = true;
             }
-            $isPermanent = $isPermanent ? 1 : 0;
+            $isPermanent = (bool)$isPermanent;
         } elseif ($GLOBALS['TYPO3_CONF_VARS']['FE']['permalogin'] == 2) {
-            $isPermanent = 1;
+            $isPermanent = true;
         } else {
-            $isPermanent = 0;
+            $isPermanent = false;
         }
         $loginData['permanent'] = $isPermanent;
         $this->is_permanent = $isPermanent;
index e367957..07150bb 100644 (file)
@@ -3532,7 +3532,7 @@ class ContentObjectRenderer
     }
 
     /**
-     * Compares values together based on the settings in the input TypoScript array and returns TRUE or FALSE based on the comparison result.
+     * Compares values together based on the settings in the input TypoScript array and returns the comparison result.
      * Implements the "if" function in TYPO3 TypoScript
      *
      * @param array $conf TypoScript properties defining what to compare
@@ -3767,11 +3767,11 @@ class ContentObjectRenderer
             if (!$inside) {
                 $len = strcspn(substr($str, $pointer), '{');
                 $newVal .= substr($str, $pointer, $len);
-                $inside = 1;
+                $inside = true;
             } else {
                 $len = strcspn(substr($str, $pointer), '}') + 1;
                 $newVal .= $this->getData(substr($str, $pointer + 1, $len - 2), $this->data);
-                $inside = 0;
+                $inside = false;
             }
             $pointer += $len;
         } while ($pointer < $totalLen);
@@ -4639,7 +4639,7 @@ class ContentObjectRenderer
             return $theValue;
         }
         // Indicates that the data is from within a tag.
-        $inside = 0;
+        $inside = false;
         // Pointer to the total string position
         $pointer = 0;
         // Loaded with the current typo-tag if any.
@@ -4748,7 +4748,7 @@ class ContentObjectRenderer
                     }
                     $contentAccum[$contentAccumP] .= $data;
                 }
-                $inside = 1;
+                $inside = true;
             } else {
                 // tags
                 $len = strcspn(substr($theValue, $pointer), '>') + 1;
@@ -4785,7 +4785,7 @@ class ContentObjectRenderer
                         // This flag indicates, that NL- (13-10-chars) should be stripped first and last.
                         $stripNL = (bool)$theConf['stripNL'];
                         // This flag indicates, that this TypoTag section should NOT be included in the nonTypoTag content.
-                        $breakOut = $theConf['breakoutTypoTagContent'] ? 1 : 0;
+                        $breakOut = (bool)$theConf['breakoutTypoTagContent'];
                         $this->parameters = array();
                         if ($currentTag[1]) {
                             $params = GeneralUtility::get_tag_attributes($currentTag[1]);
@@ -4831,7 +4831,7 @@ class ContentObjectRenderer
                         $contentAccum[$contentAccumP] .= HTMLSpecialChars($data);
                     }
                 }
-                $inside = 0;
+                $inside = false;
             }
             $pointer += $len;
         } while ($pointer < $totalLen);
@@ -4875,10 +4875,10 @@ class ContentObjectRenderer
         $str_content = '';
         foreach ($lParts as $k => $l) {
             $sameBeginEnd = 0;
-            $emptyTag = 0;
+            $emptyTag = false;
             $l = trim($l);
             $attrib = array();
-            $nWrapped = 0;
+            $nonWrapped = false;
             $tagName = '';
             if ($l[0] === '<' && substr($l, -1) === '>') {
                 $fwParts = explode('>', substr($l, 1), 2);
@@ -4889,7 +4889,7 @@ class ContentObjectRenderer
                     }
                     if (substr($fwParts[0], -1) === '/') {
                         $sameBeginEnd = 1;
-                        $emptyTag = 1;
+                        $emptyTag = true;
                         $attrib = GeneralUtility::get_tag_attributes('<' . substr($fwParts[0], 0, -1) . '>');
                     }
                 } else {
@@ -4905,9 +4905,9 @@ class ContentObjectRenderer
             } else {
                 $uTagName = strtoupper($nonWrappedTag);
                 // The line will be wrapped: $uTagName should not be an empty tag
-                $emptyTag = 0;
+                $emptyTag = false;
                 $str_content = $lParts[$k];
-                $nWrapped = 1;
+                $nonWrapped = true;
                 $attrib = array();
             }
             // Wrapping all inner-content:
@@ -4950,7 +4950,7 @@ class ContentObjectRenderer
                     }
                 }
             }
-            if ($nWrapped && $conf['wrapNonWrappedLines']) {
+            if ($nonWrapped && $conf['wrapNonWrappedLines']) {
                 $str_content = $this->wrap($str_content, $conf['wrapNonWrappedLines']);
             }
             $lParts[$k] = $str_content;
@@ -7232,26 +7232,26 @@ class ContentObjectRenderer
         } else {
             $groupList = $feUserRow['usergroup'];
         }
-        $ok = 0;
+        $ok = false;
         // Points to the field that allows further editing from frontend if not set. If set the record is locked.
         if (!$GLOBALS['TCA'][$table]['ctrl']['fe_admin_lock'] || !$row[$GLOBALS['TCA'][$table]['ctrl']['fe_admin_lock']]) {
             // Points to the field (int) that holds the fe_users-id of the creator fe_user
             if ($GLOBALS['TCA'][$table]['ctrl']['fe_cruser_id']) {
                 $rowFEUser = (int)$row[$GLOBALS['TCA'][$table]['ctrl']['fe_cruser_id']];
                 if ($rowFEUser && $rowFEUser === (int)$feUserRow['uid']) {
-                    $ok = 1;
+                    $ok = true;
                 }
             }
             // If $feEditSelf is set, fe_users may always edit them selves...
             if ($feEditSelf && $table === 'fe_users' && (int)$feUserRow['uid'] === (int)$row['uid']) {
-                $ok = 1;
+                $ok = true;
             }
             // Points to the field (int) that holds the fe_group-id of the creator fe_user's first group
             if ($GLOBALS['TCA'][$table]['ctrl']['fe_crgroup_id']) {
                 $rowFEUser = (int)$row[$GLOBALS['TCA'][$table]['ctrl']['fe_crgroup_id']];
                 if ($rowFEUser) {
                     if (GeneralUtility::inList($groupList, $rowFEUser)) {
-                        $ok = 1;
+                        $ok = true;
                     }
                 }
             }
@@ -7537,7 +7537,7 @@ class ContentObjectRenderer
      * @param string $groupBy Optional GROUP BY field(s), if none, supply blank string.
      * @param string $orderBy Optional ORDER BY field(s), if none, supply blank string.
      * @param string $limit Optional LIMIT value ([begin,]max), if none, supply blank string.
-     * @return bool|\mysqli_result SQL result pointer
+     * @return bool|\mysqli_result|object MySQLi result object / DBAL object
      * @see mm_query_uidList()
      */
     public function exec_mm_query($select, $local_table, $mm_table, $foreign_table, $whereClause = '', $groupBy = '', $orderBy = '', $limit = '')
@@ -7566,7 +7566,7 @@ class ContentObjectRenderer
      * @param string $groupBy Optional GROUP BY field(s), if none, supply blank string.
      * @param string $orderBy Optional ORDER BY field(s), if none, supply blank string.
      * @param string $limit Optional LIMIT value ([begin,]max), if none, supply blank string.
-     * @return bool|\mysqli_result SQL result pointer
+     * @return bool|\mysqli_result|object MySQLi result object / DBAL object
      * @see mm_query()
      */
     public function exec_mm_query_uidList($select, $local_table_uidlist, $mm_table, $foreign_table = '', $whereClause = '', $groupBy = '', $orderBy = '', $limit = '')
@@ -7623,7 +7623,7 @@ class ContentObjectRenderer
      *
      * @param string $table The table name
      * @param array $conf The TypoScript configuration properties
-     * @return mixed A SQL result pointer
+     * @return bool|\mysqli_result|object MySQLi result object / DBAL object
      * @see getQuery()
      */
     public function exec_getQuery($table, $conf)
index 4d8f8c4..6452cb5 100644 (file)
@@ -82,7 +82,7 @@ abstract class AbstractMenuContentObject
     /**
      * @var bool
      */
-    public $debug = 0;
+    public $debug = false;
 
     /**
      * Loaded with the parent cObj-object when a new HMENU is made
index ed30485..9f25cf7 100644 (file)
@@ -451,10 +451,10 @@ class TypoScriptFrontendController
     public $defaultBodyTag = '<body>';
 
     /**
-     * Debug flag, may output special debug html-code.
-     * @var string
+     * Debug flag. If TRUE special debug-output maybe be shown (which includes html-formatting).
+     * @var bool
      */
-    public $debug = '';
+    public $debug = false;
 
     /**
      * Default internal target
@@ -1234,7 +1234,7 @@ class TypoScriptFrontendController
         if ($this->beUserLogin || $this->doWorkspacePreview()) {
             // Backend user preview features:
             if ($this->beUserLogin && $backendUser->adminPanel instanceof AdminPanelView) {
-                $this->fePreview = (bool)$backendUser->adminPanel->extGetFeAdminValue('preview');
+                $this->fePreview = (int)$backendUser->adminPanel->extGetFeAdminValue('preview');
                 // If admin panel preview is enabled...
                 if ($this->fePreview) {
                     if ($this->fe_user->user) {
@@ -1794,24 +1794,24 @@ class TypoScriptFrontendController
     {
         // Initialize:
         $c = count($this->rootLine);
-        $disable = false;
+        $loginAllowed = true;
         // Traverse root line from root and outwards:
         for ($a = 0; $a < $c; $a++) {
             // If a value is set for login state:
             if ($this->rootLine[$a]['fe_login_mode'] > 0) {
                 // Determine state from value:
                 if ((int)$this->rootLine[$a]['fe_login_mode'] === 1) {
-                    $disable = true;
+                    $loginAllowed = false;
                     $this->loginAllowedInBranch_mode = 'all';
                 } elseif ((int)$this->rootLine[$a]['fe_login_mode'] === 3) {
-                    $disable = true;
+                    $loginAllowed = false;
                     $this->loginAllowedInBranch_mode = 'groups';
                 } else {
-                    $disable = false;
+                    $loginAllowed = true;
                 }
             }
         }
-        return !$disable;
+        return $loginAllowed;
     }
 
     /**
index 2330b4b..44e0d39 100644 (file)
@@ -70,7 +70,7 @@ class PageGenerator
         }
         // Global vars...
         $tsfe->indexedDocTitle = $tsfe->page['title'];
-        $tsfe->debug = '' . $tsfe->config['config']['debug'];
+        $tsfe->debug = !empty($tsfe->config['config']['debug']);
         // Base url:
         if (isset($tsfe->config['config']['baseURL'])) {
             $tsfe->baseUrl = $tsfe->config['config']['baseURL'];
index 3818ee1..6201516 100644 (file)
@@ -113,9 +113,9 @@ class AbstractPlugin
      * Flag that tells if the locallang file has been fetch (or tried to
      * be fetched) already.
      *
-     * @var int
+     * @var bool
      */
-    public $LOCAL_LANG_loaded = 0;
+    public $LOCAL_LANG_loaded = false;
 
     /**
      * Pointer to the language to use.
@@ -178,9 +178,9 @@ class AbstractPlugin
     public $pi_autoCacheFields = array();
 
     /**
-     * @var int
+     * @var bool
      */
-    public $pi_autoCacheEn = 0;
+    public $pi_autoCacheEn = false;
 
     /**
      * If set, then links are
@@ -543,7 +543,7 @@ class AbstractPlugin
         //                      2: only the text "Displaying results..." will be shown
         $showResultCount = (int)$showResultCount;
         // If this is set, two links named "<< First" and "LAST >>" will be shown and point to the very first or last page.
-        $showFirstLast = $this->internal['showFirstLast'];
+        $showFirstLast = !empty($this->internal['showFirstLast']);
         // If this has a value the "previous" button is always visible (will be forced if "showFirstLast" is set)
         $alwaysPrev = $showFirstLast ? 1 : $this->pi_alwaysPrev;
         if (isset($this->internal['pagefloat'])) {
@@ -1016,7 +1016,7 @@ class AbstractPlugin
                 }
             }
         }
-        $this->LOCAL_LANG_loaded = 1;
+        $this->LOCAL_LANG_loaded = true;
     }
 
     /***************************