GeneaLabs / laravel-model-caching

src/Traits/Caching.php

Spacing +15 added lines, -15 removed lines

@@ -34,18 +34,18 @@ 
 
     protected function applyScopesToInstance()
     {
-        if (! property_exists($this, "scopes")
+        if (!property_exists($this, "scopes")
             || $this->scopesAreApplied
         ) {
             return;
         }
 
         foreach ($this->scopes as $identifier => $scope) {
-            if (! isset($this->scopes[$identifier])) {
+            if (!isset($this->scopes[$identifier])) {
                 continue;
             }
 
-            $this->callScope(function () use ($scope) {
+            $this->callScope(function() use ($scope) {
                 if ($scope instanceof Closure) {
                     $scope($this);
                 }
@@ -103,7 +103,7 @@ 
             $cacheKey = "{$cachePrefix}:{$modelClassName}-cooldown:saved-at";
 
             $this->cache()
-                ->rememberForever($cacheKey, function () {
+                ->rememberForever($cacheKey, function() {
                     return (new Carbon)->now();
                 });
         }
@@ -180,7 +180,7 @@ 
 
     public function getModelCacheCooldown(Model $instance) : array
     {
-        if (! $instance->cacheCooldownSeconds) {
+        if (!$instance->cacheCooldownSeconds) {
             return [null, null, null];
         }
 
@@ -189,7 +189,7 @@ 
         [$cacheCooldown, $invalidatedAt, $savedAt] = $this
             ->getCacheCooldownDetails($instance, $cachePrefix, $modelClassName);
 
-        if (! $cacheCooldown || $cacheCooldown === 0) {
+        if (!$cacheCooldown || $cacheCooldown === 0) {
             return [null, null, null];
         }
 
@@ -218,7 +218,7 @@ 
     {
         [$cacheCooldown, $invalidatedAt] = $this->getModelCacheCooldown($instance);
 
-        if (! $cacheCooldown
+        if (!$cacheCooldown
             || (new Carbon)->now()->diffInSeconds($invalidatedAt) < $cacheCooldown
         ) {
             return;
@@ -243,7 +243,7 @@ 
     {
         [$cacheCooldown, $invalidatedAt] = $instance->getModelCacheCooldown($instance);
 
-        if (! $cacheCooldown) {
+        if (!$cacheCooldown) {
             $instance->flushCache();
 
             if ($relationship) {
@@ -270,7 +270,7 @@ 
 
     public function isCachable() : bool
     {
-        $isCacheDisabled = ! Container::getInstance()
+        $isCacheDisabled = !Container::getInstance()
             ->make("config")
             ->get("laravel-model-caching.enabled");
         $allRelationshipsAreCachable = true;
@@ -281,8 +281,8 @@ 
             $allRelationshipsAreCachable = collect($this
                 ->eagerLoad)
                 ->keys()
-                ->reduce(function ($carry, $related) {
-                    if (! method_exists($this, $related)
+                ->reduce(function($carry, $related) {
+                    if (!method_exists($this, $related)
                         || $carry === false
                     ) {
                         return $carry;
@@ -290,8 +290,8 @@ 
         
                     $relatedModel = $this->model->$related()->getRelated();
 
-                    if (! method_exists($relatedModel, "isCachable")
-                        || ! $relatedModel->isCachable()
+                    if (!method_exists($relatedModel, "isCachable")
+                        || !$relatedModel->isCachable()
                     ) {
                         return false;
                     }
@@ -302,7 +302,7 @@ 
         }
 
         return $this->isCachable
-            && ! $isCacheDisabled
+            && !$isCacheDisabled
             && $allRelationshipsAreCachable;
     }
 
@@ -313,7 +313,7 @@ 
         $cacheKey = "{$cachePrefix}:{$modelClassName}-cooldown:saved-at";
 
         $instance->cache()
-            ->rememberForever($cacheKey, function () {
+            ->rememberForever($cacheKey, function() {
                 return (new Carbon)->now();
             });
     }

src/CacheKey.php

Spacing +30 added lines, -30 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
@@ -179,7 +179,7 @@ 
             $this->currentBinding++;
 
             if ($where["type"] === "between") {
-                $values .= "_" . $this->query->bindings["where"][$this->currentBinding];
+                $values .= "_".$this->query->bindings["where"][$this->currentBinding];
                 $this->currentBinding++;
             }
         }
@@ -195,8 +195,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);
@@ -210,11 +210,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
@@ -228,7 +228,7 @@ 
 
     protected function getInAndNotInClauses(array $where) : string
     {
-        if (! in_array($where["type"], ["In", "NotIn", "InRaw"])) {
+        if (!in_array($where["type"], ["In", "NotIn", "InRaw"])) {
             return "";
         }
 
@@ -240,7 +240,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], "_");
@@ -278,7 +278,7 @@ 
                 continue;
             }
 
-            $result .= $glue . $value;
+            $result .= $glue.$value;
         }
 
         return $result;
@@ -286,7 +286,7 @@ 
 
     protected function getRawClauses(array $where) : string
     {
-        if (! in_array($where["type"], ["raw"])) {
+        if (!in_array($where["type"], ["raw"])) {
             return "";
         }
 
@@ -294,7 +294,7 @@ 
         $clause = "_{$where["boolean"]}";
 
         while (count($queryParts) > 1) {
-            $clause .= "_" . array_shift($queryParts);
+            $clause .= "_".array_shift($queryParts);
             $clause .= $this->query->bindings["where"][$this->currentBinding];
             $this->currentBinding++;
         }
@@ -302,10 +302,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
@@ -341,8 +341,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}";
             }
 
@@ -356,7 +356,7 @@ 
 
     protected function getBindingsSlug() : string
     {
-        if (! method_exists($this->model, 'query')) {
+        if (!method_exists($this->model, 'query')) {
             return '';
         }
         

src/Providers/Service.php

Spacing +2 added lines, -2 removed lines

@@ -14,7 +14,7 @@ 
 
     public function boot()
     {
-        $configPath = __DIR__ . '/../../config/laravel-model-caching.php';
+        $configPath = __DIR__.'/../../config/laravel-model-caching.php';
         $this->mergeConfigFrom($configPath, 'laravel-model-caching');
         $this->commands([
             Clear::class,
@@ -27,7 +27,7 @@ 
 
     public function register()
     {
-        if (! class_exists('GeneaLabs\LaravelModelCaching\EloquentBuilder')) {
+        if (!class_exists('GeneaLabs\LaravelModelCaching\EloquentBuilder')) {
             class_alias(
                 ModelCaching::builder(),
                 'GeneaLabs\LaravelModelCaching\EloquentBuilder'

tests/Nova/BelongsToManyTest.php

Spacing +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
         $beforeBook->title = 'new foo';
         $beforeBook->save();
 
-        $this->postJson('nova-api/books/' . $beforeBook->id . '/attach/stores' , [
+        $this->postJson('nova-api/books/'.$beforeBook->id.'/attach/stores', [
             'stores' => $beforeStore->id,
             'viaRelationship' => 'stores'
         ]);
@@ -47,7 +47,7 @@ 
         $beforeBooks = $beforeStore->books;
         $beforeBook = $beforeBooks->first();
 
-        $this->deleteJson('/nova-api/stores/detach?viaResource=books&viaResourceId=' . $beforeBook->id  . '&viaRelationship=stores' , [
+        $this->deleteJson('/nova-api/stores/detach?viaResource=books&viaResourceId='.$beforeBook->id.'&viaRelationship=stores', [
             'resources' => [$beforeStore->id],
         ]);
 
@@ -66,7 +66,7 @@ 
         $beforeStore = Store::with(['books'])->get()->first();
         $beforeBook = $beforeStore->books->first();
 
-        $this->putJson('nova-api/books/' . $beforeBook->id, [
+        $this->putJson('nova-api/books/'.$beforeBook->id, [
             'title' => 'foo',
         ]);
 

tests/Nova/UpdateTest.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
         $beforeAuthors = (new Author)->get();
         $author = $beforeAuthors->first();
 
-        $this->putJson('nova-api/authors/' . $author->id, [
+        $this->putJson('nova-api/authors/'.$author->id, [
             'name' => 'foo',
             'email' => 'test1@noemail.com',
         ]);

tests/CreatesApplication.php

Spacing +10 added lines, -10 removed lines

@@ -31,20 +31,20 @@ 
 
         $this->setUpBaseLineSqlLiteDatabase();
 
-        $databasePath = __DIR__ . "/database";
+        $databasePath = __DIR__."/database";
         $this->testingSqlitePath = "{$databasePath}/";
         $baselinePath = "{$databasePath}/baseline.sqlite";
         $testingPath = "{$databasePath}/testing.sqlite";
 
-        ! file_exists($testingPath)
+        !file_exists($testingPath)
             ?: unlink($testingPath);
         copy($baselinePath, $testingPath);
 
-        require(__DIR__ . '/routes/web.php');
+        require(__DIR__.'/routes/web.php');
 
-        $this->withFactories(__DIR__ . '/database/factories');
+        $this->withFactories(__DIR__.'/database/factories');
 
-        view()->addLocation(__DIR__ . '/resources/views', 'laravel-model-caching');
+        view()->addLocation(__DIR__.'/resources/views', 'laravel-model-caching');
 
         $this->cache = app('cache')
             ->store(config('laravel-model-caching.store'));
@@ -69,7 +69,7 @@ 
 
         self::$baseLineDatabaseMigrated = true;
 
-        $file = __DIR__ . '/database/baseline.sqlite';
+        $file = __DIR__.'/database/baseline.sqlite';
         $this->app['config']->set('database.default', 'baseline');
         $this->app['config']->set('database.connections.baseline', [
             'driver' => 'sqlite',
@@ -79,12 +79,12 @@ 
             "foreign_key_constraints" => false,
         ]);
 
-        ! file_exists($file)
+        !file_exists($file)
             ?: unlink($file);
         touch($file);
 
-        $this->withFactories(__DIR__ . '/database/factories');
-        $this->loadMigrationsFrom(__DIR__ . '/database/migrations');
+        $this->withFactories(__DIR__.'/database/factories');
+        $this->loadMigrationsFrom(__DIR__.'/database/migrations');
 
         Artisan::call('db:seed', [
             '--class' => 'DatabaseSeeder',
@@ -99,7 +99,7 @@ 
         $app['config']->set('database.default', 'testing');
         $app['config']->set('database.connections.testing', [
             'driver' => 'sqlite',
-            'database' => __DIR__ . '/database/testing.sqlite',
+            'database' => __DIR__.'/database/testing.sqlite',
             'prefix' => '',
             "foreign_key_constraints" => false,
         ]);

tests/database/seeds/DatabaseSeeder.php

Spacing +5 added lines, -5 removed lines

@@ -42,18 +42,18 @@ 
                 "commentable_id" => $post->id,
                 "commentable_type" => Post::class,
             ])
-            ->each(function ($comment) {
+            ->each(function($comment) {
                 (new Comment)->create([
                     "commentable_id" => $comment->commentable_id,
                     "commentable_type" => UncachedPost::class,
                     "description" => $comment->description,
-                    "subject" => $comment->subject . ' - uncached post',
+                    "subject" => $comment->subject.' - uncached post',
                 ]);
             });
         $publishers = factory(Publisher::class, 10)->create();
         (new Author)->observe(AuthorObserver::class);
         factory(Author::class, 10)->create()
-            ->each(function ($author) use ($publishers) {
+            ->each(function($author) use ($publishers) {
                 $profile = factory(Profile::class)
                     ->make();
                 $profile->author_id = $author->id;
@@ -63,7 +63,7 @@ 
                         "author_id" => $author->id,
                         "publisher_id" => $publishers[rand(0, 9)]->id,
                     ])
-                    ->each(function ($book) use ($author, $publishers) {
+                    ->each(function($book) use ($author, $publishers) {
                         factory(Printer::class)->create([
                             "book_id" => $book->id,
                         ]);
@@ -75,7 +75,7 @@ 
 
         $bookIds = (new Book)->all()->pluck('id');
         factory(Store::class, 10)->create()
-            ->each(function ($store) use ($bookIds) {
+            ->each(function($store) use ($bookIds) {
                 $store->books()->sync(rand($bookIds->min(), $bookIds->max()));
             });
     }

tests/database/factories/UserFactory.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 use GeneaLabs\LaravelModelCaching\Tests\Fixtures\Supplier;
 use GeneaLabs\LaravelModelCaching\Tests\Fixtures\User;
 
-$factory->define(User::class, function (Faker $faker) {
+$factory->define(User::class, function(Faker $faker) {
     return [
         'name' => $faker->name,
         "supplier_id" => factory(Supplier::class)->create()->id,

tests/database/factories/HistoryFactory.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 use GeneaLabs\LaravelModelCaching\Tests\Fixtures\History;
 use GeneaLabs\LaravelModelCaching\Tests\Fixtures\User;
 
-$factory->define(History::class, function (Faker $faker) {
+$factory->define(History::class, function(Faker $faker) {
     return [
         'name' => $faker->name,
         "user_id" => factory(User::class)->create()->id,