Completed
Push — master ( c99a2b...4f91fd )
by Alexey
06:07
created
system/modules/Ui/objects/DataManager.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
      * Construct new data manager
29 29
      * 
30 30
      * @param string $modelName
31
-     * @param string|array $dataManager
31
+     * @param string $dataManager
32 32
      * @throws Exception
33 33
      */
34 34
     public function __construct($modelName, $dataManager = 'manager')
@@ -330,6 +330,9 @@  discard block
 block discarded – undo
330 330
         return $rows;
331 331
     }
332 332
 
333
+    /**
334
+     * @param DataManager $dataManager
335
+     */
333 336
     public static function drawCol($item, $colName, $params = [], $dataManager = null, $originalCol = '', $originalItem = null)
334 337
     {
335 338
         $modelName = get_class($item);
Please login to merge, or discard this patch.
system/modules/Db/objects/Mysql/Query.php 1 patch
Braces   +51 added lines, -40 removed lines patch added patch discarded remove patch
@@ -126,12 +126,14 @@  discard block
 block discarded – undo
126 126
         if (!is_array($order)) {
127 127
             $this->order[] = "{$order} {$type}";
128 128
         } else {
129
-            foreach ($order as $item)
130
-                if (!is_array($item)) {
129
+            foreach ($order as $item) {
130
+                            if (!is_array($item)) {
131 131
                     call_user_func_array(array($this, 'order'), $order);
132
+            }
132 133
                     break;
133
-                } else
134
-                    $this->order($item);
134
+                } else {
135
+                                    $this->order($item);
136
+                }
135 137
         }
136 138
     }
137 139
 
@@ -140,8 +142,9 @@  discard block
 block discarded – undo
140 142
         $start = intval($start);
141 143
         $len = intval($len);
142 144
         $this->limit = "LIMIT {$start}";
143
-        if ($len !== 0)
144
-            $this->limit .= ",{$len}";
145
+        if ($len !== 0) {
146
+                    $this->limit .= ",{$len}";
147
+        }
145 148
     }
146 149
 
147 150
     public function buildJoin($table, $where = false, $type = 'LEFT', $alias = '')
@@ -154,10 +157,12 @@  discard block
 block discarded – undo
154 157
             }
155 158
         } else {
156 159
             $join .= " {$type} JOIN {$this->curInstance->table_prefix}{$table}";
157
-            if ($alias)
158
-                $join .= " AS `{$alias}`";
159
-            if ($where)
160
-                $join .= " ON {$where}";
160
+            if ($alias) {
161
+                            $join .= " AS `{$alias}`";
162
+            }
163
+            if ($where) {
164
+                            $join .= " ON {$where}";
165
+            }
161 166
         }
162 167
         return $join;
163 168
     }
@@ -177,31 +182,33 @@  discard block
 block discarded – undo
177 182
                 $operation = '=';
178 183
             }
179 184
 
180
-            if ($concatenation === false)
181
-                $concatenation = 'AND';
182
-            elseif ($concatenation === true)
183
-                $concatenation = '';
185
+            if ($concatenation === false) {
186
+                            $concatenation = 'AND';
187
+            } elseif ($concatenation === true) {
188
+                            $concatenation = '';
189
+            }
184 190
 
185
-            if ($this->whereString == NULL)
186
-                $this->whereString = ' WHERE ';
191
+            if ($this->whereString == NULL) {
192
+                            $this->whereString = ' WHERE ';
193
+            }
187 194
 
188 195
             if (stristr($operation, 'IN') || stristr($operation, 'NOT IN')) {
189
-                if (!preg_match('!\(!', $value) && !preg_match('![^0-9,\.\(\) ]!', $value))
190
-                    $value = "({$value})";
191
-                elseif (preg_match('!\(!', $value) && preg_match('![^0-9,\.\(\) ]!', $value))
192
-                    $value = "\"{$value}\"";
193
-            }
194
-            elseif (!in_array($value, array('CURRENT_TIMESTAMP'))) {
196
+                if (!preg_match('!\(!', $value) && !preg_match('![^0-9,\.\(\) ]!', $value)) {
197
+                                    $value = "({$value})";
198
+                } elseif (preg_match('!\(!', $value) && preg_match('![^0-9,\.\(\) ]!', $value)) {
199
+                                    $value = "\"{$value}\"";
200
+                }
201
+            } elseif (!in_array($value, array('CURRENT_TIMESTAMP'))) {
195 202
                 $this->params[] = $value;
196 203
                 $value = "?";
197 204
             }
198 205
 
199
-            if (substr($this->whereString, -1, 1) == '(' || substr($this->whereString, -2, 2) == 'E ')
200
-                $this->whereString .= " {$where} {$operation} {$value} ";
201
-            else
202
-                $this->whereString .= "{$concatenation} {$where} {$operation} {$value} ";
203
-        }
204
-        else {
206
+            if (substr($this->whereString, -1, 1) == '(' || substr($this->whereString, -2, 2) == 'E ') {
207
+                            $this->whereString .= " {$where} {$operation} {$value} ";
208
+            } else {
209
+                            $this->whereString .= "{$concatenation} {$where} {$operation} {$value} ";
210
+            }
211
+        } else {
205 212
             $i = -1;
206 213
             while (isset($where[++$i])) {
207 214
                 $item = $where[$i];
@@ -216,28 +223,32 @@  discard block
 block discarded – undo
216 223
                         $this->whereString .= "{$concatenation} ";
217 224
                     }
218 225
 
219
-                    if ($this->whereString != NULL)
220
-                        $this->whereString .= '(';
221
-                    else
222
-                        $this->whereString = 'WHERE (';
226
+                    if ($this->whereString != NULL) {
227
+                                            $this->whereString .= '(';
228
+                    } else {
229
+                                            $this->whereString = 'WHERE (';
230
+                    }
223 231
                 }
224 232
 
225 233
                 if (!is_array($item)) {
226 234
                     call_user_func_array(array($this, 'buildWhere'), $where);
227 235
                     break;
228 236
                 } else {
229
-                    if ($this->whereString != NULL && substr($this->whereString, -1, 1) != '(')
230
-                        if (!isset($item[3]))
237
+                    if ($this->whereString != NULL && substr($this->whereString, -1, 1) != '(') {
238
+                                            if (!isset($item[3]))
231 239
                             $concatenation = 'AND';
232
-                        else
233
-                            $concatenation = $item[3];
234
-                    elseif (substr($this->whereString, -1, 1) != '(')
235
-                        $this->whereString = 'WHERE ';
240
+                    } else {
241
+                                                    $concatenation = $item[3];
242
+                        }
243
+                    elseif (substr($this->whereString, -1, 1) != '(') {
244
+                                            $this->whereString = 'WHERE ';
245
+                    }
236 246
 
237 247
                     $this->buildWhere($item);
238 248
                 }
239
-                if (!isset($where[$i + 1]) && isset($where[$i - 1]))
240
-                    $this->whereString .= ') ';
249
+                if (!isset($where[$i + 1]) && isset($where[$i - 1])) {
250
+                                    $this->whereString .= ') ';
251
+                }
241 252
             }
242 253
         }
243 254
     }
Please login to merge, or discard this patch.