Passed
Pull Request — master (#303)
by Ben
35:24
created
src/Sets/SetReference.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
     {
40 40
         // Constraints
41 41
         if (!isset($values['action'])) {
42
-            throw new \InvalidArgumentException('Set reference array is missing required values for the "action" keys. Given: ' . print_r($values, true));
42
+            throw new \InvalidArgumentException('Set reference array is missing required values for the "action" keys. Given: '.print_r($values, true));
43 43
         }
44 44
 
45 45
         return new static(
@@ -54,14 +54,14 @@  discard block
 block discarded – undo
54 54
     {
55 55
         $sets = config('thinktomorrow.chief.sets', []);
56 56
 
57
-        return collect($sets)->map(function ($set, $key) {
57
+        return collect($sets)->map(function($set, $key) {
58 58
             return SetReference::fromArray($key, $set);
59 59
         });
60 60
     }
61 61
 
62 62
     public static function find($key): ?SetReference
63 63
     {
64
-        return static::all()->filter(function ($ref) use ($key) {
64
+        return static::all()->filter(function($ref) use ($key) {
65 65
             return $ref->key() == $key;
66 66
         })->first();
67 67
     }
@@ -78,13 +78,13 @@  discard block
 block discarded – undo
78 78
 
79 79
         $this->validateAction($class, $method);
80 80
 
81
-        $result = call_user_func_array([app($class),$method], $this->parameters($class, $method, $parent));
81
+        $result = call_user_func_array([app($class), $method], $this->parameters($class, $method, $parent));
82 82
 
83
-        if (! $result instanceof Set && $result instanceof Collection) {
83
+        if (!$result instanceof Set && $result instanceof Collection) {
84 84
             return new Set($result->all(), $this->key);
85 85
         }
86 86
 
87
-        if (! $result instanceof Set && $result instanceof Paginator) {
87
+        if (!$result instanceof Set && $result instanceof Paginator) {
88 88
             return new Set($result->all(), $this->key, [
89 89
                 'paginate' => [
90 90
                     'total' => $result->total(),
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
 
153 153
     private static function validateAction($class, $method)
154 154
     {
155
-        if (! class_exists($class)) {
155
+        if (!class_exists($class)) {
156 156
             throw new \InvalidArgumentException('The class ['.$class.'] isn\'t a valid class reference or does not exist in the chief-settings.sets config entry.');
157 157
         }
158 158
 
Please login to merge, or discard this patch.
src/Urls/UrlHelper.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
      */
40 40
     public static function onlineModels(bool $onlySingles = false, Model $ignoredModel = null): Collection
41 41
     {
42
-        $models = chiefMemoize('all-online-models', function () use ($onlySingles) {
42
+        $models = chiefMemoize('all-online-models', function() use ($onlySingles) {
43 43
             $builder = UrlRecord::whereNull('redirect_id')
44 44
                 ->select('model_type', 'model_id')
45 45
                 ->groupBy('model_type', 'model_id');
@@ -48,17 +48,17 @@  discard block
 block discarded – undo
48 48
                 $builder->where('model_type', 'singles');
49 49
             }
50 50
 
51
-            return $builder->get()->mapToGroups(function ($record) {
51
+            return $builder->get()->mapToGroups(function($record) {
52 52
                 return [$record->model_type => $record->model_id];
53
-            })->map(function ($record, $key) {
53
+            })->map(function($record, $key) {
54 54
                 return Morphables::instance($key)->find($record->toArray());
55
-            })->map->reject(function ($model) {
55
+            })->map->reject(function($model) {
56 56
                 return is_null($model) || !$model->isPublished(); // Invalid references to archived or removed models where url record still exists.
57 57
             })->flatten();
58 58
         }, [$onlySingles]);
59 59
 
60 60
         if ($ignoredModel) {
61
-            $models = $models->reject(function ($model) use ($ignoredModel) {
61
+            $models = $models->reject(function($model) use ($ignoredModel) {
62 62
                 return (get_class($model) === get_class($ignoredModel) && $model->id === $ignoredModel->id);
63 63
             });
64 64
         }
Please login to merge, or discard this patch.
src/Fields/Types/MediaField.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types = 1);
3
+declare(strict_types=1);
4 4
 
5 5
 namespace Thinktomorrow\Chief\Fields\Types;
6 6
 
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
             return $this;
41 41
         }
42 42
 
43
-        return 'files['. ($this->values['name'] ?? $this->key()).']';
43
+        return 'files['.($this->values['name'] ?? $this->key()).']';
44 44
     }
45 45
 
46 46
     public function sluggifyName()
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
         $images = [];
59 59
         $locale = $locale ?? app()->getLocale();
60 60
 
61
-        $assets = $model->assetRelation->where('pivot.type', $this->key())->filter(function ($asset) use ($locale) {
61
+        $assets = $model->assetRelation->where('pivot.type', $this->key())->filter(function($asset) use ($locale) {
62 62
             return $asset->pivot->locale == $locale;
63 63
         })->sortBy('pivot.order');
64 64
 
Please login to merge, or discard this patch.
src/Management/Application/DeleteManagedModel.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
             DB::beginTransaction();
22 22
 
23 23
             // For stateful transitions we will apply this deletion as a state
24
-            if($model instanceof StatefulContract) {
24
+            if ($model instanceof StatefulContract) {
25 25
                 (new PageState($model))->apply('delete');
26 26
                 $model->save();
27 27
             }
@@ -29,17 +29,17 @@  discard block
 block discarded – undo
29 29
             Relation::deleteRelationsOf($model->getMorphClass(), $model->id);
30 30
 
31 31
             // Mark the slug as deleted to avoid any conflict with newly created modules with the same slug.
32
-            if($model instanceof Module) {
32
+            if ($model instanceof Module) {
33 33
                 $model->update([
34
-                    'slug' => $model->slug . $this->appendDeleteMarker(),
34
+                    'slug' => $model->slug.$this->appendDeleteMarker(),
35 35
                 ]);
36 36
             }
37 37
 
38
-            if($model instanceof ProvidesUrl){
38
+            if ($model instanceof ProvidesUrl) {
39 39
                 UrlRecord::getByModel($model)->each->delete();
40 40
             }
41 41
 
42
-            if($model instanceof Page){
42
+            if ($model instanceof Page) {
43 43
                 Module::where('page_id', $model->id)->delete();
44 44
             }
45 45
 
@@ -58,6 +58,6 @@  discard block
 block discarded – undo
58 58
 
59 59
     private function appendDeleteMarker(): string
60 60
     {
61
-        return '_DELETED_' . time();
61
+        return '_DELETED_'.time();
62 62
     }
63 63
 }
Please login to merge, or discard this patch.
src/States/Archivable/Archivable.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
 
24 24
     public function scopeUnarchived($query)
25 25
     {
26
-        $query->withoutGlobalScope(ArchiveScope::class)->where('current_state','<>', PageState::ARCHIVED);
26
+        $query->withoutGlobalScope(ArchiveScope::class)->where('current_state', '<>', PageState::ARCHIVED);
27 27
     }
28 28
 
29 29
     public function scopeWithArchived($query)
Please login to merge, or discard this patch.
src/States/Archivable/ArchiveScope.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types = 1);
3
+declare(strict_types=1);
4 4
 
5 5
 namespace Thinktomorrow\Chief\States\Archivable;
6 6
 
@@ -13,6 +13,6 @@  discard block
 block discarded – undo
13 13
 {
14 14
     public function apply(Builder $builder, Model $model)
15 15
     {
16
-        $builder->where('current_state','<>', PageState::ARCHIVED);
16
+        $builder->where('current_state', '<>', PageState::ARCHIVED);
17 17
     }
18 18
 }
Please login to merge, or discard this patch.