chamilo / chamilo-lms

main/inc/lib/pear/HTML/QuickForm/static.php

Spacing +1 added lines, -1 removed lines

@@ -131,7 +131,7 @@
      */
     public function toHtml()
     {
-        return $this->_getTabs() . $this->_text;
+        return $this->_getTabs().$this->_text;
     } //end func toHtml
 
     // }}}

main/inc/lib/pear/HTML/QuickForm/RuleRegistry.php

Spacing +94 added lines, -94 removed lines

@@ -89,7 +89,7 @@ 
         $type = strtolower($type);
         if ($type == 'regex') {
             // Regular expression
-            $rule =& $this->getRule('regex');
+            $rule = & $this->getRule('regex');
             $rule->addData($ruleName, $data1);
 
         } elseif ($type == 'function' || $type == 'callback') {
@@ -208,7 +208,7 @@ 
      */
     function getValidationScript(&$element, $elementName, $ruleData)
     {
-        $reset =  (isset($ruleData['reset'])) ? $ruleData['reset'] : false;
+        $reset = (isset($ruleData['reset'])) ? $ruleData['reset'] : false;
         $rule  = $this->getRule($ruleData['type']);
         if (!is_array($element)) {
             list($jsValue, $jsReset) = $this->_getJsValue($element, $elementName, $reset, null);
@@ -217,31 +217,31 @@ 
             $jsReset = '';
             for ($i = 0; $i < count($element); $i++) {
                 list($tmp_value, $tmp_reset) = $this->_getJsValue($element[$i], $element[$i]->getName(), $reset, $i);
-                $jsValue .= "\n" . $tmp_value;
+                $jsValue .= "\n".$tmp_value;
                 $jsReset .= $tmp_reset;
             }
         }
-        $jsField = isset($ruleData['group'])? $ruleData['group']: $elementName;
+        $jsField = isset($ruleData['group']) ? $ruleData['group'] : $elementName;
         list ($jsPrefix, $jsCheck) = $rule->getValidationScript($ruleData['format']);
         if (!isset($ruleData['howmany'])) {
-            $js = $jsValue . "\n" . $jsPrefix .
-                  "  if (" . str_replace('{jsVar}', 'value', $jsCheck) . " && !errFlag['{$jsField}']) {\n" .
-                  "    errFlag['{$jsField}'] = true;\n" .
-                  "    _qfMsg = _qfMsg + '\\n - {$ruleData['message']}';\n" .
-                  $jsReset .
+            $js = $jsValue."\n".$jsPrefix.
+                  "  if (".str_replace('{jsVar}', 'value', $jsCheck)." && !errFlag['{$jsField}']) {\n".
+                  "    errFlag['{$jsField}'] = true;\n".
+                  "    _qfMsg = _qfMsg + '\\n - {$ruleData['message']}';\n".
+                  $jsReset.
                   "  }\n";
         } else {
-            $js = $jsValue . "\n" . $jsPrefix .
-                  "  var res = 0;\n" .
-                  "  for (var i = 0; i < value.length; i++) {\n" .
-                  "    if (!(" . str_replace('{jsVar}', 'value[i]', $jsCheck) . ")) {\n" .
-                  "      res++;\n" .
-                  "    }\n" .
-                  "  }\n" .
-                  "  if (res < {$ruleData['howmany']} && !errFlag['{$jsField}']) {\n" .
-                  "    errFlag['{$jsField}'] = true;\n" .
-                  "    _qfMsg = _qfMsg + '\\n - {$ruleData['message']}';\n" .
-                  $jsReset .
+            $js = $jsValue."\n".$jsPrefix.
+                  "  var res = 0;\n".
+                  "  for (var i = 0; i < value.length; i++) {\n".
+                  "    if (!(".str_replace('{jsVar}', 'value[i]', $jsCheck).")) {\n".
+                  "      res++;\n".
+                  "    }\n".
+                  "  }\n".
+                  "  if (res < {$ruleData['howmany']} && !errFlag['{$jsField}']) {\n".
+                  "    errFlag['{$jsField}'] = true;\n".
+                  "    _qfMsg = _qfMsg + '\\n - {$ruleData['message']}';\n".
+                  $jsReset.
                   "  }\n";
         }
         return $js;
@@ -262,72 +262,72 @@ 
     */
     function _getJsValue(&$element, $elementName, $reset = false, $index = null)
     {
-        $jsIndex = isset($index)? '[' . $index . ']': '';
-        $tmp_reset = $reset? "    var field = frm.elements['$elementName'];\n": '';
+        $jsIndex = isset($index) ? '['.$index.']' : '';
+        $tmp_reset = $reset ? "    var field = frm.elements['$elementName'];\n" : '';
         if (is_a($element, 'html_quickform_group')) {
             $value = "  _qfGroups['{$elementName}'] = {";
-            $elements =& $element->getElements();
+            $elements = & $element->getElements();
             for ($i = 0, $count = count($elements); $i < $count; $i++) {
-                $append = ($elements[$i]->getType() == 'select' && $elements[$i]->getMultiple())? '[]': '';
-                $value .= "'" . $element->getElementName($i) . $append . "': true" .
-                          ($i < $count - 1? ', ': '');
+                $append = ($elements[$i]->getType() == 'select' && $elements[$i]->getMultiple()) ? '[]' : '';
+                $value .= "'".$element->getElementName($i).$append."': true".
+                          ($i < $count - 1 ? ', ' : '');
             }
             $value .=
-                "};\n" .
-                "  value{$jsIndex} = new Array();\n" .
-                "  var valueIdx = 0;\n" .
-                "  for (var i = 0; i < frm.elements.length; i++) {\n" .
-                "    var _element = frm.elements[i];\n" .
-                "    if (_element.name in _qfGroups['{$elementName}']) {\n" .
-                "      switch (_element.type) {\n" .
-                "        case 'checkbox':\n" .
-                "        case 'radio':\n" .
-                "          if (_element.checked) {\n" .
-                "            value{$jsIndex}[valueIdx++] = _element.value;\n" .
-                "          }\n" .
-                "          break;\n" .
-                "        case 'select-one':\n" .
-                "          if (-1 != _element.selectedIndex) {\n" .
-                "            value{$jsIndex}[valueIdx++] = _element.options[_element.selectedIndex].value;\n" .
-                "          }\n" .
-                "          break;\n" .
-                "        case 'select-multiple':\n" .
-                "          var tmpVal = new Array();\n" .
-                "          var tmpIdx = 0;\n" .
-                "          for (var j = 0; j < _element.options.length; j++) {\n" .
-                "            if (_element.options[j].selected) {\n" .
-                "              tmpVal[tmpIdx++] = _element.options[j].value;\n" .
-                "            }\n" .
-                "          }\n" .
-                "          if (tmpIdx > 0) {\n" .
-                "            value{$jsIndex}[valueIdx++] = tmpVal;\n" .
-                "          }\n" .
-                "          break;\n" .
-                "        default:\n" .
-                "          value{$jsIndex}[valueIdx++] = _element.value;\n" .
-                "      }\n" .
-                "    }\n" .
+                "};\n".
+                "  value{$jsIndex} = new Array();\n".
+                "  var valueIdx = 0;\n".
+                "  for (var i = 0; i < frm.elements.length; i++) {\n".
+                "    var _element = frm.elements[i];\n".
+                "    if (_element.name in _qfGroups['{$elementName}']) {\n".
+                "      switch (_element.type) {\n".
+                "        case 'checkbox':\n".
+                "        case 'radio':\n".
+                "          if (_element.checked) {\n".
+                "            value{$jsIndex}[valueIdx++] = _element.value;\n".
+                "          }\n".
+                "          break;\n".
+                "        case 'select-one':\n".
+                "          if (-1 != _element.selectedIndex) {\n".
+                "            value{$jsIndex}[valueIdx++] = _element.options[_element.selectedIndex].value;\n".
+                "          }\n".
+                "          break;\n".
+                "        case 'select-multiple':\n".
+                "          var tmpVal = new Array();\n".
+                "          var tmpIdx = 0;\n".
+                "          for (var j = 0; j < _element.options.length; j++) {\n".
+                "            if (_element.options[j].selected) {\n".
+                "              tmpVal[tmpIdx++] = _element.options[j].value;\n".
+                "            }\n".
+                "          }\n".
+                "          if (tmpIdx > 0) {\n".
+                "            value{$jsIndex}[valueIdx++] = tmpVal;\n".
+                "          }\n".
+                "          break;\n".
+                "        default:\n".
+                "          value{$jsIndex}[valueIdx++] = _element.value;\n".
+                "      }\n".
+                "    }\n".
                 "  }\n";
             if ($reset) {
                 $tmp_reset =
-                    "    for (var i = 0; i < frm.elements.length; i++) {\n" .
-                    "      var _element = frm.elements[i];\n" .
-                    "      if (_element.name in _qfGroups['{$elementName}']) {\n" .
-                    "        switch (_element.type) {\n" .
-                    "          case 'checkbox':\n" .
-                    "          case 'radio':\n" .
-                    "            _element.checked = _element.defaultChecked;\n" .
-                    "            break;\n" .
-                    "          case 'select-one':\n" .
-                    "          case 'select-multiple':\n" .
-                    "            for (var j = 0; j < _element.options.length; j++) {\n" .
-                    "              _element.options[j].selected = _element.options[j].defaultSelected;\n" .
-                    "            }\n" .
-                    "            break;\n" .
-                    "          default:\n" .
-                    "            _element.value = _element.defaultValue;\n" .
-                    "        }\n" .
-                    "      }\n" .
+                    "    for (var i = 0; i < frm.elements.length; i++) {\n".
+                    "      var _element = frm.elements[i];\n".
+                    "      if (_element.name in _qfGroups['{$elementName}']) {\n".
+                    "        switch (_element.type) {\n".
+                    "          case 'checkbox':\n".
+                    "          case 'radio':\n".
+                    "            _element.checked = _element.defaultChecked;\n".
+                    "            break;\n".
+                    "          case 'select-one':\n".
+                    "          case 'select-multiple':\n".
+                    "            for (var j = 0; j < _element.options.length; j++) {\n".
+                    "              _element.options[j].selected = _element.options[j].defaultSelected;\n".
+                    "            }\n".
+                    "            break;\n".
+                    "          default:\n".
+                    "            _element.value = _element.defaultValue;\n".
+                    "        }\n".
+                    "      }\n".
                     "    }\n";
             }
 
@@ -335,20 +335,20 @@ 
             if ($element->getMultiple()) {
                 $elementName .= '[]';
                 $value =
-                    "  value{$jsIndex} = new Array();\n" .
-                    "  var valueIdx = 0;\n" .
-                    "  for (var i = 0; i < frm.elements['{$elementName}'].options.length; i++) {\n" .
-                    "    if (frm.elements['{$elementName}'].options[i].selected) {\n" .
-                    "      value{$jsIndex}[valueIdx++] = frm.elements['{$elementName}'].options[i].value;\n" .
-                    "    }\n" .
+                    "  value{$jsIndex} = new Array();\n".
+                    "  var valueIdx = 0;\n".
+                    "  for (var i = 0; i < frm.elements['{$elementName}'].options.length; i++) {\n".
+                    "    if (frm.elements['{$elementName}'].options[i].selected) {\n".
+                    "      value{$jsIndex}[valueIdx++] = frm.elements['{$elementName}'].options[i].value;\n".
+                    "    }\n".
                     "  }\n";
             } else {
                 $value = "  value{$jsIndex} = frm.elements['{$elementName}'].selectedIndex == -1? '': frm.elements['{$elementName}'].options[frm.elements['{$elementName}'].selectedIndex].value;\n";
             }
             if ($reset) {
                 $tmp_reset .=
-                    "    for (var i = 0; i < field.options.length; i++) {\n" .
-                    "      field.options[i].selected = field.options[i].defaultSelected;\n" .
+                    "    for (var i = 0; i < field.options.length; i++) {\n".
+                    "      field.options[i].selected = field.options[i].defaultSelected;\n".
                     "    }\n";
             }
 
@@ -362,17 +362,17 @@ 
             }
 
         } elseif ($element->getType() == 'radio') {
-            $value = "  value{$jsIndex} = '';\n" .
+            $value = "  value{$jsIndex} = '';\n".
                      // Fix for bug #5644
-                     "  var els = 'length' in frm.elements['$elementName']? frm.elements['$elementName']: [ frm.elements['$elementName'] ];\n" .
-                     "  for (var i = 0; i < els.length; i++) {\n" .
-                     "    if (els[i].checked) {\n" .
-                     "      value{$jsIndex} = els[i].value;\n" .
-                     "    }\n" .
+                     "  var els = 'length' in frm.elements['$elementName']? frm.elements['$elementName']: [ frm.elements['$elementName'] ];\n".
+                     "  for (var i = 0; i < els.length; i++) {\n".
+                     "    if (els[i].checked) {\n".
+                     "      value{$jsIndex} = els[i].value;\n".
+                     "    }\n".
                      "  }";
             if ($reset) {
-                $tmp_reset .= "    for (var i = 0; i < field.length; i++) {\n" .
-                              "      field[i].checked = field[i].defaultChecked;\n" .
+                $tmp_reset .= "    for (var i = 0; i < field.length; i++) {\n".
+                              "      field[i].checked = field[i].defaultChecked;\n".
                               "    }";
             }
 

main/inc/lib/pear/HTML/QuickForm/checkbox.php

Spacing +6 added lines, -6 removed lines

@@ -124,7 +124,7 @@ 
      */
     function getChecked()
     {
-        return (bool)$this->getAttribute('checked');
+        return (bool) $this->getAttribute('checked');
     } //end func getChecked
 
     // }}}
@@ -147,9 +147,9 @@ 
             $labelClass = $this->labelClass;
             $checkboxClass = $this->checkboxClass;
 
-            $label ='
+            $label = '
                 <div class="'.$checkboxClass.'">
-                <label class="'.$labelClass.'">' .
+                <label class="'.$labelClass.'">'.
                     HTML_QuickForm_input::toHtml().$this->_text.
                 '</label>
                 </div>
@@ -158,7 +158,7 @@ 
             return $label;
         }
 
-        return HTML_QuickForm_input::toHtml() . $label;
+        return HTML_QuickForm_input::toHtml().$label;
     } //end func toHtml
 
     // }}}
@@ -174,7 +174,7 @@ 
     function getFrozenHtml()
     {
         if ($this->getChecked()) {
-            return '<code>[x]</code>' .
+            return '<code>[x]</code>'.
                    $this->_getPersistantData();
         } else {
             return '<code>[ ]</code>';
@@ -295,7 +295,7 @@ 
     {
         $value = $this->_findValue($submitValues);
         if (null === $value) {
-            $value = $this->getChecked()? true: null;
+            $value = $this->getChecked() ? true : null;
         }
         return $this->_prepareValue($value, $assoc);
     }

main/inc/lib/pear/HTML/QuickForm/Rule/Email.php

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
     function validate($email, $checkDomain = false)
     {
         // Fix for bug #10799: add 'D' modifier to regex
-        if (preg_match($this->regex . 'D', $email)) {
+        if (preg_match($this->regex.'D', $email)) {
             if ($checkDomain && function_exists('checkdnsrr')) {
                 $tokens = explode('@', $email);
                 if (checkdnsrr($tokens[1], 'MX') || checkdnsrr($tokens[1], 'A')) {
@@ -61,7 +61,7 @@ 
 
     function getValidationScript($options = null)
     {
-        return array("  var regex = " . $this->regex . ";\n", "{jsVar} != '' && !regex.test({jsVar})");
+        return array("  var regex = ".$this->regex.";\n", "{jsVar} != '' && !regex.test({jsVar})");
     } // end func getValidationScript
 
 } // end class HTML_QuickForm_Rule_Email

main/inc/lib/pear/HTML/QuickForm/Rule/Callback.php

Spacing +3 added lines, -3 removed lines

@@ -106,10 +106,10 @@
     {
         if (isset($this->_data[$this->name])) {
             $callback = $this->_data[$this->name][0];
-            $params   = ($this->_BCMode[$this->name]? "'', {jsVar}": '{jsVar}') .
-                        (isset($options)? ", '{$options}'": '');
+            $params   = ($this->_BCMode[$this->name] ? "'', {jsVar}" : '{jsVar}').
+                        (isset($options) ? ", '{$options}'" : '');
         } else {
-            $callback = is_array($options)? $options[1]: $options;
+            $callback = is_array($options) ? $options[1] : $options;
             $params   = '{jsVar}';
         }
         return array('', "{jsVar} != '' && !{$callback}({$params})");

main/inc/lib/pear/HTML/QuickForm/Rule/Required.php

Spacing +2 added lines, -2 removed lines

@@ -48,14 +48,14 @@
                 isset($value['tmp_name']) &&

                 isset($value['size']) &&

                 isset($value['error'])

-            ){

+            ) {

                 if (empty($value['tmp_name'])) {

 

                     return false;

                 }

             }

         } else {

-            if ((string)$value == '') {

+            if ((string) $value == '') {

 

                 return false;

             }

main/inc/lib/pear/HTML/QuickForm/Rule/Compare.php

Spacing +2 added lines, -2 removed lines

@@ -78,9 +78,9 @@
         $operator = $this->_findOperator($operator);
 
         if ('===' != $operator && '!==' != $operator) {
-            $compareFn = create_function('$a, $b', 'return floatval($a) ' . $operator . ' floatval($b);');
+            $compareFn = create_function('$a, $b', 'return floatval($a) '.$operator.' floatval($b);');
         } else {
-            $compareFn = create_function('$a, $b', 'return strval($a) ' . $operator . ' strval($b);');
+            $compareFn = create_function('$a, $b', 'return strval($a) '.$operator.' strval($b);');
         }
 
         return $compareFn($values[0], $values[1]);

main/inc/lib/pear/HTML/QuickForm/Rule/Regex.php

Spacing +4 added lines, -4 removed lines

@@ -61,11 +61,11 @@ 
     {
         // Fix for bug #10799: add 'D' modifier to regex
         if (isset($this->_data[$this->name])) {
-            if (!preg_match($this->_data[$this->name] . 'D', $value)) {
+            if (!preg_match($this->_data[$this->name].'D', $value)) {
                 return false;
             }
         } else {
-            if (!preg_match($regex . 'D', $value)) {
+            if (!preg_match($regex.'D', $value)) {
                 return false;
             }
         }
@@ -91,11 +91,11 @@ 
 
         // bug #12376, converting unicode escapes and stripping 'u' modifier
         if ($pos = strpos($regex, 'u', strrpos($regex, '/'))) {
-            $regex = substr($regex, 0, $pos) . substr($regex, $pos + 1);
+            $regex = substr($regex, 0, $pos).substr($regex, $pos + 1);
             $regex = preg_replace('/(?<!\\\\)(?>\\\\\\\\)*\\\\x{([a-fA-F0-9]+)}/', '\\u$1', $regex);
         }
 
-        return array("  var regex = " . $regex . ";\n", "{jsVar} != '' && !regex.test({jsVar})");
+        return array("  var regex = ".$regex.";\n", "{jsVar} != '' && !regex.test({jsVar})");
     } // end func getValidationScript
 
 } // end class HTML_QuickForm_Rule_Regex

main/inc/lib/pear/HTML/QuickForm/select.php

Spacing +24 added lines, -24 removed lines

@@ -86,7 +86,7 @@ 
             if (!empty($attributes['class'])) {
                 $oldClass = $attributes['class'];
             }
-            $attributes['class'] = $oldClass . ' selectpicker show-tick form-control';
+            $attributes['class'] = $oldClass.' selectpicker show-tick form-control';
             $attributes['data-live-search'] = 'true';
         }
         $columnsSize = isset($attributes['cols-size']) ? $attributes['cols-size'] : null;
@@ -196,7 +196,7 @@ 
     function getPrivateName()
     {
         if ($this->getAttribute('multiple')) {
-            return $this->getName() . '[]';
+            return $this->getName().'[]';
         } else {
             return $this->getName();
         }
@@ -312,7 +312,7 @@ 
      */
     function getMultiple()
     {
-        return (bool)$this->getAttribute('multiple');
+        return (bool) $this->getAttribute('multiple');
     } //end func getMultiple
 
     // }}}
@@ -332,7 +332,7 @@ 
     function addOption($text, $value, $attributes = null, $return_array = false)
     {
         if (null === $attributes) {
-            $attributes = array('value' => (string)$value);
+            $attributes = array('value' => (string) $value);
         } else {
             $attributes = $this->_parseAttributes($attributes);
             if (isset($attributes['selected'])) {
@@ -344,7 +344,7 @@ 
                     $this->_values[] = $value;
                 }
             }
-            $this->_updateAttrArray($attributes, array('value' => (string)$value));
+            $this->_updateAttrArray($attributes, array('value' => (string) $value));
         }
         if ($return_array) {
             return array('text' => $text, 'attr' => $attributes);
@@ -383,7 +383,7 @@ 
      * @return    PEAR_Error on error or true
      * @throws    PEAR_Error
      */
-    function loadArray($arr, $values=null)
+    function loadArray($arr, $values = null)
     {
         if (!is_array($arr)) {
             return PEAR::raiseError('Argument 1 of HTML_Select::loadArray is not a valid array');
@@ -420,7 +420,7 @@ 
      * @return    PEAR_Error on error or true
      * @throws    PEAR_Error
      */
-    function loadDbResult(&$result, $textCol=null, $valueCol=null, $values=null)
+    function loadDbResult(&$result, $textCol = null, $valueCol = null, $values = null)
     {
         if (!is_object($result) || !is_a($result, 'db_result')) {
             return PEAR::raiseError('Argument 1 of HTML_Select::loadDbResult is not a valid DB_result');
@@ -429,7 +429,7 @@ 
             $this->setValue($values);
         }
         $fetchMode = ($textCol && $valueCol) ? DB_FETCHMODE_ASSOC : DB_FETCHMODE_ORDERED;
-        while (is_array($row = $result->fetchRow($fetchMode)) ) {
+        while (is_array($row = $result->fetchRow($fetchMode))) {
             if ($fetchMode == DB_FETCHMODE_ASSOC) {
                 $this->addOption($row[$textCol], $row[$valueCol]);
             } else {
@@ -455,7 +455,7 @@ 
      * @return    void
      * @throws    PEAR_Error
      */
-    function loadQuery(&$conn, $sql, $textCol=null, $valueCol=null, $values=null)
+    function loadQuery(&$conn, $sql, $textCol = null, $valueCol = null, $values = null)
     {
         if (is_string($conn)) {
             require_once('DB.php');
@@ -502,7 +502,7 @@ 
      * @return    PEAR_Error on error or true
      * @throws    PEAR_Error
      */
-    function load(&$options, $param1=null, $param2=null, $param3=null, $param4=null)
+    function load(&$options, $param1 = null, $param2 = null, $param3 = null, $param4 = null)
     {
         switch (true) {
             case is_array($options):
@@ -539,32 +539,32 @@ 
             $strHtml = '';
 
             if ($this->getComment() != '') {
-                $strHtml .= $tabs . '<!-- ' . $this->getComment() . " //-->\n";
+                $strHtml .= $tabs.'<!-- '.$this->getComment()." //-->\n";
             }
 
             if (!$this->getMultiple()) {
                 $attrString = $this->_getAttrString($this->_attributes);
             } else {
                 $myName = $this->getName();
-                $this->setName($myName . '[]');
+                $this->setName($myName.'[]');
                 $attrString = $this->_getAttrString($this->_attributes);
                 $this->setName($myName);
             }
 
-            $strHtml .= $tabs . '<select ' . $attrString . ">\n";
+            $strHtml .= $tabs.'<select '.$attrString.">\n";
 
-            $strValues = is_array($this->_values)? array_map('strval', $this->_values): array();
+            $strValues = is_array($this->_values) ? array_map('strval', $this->_values) : array();
 
             foreach ($this->_options as $option) {
 
                 if (!empty($strValues) && in_array($option['attr']['value'], $strValues, true)) {
                     $option['attr']['selected'] = 'selected';
                 }
-                $strHtml .= $tabs . "<option" . $this->_getAttrString($option['attr']) . '>' .
-                    $option['text'] . "</option>";
+                $strHtml .= $tabs."<option".$this->_getAttrString($option['attr']).'>'.
+                    $option['text']."</option>";
             }
             foreach ($this->_optgroups as $optgroup) {
-                $strHtml .= $tabs . '<optgroup label="' . $optgroup['label'] . '">';
+                $strHtml .= $tabs.'<optgroup label="'.$optgroup['label'].'">';
                 foreach ($optgroup['options'] as $option) {
                     $text = $option['text'];
                     unset($option['text']);
@@ -573,11 +573,11 @@ 
                         $option['selected'] = 'selected';
                     }
 
-                    $strHtml .= $tabs . " <option" . $this->_getAttrString($option) . '>' .$text . "</option>";
+                    $strHtml .= $tabs." <option".$this->_getAttrString($option).'>'.$text."</option>";
                 }
                 $strHtml .= "</optgroup>";
             }
-            return $strHtml . $tabs . '</select>';
+            return $strHtml.$tabs.'</select>';
         }
     }
 
@@ -602,22 +602,22 @@ 
                 }
             }
         }
-        $html = empty($value)? '&nbsp;': join('<br />', $value);
+        $html = empty($value) ? '&nbsp;' : join('<br />', $value);
         if ($this->_persistantFreeze) {
             $name = $this->getPrivateName();
             // Only use id attribute if doing single hidden input
             if (1 == count($value)) {
                 $id     = $this->getAttribute('id');
-                $idAttr = isset($id)? array('id' => $id): array();
+                $idAttr = isset($id) ? array('id' => $id) : array();
             } else {
                 $idAttr = array();
             }
             foreach ($value as $key => $item) {
-                $html .= '<input' . $this->_getAttrString(array(
+                $html .= '<input'.$this->_getAttrString(array(
                              'type'  => 'hidden',
                              'name'  => $name,
                              'value' => $this->_values[$key]
-                         ) + $idAttr) . ' />';
+                         ) + $idAttr).' />';
             }
         }
         return $html;
@@ -635,7 +635,7 @@ 
         $value = $this->_findValue($submitValues);
         if (is_null($value)) {
             $value = $this->getValue();
-        } elseif(!is_array($value)) {
+        } elseif (!is_array($value)) {
             $value = array($value);
         }
         if (is_array($value) && !empty($this->_options)) {