Passed
Push — dev_2x ( fedccd...fb9ebe )
by Adrian
02:16
created
src/Blueprint/Orm.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -39,27 +39,27 @@
 block discarded – undo
39 39
     {
40 40
         $errors = [];
41 41
 
42
-        if (! $this->entityNamespace) {
42
+        if (!$this->entityNamespace) {
43 43
             $errors[] = 'Missing entity namespace property';
44 44
         }
45 45
 
46
-        if (! $this->entityDestination) {
46
+        if (!$this->entityDestination) {
47 47
             $errors[] = 'Missing entity destination property';
48
-        } elseif (! is_dir($this->entityDestination)) {
48
+        } elseif (!is_dir($this->entityDestination)) {
49 49
             $errors[] = sprintf('%s is not a valid directory', $this->entityDestination);
50
-        } elseif (! is_writable($this->entityDestination)) {
50
+        } elseif (!is_writable($this->entityDestination)) {
51 51
             $errors[] = sprintf('%s is not writable', $this->entityDestination);
52 52
         }
53 53
 
54
-        if (! $this->mapperNamespace) {
54
+        if (!$this->mapperNamespace) {
55 55
             $errors[] = 'Missing mapper namespace property';
56 56
         }
57 57
 
58
-        if (! $this->mapperDestination) {
58
+        if (!$this->mapperDestination) {
59 59
             $errors[] = 'Missing entity destination property';
60
-        } elseif (! is_dir($this->mapperDestination)) {
60
+        } elseif (!is_dir($this->mapperDestination)) {
61 61
             $errors[] = sprintf('%s is not a valid directory', $this->mapperDestination);
62
-        } elseif (! is_writable($this->mapperDestination)) {
62
+        } elseif (!is_writable($this->mapperDestination)) {
63 63
             $errors[] = sprintf('%s is not writable', $this->mapperDestination);
64 64
         }
65 65
 
Please login to merge, or discard this patch.
src/Blueprint/Mapper.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -68,30 +68,30 @@  discard block
 block discarded – undo
68 68
     {
69 69
         $errors = [];
70 70
 
71
-        if (! $this->table) {
71
+        if (!$this->table) {
72 72
             $errors[] = 'Missing table property';
73 73
         }
74 74
 
75
-        if (! $this->className) {
75
+        if (!$this->className) {
76 76
             $errors[] = 'Missing class name property';
77 77
         }
78 78
 
79 79
         if ($this->destination) {
80
-            if (! is_dir($this->destination)) {
80
+            if (!is_dir($this->destination)) {
81 81
                 $errors[] = sprintf('%s is not a valid directory', $this->destination);
82
-            } elseif (! is_writable($this->destination)) {
82
+            } elseif (!is_writable($this->destination)) {
83 83
                 $errors[] = sprintf('%s is not writable', $this->destination);
84 84
             }
85 85
         }
86 86
 
87
-        if (! $this->entityClass) {
87
+        if (!$this->entityClass) {
88 88
             $errors[] = 'Missing entity class name property';
89 89
         }
90 90
 
91 91
         if ($this->entityDestination) {
92
-            if (! is_dir($this->entityDestination)) {
92
+            if (!is_dir($this->entityDestination)) {
93 93
                 $errors[] = sprintf('%s is not a valid directory', $this->entityDestination);
94
-            } elseif (! is_writable($this->entityDestination)) {
94
+            } elseif (!is_writable($this->entityDestination)) {
95 95
                 $errors[] = sprintf('%s is not writable', $this->entityDestination);
96 96
             }
97 97
         }
@@ -353,7 +353,7 @@  discard block
 block discarded – undo
353 353
     public function addRelation($name, Relation $relation): Mapper
354 354
     {
355 355
         $relation->setMapper($this);
356
-        if (! $relation->getForeignMapper()) {
356
+        if (!$relation->getForeignMapper()) {
357 357
             $relation->setForeignMapper(Inflector::pluralize($name));
358 358
         }
359 359
         $relation->setName($name);
@@ -386,19 +386,19 @@  discard block
 block discarded – undo
386 386
 
387 387
         $singular = Inflector::singularize($name);
388 388
 
389
-        if (! $this->table) {
389
+        if (!$this->table) {
390 390
             $this->setTable($name);
391 391
         }
392 392
 
393
-        if ($this->table !== $name && ! $this->tableAlias) {
393
+        if ($this->table !== $name && !$this->tableAlias) {
394 394
             $this->setTableAlias($name);
395 395
         }
396 396
 
397
-        if (! $this->className) {
397
+        if (!$this->className) {
398 398
             $this->setClassName(Str::className($singular) . 'Mapper');
399 399
         }
400 400
 
401
-        if (! $this->entityClass) {
401
+        if (!$this->entityClass) {
402 402
             $this->setEntityClass(Str::className($singular));
403 403
         }
404 404
 
Please login to merge, or discard this patch.
src/Blueprint/Relation.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -49,28 +49,28 @@  discard block
 block discarded – undo
49 49
     {
50 50
         $errors = [];
51 51
 
52
-        if (! $this->name) {
52
+        if (!$this->name) {
53 53
             $errors[] = "Unknown relation name";
54 54
         }
55 55
 
56
-        if (! $this->type) {
56
+        if (!$this->type) {
57 57
             $errors[] = "Unknown relation type";
58 58
         }
59 59
 
60
-        if (! $this->nativeKey) {
60
+        if (!$this->nativeKey) {
61 61
             $errors[] = "Missing native key column";
62 62
         }
63 63
 
64
-        if (! $this->foreignMapper) {
64
+        if (!$this->foreignMapper) {
65 65
             $errors[] = "Missing foreign mapper name";
66 66
         }
67 67
 
68
-        if (! $this->foreignKey) {
68
+        if (!$this->foreignKey) {
69 69
             $errors[] = "Missing foreign key";
70 70
         }
71 71
 
72 72
         $strategies = [RelationConfig::LOAD_LAZY, RelationConfig::LOAD_EAGER, RelationConfig::LOAD_NONE];
73
-        if (! in_array($this->loadStrategy, $strategies)) {
73
+        if (!in_array($this->loadStrategy, $strategies)) {
74 74
             $errors[] = sprintf("Relation loading strategy is not valid (allowed values: %s)", implode(', ', $strategies));
75 75
         }
76 76
 
@@ -205,7 +205,7 @@  discard block
 block discarded – undo
205 205
                 $value !== 0 &&
206 206
                 $value !== '' &&
207 207
                 $value !== [] &&
208
-                ! is_object($value)) {
208
+                !is_object($value)) {
209 209
                 $result[Str::underscore($prop)] = $value;
210 210
             }
211 211
             if (is_object($value) && is_callable($value)) {
Please login to merge, or discard this patch.
src/Blueprint/Column.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -220,11 +220,11 @@
 block discarded – undo
220 220
     {
221 221
         $errors = [];
222 222
 
223
-        if (! $this->name) {
223
+        if (!$this->name) {
224 224
             $errors[] = 'Column requires a name';
225 225
         }
226 226
 
227
-        if (! $this->type) {
227
+        if (!$this->type) {
228 228
             $errors[] = 'Column requires a type';
229 229
         } elseif (($type = $this->getConstantByValue($this->type)) && substr($type, 0, 5) !== 'TYPE_') {
230 230
             $errors[] = sprintf('Column does not have a valid type (%s)', $this->type);
Please login to merge, or discard this patch.
src/Blueprint/ClassMethod.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -32,11 +32,11 @@
 block discarded – undo
32 32
     {
33 33
         $errors = [];
34 34
 
35
-        if (! $this->name) {
35
+        if (!$this->name) {
36 36
             $errors[] = 'Method requires a name';
37 37
         }
38 38
 
39
-        if (! in_array($this->visibility, [ClassType::VISIBILITY_PUBLIC, ClassType::VISIBILITY_PROTECTED])) {
39
+        if (!in_array($this->visibility, [ClassType::VISIBILITY_PUBLIC, ClassType::VISIBILITY_PROTECTED])) {
40 40
             $errors[] = 'Wrong method visilibity type. Only `public` and `protected` are allowed.';
41 41
         }
42 42
 
Please login to merge, or discard this patch.
src/Blueprint/Relation/ManyToMany.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -58,11 +58,11 @@  discard block
 block discarded – undo
58 58
 
59 59
     protected function maybeSetAdditionalProperties()
60 60
     {
61
-        if (! $this->mapper || ! $this->foreignMapper) {
61
+        if (!$this->mapper || !$this->foreignMapper) {
62 62
             return;
63 63
         }
64 64
 
65
-        if (! $this->throughTable) {
65
+        if (!$this->throughTable) {
66 66
             $tablePrefix = $this->mapper->getTableAlias() ?
67 67
                 str_replace($this->mapper->getTable(), '', $this->mapper->getTableAlias())
68 68
                 : '';
@@ -77,11 +77,11 @@  discard block
 block discarded – undo
77 77
             }
78 78
         }
79 79
 
80
-        if (! $this->throughNativeColumn) {
80
+        if (!$this->throughNativeColumn) {
81 81
             $this->throughNativeColumn = Inflector::singularize($this->mapper->getName()) . '_id';
82 82
         }
83 83
 
84
-        if (! $this->throughForeignColumn) {
84
+        if (!$this->throughForeignColumn) {
85 85
             $this->throughForeignColumn = Inflector::singularize($this->foreignMapper) . '_id';
86 86
         }
87 87
     }
Please login to merge, or discard this patch.
src/Query.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
             return $this;
139 139
         }
140 140
 
141
-        if (! $this->mapper->hasRelation($name)) {
141
+        if (!$this->mapper->hasRelation($name)) {
142 142
             throw new \InvalidArgumentException(
143 143
                 sprintf("Relation %s, not defined for %s", $name, $this->mapper->getConfig()->getTable())
144 144
             );
@@ -200,8 +200,8 @@  discard block
 block discarded – undo
200 200
             return null;
201 201
         }
202 202
 
203
-        $receivedTracker = ! ! $tracker;
204
-        if (! $tracker) {
203
+        $receivedTracker = !!$tracker;
204
+        if (!$tracker) {
205 205
             $receivedTracker = false;
206 206
             $tracker         = new Tracker([$data]);
207 207
         }
@@ -211,7 +211,7 @@  discard block
 block discarded – undo
211 211
         $this->injectAggregates($entity, $tracker, $load);
212 212
         $entity->setState(StateEnum::SYNCHRONIZED);
213 213
 
214
-        if (! $receivedTracker) {
214
+        if (!$receivedTracker) {
215 215
             $tracker->replaceRows([$entity]);
216 216
         }
217 217
 
@@ -262,7 +262,7 @@  discard block
 block discarded – undo
262 262
             $queryCallback = $eagerLoad[$name] ?? null;
263 263
             $nextLoad      = Arr::getChildren($eagerLoad, $name);
264 264
 
265
-            if (! $tracker->hasRelation($name)) {
265
+            if (!$tracker->hasRelation($name)) {
266 266
                 $tracker->setRelation($name, $relation, $queryCallback, $nextLoad);
267 267
             }
268 268
 
@@ -278,7 +278,7 @@  discard block
 block discarded – undo
278 278
     {
279 279
         foreach ($this->mapper->getRelations() as $name) {
280 280
             $relation = $this->mapper->getRelation($name);
281
-            if (! method_exists($relation, 'getAggregates')) {
281
+            if (!method_exists($relation, 'getAggregates')) {
282 282
                 continue;
283 283
             }
284 284
             $aggregates = $relation->getAggregates();
@@ -307,7 +307,7 @@  discard block
 block discarded – undo
307 307
      */
308 308
     public function chunk(int $size, callable $callback, int $limit = 100000)
309 309
     {
310
-        if (! $this->orderBy->build()) {
310
+        if (!$this->orderBy->build()) {
311 311
             $this->orderBy(...(array)$this->mapper->getConfig()->getPrimaryKey());
312 312
         }
313 313
 
Please login to merge, or discard this patch.