Test Setup Failed
Push — a-simpler-manager ( 9beba5...d99f82 )
by Ben
06:26
created
src/Managers/Assistants/CrudAssistant.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -100,14 +100,14 @@  discard block
 block discarded – undo
100 100
         // Default sorted by publish date
101 101
         $modelClass = $this->managedModelClass();
102 102
         if (Schema::hasColumn((new $modelClass)->getTable(), 'published_at')) {
103
-            $filters = $filters->add(HiddenFilter::make('publish', function ($query) {
103
+            $filters = $filters->add(HiddenFilter::make('publish', function($query) {
104 104
                 return $query->orderBy('published_at', 'DESC');
105 105
             }));
106 106
         }
107 107
 
108 108
         // if model has no timestamps, updated_at doesn't exist
109 109
         if ((new $modelClass)->timestamps) {
110
-            $filters = $filters->add(HiddenFilter::make('updated', function ($query) {
110
+            $filters = $filters->add(HiddenFilter::make('updated', function($query) {
111 111
                 return $query->orderBy('updated_at', 'DESC');
112 112
             }));
113 113
         }
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
         $model->saveFields($model->fields()->tagged('create'), $request->all(), $request->allFiles());
142 142
 
143 143
         return redirect()->to($this->route('edit', $model))
144
-            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "' . $model->adminLabel('title') . '" is toegevoegd');
144
+            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "'.$model->adminLabel('title').'" is toegevoegd');
145 145
     }
146 146
 
147 147
     public function edit(Request $request, $id)
@@ -172,7 +172,7 @@  discard block
 block discarded – undo
172 172
         $model->saveFields($model->fields(), $request->all(), $request->allFiles());
173 173
 
174 174
         return redirect()->to($this->route('index'))
175
-            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  <a href="' . $this->route('edit', $model) . '">' . $model->adminLabel('title') . '</a> is aangepast');
175
+            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  <a href="'.$this->route('edit', $model).'">'.$model->adminLabel('title').'</a> is aangepast');
176 176
     }
177 177
 
178 178
     public function delete(Request $request, $id)
@@ -182,12 +182,12 @@  discard block
 block discarded – undo
182 182
         $model = $this->managedModelClass()::findOrFail($id);
183 183
 
184 184
         if ($request->get('deleteconfirmation') !== 'DELETE') {
185
-            return redirect()->back()->with('messages.warning', $model->adminLabel('title') . ' is niet verwijderd.');
185
+            return redirect()->back()->with('messages.warning', $model->adminLabel('title').' is niet verwijderd.');
186 186
         }
187 187
 
188 188
         app(DeleteManagedModel::class)->handle($model);
189 189
 
190 190
         return redirect()->to($this->route('index'))
191
-            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "' . $model->adminLabel('title') . '" is verwijderd.');
191
+            ->with('messages.success', '<i class="fa fa-fw fa-check-circle"></i>  "'.$model->adminLabel('title').'" is verwijderd.');
192 192
     }
193 193
 }
Please login to merge, or discard this patch.
src/Legacy/Fragments/Fragment.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
 
35 35
     public static function fromModel(FragmentModel $fragmentModel): self
36 36
     {
37
-        return new static($fragmentModel->key, $fragmentModel->values->all(), new Fields(), (int) $fragmentModel->id);
37
+        return new static($fragmentModel->key, $fragmentModel->values->all(), new Fields(), (int)$fragmentModel->id);
38 38
     }
39 39
 
40 40
     public static function fromNew(string $key, array $values): self
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
         $modelId = $payload['modelId'];
57 57
         unset($payload['modelId']);
58 58
 
59
-        return new static($key, $payload, new Fields(), (int) $modelId);
59
+        return new static($key, $payload, new Fields(), (int)$modelId);
60 60
     }
61 61
 
62 62
     public function setFields(Fields $fields): self
Please login to merge, or discard this patch.
src/Legacy/Fragments/ManagesFragments.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -13,15 +13,15 @@
 block discarded – undo
13 13
     public function saveFragmentFields(FragmentField $fragmentField, Request $request)
14 14
     {
15 15
         $payload = $request->input($fragmentField->getDottedName(), []);
16
-        $imagePayload = $request->input('images.' . $fragmentField->getDottedName(), []);
16
+        $imagePayload = $request->input('images.'.$fragmentField->getDottedName(), []);
17 17
 
18 18
         // Compose Fragment instances for each payload entry
19
-        $fragments = array_map(function ($fragmentPayload) use ($fragmentField) {
19
+        $fragments = array_map(function($fragmentPayload) use ($fragmentField) {
20 20
             return Fragment::fromRequestPayload($fragmentField->getKey(), $fragmentPayload); // (new Fragment($field->getKey(), $fragmentPayload));
21 21
         }, $payload);
22 22
 
23 23
         // remove all dead fragments
24
-        $this->existingModel()->removeAllFragments($fragmentField->getKey(), array_map(function (Fragment $fragment) {
24
+        $this->existingModel()->removeAllFragments($fragmentField->getKey(), array_map(function(Fragment $fragment) {
25 25
             return $fragment->hasModelId() ? $fragment->getModelId() : null;
26 26
         }, $fragments));
27 27
 
Please login to merge, or discard this patch.
src/Legacy/Fragments/FragmentField.php 2 patches
Indentation   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -63,22 +63,22 @@
 block discarded – undo
63 63
                 ->setModelIdInputName($this->name . '[' . $k . '][modelId]')
64 64
                 ->setFields($this->fields->clone()->map(function (Field $field) use ($k, $fragment) {
65 65
                     return $field->name($this->name . '.' . $k . '.' . $field->getName())
66
-                                 ->localizedFormat(':name.:locale')
67
-                                 ->valueResolver(function ($model = null, $locale = null, $field) use ($fragment) {
68
-                                     if (isset($field->value)) {
69
-                                         return $field->value;
70
-                                     }
71
-
72
-                                     if ($field instanceof MediaField) {
73
-                                         if (!$fragment->hasModelId()) {
74
-                                             return [];
75
-                                         }
76
-
77
-                                         return $field->getMedia(FragmentModel::find($fragment->getModelId()), $locale);
78
-                                     }
79
-
80
-                                     return $fragment->getValue($field->getColumn(), $locale);
81
-                                 });
66
+                                    ->localizedFormat(':name.:locale')
67
+                                    ->valueResolver(function ($model = null, $locale = null, $field) use ($fragment) {
68
+                                        if (isset($field->value)) {
69
+                                            return $field->value;
70
+                                        }
71
+
72
+                                        if ($field instanceof MediaField) {
73
+                                            if (!$fragment->hasModelId()) {
74
+                                                return [];
75
+                                            }
76
+
77
+                                            return $field->getMedia(FragmentModel::find($fragment->getModelId()), $locale);
78
+                                        }
79
+
80
+                                        return $fragment->getValue($field->getColumn(), $locale);
81
+                                    });
82 82
                 }));
83 83
         }
84 84
 
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -48,11 +48,11 @@  discard block
 block discarded – undo
48 48
         // Model is auto-injected by Manager::editFields() method.
49 49
         if (($this->model)) {
50 50
             if (!method_exists($this->model, 'getFragments')) {
51
-                throw new \RuntimeException(get_class($this->model) . ' is missing the ' . HasFragments::class . ' trait.');
51
+                throw new \RuntimeException(get_class($this->model).' is missing the '.HasFragments::class.' trait.');
52 52
             }
53 53
 
54 54
             if (count($modelFragments = $this->model->getFragments($this->getKey())) > 0) {
55
-                $fragments = $modelFragments->map(function (FragmentModel $fragmentModel) {
55
+                $fragments = $modelFragments->map(function(FragmentModel $fragmentModel) {
56 56
                     return Fragment::fromModel($fragmentModel);
57 57
                 })->all();
58 58
             }
@@ -60,11 +60,11 @@  discard block
 block discarded – undo
60 60
 
61 61
         foreach ($fragments as $k => $fragment) {
62 62
             $fragments[$k] = $fragments[$k]
63
-                ->setModelIdInputName($this->name . '[' . $k . '][modelId]')
64
-                ->setFields($this->fields->clone()->map(function (Field $field) use ($k, $fragment) {
65
-                    return $field->name($this->name . '.' . $k . '.' . $field->getName())
63
+                ->setModelIdInputName($this->name.'['.$k.'][modelId]')
64
+                ->setFields($this->fields->clone()->map(function(Field $field) use ($k, $fragment) {
65
+                    return $field->name($this->name.'.'.$k.'.'.$field->getName())
66 66
                                  ->localizedFormat(':name.:locale')
67
-                                 ->valueResolver(function ($model = null, $locale = null, $field) use ($fragment) {
67
+                                 ->valueResolver(function($model = null, $locale = null, $field) use ($fragment) {
68 68
                                      if (isset($field->value)) {
69 69
                                          return $field->value;
70 70
                                      }
@@ -110,8 +110,8 @@  discard block
 block discarded – undo
110 110
         }
111 111
 
112 112
         // Take the fields from the first fragment as a starting point for duplication
113
-        return array_map(function (\Thinktomorrow\Chief\ManagedModels\Fields\Types\Field $field) {
114
-            return $field->valueResolver(function ($model = null, $locale = null, $field) {
113
+        return array_map(function(\Thinktomorrow\Chief\ManagedModels\Fields\Types\Field $field) {
114
+            return $field->valueResolver(function($model = null, $locale = null, $field) {
115 115
                 if ($field instanceof \Thinktomorrow\Chief\ManagedModels\Fields\Types\MediaField) {
116 116
                     return [];
117 117
                 }
Please login to merge, or discard this patch.
src/Legacy/Fragments/HasFragments.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
24 24
         ], $fragment->getValues());
25 25
 
26 26
         if ($fragment->hasModelId()) {
27
-            $model =FragmentModel::find($fragment->getModelId());
27
+            $model = FragmentModel::find($fragment->getModelId());
28 28
             $model->update($values);
29 29
 
30 30
             return $model->id;
Please login to merge, or discard this patch.
src/Modules/Modules.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -33,12 +33,12 @@  discard block
 block discarded – undo
33 33
         // TODO: how to assign for pagebuilder?? with tag, via the chief relations config or is this default
34 34
         $moduleManagerClasses = $this->registry->tagged('module')->get();
35 35
 trap('working on it');
36
-        $managers = collect(array_map(function($moduleManagerClass){
36
+        $managers = collect(array_map(function($moduleManagerClass) {
37 37
             return app($moduleManagerClass);
38 38
         }, $moduleManagerClasses));
39 39
 
40 40
         return $managers
41
-            ->reject(function($manager){ return !$manager->can('create'); })
41
+            ->reject(function($manager) { return !$manager->can('create'); })
42 42
             ->map(function($manager) {
43 43
                 return [
44 44
                     'label' => $manager->adminLabel('page_title'),
@@ -53,13 +53,13 @@  discard block
 block discarded – undo
53 53
         // Get the modules that are set to be shareable.
54 54
         $moduleManagerClasses = $this->shareableModules();
55 55
 
56
-        $managers = collect(array_map(function($moduleManagerClass){
56
+        $managers = collect(array_map(function($moduleManagerClass) {
57 57
             return app($moduleManagerClass);
58 58
         }, $moduleManagerClasses));
59 59
 
60 60
         return $managers
61
-            ->reject(function($manager){ return !$manager->can('create-shared'); })
62
-            ->map(function($manager){
61
+            ->reject(function($manager) { return !$manager->can('create-shared'); })
62
+            ->map(function($manager) {
63 63
                 return [
64 64
                     'label' => $manager->adminLabel('page_title'),
65 65
                     'value' => $manager->route('create-shared')
Please login to merge, or discard this patch.
src/Fragments/Actions/CreateFragmentModel.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@
 block discarded – undo
29 29
      */
30 30
     public function create(FragmentsOwner $owner, Fragmentable $fragmentable, int $order, array $data = []): FragmentModel
31 31
     {
32
-        if(!$context = ContextModel::ownedBy($owner)) {
32
+        if (!$context = ContextModel::ownedBy($owner)) {
33 33
             $context = ContextModel::createForOwner($owner);
34 34
         }
35 35
 
Please login to merge, or discard this patch.
src/Fragments/Actions/RenderFragments.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@  discard block
 block discarded – undo
15 15
     public function render(Collection $fragmentables, FragmentsOwner $owner, $viewData = []): string
16 16
     {
17 17
         // Validate each entry as a valid fragment object.
18
-        $fragmentables->each(function(Fragmentable $fragmentable){});
18
+        $fragmentables->each(function(Fragmentable $fragmentable) {});
19 19
 
20 20
         // Init new loop object
21 21
         $this->loopsStack = []; $this->addLoop($fragmentables);
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
             $this->incrementLoopIndices();
25 25
             $loop = $this->getLastLoop();
26 26
 
27
-            return $carry . $fragmentable->renderFragment($owner, $loop, $fragmentables, $viewData);
27
+            return $carry.$fragmentable->renderFragment($owner, $loop, $fragmentables, $viewData);
28 28
         }, '');
29 29
     }
30 30
 }
Please login to merge, or discard this patch.
src/Fragments/Database/FragmentRepository.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
      */
27 27
     public function getByOwner(FragmentsOwner $owner): Collection
28 28
     {
29
-        if(!$context = ContextModel::ownedBy($owner)) return collect();
29
+        if (!$context = ContextModel::ownedBy($owner)) return collect();
30 30
 
31 31
         $fragmentModels = $context->fragments()->get();
32 32
 
@@ -44,16 +44,16 @@  discard block
 block discarded – undo
44 44
 
45 45
     private function prefetchRecords(Collection $fragmentModels)
46 46
     {
47
-        $fragmentModels->mapToGroups(function(FragmentModel $fragmentModel){
47
+        $fragmentModels->mapToGroups(function(FragmentModel $fragmentModel) {
48 48
             return [ModelReference::fromString($fragmentModel->model_reference)->className() => ModelReference::fromString($fragmentModel->model_reference)->id()];
49
-        })->reject(function($modelIds, $className){
49
+        })->reject(function($modelIds, $className) {
50 50
             $reflection = new ReflectionClass($className);
51 51
             return !$reflection->isSubclassOf(Model::class);
52
-        })->each(function($modelIds, $className){
52
+        })->each(function($modelIds, $className) {
53 53
             $modelIds = $modelIds->filter(fn($modelId) => $modelId !== 0);
54 54
             $records = $className::withoutGlobalScopes()->whereIn('id', $modelIds->toArray())->get();
55 55
 
56
-            $records->each(function($record){
56
+            $records->each(function($record) {
57 57
                 $this->prefetchedRecords[$record->modelReference()->get()] = $record;
58 58
             });
59 59
         });
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
     private function fragmentFactory(FragmentModel $fragmentModel): Fragmentable
63 63
     {
64 64
         return $this->prefetchedRecords
65
-            ->get($fragmentModel->model_reference,  fn() => ModelReference::fromString($fragmentModel->model_reference)->instance())
65
+            ->get($fragmentModel->model_reference, fn() => ModelReference::fromString($fragmentModel->model_reference)->instance())
66 66
             ->setFragmentModel($fragmentModel);
67 67
     }
68 68
 
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -26,7 +26,9 @@
 block discarded – undo
26 26
      */
27 27
     public function getByOwner(FragmentsOwner $owner): Collection
28 28
     {
29
-        if(!$context = ContextModel::ownedBy($owner)) return collect();
29
+        if(!$context = ContextModel::ownedBy($owner)) {
30
+            return collect();
31
+        }
30 32
 
31 33
         $fragmentModels = $context->fragments()->get();
32 34
 
Please login to merge, or discard this patch.