Completed
Push — master ( 960e79...53cd45 )
by Michael
48:22 queued 33:24
created
xoops_lib/Xoops/Form/Select.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
      */
73 73
     public function getSize()
74 74
     {
75
-        return (int) $this->get('size');
75
+        return (int)$this->get('size');
76 76
     }
77 77
 
78 78
      /**
@@ -99,11 +99,11 @@  discard block
 block discarded – undo
99 99
      */
100 100
     public function renderOption($optionValue, $optionDisplay, $selected)
101 101
     {
102
-        $rendered = '<option value="' . htmlspecialchars($optionValue, ENT_QUOTES) . '"';
102
+        $rendered = '<option value="'.htmlspecialchars($optionValue, ENT_QUOTES).'"';
103 103
         if (in_array($optionValue, $selected)) {
104 104
             $rendered .= ' selected';
105 105
         }
106
-        $rendered .= '>' . $optionDisplay . '</option>' . "\n";
106
+        $rendered .= '>'.$optionDisplay.'</option>'."\n";
107 107
 
108 108
         return $rendered;
109 109
     }
@@ -115,18 +115,18 @@  discard block
 block discarded – undo
115 115
      */
116 116
     public function defaultRender()
117 117
     {
118
-        $selected = (array) $this->getValue();
118
+        $selected = (array)$this->getValue();
119 119
 
120 120
         $ele_options = $this->getOptions();
121 121
 
122
-        $extra = ($this->getExtra() != '' ? " " . $this->getExtra() : '');
122
+        $extra = ($this->getExtra() != '' ? " ".$this->getExtra() : '');
123 123
         $attributes = $this->renderAttributeString();
124
-        $rendered = '<select ' . $attributes . $extra .' >' . "\n";
124
+        $rendered = '<select '.$attributes.$extra.' >'."\n";
125 125
 
126 126
         if (empty($ele_optgroup)) {
127 127
             foreach ($ele_options as $value => $display) {
128 128
                 if (is_array($display)) {
129
-                    $rendered .= '<optgroup label="' . $value . '">' . "\n";
129
+                    $rendered .= '<optgroup label="'.$value.'">'."\n";
130 130
                     foreach ($display as $optvalue => $optdisplay) {
131 131
                         $rendered .= $this->renderOption($optvalue, $optdisplay, $selected);
132 132
                     }
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
                 }
136 136
             }
137 137
         }
138
-        $rendered .= '</select>' . "\n";
138
+        $rendered .= '</select>'."\n";
139 139
 
140 140
         return $rendered;
141 141
     }
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
             return "\nvar hasSelected = false; var selectBox = myform.{$eltname};"
162 162
                 . "for (i = 0; i < selectBox.options.length; i++ ) { "
163 163
                 . "if (selectBox.options[i].selected == true && selectBox.options[i].value != '') "
164
-                . "{ hasSelected = true; break; } }" . "if (!hasSelected) "
164
+                . "{ hasSelected = true; break; } }"."if (!hasSelected) "
165 165
                 . "{ window.alert(\"{$eltmsg}\"); selectBox.focus(); return false; }";
166 166
         }
167 167
         return '';
Please login to merge, or discard this patch.