[BUGFIX] Missing initialization of variables
authorSteffen Kamper <info@sk-typo3.de>
Sun, 13 Mar 2011 13:09:57 +0000 (14:09 +0100)
committerSteffen Kamper <info@sk-typo3.de>
Tue, 22 Mar 2011 18:13:19 +0000 (19:13 +0100)
Initialize variables, which are used, but not initialized in every case before.

Change-Id: Iccc6f2c31a4ade6485072f56afd7f1b227d1f516
Resolves: #M17934
Reviewed-on: http://review.typo3.org/1144
Reviewed-by: Steffen Gebert <steffen.gebert@typo3.org>
Tested-by: Steffen Gebert <steffen.gebert@typo3.org>
Reviewed-by: Sebastian Fischer <typo3@evoweb.de>
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
t3lib/class.t3lib_admin.php
t3lib/class.t3lib_befunc.php
t3lib/class.t3lib_fullsearch.php
t3lib/class.t3lib_positionmap.php
t3lib/class.t3lib_querygenerator.php
t3lib/class.t3lib_superadmin.php
t3lib/class.t3lib_tceforms.php
t3lib/class.t3lib_tceforms_inline.php
t3lib/class.t3lib_timetrack.php

index 8fdd380..0eb7f64 100644 (file)
@@ -674,6 +674,7 @@ class t3lib_admin {
         */
        function testDBRefs($theArray) {
                global $TCA;
+               $result = '';
                foreach ($theArray as $table => $dbArr) {
                        if ($TCA[$table]) {
                                $idlist = array_keys($dbArr);
index 9f3bb50..2d041a1 100644 (file)
@@ -2298,7 +2298,7 @@ final class t3lib_BEfunc {
 
                                // If this field is a password field, then hide the password by changing it to a random number of asterisk (*)
                        if (stristr($theColConf['eval'], 'password')) {
-                               unset($l);
+                               $l = '';
                                $randomNumber = rand(5, 12);
                                for ($i = 0; $i < $randomNumber; $i++) {
                                        $l .= '*';
index 4755aa9..66bbf95 100644 (file)
@@ -361,6 +361,7 @@ class t3lib_fullsearch {
        function queryMaker() {
                global $TCA;
 
+               $output = '';
                if (is_array($GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['t3lib_fullsearch'])) {
                        $this->hookArray = $GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['t3lib_fullsearch'];
                }
@@ -412,7 +413,7 @@ class t3lib_fullsearch {
 
                                $res = @$GLOBALS['TYPO3_DB']->sql_query($qExplain);
                                if ($GLOBALS['TYPO3_DB']->sql_error()) {
-                                       $out .= '<BR><strong>Error:</strong><BR><font color="red"><strong>' . $GLOBALS['TYPO3_DB']->sql_error() . '</strong></font>';
+                                       $out = '<BR><strong>Error:</strong><BR><font color="red"><strong>' . $GLOBALS['TYPO3_DB']->sql_error() . '</strong></font>';
                                        $output .= $GLOBALS['SOBE']->doc->section('SQL error', $out, 0, 1);
                                } else {
                                        $cPR = $this->getQueryResultCode($mQ, $res, $qGen->table);
@@ -434,7 +435,7 @@ class t3lib_fullsearch {
         */
        function getQueryResultCode($mQ, $res, $table) {
                global $TCA;
-               $output = '';
+               $out = '';
                $cPR = array();
                switch ($mQ) {
                        case 'count':
@@ -586,6 +587,7 @@ class t3lib_fullsearch {
                $SET = $GLOBALS['SOBE']->MOD_SETTINGS;
                $swords = $SET['sword'];
 
+               $out = '';
                $limit = 200;
                $showAlways = 0;
                if ($swords) {
index 66813cc..83355a6 100644 (file)
@@ -120,6 +120,7 @@ class t3lib_positionMap {
        function positionTree($id, $pageinfo, $perms_clause, $R_URI) {
                global $LANG, $BE_USER;
 
+               $code = '';
                        // Make page tree object:
                $t3lib_pageTree = t3lib_div::makeInstance('localPageTree');
                $t3lib_pageTree->init(' AND ' . $perms_clause);
@@ -234,7 +235,7 @@ class t3lib_positionMap {
         * @return      string          <script> section
         */
        function JSimgFunc($prefix = '') {
-               $code .= $GLOBALS['TBE_TEMPLATE']->wrapScriptTags('
+               $code = $GLOBALS['TBE_TEMPLATE']->wrapScriptTags('
 
                        var img_newrecord_marker=new Image();
                        img_newrecord_marker.src = "' . t3lib_iconWorks::skinImg($this->backPath, 'gfx/newrecord' . $prefix . '_marker.gif', '', 1) . '";
index 7d3b0b9..74266ea 100644 (file)
@@ -749,6 +749,7 @@ class t3lib_queryGenerator {
         * @return      [type]          ...
         */
        function makeOptionList($fN, $conf, $table) {
+               $out = '';
                $fieldSetup = $this->fields[$fN];
                if ($fieldSetup['type'] == 'files') {
                        if ($conf['comparison'] == 66 || $conf['comparison'] == 67) {
@@ -1004,7 +1005,7 @@ class t3lib_queryGenerator {
                        $out .= '<option value="OR"' . ($op == 'OR' ? ' selected' : '') . '>' . $this->lang['OR'] . '</option>';
                        $out .= '</select>';
                } else {
-                       $out .= '<input type="hidden" value="' . $op . '" name="' . $name . '[operator]">';
+                       $out = '<input type="hidden" value="' . $op . '" name="' . $name . '[operator]">';
                        $out .= '<img src="clear.gif" height="1" width="47">';
 
                }
@@ -1215,6 +1216,7 @@ class t3lib_queryGenerator {
         * @return      [type]          ...
         */
        function getQuerySingle($conf, $first) {
+               $qs = '';
                $prefix = $this->enablePrefix ? $this->table . '.' : '';
                if (!$first) {
                                // Is it OK to insert the AND operator if none is set?
@@ -1522,6 +1524,7 @@ class t3lib_queryGenerator {
         * @return      [type]          ...
         */
        function JSbottom($formname) {
+               $out = '';
                if ($this->extJSCODE) {
                        $out .= '
                        <script language="javascript" type="text/javascript" src="' . $GLOBALS['BACK_PATH'] . '../t3lib/jsfunc.evalfield.js"></script>
@@ -1532,8 +1535,8 @@ class t3lib_queryGenerator {
                                TBE_EDITOR.backend_interface = "' . $GLOBALS['BE_USER']->uc['interfaceSetup'] . '";
                                ' . $this->extJSCODE . '
                        </script>';
-                       return $out;
                }
+               return $out;
        }
 
        /**
index bbf0cc8..b2a96c5 100644 (file)
@@ -480,6 +480,7 @@ class t3lib_superadmin {
         * @see initProcess()
         */
        function processSiteDir($path, $dir) {
+               $out = '';
                if (@is_dir($path)) {
                        $localconf = $path . '/typo3conf/localconf.php';
                        if (@is_file($localconf)) {
@@ -1307,6 +1308,7 @@ class t3lib_superadmin {
                $pass = trim(t3lib_div::_POST('NEWPASS'));
                $passMD5 = t3lib_div::_POST('NEWPASS_md5');
 
+               $content = '';
                $updatedFlag = 0;
                if (($pass || $passMD5) && is_array($whichFields)) {
                        $pass = $passMD5 ? $passMD5 : md5($pass);
index ce88bca..1bd74b4 100644 (file)
@@ -1051,6 +1051,7 @@ class t3lib_TCEforms {
                $classAndStyleAttributes = $this->formWidthAsArray($size);
 
                $fieldAppendix = '';
+               $item = '';
                $cssClasses = array($classAndStyleAttributes['class']);
                $cssStyle = $classAndStyleAttributes['style'];
 
@@ -1384,6 +1385,7 @@ class t3lib_TCEforms {
        function getSingleField_typeCheck($table, $field, $row, &$PA) {
                $config = $PA['fieldConf']['config'];
 
+               $item = '';
                $disabled = '';
                if ($this->renderReadonly || $config['readOnly']) {
                        $disabled = ' disabled="disabled"';
@@ -1460,6 +1462,7 @@ class t3lib_TCEforms {
        function getSingleField_typeRadio($table, $field, $row, &$PA) {
                $config = $PA['fieldConf']['config'];
 
+               $item = '';
                $disabled = '';
                if ($this->renderReadonly || $config['readOnly']) {
                        $disabled = ' disabled="disabled"';
@@ -1614,7 +1617,10 @@ class t3lib_TCEforms {
                $size = intval($config['size']);
                $selectedStyle = ''; // Style set on <select/>
 
+               $item = '';
                $disabled = '';
+               $onChangeIcon = ''; // TODO: icon is used but never assigned
+
                if ($this->renderReadonly || $config['readOnly']) {
                        $disabled = ' disabled="disabled"';
                        $onlySelectedIconShown = 1;
@@ -1773,8 +1779,7 @@ class t3lib_TCEforms {
         * @return      string          The HTML code for the item
         * @see getSingleField_typeSelect()
         */
-       function getSingleField_typeSelect_checkbox(
-               $table, $field, $row, &$PA, $config, $selItems, $nMV_label) {
+       function getSingleField_typeSelect_checkbox($table, $field, $row, &$PA, $config, $selItems, $nMV_label) {
 
                if (empty($selItems)) {
                        return '';
@@ -1783,6 +1788,7 @@ class t3lib_TCEforms {
                        // Get values in an array (and make unique, which is fine because there can be no duplicates anyway):
                $itemArray = array_flip($this->extractValuesOnlyFromValueLabelList($PA['itemFormElValue']));
 
+               $item = '';
                $disabled = '';
                if ($this->renderReadonly || $config['readOnly']) {
                        $disabled = ' disabled="disabled"';
@@ -1931,6 +1937,7 @@ class t3lib_TCEforms {
                        // Get values in an array (and make unique, which is fine because there can be no duplicates anyway):
                $itemArray = array_flip($this->extractValuesOnlyFromValueLabelList($PA['itemFormElValue']));
 
+               $item = '';
                $disabled = '';
                if ($this->renderReadonly || $config['readOnly']) {
                        $disabled = ' disabled="disabled"';
@@ -2039,6 +2046,7 @@ class t3lib_TCEforms {
         */
        function getSingleField_typeSelect_multiple($table, $field, $row, &$PA, $config, $selItems, $nMV_label) {
 
+               $item = '';
                $disabled = '';
                if ($this->renderReadonly || $config['readOnly']) {
                        $disabled = ' disabled="disabled"';
@@ -2172,6 +2180,7 @@ class t3lib_TCEforms {
                $allowed = trim($config['allowed']);
                $disallowed = trim($config['disallowed']);
 
+               $item = '';
                $disabled = '';
                if ($this->renderReadonly || $config['readOnly']) {
                        $disabled = ' disabled="disabled"';
@@ -2192,7 +2201,7 @@ class t3lib_TCEforms {
                        case 'file': // If the element is of the internal type "file":
 
                                        // Creating string showing allowed types:
-                               $tempFT = t3lib_div::trimExplode(',', $allowed, 1);
+                               $tempFT = t3lib_div::trimExplode(',', $allowed, TRUE);
                                if (!count($tempFT)) {
                                        $info .= '*';
                                }
@@ -2202,7 +2211,7 @@ class t3lib_TCEforms {
                                        }
                                }
                                        // Creating string, showing disallowed types:
-                               $tempFT_dis = t3lib_div::trimExplode(',', $disallowed, 1);
+                               $tempFT_dis = t3lib_div::trimExplode(',', $disallowed, TRUE);
                                if (count($tempFT_dis)) {
                                        $info .= '<br />';
                                }
@@ -2213,7 +2222,7 @@ class t3lib_TCEforms {
                                }
 
                                        // Making the array of file items:
-                               $itemArray = t3lib_div::trimExplode(',', $PA['itemFormElValue'], 1);
+                               $itemArray = t3lib_div::trimExplode(',', $PA['itemFormElValue'], TRUE);
 
                                        // Showing thumbnails:
                                $thumbsnail = '';
@@ -2471,6 +2480,7 @@ class t3lib_TCEforms {
 
                        // Data Structure:
                $dataStructArray = t3lib_BEfunc::getFlexFormDS($PA['fieldConf']['config'], $row, $table);
+               $item = '';
 
                        // Manipulate Flexform DS via TSConfig and group access lists
                if (is_array($dataStructArray)) {
@@ -3578,13 +3588,15 @@ class t3lib_TCEforms {
         * @see getSingleField(), registerDefaultLanguageData()
         */
        function renderVDEFDiff($vArray, $vDEFkey) {
+               $item = NULL;
+
                if ($GLOBALS['TYPO3_CONF_VARS']['BE']['flexFormXMLincludeDiffBase'] && isset($vArray[$vDEFkey . '.vDEFbase']) && strcmp($vArray[$vDEFkey . '.vDEFbase'], $vArray['vDEF'])) {
 
                                // Create diff-result:
                        $t3lib_diff_Obj = t3lib_div::makeInstance('t3lib_diff');
                        $diffres = $t3lib_diff_Obj->makeDiffDisplay($vArray[$vDEFkey . '.vDEFbase'], $vArray['vDEF']);
 
-                       $item .= '<div class="typo3-TCEforms-diffBox">' .
+                       $item = '<div class="typo3-TCEforms-diffBox">' .
                                         '<div class="typo3-TCEforms-diffBox-header">' . htmlspecialchars($this->getLL('l_changeInOrig')) . ':</div>' .
                                         $diffres .
                                         '</div>';
@@ -4314,17 +4326,16 @@ class t3lib_TCEforms {
         */
        function getSingleHiddenField($table, $field, $row) {
                global $TCA;
-               $out = '';
+               $item = '';
                t3lib_div::loadTCA($table);
                if ($TCA[$table]['columns'][$field]) {
 
                        $uid = $row['uid'];
                        $itemName = $this->prependFormFieldNames . '[' . $table . '][' . $uid . '][' . $field . ']';
                        $itemValue = $row[$field];
-                       $item .= '<input type="hidden" name="' . $itemName . '" value="' . htmlspecialchars($itemValue) . '" />';
-                       $out = $item;
+                       $item = '<input type="hidden" name="' . $itemName . '" value="' . htmlspecialchars($itemValue) . '" />';
                }
-               return $out;
+               return $item;
        }
 
        /**
@@ -5428,6 +5439,7 @@ class t3lib_TCEforms {
        function JSbottom($formname = 'forms[0]', $update = FALSE) {
                $jsFile = array();
                $elements = array();
+               $out = '';
 
                        // required:
                foreach ($this->requiredFields as $itemImgName => $itemName) {
index 6915cbd..aaaff98 100644 (file)
@@ -176,6 +176,8 @@ class t3lib_TCEforms_inline {
                        return FALSE;
                }
 
+               $item = '';
+
                        // count the number of processed inline elements
                $this->inlineCount++;
 
@@ -1312,6 +1314,7 @@ class t3lib_TCEforms_inline {
         * @return      array           An array to be used for JSON
         */
        protected function getExecuteChangesJsonArray($oldItemList, $newItemList) {
+               $data = '';
                $parent = $this->getStructureLevel(-1);
                $current = $this->inlineStructure['unstable'];
 
@@ -2072,11 +2075,15 @@ class t3lib_TCEforms_inline {
         * @return      string          The wrapped HTML code
         */
        function wrapFormsSection($section, $styleAttrs = array(), $tableAttrs = array()) {
+               $style = '';
+               $table = '';
                if (!$styleAttrs['margin-right']) {
                        $styleAttrs['margin-right'] = $this->inlineStyles['margin-right'] . 'px';
                }
 
-               foreach ($styleAttrs as $key => $value) $style .= ($style ? ' ' : '') . $key . ': ' . htmlspecialchars($value) . '; ';
+               foreach ($styleAttrs as $key => $value) {
+                       $style .= ($style ? ' ' : '') . $key . ': ' . htmlspecialchars($value) . '; ';
+               }
                if ($style) {
                        $style = ' style="' . $style . '"';
                }
@@ -2100,7 +2107,9 @@ class t3lib_TCEforms_inline {
                        $tableAttrs['class'] = $this->borderStyle[3];
                }
 
-               foreach ($tableAttrs as $key => $value) $table .= ($table ? ' ' : '') . $key . '="' . htmlspecialchars($value) . '"';
+               foreach ($tableAttrs as $key => $value) {
+                       $table .= ($table ? ' ' : '') . $key . '="' . htmlspecialchars($value) . '"';
+               }
 
                $out = '<table ' . $table . $style . '>' . $section . '</table>';
                return $out;
index 1226834..d34a8e0 100644 (file)
@@ -366,7 +366,7 @@ class t3lib_timeTrack {
 
                                // key value:
                        $keyValue = $data['value'];
-                       $item .= '<td class="' . $logRowClass . ' typo3-adminPanel-tsLogTime" style="' . $bgColor . '">' . $this->fw(htmlspecialchars($keyValue)) . '</td>';
+                       $item .= '<td class="' . $logRowClass . ' typo3-adminPanel-tsLogTime">' . $this->fw(htmlspecialchars($keyValue)) . '</td>';
 
                        if ($this->printConf['allTime']) {
                                $item .= '<td class="' . $logRowClass . ' typo3-adminPanel-tsLogTime"> ' . $this->fw($data['starttime']) . '</td>';