Test Failed
Pull Request — master (#409)
by Philippe
14:51 queued 06:28
created
src/Management/Managers.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
     {
51 51
         $registrations = collect($this->register->filterByTag($tag)->all());
52 52
 
53
-        return $registrations->map(function ($registration) {
53
+        return $registrations->map(function($registration) {
54 54
             return $this->instance($registration);
55 55
         });
56 56
     }
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
     {
66 66
         $registrations = collect($this->register->filterByTag($tag)->all());
67 67
 
68
-        return $registrations->map(function ($registration) {
68
+        return $registrations->map(function($registration) {
69 69
             return $this->instance($registration)->details();
70 70
         });
71 71
     }
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
         //return array with group name and values
77 77
         $grouped = [];
78 78
 
79
-        $managers = $managers->map(function (Manager $item) {
79
+        $managers = $managers->map(function(Manager $item) {
80 80
             return [
81 81
                 'id'    => $item->details()->id,
82 82
                 'group' => $item->details()->plural,
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
     {
91 91
         $registrations = collect($this->register->all());
92 92
 
93
-        return $registrations->map(function ($registration) {
93
+        return $registrations->map(function($registration) {
94 94
             return $this->instance($registration);
95 95
         });
96 96
     }
Please login to merge, or discard this patch.
src/Management/ManagesPagebuilder.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
         
49 49
         $availableChildren = AvailableChildren::forParent($model);
50 50
         
51
-        $modules = $availableChildren->onlyModules()->reject(function ($module) use ($model) {
51
+        $modules = $availableChildren->onlyModules()->reject(function($module) use ($model) {
52 52
             return $module->page_id != null && $module->page_id != $model->id;
53 53
         });
54 54
 
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
         $available_sets = FlatReferencePresenter::toGroupedSelectValues($availableChildren->onlySets())->toArray();
58 58
 
59 59
         // Current sections
60
-        $sections = $model->children()->map(function ($section, $index) {
60
+        $sections = $model->children()->map(function($section, $index) {
61 61
             if ($section instanceof TranslatableContract) {
62 62
                 $section->injectTranslationForForm();
63 63
             }
Please login to merge, or discard this patch.
src/Management/Assistants/ManagesAssistants.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -51,7 +51,7 @@
 block discarded – undo
51 51
     public function assistant(string $assistantKey): Assistant
52 52
     {
53 53
         if (!$this->isAssistedBy($assistantKey)) {
54
-            throw new MissingAssistant('No assistant [' . $assistantKey . '] registered on manager ' . static::class);
54
+            throw new MissingAssistant('No assistant ['.$assistantKey.'] registered on manager '.static::class);
55 55
         }
56 56
 
57 57
         $instance = app($this->getAssistantClass($assistantKey));
Please login to merge, or discard this patch.
src/Management/Application/DeleteManagedModel.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
             // Mark the slug as deleted to avoid any conflict with newly created modules with the same slug.
32 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
 
@@ -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/Fields/Types/ImageField.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@
 block discarded – undo
25 25
         $images = [];
26 26
         $locale = $locale ?? app()->getLocale();
27 27
 
28
-        $assets = $model->assetRelation->where('pivot.type', $this->getKey())->filter(function ($asset) use ($locale) {
28
+        $assets = $model->assetRelation->where('pivot.type', $this->getKey())->filter(function($asset) use ($locale) {
29 29
             return $asset->pivot->locale == $locale;
30 30
         })->sortBy('pivot.order');
31 31
 
Please login to merge, or discard this patch.
src/Modules/ModuleManager.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
          * Therefore the index of these modules is at the modules tab of this page model.
34 34
          */
35 35
         if ($verb == 'index' && $this->model->isPageSpecific()) {
36
-            return app(Managers::class)->findByModel($this->model->page)->route('edit') . '#eigen-modules';
36
+            return app(Managers::class)->findByModel($this->model->page)->route('edit').'#eigen-modules';
37 37
         }
38 38
 
39 39
         $routes = [
@@ -153,7 +153,7 @@  discard block
 block discarded – undo
153 153
             if (is_array_empty($translation)) {
154 154
 
155 155
                 // Nullify all values
156
-                $trans[$locale] = array_map(function ($value) {
156
+                $trans[$locale] = array_map(function($value) {
157 157
                     return null;
158 158
                 }, $translation);
159 159
                 continue;
Please login to merge, or discard this patch.
src/Modules/Module.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
             return static::$managedModelKey;
70 70
         }
71 71
 
72
-        throw new \Exception('Missing required static property \'managedModelKey\' on ' . static::class . '.');
72
+        throw new \Exception('Missing required static property \'managedModelKey\' on '.static::class.'.');
73 73
     }
74 74
 
75 75
     /**
@@ -79,9 +79,9 @@  discard block
 block discarded – undo
79 79
      */
80 80
     public static function availableForCreation(): Collection
81 81
     {
82
-        $managers = app(Managers::class)->findByTag('module')->filter(function ($manager) {
82
+        $managers = app(Managers::class)->findByTag('module')->filter(function($manager) {
83 83
             return $manager->can('create');
84
-        })->map(function ($manager) {
84
+        })->map(function($manager) {
85 85
             return $manager->details();
86 86
         });
87 87
 
Please login to merge, or discard this patch.
src/Management/Assistants/UrlAssistant.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -69,9 +69,9 @@  discard block
 block discarded – undo
69 69
 
70 70
         // Push update to homepage setting value
71 71
         // TODO: we should just fetch the homepages and push that instead...
72
-        UrlRecord::getByModel($this->manager->existingModel())->reject(function ($record) {
72
+        UrlRecord::getByModel($this->manager->existingModel())->reject(function($record) {
73 73
             return ($record->isRedirect() || !$record->isHomepage());
74
-        })->each(function ($record) {
74
+        })->each(function($record) {
75 75
             app(ChangeHomepage::class)->onUrlChanged($record);
76 76
         });
77 77
     }
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
     private function validateModel()
85 85
     {
86 86
         if (!$this->manager->existingModel() instanceof ProvidesUrl) {
87
-            throw new \Exception('UrlAssistant requires the model interfaced by ' . ProvidesUrl::class . '.');
87
+            throw new \Exception('UrlAssistant requires the model interfaced by '.ProvidesUrl::class.'.');
88 88
         }
89 89
     }
90 90
 }
Please login to merge, or discard this patch.
src/Management/Assistants/ArchiveAssistant.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
 
50 50
     public function findAllArchived(): Collection
51 51
     {
52
-        return $this->manager->modelInstance()::archived()->get()->map(function ($model) {
52
+        return $this->manager->modelInstance()::archived()->get()->map(function($model) {
53 53
             return $this->managers->findByModel($model);
54 54
         });
55 55
     }
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
     private function guard($verb): Assistant
91 91
     {
92 92
         if (!$this->manager->existingModel() instanceof StatefulContract) {
93
-            throw new \InvalidArgumentException('ArchiveAssistant requires the model to implement the StatefulContract. [' . get_class($this->manager->existingModel()) . '] given instead.');
93
+            throw new \InvalidArgumentException('ArchiveAssistant requires the model to implement the StatefulContract. ['.get_class($this->manager->existingModel()).'] given instead.');
94 94
         }
95 95
 
96 96
         if (!$this->can($verb)) {
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 
121 121
             // Ok now get all urls from this model and point them to the new records
122 122
             foreach (UrlRecord::getByModel($this->manager->existingModel()) as $urlRecord) {
123
-                if ($targetRecord = $targetRecords->first(function ($record) use ($urlRecord) {
123
+                if ($targetRecord = $targetRecords->first(function($record) use ($urlRecord) {
124 124
                     return ($record->locale == $urlRecord->locale && !$record->isRedirect());
125 125
                 })) {
126 126
                     $urlRecord->redirectTo($targetRecord);
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
             ->performedOn($this->manager->existingModel())
138 138
             ->log('archived');
139 139
 
140
-        return redirect()->to($this->manager->route('index'))->with('messages.success', $this->manager->details()->title . ' is gearchiveerd.');
140
+        return redirect()->to($this->manager->route('index'))->with('messages.success', $this->manager->details()->title.' is gearchiveerd.');
141 141
     }
142 142
 
143 143
     public function unarchive()
@@ -153,6 +153,6 @@  discard block
 block discarded – undo
153 153
             ->performedOn($this->manager->existingModel())
154 154
             ->log('unarchived');
155 155
 
156
-        return redirect()->to($this->manager->route('index'))->with('messages.success', $this->manager->details()->title . ' is hersteld.');
156
+        return redirect()->to($this->manager->route('index'))->with('messages.success', $this->manager->details()->title.' is hersteld.');
157 157
     }
158 158
 }
Please login to merge, or discard this patch.