Passed
Pull Request — master (#223)
by
unknown
05:06
created
src/Traits/BuilderCaching.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -6,7 +6,7 @@
 block discarded – undo
6 6
 {
7 7
     public function all($columns = ['*']) : Collection
8 8
     {
9
-        if (! $this->isCachable()) {
9
+        if (!$this->isCachable()) {
10 10
             $this->model->disableModelCaching();
11 11
         }
12 12
 
Please login to merge, or discard this patch.
src/Providers/Service.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -9,7 +9,7 @@
 block discarded – undo
9 9
 
10 10
     public function boot()
11 11
     {
12
-        $configPath = __DIR__ . '/../../config/laravel-model-caching.php';
12
+        $configPath = __DIR__.'/../../config/laravel-model-caching.php';
13 13
         $this->mergeConfigFrom($configPath, 'laravel-model-caching');
14 14
         $this->commands(Clear::class);
15 15
     }
Please login to merge, or discard this patch.
src/CachedBuilder.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@  discard block
 block discarded – undo
15 15
 
16 16
     public function avg($column)
17 17
     {
18
-        if (! $this->isCachable()) {
18
+        if (!$this->isCachable()) {
19 19
             return parent::avg($column);
20 20
         }
21 21
 
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
 
27 27
     public function count($columns = "*")
28 28
     {
29
-        if (! $this->isCachable()) {
29
+        if (!$this->isCachable()) {
30 30
             return parent::count($columns);
31 31
         }
32 32
 
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
      */
49 49
     public function find($id, $columns = ["*"])
50 50
     {
51
-        if (! $this->isCachable()) {
51
+        if (!$this->isCachable()) {
52 52
             return parent::find($id, $columns);
53 53
         }
54 54
 
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
 
61 61
     public function first($columns = ["*"])
62 62
     {
63
-        if (! $this->isCachable()) {
63
+        if (!$this->isCachable()) {
64 64
             return parent::first($columns);
65 65
         }
66 66
 
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
 
72 72
     public function get($columns = ["*"])
73 73
     {
74
-        if (! $this->isCachable()) {
74
+        if (!$this->isCachable()) {
75 75
             return parent::get($columns);
76 76
         }
77 77
 
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
 
97 97
     public function max($column)
98 98
     {
99
-        if (! $this->isCachable()) {
99
+        if (!$this->isCachable()) {
100 100
             return parent::max($column);
101 101
         }
102 102
 
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
 
108 108
     public function min($column)
109 109
     {
110
-        if (! $this->isCachable()) {
110
+        if (!$this->isCachable()) {
111 111
             return parent::min($column);
112 112
         }
113 113
 
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
         $pageName = "page",
123 123
         $page = null
124 124
     ) {
125
-        if (! $this->isCachable()) {
125
+        if (!$this->isCachable()) {
126 126
             return parent::paginate($perPage, $columns, $pageName, $page);
127 127
         }
128 128
 
@@ -143,12 +143,12 @@  discard block
 block discarded – undo
143 143
 
144 144
         foreach ($items as $key => $value) {
145 145
             if (is_array($value)) {
146
-                $result .= $key . $glue . $this->recursiveImplodeWithKey($value, $glue);
146
+                $result .= $key.$glue.$this->recursiveImplodeWithKey($value, $glue);
147 147
 
148 148
                 continue;
149 149
             }
150 150
 
151
-            $result .= $glue . $key . $glue . $value;
151
+            $result .= $glue.$key.$glue.$value;
152 152
         }
153 153
 
154 154
         return $result;
@@ -156,11 +156,11 @@  discard block
 block discarded – undo
156 156
 
157 157
     public function pluck($column, $key = null)
158 158
     {
159
-        if (! $this->isCachable()) {
159
+        if (!$this->isCachable()) {
160 160
             return parent::pluck($column, $key);
161 161
         }
162 162
 
163
-        $keyDifferentiator = "-pluck_{$column}" . ($key ? "_{$key}" : "");
163
+        $keyDifferentiator = "-pluck_{$column}".($key ? "_{$key}" : "");
164 164
         $cacheKey = $this->makeCacheKey([$column], null, $keyDifferentiator);
165 165
 
166 166
         return $this->cachedValue(func_get_args(), $cacheKey);
@@ -168,7 +168,7 @@  discard block
 block discarded – undo
168 168
 
169 169
     public function sum($column)
170 170
     {
171
-        if (! $this->isCachable()) {
171
+        if (!$this->isCachable()) {
172 172
             return parent::sum($column);
173 173
         }
174 174
 
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
 
187 187
     public function value($column)
188 188
     {
189
-        if (! $this->isCachable()) {
189
+        if (!$this->isCachable()) {
190 190
             return parent::value($column);
191 191
         }
192 192
 
@@ -255,7 +255,7 @@  discard block
 block discarded – undo
255 255
         return $this->cache($cacheTags)
256 256
             ->rememberForever(
257 257
                 $hashedCacheKey,
258
-                function () use ($arguments, $cacheKey, $method) {
258
+                function() use ($arguments, $cacheKey, $method) {
259 259
                     return [
260 260
                         "key" => $cacheKey,
261 261
                         "value" => parent::{$method}(...$arguments),
Please login to merge, or discard this patch.
src/Traits/ModelCaching.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -18,22 +18,22 @@  discard block
 block discarded – undo
18 18
         $key = $instance->makeCacheKey();
19 19
 
20 20
         return $instance->cache($tags)
21
-            ->rememberForever($key, function () use ($columns) {
21
+            ->rememberForever($key, function() use ($columns) {
22 22
                 return parent::all($columns);
23 23
             });
24 24
     }
25 25
 
26 26
     public static function bootCachable()
27 27
     {
28
-        static::created(function ($instance) {
28
+        static::created(function($instance) {
29 29
             $instance->checkCooldownAndFlushAfterPersisting($instance);
30 30
         });
31 31
 
32
-        static::deleted(function ($instance) {
32
+        static::deleted(function($instance) {
33 33
             $instance->checkCooldownAndFlushAfterPersisting($instance);
34 34
         });
35 35
 
36
-        static::saved(function ($instance) {
36
+        static::saved(function($instance) {
37 37
             $instance->checkCooldownAndFlushAfterPersisting($instance);
38 38
         });
39 39
 
@@ -42,15 +42,15 @@  discard block
 block discarded – undo
42 42
         //     $instance->checkCooldownAndFlushAfterPersisting($instance);
43 43
         // });
44 44
 
45
-        static::pivotAttached(function ($instance) {
45
+        static::pivotAttached(function($instance) {
46 46
             $instance->checkCooldownAndFlushAfterPersisting($instance);
47 47
         });
48 48
 
49
-        static::pivotDetached(function ($instance) {
49
+        static::pivotDetached(function($instance) {
50 50
             $instance->checkCooldownAndFlushAfterPersisting($instance);
51 51
         });
52 52
 
53
-        static::pivotUpdated(function ($instance) {
53
+        static::pivotUpdated(function($instance) {
54 54
             $instance->checkCooldownAndFlushAfterPersisting($instance);
55 55
         });
56 56
     }
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
 
67 67
     public function newEloquentBuilder($query)
68 68
     {
69
-        if (! $this->isCachable()) {
69
+        if (!$this->isCachable()) {
70 70
             $this->isCachable = true;
71 71
 
72 72
             return new EloquentBuilder($query);
@@ -93,13 +93,13 @@  discard block
 block discarded – undo
93 93
         $cacheKey = "{$cachePrefix}:{$modelClassName}-cooldown:seconds";
94 94
 
95 95
         $this->cache()
96
-            ->rememberForever($cacheKey, function () use ($seconds) {
96
+            ->rememberForever($cacheKey, function() use ($seconds) {
97 97
                 return $seconds;
98 98
             });
99 99
 
100 100
         $cacheKey = "{$cachePrefix}:{$modelClassName}-cooldown:invalidated-at";
101 101
         $this->cache()
102
-            ->rememberForever($cacheKey, function () {
102
+            ->rememberForever($cacheKey, function() {
103 103
                 return (new Carbon)->now();
104 104
             });
105 105
 
Please login to merge, or discard this patch.
src/CacheKey.php 1 patch
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
 
54 54
     protected function getLimitClause() : string
55 55
     {
56
-        if (! $this->query->limit) {
56
+        if (!$this->query->limit) {
57 57
             return "";
58 58
         }
59 59
 
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 
74 74
     protected function getOffsetClause() : string
75 75
     {
76
-        if (! $this->query->offset) {
76
+        if (!$this->query->offset) {
77 77
             return "";
78 78
         }
79 79
 
@@ -85,12 +85,12 @@  discard block
 block discarded – undo
85 85
         $orders = collect($this->query->orders);
86 86
 
87 87
         return $orders
88
-            ->reduce(function ($carry, $order) {
88
+            ->reduce(function($carry, $order) {
89 89
                 if (($order["type"] ?? "") === "Raw") {
90
-                    return $carry . "_orderByRaw_" . str_slug($order["sql"]);
90
+                    return $carry."_orderByRaw_".str_slug($order["sql"]);
91 91
                 }
92 92
 
93
-                return $carry . "_orderBy_" . $order["column"] . "_" . $order["direction"];
93
+                return $carry."_orderBy_".$order["column"]."_".$order["direction"];
94 94
             })
95 95
             ?: "";
96 96
     }
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
             return "";
102 102
         }
103 103
 
104
-        return "_" . implode("_", $columns);
104
+        return "_".implode("_", $columns);
105 105
     }
106 106
 
107 107
     protected function getTypeClause($where) : string
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
         $values = $this->getValuesFromWhere($where);
123 123
         $values = $this->getValuesFromBindings($where, $values);
124 124
 
125
-        return "_" . $values;
125
+        return "_".$values;
126 126
     }
127 127
 
128 128
     protected function getValuesFromWhere(array $where) : string
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
             $subKey = str_replace($prefix, "", $subKey);
135 135
             $subKey = str_replace($this->getModelSlug(), "", $subKey);
136 136
             $classParts = explode("\\", get_class($this->model));
137
-            $subKey = strtolower(array_pop($classParts)) . $subKey;
137
+            $subKey = strtolower(array_pop($classParts)).$subKey;
138 138
 
139 139
             return $subKey;
140 140
         }
@@ -154,7 +154,7 @@  discard block
 block discarded – undo
154 154
             $this->currentBinding++;
155 155
 
156 156
             if ($where["type"] === "between") {
157
-                $values .= "_" . $this->query->bindings["where"][$this->currentBinding];
157
+                $values .= "_".$this->query->bindings["where"][$this->currentBinding];
158 158
                 $this->currentBinding++;
159 159
             }
160 160
         }
@@ -164,8 +164,8 @@  discard block
 block discarded – undo
164 164
 
165 165
     protected function getWhereClauses(array $wheres = []) : string
166 166
     {
167
-        return "" . $this->getWheres($wheres)
168
-            ->reduce(function ($carry, $where) {
167
+        return "".$this->getWheres($wheres)
168
+            ->reduce(function($carry, $where) {
169 169
                 $value = $carry;
170 170
                 $value .= $this->getNestedClauses($where);
171 171
                 $value .= $this->getColumnClauses($where);
@@ -179,11 +179,11 @@  discard block
 block discarded – undo
179 179
 
180 180
     protected function getNestedClauses(array $where) : string
181 181
     {
182
-        if (! in_array($where["type"], ["Exists", "Nested", "NotExists"])) {
182
+        if (!in_array($where["type"], ["Exists", "Nested", "NotExists"])) {
183 183
             return "";
184 184
         }
185 185
 
186
-        return "-" . strtolower($where["type"]) . $this->getWhereClauses($where["query"]->wheres);
186
+        return "-".strtolower($where["type"]).$this->getWhereClauses($where["query"]->wheres);
187 187
     }
188 188
 
189 189
     protected function getColumnClauses(array $where) : string
@@ -197,7 +197,7 @@  discard block
 block discarded – undo
197 197
 
198 198
     protected function getInClauses(array $where) : string
199 199
     {
200
-        if (! in_array($where["type"], ["In"])) {
200
+        if (!in_array($where["type"], ["In"])) {
201 201
             return "";
202 202
         }
203 203
 
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
 
210 210
     protected function getInAndNotInClauses(array $where) : string
211 211
     {
212
-        if (! in_array($where["type"], ["In", "NotIn"])) {
212
+        if (!in_array($where["type"], ["In", "NotIn"])) {
213 213
             return "";
214 214
         }
215 215
 
@@ -247,7 +247,7 @@  discard block
 block discarded – undo
247 247
                 continue;
248 248
             }
249 249
 
250
-            $result .= $glue . $value;
250
+            $result .= $glue.$value;
251 251
         }
252 252
 
253 253
         return $result;
@@ -263,7 +263,7 @@  discard block
 block discarded – undo
263 263
         $clause = "_{$where["boolean"]}";
264 264
 
265 265
         while (count($queryParts) > 1) {
266
-            $clause .= "_" . array_shift($queryParts);
266
+            $clause .= "_".array_shift($queryParts);
267 267
             $clause .= $this->query->bindings["where"][$this->currentBinding];
268 268
             $this->currentBinding++;
269 269
         }
@@ -271,10 +271,10 @@  discard block
 block discarded – undo
271 271
         $lastPart = array_shift($queryParts);
272 272
 
273 273
         if ($lastPart) {
274
-            $clause .= "_" . $lastPart;
274
+            $clause .= "_".$lastPart;
275 275
         }
276 276
 
277
-        return "-" . str_replace(" ", "_", $clause);
277
+        return "-".str_replace(" ", "_", $clause);
278 278
     }
279 279
 
280 280
     protected function getOtherClauses(array $where) : string
@@ -308,8 +308,8 @@  discard block
 block discarded – undo
308 308
             return "";
309 309
         }
310 310
 
311
-        return $eagerLoads->keys()->reduce(function ($carry, $related) {
312
-            if (! method_exists($this->model, $related)) {
311
+        return $eagerLoads->keys()->reduce(function($carry, $related) {
312
+            if (!method_exists($this->model, $related)) {
313 313
                 return "{$carry}-{$related}";
314 314
             }
315 315
 
Please login to merge, or discard this patch.
src/CacheTags.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
     {
29 29
         $tags = collect($this->eagerLoad)
30 30
             ->keys()
31
-            ->map(function ($relationName) {
31
+            ->map(function($relationName) {
32 32
                 $relation = $this->getRelation($relationName);
33 33
 
34 34
                 return $this->getCachePrefix()
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
     protected function getRelation(string $relationName) : Relation
54 54
     {
55 55
         return collect(explode('.', $relationName))
56
-            ->reduce(function ($carry, $name) {
56
+            ->reduce(function($carry, $name) {
57 57
                 $carry = $carry ?: $this->model;
58 58
                 $carry = $this->getRelatedModel($carry);
59 59
 
Please login to merge, or discard this patch.
src/Traits/CachePrefixing.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -5,10 +5,10 @@
 block discarded – undo
5 5
     protected function getCachePrefix() : string
6 6
     {
7 7
         return "genealabs:laravel-model-caching:"
8
-            . $this->getConnectionName() . ":"
9
-            . $this->getDatabaseName() . ":"
8
+            . $this->getConnectionName().":"
9
+            . $this->getDatabaseName().":"
10 10
             . (config("laravel-model-caching.cache-prefix")
11
-                ? config("laravel-model-caching.cache-prefix", "") . ":"
11
+                ? config("laravel-model-caching.cache-prefix", "").":"
12 12
                 : "");
13 13
     }
14 14
 
Please login to merge, or discard this patch.
config/laravel-model-caching.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,6 +10,6 @@
 block discarded – undo
10 10
     'enable-cooldown' => env('MODEL_CACHE_COOLDOWN_ENABLE', true),
11 11
 
12 12
     'cooldown-disable' => [
13
-		//put full qualified classnames here
13
+        //put full qualified classnames here
14 14
     ]
15 15
 ];
Please login to merge, or discard this patch.
src/Console/Commands/Clear.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@  discard block
 block discarded – undo
12 12
     {
13 13
         $option = $this->option('model');
14 14
 
15
-        if (! $option) {
15
+        if (!$option) {
16 16
             return $this->flushEntireCache();
17 17
         }
18 18
 
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
         $usesCachableTrait = Caching::getAllTraitsUsedByClass($option)
37 37
             ->contains("GeneaLabs\LaravelModelCaching\Traits\Cachable");
38 38
 
39
-        if (! $usesCachableTrait) {
39
+        if (!$usesCachableTrait) {
40 40
             $this->error("'{$option}' is not an instance of CachedModel.");
41 41
             $this->line("Only CachedModel instances can be flushed.");
42 42
 
Please login to merge, or discard this patch.