Completed
Push — work-fleets ( a5ca82...ab0738 )
by SuperNova.WS
05:46
created
includes/classes/DBStaticRecord.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -55,8 +55,8 @@
 block discarded – undo
55 55
   }
56 56
 
57 57
   /**
58
-   * @param int|string  $recordId
59
-   * @param mixed|array $fieldList
58
+   * @param integer  $recordId
59
+   * @param string[] $fieldList
60 60
    * @param bool        $forUpdate
61 61
    *
62 62
    * @return array|null
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
    */
64 64
   public static function getRecordById($recordId, $fieldList = '*', $forUpdate = false) {
65 65
 //    return static::getRecord(array(static::$_idField => $recordId), $fieldList, $forUpdate);
66
-    return static::getRecord(array(static::$_idField . '=' . $recordId), $fieldList, $forUpdate);
66
+    return static::getRecord(array(static::$_idField.'='.$recordId), $fieldList, $forUpdate);
67 67
   }
68 68
 
69 69
 
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
         DbSqlStatement::build(null, get_called_class())
85 85
           ->select()
86 86
           ->fields(static::$_idField)
87
-          ->where(array("`" . static::$_idField . "` IN (" . implode(',', $idList) . ")"))
87
+          ->where(array("`".static::$_idField."` IN (".implode(',', $idList).")"))
88 88
       );
89 89
     }
90 90
 
Please login to merge, or discard this patch.
includes/classes/DbSqlStatement.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
   // TODO - fields should be escaped !!
175 175
   // TODO - $where should be validated and checked!
176 176
   public function where($where = array()) {
177
-    if(!is_array($where)) {
177
+    if (!is_array($where)) {
178 178
       throw new ExceptionDbSqlWhereNotAnArray();
179 179
     }
180 180
     $this->where = $where;
@@ -209,24 +209,24 @@  discard block
 block discarded – undo
209 209
     $result = '';
210 210
     $result .= $this->stringEscape($this->operation);
211 211
 
212
-    $result .= ' ' . $this->selectFieldsToString($this->fields);
212
+    $result .= ' '.$this->selectFieldsToString($this->fields);
213 213
 
214 214
     $result .= ' FROM';
215
-    $result .= ' `{{' . $this->stringEscape($this->table) . '}}`';
216
-    $result .= !empty($this->alias) ? ' AS `' . $this->stringEscape($this->alias) . '`' : '';
215
+    $result .= ' `{{'.$this->stringEscape($this->table).'}}`';
216
+    $result .= !empty($this->alias) ? ' AS `'.$this->stringEscape($this->alias).'`' : '';
217 217
 
218 218
     // TODO - fields should be escaped !!
219
-    $result .= !empty($this->where) ? ' WHERE ' . implode(' AND ', $this->where) : '';
219
+    $result .= !empty($this->where) ? ' WHERE '.implode(' AND ', $this->where) : '';
220 220
 
221 221
     // TODO - fields should be escaped !!
222
-    $result .= !empty($this->group) ? ' GROUP BY ' . implode(',', $this->group) : '';
222
+    $result .= !empty($this->group) ? ' GROUP BY '.implode(',', $this->group) : '';
223 223
 
224 224
     // TODO - fields should be escaped !!
225
-    $result .= !empty($this->order) ? ' ORDER BY ' . implode(',', $this->order) : '';
225
+    $result .= !empty($this->order) ? ' ORDER BY '.implode(',', $this->order) : '';
226 226
 
227 227
     // TODO - fields should be escaped !!
228 228
     // TODO - separate offset and row_count
229
-    $result .= !empty($this->limit) ? ' LIMIT ' . implode(' OFFSET ', $this->limit) : '';
229
+    $result .= !empty($this->limit) ? ' LIMIT '.implode(' OFFSET ', $this->limit) : '';
230 230
 
231 231
     // TODO - protect from double escape!
232 232
 
@@ -264,7 +264,7 @@  discard block
 block discarded – undo
264 264
    */
265 265
   protected function processField($fieldName) {
266 266
     if (is_bool($fieldName)) {
267
-      $result = (string)intval($fieldName);
267
+      $result = (string) intval($fieldName);
268 268
     } elseif (is_null($fieldName)) {
269 269
       $result = 'NULL';
270 270
     } elseif ($fieldName === '*') {
@@ -282,7 +282,7 @@  discard block
 block discarded – undo
282 282
    * @return string
283 283
    */
284 284
   protected function processFieldDefault($fieldName) {
285
-    $result = (string)$fieldName;
285
+    $result = (string) $fieldName;
286 286
     if (
287 287
       $result != ''
288 288
       &&
@@ -296,7 +296,7 @@  discard block
 block discarded – undo
296 296
       !is_numeric($fieldName)
297 297
     ) {
298 298
       // Other should be formatted
299
-      $result = '`' . $this->stringEscape($result) . '`';
299
+      $result = '`'.$this->stringEscape($result).'`';
300 300
     }
301 301
 
302 302
     return $result;
Please login to merge, or discard this patch.