GeneaLabs / laravel-model-caching

src/CacheKey.php

Spacing +31 added lines, -31 removed lines

@@ -53,7 +53,7 @@ 
 
     protected function getBindingsSlug() : string
     {
-        if (! method_exists($this->model, 'query')) {
+        if (!method_exists($this->model, 'query')) {
             return '';
         }
 
@@ -81,7 +81,7 @@ 
 
     protected function getInAndNotInClauses(array $where) : string
     {
-        if (! in_array($where["type"], ["In", "NotIn", "InRaw"])) {
+        if (!in_array($where["type"], ["In", "NotIn", "InRaw"])) {
             return "";
         }
 
@@ -93,7 +93,7 @@ 
             $this->currentBinding += count($where["values"]);
         }
 
-        if (! is_numeric($subquery) && ! is_numeric(str_replace("_", "", $subquery))) {
+        if (!is_numeric($subquery) && !is_numeric(str_replace("_", "", $subquery))) {
             try {
                 $subquery = Uuid::fromBytes($subquery);
                 $values = $this->recursiveImplode([$subquery], "_");
@@ -113,8 +113,8 @@ 
 
     protected function getLimitClause() : string
     {
-        if (! property_exists($this->query, "limit")
-            || ! $this->query->limit
+        if (!property_exists($this->query, "limit")
+            || !$this->query->limit
         ) {
             return "";
         }
@@ -129,17 +129,17 @@ 
 
     protected function getNestedClauses(array $where) : string
     {
-        if (! in_array($where["type"], ["Exists", "Nested", "NotExists"])) {
+        if (!in_array($where["type"], ["Exists", "Nested", "NotExists"])) {
             return "";
         }
 
-        return "-" . strtolower($where["type"]) . $this->getWhereClauses($where["query"]->wheres);
+        return "-".strtolower($where["type"]).$this->getWhereClauses($where["query"]->wheres);
     }
 
     protected function getOffsetClause() : string
     {
-        if (! property_exists($this->query, "offset")
-            || ! $this->query->offset
+        if (!property_exists($this->query, "offset")
+            || !$this->query->offset
         ) {
             return "";
         }
@@ -149,8 +149,8 @@ 
 
     protected function getOrderByClauses() : string
     {
-        if (! property_exists($this->query, "orders")
-            || ! $this->query->orders
+        if (!property_exists($this->query, "orders")
+            || !$this->query->orders
         ) {
             return "";
         }
@@ -158,12 +158,12 @@ 
         $orders = collect($this->query->orders);
 
         return $orders
-            ->reduce(function ($carry, $order) {
+            ->reduce(function($carry, $order) {
                 if (($order["type"] ?? "") === "Raw") {
-                    return $carry . "_orderByRaw_" . (new Str)->slug($order["sql"]);
+                    return $carry."_orderByRaw_".(new Str)->slug($order["sql"]);
                 }
 
-                return $carry . "_orderBy_" . $order["column"] . "_" . $order["direction"];
+                return $carry."_orderBy_".$order["column"]."_".$order["direction"];
             })
             ?: "";
     }
@@ -184,8 +184,8 @@ 
     {
         if (($columns === ["*"]
                 || $columns === [])
-            && (! property_exists($this->query, "columns")
-                || ! $this->query->columns)
+            && (!property_exists($this->query, "columns")
+                || !$this->query->columns)
         ) {
             return "";
         }
@@ -193,15 +193,15 @@ 
         if (property_exists($this->query, "columns")
             && $this->query->columns
         ) {
-            return "_" . implode("_", $this->query->columns);
+            return "_".implode("_", $this->query->columns);
         }
 
-        return "_" . implode("_", $columns);
+        return "_".implode("_", $columns);
     }
 
     protected function getRawClauses(array $where) : string
     {
-        if (! in_array($where["type"], ["raw"])) {
+        if (!in_array($where["type"], ["raw"])) {
             return "";
         }
 
@@ -209,7 +209,7 @@ 
         $clause = "_{$where["boolean"]}";
 
         while (count($queryParts) > 1) {
-            $clause .= "_" . array_shift($queryParts);
+            $clause .= "_".array_shift($queryParts);
             $clause .= $this->getCurrentBinding("where");
             $this->currentBinding++;
         }
@@ -217,10 +217,10 @@ 
         $lastPart = array_shift($queryParts);
 
         if ($lastPart) {
-            $clause .= "_" . $lastPart;
+            $clause .= "_".$lastPart;
         }
 
-        return "-" . str_replace(" ", "_", $clause);
+        return "-".str_replace(" ", "_", $clause);
     }
 
     protected function getTableSlug() : string
@@ -240,7 +240,7 @@ 
 
     protected function getValuesClause(array $where = []) : string
     {
-        if (! $where
+        if (!$where
             || in_array($where["type"], ["NotNull", "Null"])
         ) {
             return "";
@@ -249,7 +249,7 @@ 
         $values = $this->getValuesFromWhere($where);
         $values = $this->getValuesFromBindings($where, $values);
 
-        return "_" . $values;
+        return "_".$values;
     }
 
     protected function getValuesFromWhere(array $where) : string
@@ -274,7 +274,7 @@ 
 
     protected function getValuesFromBindings(array $where, string $values) : string
     {
-        $bindingFallback = __CLASS__ . ':UNKNOWN_BINDING';
+        $bindingFallback = __CLASS__.':UNKNOWN_BINDING';
         $currentBinding = $this->getCurrentBinding("where", $bindingFallback);
 
         if ($currentBinding !== $bindingFallback) {
@@ -282,7 +282,7 @@ 
             $this->currentBinding++;
 
             if ($where["type"] === "between") {
-                $values .= "_" . $this->getCurrentBinding("where");
+                $values .= "_".$this->getCurrentBinding("where");
                 $this->currentBinding++;
             }
         }
@@ -298,8 +298,8 @@ 
 
     protected function getWhereClauses(array $wheres = []) : string
     {
-        return "" . $this->getWheres($wheres)
-            ->reduce(function ($carry, $where) {
+        return "".$this->getWheres($wheres)
+            ->reduce(function($carry, $where) {
                 $value = $carry;
                 $value .= $this->getNestedClauses($where);
                 $value .= $this->getColumnClauses($where);
@@ -332,8 +332,8 @@ 
             return "";
         }
 
-        return $eagerLoads->keys()->reduce(function ($carry, $related) {
-            if (! method_exists($this->model, $related)) {
+        return $eagerLoads->keys()->reduce(function($carry, $related) {
+            if (!method_exists($this->model, $related)) {
                 return "{$carry}-{$related}";
             }
 
@@ -365,7 +365,7 @@ 
                 continue;
             }
 
-            $result .= $glue . $value;
+            $result .= $glue.$value;
         }
 
         return $result;

tests/Integration/CachedBuilder/ScopeTest.php

Spacing +10 added lines, -10 removed lines

@@ -111,7 +111,7 @@ 
         $this->actingAs($user);
         $authorsA = (new AuthorBeginsWithScoped)
             ->all()
-            ->map(function ($author) {
+            ->map(function($author) {
                 return (new Str)->substr($author->name, 0, 1);
             })
             ->unique();
@@ -119,7 +119,7 @@ 
         $this->actingAs($user);
         $authorsB = (new AuthorBeginsWithScoped)
             ->all()
-            ->map(function ($author) {
+            ->map(function($author) {
                 return (new Str)->substr($author->name, 0, 1);
             })
             ->unique();
@@ -137,7 +137,7 @@ 
         $this->actingAs($user);
         $authorsA = (new AuthorBeginsWithScoped)
             ->get()
-            ->map(function ($author) {
+            ->map(function($author) {
                 return (new Str)->substr($author->name, 0, 1);
             })
             ->unique();
@@ -145,7 +145,7 @@ 
         $this->actingAs($user);
         $authorsB = (new AuthorBeginsWithScoped)
             ->get()
-            ->map(function ($author) {
+            ->map(function($author) {
                 return (new Str)->substr($author->name, 0, 1);
             })
             ->unique();
@@ -164,7 +164,7 @@ 
         $authorsA = (new AuthorBeginsWithScoped)
             ->withoutGlobalScopes()
             ->get()
-            ->map(function ($author) {
+            ->map(function($author) {
                 return (new Str)->substr($author->name, 0, 1);
             })
             ->unique();
@@ -173,7 +173,7 @@ 
         $authorsB = (new AuthorBeginsWithScoped)
             ->withoutGlobalScopes(['GeneaLabs\LaravelModelCaching\Tests\Fixtures\Scopes\NameBeginsWith'])
             ->get()
-            ->map(function ($author) {
+            ->map(function($author) {
                 return (new Str)->substr($author->name, 0, 1);
             })
             ->unique();
@@ -190,7 +190,7 @@ 
         $authorsA = (new AuthorBeginsWithScoped)
             ->withoutGlobalScope('GeneaLabs\LaravelModelCaching\Tests\Fixtures\Scopes\NameBeginsWith')
             ->get()
-            ->map(function ($author) {
+            ->map(function($author) {
                 return (new Str)->substr($author->name, 0, 1);
             })
             ->unique();
@@ -199,7 +199,7 @@ 
         $authorsB = (new AuthorBeginsWithScoped)
             ->withoutGlobalScope('GeneaLabs\LaravelModelCaching\Tests\Fixtures\Scopes\NameBeginsWith')
             ->get()
-            ->map(function ($author) {
+            ->map(function($author) {
                 return (new Str)->substr($author->name, 0, 1);
             })
             ->unique();
@@ -213,13 +213,13 @@ 
         $first = "A";
         $second = "B";
         $authors1 = (new Author)
-            ->with(['books' => static function (HasMany $model) use ($first) {
+            ->with(['books' => static function(HasMany $model) use ($first) {
                 $model->startsWith($first);
             }])
             ->get();
         $authors2 = (new Author)
             ->disableModelCaching()
-            ->with(['books' => static function (HasMany $model) use ($second) {
+            ->with(['books' => static function(HasMany $model) use ($second) {
                 $model->startsWith($second);
             }])
             ->get();

src/Traits/Caching.php

Spacing +15 added lines, -15 removed lines

@@ -45,7 +45,7 @@ 
 
     protected function applyScopesToInstance()
     {
-        if (! property_exists($this, "scopes")
+        if (!property_exists($this, "scopes")
             || $this->scopesAreApplied
             || $this->withoutAllGlobalScopes
         ) {
@@ -53,11 +53,11 @@ 
         }
 
         foreach ($this->scopes as $identifier => $scope) {
-            if (! isset($this->scopes[$identifier]) || isset($this->withoutGlobalScopes[$identifier])) {
+            if (!isset($this->scopes[$identifier]) || isset($this->withoutGlobalScopes[$identifier])) {
                 continue;
             }
 
-            $this->callScope(function () use ($scope) {
+            $this->callScope(function() use ($scope) {
                 if ($scope instanceof Closure) {
                     $scope($this);
                 }
@@ -115,7 +115,7 @@ 
             $cacheKey = "{$cachePrefix}:{$modelClassName}-cooldown:saved-at";
 
             $this->cache()
-                ->rememberForever($cacheKey, function () {
+                ->rememberForever($cacheKey, function() {
                     return (new Carbon)->now();
                 });
         }
@@ -192,7 +192,7 @@ 
 
     public function getModelCacheCooldown(Model $instance) : array
     {
-        if (! $instance->cacheCooldownSeconds) {
+        if (!$instance->cacheCooldownSeconds) {
             return [null, null, null];
         }
 
@@ -201,7 +201,7 @@ 
         [$cacheCooldown, $invalidatedAt, $savedAt] = $this
             ->getCacheCooldownDetails($instance, $cachePrefix, $modelClassName);
 
-        if (! $cacheCooldown || $cacheCooldown === 0) {
+        if (!$cacheCooldown || $cacheCooldown === 0) {
             return [null, null, null];
         }
 
@@ -230,7 +230,7 @@ 
     {
         [$cacheCooldown, $invalidatedAt] = $this->getModelCacheCooldown($instance);
 
-        if (! $cacheCooldown
+        if (!$cacheCooldown
             || (new Carbon)->now()->diffInSeconds($invalidatedAt) < $cacheCooldown
         ) {
             return;
@@ -255,7 +255,7 @@ 
     {
         [$cacheCooldown, $invalidatedAt] = $instance->getModelCacheCooldown($instance);
 
-        if (! $cacheCooldown) {
+        if (!$cacheCooldown) {
             $instance->flushCache();
 
             if ($relationship) {
@@ -282,7 +282,7 @@ 
 
     public function isCachable() : bool
     {
-        $isCacheDisabled = ! Container::getInstance()
+        $isCacheDisabled = !Container::getInstance()
             ->make("config")
             ->get("laravel-model-caching.enabled");
         $allRelationshipsAreCachable = true;
@@ -293,8 +293,8 @@ 
             $allRelationshipsAreCachable = collect($this
                 ->eagerLoad)
                 ->keys()
-                ->reduce(function ($carry, $related) {
-                    if (! method_exists($this->model, $related)
+                ->reduce(function($carry, $related) {
+                    if (!method_exists($this->model, $related)
                         || $carry === false
                     ) {
                         return $carry;
@@ -302,8 +302,8 @@ 
         
                     $relatedModel = $this->model->$related()->getRelated();
 
-                    if (! method_exists($relatedModel, "isCachable")
-                        || ! $relatedModel->isCachable()
+                    if (!method_exists($relatedModel, "isCachable")
+                        || !$relatedModel->isCachable()
                     ) {
                         return false;
                     }
@@ -314,7 +314,7 @@ 
         }
 
         return $this->isCachable
-            && ! $isCacheDisabled
+            && !$isCacheDisabled
             && $allRelationshipsAreCachable;
     }
 
@@ -325,7 +325,7 @@ 
         $cacheKey = "{$cachePrefix}:{$modelClassName}-cooldown:saved-at";
 
         $instance->cache()
-            ->rememberForever($cacheKey, function () {
+            ->rememberForever($cacheKey, function() {
                 return (new Carbon)->now();
             });
     }