GeneaLabs / laravel-model-caching

src/Traits/Buildable.php

Spacing +15 added lines, -15 removed lines

@@ -9,7 +9,7 @@ 
 {
     public function avg($column)
     {
-        if (! $this->isCachable()) {
+        if (!$this->isCachable()) {
             return parent::avg($column);
         }
 
@@ -20,7 +20,7 @@ 
 
     public function count($columns = "*")
     {
-        if (! $this->isCachable()) {
+        if (!$this->isCachable()) {
             return parent::count($columns);
         }
 
@@ -31,7 +31,7 @@ 
 
     public function exists()
     {
-        if (! $this->isCachable()) {
+        if (!$this->isCachable()) {
             return parent::exists();
         }
 
@@ -61,7 +61,7 @@ 
      */
     public function find($id, $columns = ["*"])
     {
-        if (! $this->isCachable()) {
+        if (!$this->isCachable()) {
             return parent::find($id, $columns);
         }
 
@@ -78,7 +78,7 @@ 
 
     public function first($columns = ["*"])
     {
-        if (! $this->isCachable()) {
+        if (!$this->isCachable()) {
             return parent::first($columns);
         }
 
@@ -98,7 +98,7 @@ 
 
     public function get($columns = ["*"])
     {
-        if (! $this->isCachable()) {
+        if (!$this->isCachable()) {
             return parent::get($columns);
         }
 
@@ -134,7 +134,7 @@ 
 
     public function max($column)
     {
-        if (! $this->isCachable()) {
+        if (!$this->isCachable()) {
             return parent::max($column);
         }
 
@@ -145,7 +145,7 @@ 
 
     public function min($column)
     {
-        if (! $this->isCachable()) {
+        if (!$this->isCachable()) {
             return parent::min($column);
         }
 
@@ -160,7 +160,7 @@ 
         $pageName = "page",
         $page = null
     ) {
-        if (! $this->isCachable()) {
+        if (!$this->isCachable()) {
             return parent::paginate($perPage, $columns, $pageName, $page);
         }
 
@@ -184,7 +184,7 @@ 
         $result = "";
 
         foreach ($items as $key => $value) {
-            $result .= $glue . $key . $glue . $value;
+            $result .= $glue.$key.$glue.$value;
         }
 
         return $result;
@@ -192,11 +192,11 @@ 
 
     public function pluck($column, $key = null)
     {
-        if (! $this->isCachable()) {
+        if (!$this->isCachable()) {
             return parent::pluck($column, $key);
         }
 
-        $keyDifferentiator = "-pluck_{$column}" . ($key ? "_{$key}" : "");
+        $keyDifferentiator = "-pluck_{$column}".($key ? "_{$key}" : "");
         $cacheKey = $this->makeCacheKey([$column], null, $keyDifferentiator);
 
         return $this->cachedValue(func_get_args(), $cacheKey);
@@ -204,7 +204,7 @@ 
 
     public function sum($column)
     {
-        if (! $this->isCachable()) {
+        if (!$this->isCachable()) {
             return parent::sum($column);
         }
 
@@ -224,7 +224,7 @@ 
 
     public function value($column)
     {
-        if (! $this->isCachable()) {
+        if (!$this->isCachable()) {
             return parent::value($column);
         }
 
@@ -299,7 +299,7 @@ 
         return $this->cache($cacheTags)
             ->rememberForever(
                 $hashedCacheKey,
-                function () use ($arguments, $cacheKey, $method) {
+                function() use ($arguments, $cacheKey, $method) {
                     return [
                         "key" => $cacheKey,
                         "value" => parent::{$method}(...$arguments),

tests/Integration/CachedBuilder/BooleanTest.php

Spacing +2 added lines, -2 removed lines

@@ -67,7 +67,7 @@ 
         factory(Book::class)->create(['author_id' => $expectedAuthor->getKey(), 'title' => 'Mixed Clause']);
 
         $books = (new Book)
-            ->whereHas('author', function ($query) {
+            ->whereHas('author', function($query) {
                 return $query->where('is_famous', false);
             })
             ->whereRaw("title = ?", ['Mixed Clause']) // Test ensures this binding is included in the key
@@ -76,7 +76,7 @@ 
             ->tags($tags)
             ->get($key)['value'];
         $liveResults = (new UncachedBook)
-            ->whereHas('author', function ($query) {
+            ->whereHas('author', function($query) {
                 return $query->where('is_famous', false);
             })
             ->whereRaw("title = ?", ['Mixed Clause'])

src/CacheKey.php

Spacing +31 added lines, -31 removed lines

@@ -58,8 +58,8 @@ 
 
     protected function getLimitClause() : string
     {
-        if (! property_exists($this->query, "limit")
-            || ! $this->query->limit
+        if (!property_exists($this->query, "limit")
+            || !$this->query->limit
         ) {
             return "";
         }
@@ -80,8 +80,8 @@ 
 
     protected function getOffsetClause() : string
     {
-        if (! property_exists($this->query, "offset")
-            || ! $this->query->offset
+        if (!property_exists($this->query, "offset")
+            || !$this->query->offset
         ) {
             return "";
         }
@@ -91,8 +91,8 @@ 
 
     protected function getOrderByClauses() : string
     {
-        if (! property_exists($this->query, "orders")
-            || ! $this->query->orders
+        if (!property_exists($this->query, "orders")
+            || !$this->query->orders
         ) {
             return "";
         }
@@ -100,12 +100,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"];
             })
             ?: "";
     }
@@ -114,8 +114,8 @@ 
     {
         if (($columns === ["*"]
                 || $columns === [])
-            && (! property_exists($this->query, "columns")
-                || ! $this->query->columns)
+            && (!property_exists($this->query, "columns")
+                || !$this->query->columns)
         ) {
             return "";
         }
@@ -123,10 +123,10 @@ 
         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 getTypeClause($where) : string
@@ -140,7 +140,7 @@ 
 
     protected function getValuesClause(array $where = []) : string
     {
-        if (! $where
+        if (!$where
             || in_array($where["type"], ["NotNull", "Null"])
         ) {
             return "";
@@ -149,7 +149,7 @@ 
         $values = $this->getValuesFromWhere($where);
         $values = $this->getValuesFromBindings($where, $values);
 
-        return "_" . $values;
+        return "_".$values;
     }
 
     protected function getValuesFromWhere(array $where) : string
@@ -175,7 +175,7 @@ 
     protected function getValuesFromBindings(array $where, string $values) : string
     {
         // Fallback to this when the current binding does not exist in the bindings array
-        $bindingFallback = __CLASS__ . ':UNKNOWN_BINDING';
+        $bindingFallback = __CLASS__.':UNKNOWN_BINDING';
         $currentBinding = $this->getCurrentBinding('where') ?? $bindingFallback;
 
         if ($currentBinding !== $bindingFallback) {
@@ -183,7 +183,7 @@ 
             $this->currentBinding++;
 
             if ($where["type"] === "between") {
-                $values .= "_" . $this->getCurrentBinding('where');
+                $values .= "_".$this->getCurrentBinding('where');
                 $this->currentBinding++;
             }
         }
@@ -199,8 +199,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);
@@ -214,11 +214,11 @@ 
 
     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 getColumnClauses(array $where) : string
@@ -232,7 +232,7 @@ 
 
     protected function getInAndNotInClauses(array $where) : string
     {
-        if (! in_array($where["type"], ["In", "NotIn", "InRaw"])) {
+        if (!in_array($where["type"], ["In", "NotIn", "InRaw"])) {
             return "";
         }
 
@@ -244,7 +244,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], "_");
@@ -282,7 +282,7 @@ 
                 continue;
             }
 
-            $result .= $glue . $value;
+            $result .= $glue.$value;
         }
 
         return $result;
@@ -290,7 +290,7 @@ 
 
     protected function getRawClauses(array $where) : string
     {
-        if (! in_array($where["type"], ["raw"])) {
+        if (!in_array($where["type"], ["raw"])) {
             return "";
         }
 
@@ -298,7 +298,7 @@ 
         $clause = "_{$where["boolean"]}";
 
         while (count($queryParts) > 1) {
-            $clause .= "_" . array_shift($queryParts);
+            $clause .= "_".array_shift($queryParts);
             $clause .= $this->getCurrentBinding('where');
             $this->currentBinding++;
         }
@@ -306,10 +306,10 @@ 
         $lastPart = array_shift($queryParts);
 
         if ($lastPart) {
-            $clause .= "_" . $lastPart;
+            $clause .= "_".$lastPart;
         }
 
-        return "-" . str_replace(" ", "_", $clause);
+        return "-".str_replace(" ", "_", $clause);
     }
 
     protected function getOtherClauses(array $where) : string
@@ -345,8 +345,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}";
             }
 
@@ -360,7 +360,7 @@ 
 
     protected function getBindingsSlug() : string
     {
-        if (! method_exists($this->model, 'query')) {
+        if (!method_exists($this->model, 'query')) {
             return '';
         }