Completed
Push — 0.3 ( da43ab...625b56 )
by Ben
96:17 queued 52:29
created
src/Media/UploadMedia.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
         if (is_string($file)) {
60 60
             $image_name = json_decode($file)->output->name;
61 61
             $asset      = $this->addAsset(json_decode($file)->output->image, $type, null, $image_name, $model);
62
-        } else {
62
+        }else {
63 63
             $image_name = $file->getClientOriginalName();
64 64
             $asset      = $this->addAsset($file, $type, null, $image_name, $model);
65 65
         }
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 
83 83
         if (is_string($file)) {
84 84
             $asset = AssetUploader::uploadFromBase64($file, $filename);
85
-        } else {
85
+        }else {
86 86
             $asset = AssetUploader::upload($file, $filename);
87 87
         }
88 88
 
@@ -126,9 +126,9 @@  discard block
 block discarded – undo
126 126
      */
127 127
     private function sluggifyFilename($filename): string
128 128
     {
129
-        $extension = substr($filename, strrpos($filename, '.') + 1);
129
+        $extension = substr($filename, strrpos($filename, '.')+1);
130 130
         $filename  = substr($filename, 0, strrpos($filename, '.'));
131
-        $filename  = str_slug($filename) . '.' . $extension;
131
+        $filename  = str_slug($filename).'.'.$extension;
132 132
 
133 133
         return $filename;
134 134
     }
@@ -144,9 +144,9 @@  discard block
 block discarded – undo
144 144
                 if ($file instanceof UploadedFile && !$file->isValid()) {
145 145
                     if ($file->getError() == UPLOAD_ERR_INI_SIZE) {
146 146
                         throw new FileTooBigException(
147
-                            'Cannot upload file because it exceeded the allowed upload_max_filesize: upload_max_filesize is smaller than post size. ' .
148
-                            'upload_max_filesize: ' . (int)ini_get('upload_max_filesize') . 'MB, ' .
149
-                            'post_max_size: ' . (int)(ini_get('post_max_size')) . 'MB'
147
+                            'Cannot upload file because it exceeded the allowed upload_max_filesize: upload_max_filesize is smaller than post size. '.
148
+                            'upload_max_filesize: '.(int)ini_get('upload_max_filesize').'MB, '.
149
+                            'post_max_size: '.(int)(ini_get('post_max_size')).'MB'
150 150
                         );
151 151
                     }
152 152
                 }
Please login to merge, or discard this patch.
src/PageBuilder/PresentSections.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
             $this->addModelToCollection($i, $child);
79 79
         }
80 80
 
81
-        return $this->sets->values()->map(function (ViewableContract $child) {
81
+        return $this->sets->values()->map(function(ViewableContract $child) {
82 82
             return ($this->withSnippets && method_exists($child, 'withSnippets'))
83 83
                 ? $child->withSnippets()->setViewParent($this->parent)->renderView()
84 84
                 : $child->setViewParent($this->parent)->renderView();
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
     {
96 96
         // Only published pages you fool!
97 97
         // TODO: check for assistant instead of method existence
98
-        if (method_exists($model, 'isPublished') && ! $model->isPublished()) {
98
+        if (method_exists($model, 'isPublished') && !$model->isPublished()) {
99 99
             return;
100 100
         }
101 101
 
Please login to merge, or discard this patch.
src/Concerns/Viewable/ViewPath.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 function get(): string
36 36
     {
37
-        $basePath = $this->viewBasePath ? $this->viewBasePath . '.' : '';
37
+        $basePath = $this->viewBasePath ? $this->viewBasePath.'.' : '';
38 38
         $guessedParentViewName = $this->parent ? $this->parent->viewKey() : '';
39 39
         $guessedViewName = $this->viewable->viewKey();
40 40
 
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
         }
54 54
 
55 55
         foreach ($viewPaths as $viewPath) {
56
-            if (! view()->exists($viewPath)) {
56
+            if (!view()->exists($viewPath)) {
57 57
                 continue;
58 58
             }
59 59
 
Please login to merge, or discard this patch.
src/Pages/PageManager.php 2 patches
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -87,11 +87,11 @@
 block discarded – undo
87 87
         return parent::fields()->add(
88 88
             $this->pageBuilderField(),
89 89
             InputField::make('title')->translatable($this->model->availableLocales())
90
-                                     ->validation('required-fallback-locale|max:200', [], [
91
-                                         'trans.'.config('app.fallback_locale', 'nl').'.title' => 'title',
92
-                                     ])
93
-                                     ->label('De titel van je '.$this->model->labelSingular ?? 'pagina')
94
-                                     ->description('Dit is de titel die zal worden getoond in de overzichten en modules.'),
90
+                                        ->validation('required-fallback-locale|max:200', [], [
91
+                                            'trans.'.config('app.fallback_locale', 'nl').'.title' => 'title',
92
+                                        ])
93
+                                        ->label('De titel van je '.$this->model->labelSingular ?? 'pagina')
94
+                                        ->description('Dit is de titel die zal worden getoond in de overzichten en modules.'),
95 95
             InputField::make('seo_title')
96 96
                 ->translatable($this->model->availableLocales())
97 97
                 ->label('Zoekmachine titel'),
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -59,15 +59,15 @@  discard block
 block discarded – undo
59 59
     {
60 60
         $permission = 'update-page';
61 61
 
62
-        if (in_array($verb, ['index','show'])) {
62
+        if (in_array($verb, ['index', 'show'])) {
63 63
             $permission = 'view-page';
64
-        } elseif (in_array($verb, ['create','store'])) {
64
+        } elseif (in_array($verb, ['create', 'store'])) {
65 65
             $permission = 'create-page';
66 66
         } elseif (in_array($verb, ['delete'])) {
67 67
             $permission = 'delete-page';
68 68
         }
69 69
 
70
-        if (! auth()->guard('chief')->user()->hasPermissionTo($permission)) {
70
+        if (!auth()->guard('chief')->user()->hasPermissionTo($permission)) {
71 71
             throw NotAllowedManagerRoute::notAllowedPermission($permission, $this);
72 72
         }
73 73
     }
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
     public function fieldArrangement($key = null): FieldArrangement
129 129
     {
130 130
         if ($key == 'create') {
131
-            return new FieldArrangement($this->fieldsWithAssistantFields()->filterBy(function ($field) {
131
+            return new FieldArrangement($this->fieldsWithAssistantFields()->filterBy(function($field) {
132 132
                 return in_array($field->key, ['title']);
133 133
             }));
134 134
         }
@@ -147,8 +147,8 @@  discard block
 block discarded – undo
147 147
         if ($this->model->id) {
148 148
             return parent::details()
149 149
                 ->set('title', $this->model->title)
150
-                ->set('intro', 'laatst aangepast op ' . $this->model->updated_at->format('d/m/Y H:i'))
151
-                ->set('context', '<span class="inline-s">' . $this->assistant('publish')->publicationStatusAsLabel() . '</span>');
150
+                ->set('intro', 'laatst aangepast op '.$this->model->updated_at->format('d/m/Y H:i'))
151
+                ->set('context', '<span class="inline-s">'.$this->assistant('publish')->publicationStatusAsLabel().'</span>');
152 152
         }
153 153
 
154 154
         return parent::details();
@@ -157,7 +157,7 @@  discard block
 block discarded – undo
157 157
     public function saveFields(): Manager
158 158
     {
159 159
         // Store the morph_key upon creation
160
-        if (! $this->model->morph_key) {
160
+        if (!$this->model->morph_key) {
161 161
             $this->model->morph_key = $this->model->morphKey();
162 162
         }
163 163
 
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
             if (is_array_empty($translation)) {
203 203
 
204 204
                 // Nullify all values
205
-                $trans[$locale] = array_map(function ($value) {
205
+                $trans[$locale] = array_map(function($value) {
206 206
                     return null;
207 207
                 }, $translation);
208 208
                 continue;
Please login to merge, or discard this patch.
src/Sets/Set.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 
47 47
         // If no view has been created for this page collection, we try once again to fetch the content value if any. This will silently fail
48 48
         // if no content value is present. We don't consider the 'content' attribute to be a default as we do for module.
49
-        return $this->map(function ($item) {
49
+        return $this->map(function($item) {
50 50
             return ($this->viewParent && $item instanceof ViewableContract)
51 51
                 ? $item->setViewParent($this->viewParent)->renderView()
52 52
                 : ($item->content ?? '');
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
     {
80 80
         $currentPage = $currentPage ?? request()->get('page', 1);
81 81
         $path = request()->path();
82
-        $items = array_slice($this->all(), ($currentPage - 1) * $perPage);
82
+        $items = array_slice($this->all(), ($currentPage-1) * $perPage);
83 83
 
84 84
         return (new \Illuminate\Pagination\Paginator($items, $perPage, $currentPage))->setPath($path);
85 85
     }
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
     {
97 97
         $currentPage = $currentPage ?? request()->get('page', 1);
98 98
         $path = request()->path();
99
-        $items = array_slice($this->all(), ($currentPage - 1) * $perPage, $perPage);
99
+        $items = array_slice($this->all(), ($currentPage-1) * $perPage, $perPage);
100 100
 
101 101
         return (new \Illuminate\Pagination\LengthAwarePaginator($items, $this->count(), $perPage, $currentPage))->setPath($path);
102 102
     }
Please login to merge, or discard this patch.
src/Urls/UrlRecord.php 2 patches
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -65,8 +65,8 @@  discard block
 block discarded – undo
65 65
     public static function getByModel(Model $model)
66 66
     {
67 67
         return static::where('model_type', $model->getMorphClass())
68
-                     ->where('model_id', $model->id)
69
-                     ->get();
68
+                        ->where('model_id', $model->id)
69
+                        ->get();
70 70
     }
71 71
 
72 72
     public function replaceAndRedirect(array $values): UrlRecord
@@ -119,9 +119,9 @@  discard block
 block discarded – undo
119 119
         if ($ignoredModel) {
120 120
             $builder->whereNotIn('id', function ($query) use ($ignoredModel) {
121 121
                 $query->select('id')
122
-                      ->from('chief_urls')
123
-                      ->where('model_type', '=', $ignoredModel->getMorphClass())
124
-                      ->where('model_id', '=', $ignoredModel->id);
122
+                        ->from('chief_urls')
123
+                        ->where('model_type', '=', $ignoredModel->getMorphClass())
124
+                        ->where('model_id', '=', $ignoredModel->id);
125 125
             });
126 126
         }
127 127
 
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -112,12 +112,12 @@
 block discarded – undo
112 112
             $builder->where('locale', $locale);
113 113
         }
114 114
 
115
-        if (! $includeRedirects) {
115
+        if (!$includeRedirects) {
116 116
             $builder->whereNull('redirect_id');
117 117
         }
118 118
 
119 119
         if ($ignoredModel) {
120
-            $builder->whereNotIn('id', function ($query) use ($ignoredModel) {
120
+            $builder->whereNotIn('id', function($query) use ($ignoredModel) {
121 121
                 $query->select('id')
122 122
                       ->from('chief_urls')
123 123
                       ->where('model_type', '=', $ignoredModel->getMorphClass())
Please login to merge, or discard this patch.
src/Urls/MemoizedUrlRecord.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -8,14 +8,14 @@
 block discarded – undo
8 8
 {
9 9
     public static function findByModel(Model $model, string $locale = null): UrlRecord
10 10
     {
11
-        return chiefMemoize('url-records-find-by-model', function ($model, $locale = null) {
11
+        return chiefMemoize('url-records-find-by-model', function($model, $locale = null) {
12 12
             return parent::findByModel($model, $locale);
13 13
         }, [$model, $locale]);
14 14
     }
15 15
 
16 16
     public static function getByModel(Model $model)
17 17
     {
18
-        return chiefMemoize('url-records-get-by-model', function ($model) {
18
+        return chiefMemoize('url-records-get-by-model', function($model) {
19 19
             return parent::getByModel($model);
20 20
         }, [$model]);
21 21
     }
Please login to merge, or discard this patch.
src/Urls/SaveUrlSlugs.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
     private function saveRecord(string $locale, ?string $slug)
40 40
     {
41 41
         // Existing ones for this locale?
42
-        $nonRedirectsWithSameLocale = $this->existingRecords->filter(function ($record) use ($locale) {
42
+        $nonRedirectsWithSameLocale = $this->existingRecords->filter(function($record) use ($locale) {
43 43
             return (
44 44
                 $record->locale == $locale &&
45 45
                 !$record->isRedirect()
@@ -55,14 +55,14 @@  discard block
 block discarded – undo
55 55
 
56 56
         // If slug entry is left empty, all existing records will be deleted
57 57
         if (!$slug) {
58
-            $nonRedirectsWithSameLocale->each(function ($existingRecord) {
58
+            $nonRedirectsWithSameLocale->each(function($existingRecord) {
59 59
                 $existingRecord->delete();
60 60
             });
61 61
         } elseif ($nonRedirectsWithSameLocale->isEmpty()) {
62 62
             $this->createRecord($locale, $slug);
63
-        } else {
63
+        }else {
64 64
             // Only replace the existing records that differ from the current passed slugs
65
-            $nonRedirectsWithSameLocale->each(function ($existingRecord) use ($slug) {
65
+            $nonRedirectsWithSameLocale->each(function($existingRecord) use ($slug) {
66 66
                 if ($existingRecord->slug != $slug) {
67 67
                     $existingRecord->replaceAndRedirect(['slug' => $slug]);
68 68
                 }
@@ -87,12 +87,12 @@  discard block
 block discarded – undo
87 87
      */
88 88
     private function deleteIdenticalRedirects($existingRecords, $locale, $slug): void
89 89
     {
90
-        $existingRecords->filter(function ($record) use ($locale) {
90
+        $existingRecords->filter(function($record) use ($locale) {
91 91
             return (
92 92
                 $record->locale == $locale &&
93 93
                 $record->isRedirect()
94 94
             );
95
-        })->each(function ($existingRecord) use ($slug) {
95
+        })->each(function($existingRecord) use ($slug) {
96 96
             if ($existingRecord->slug == $slug) {
97 97
                 $existingRecord->delete();
98 98
             }
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
      */
107 107
     private function prependBaseUrlSegment(string $slug, $locale): string
108 108
     {
109
-        $slugWithBaseSegment = $this->model->baseUrlSegment($locale) . '/' . $slug;
109
+        $slugWithBaseSegment = $this->model->baseUrlSegment($locale).'/'.$slug;
110 110
         $slugWithBaseSegment = trim($slugWithBaseSegment, '/');
111 111
 
112 112
         // If slug with base segment is empty string, it means that the passed slug was probably a "/" character.
Please login to merge, or discard this patch.
src/Urls/UrlSlugFields.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -42,10 +42,10 @@  discard block
 block discarded – undo
42 42
         $fields = new static([]);
43 43
 
44 44
         foreach ($locales as $locale) {
45
-            $fields['url-slugs.' . $locale] = UrlSlugField::make('url-slugs.' . $locale)
45
+            $fields['url-slugs.'.$locale] = UrlSlugField::make('url-slugs.'.$locale)
46 46
                                                 ->setBaseUrlSegment($model->baseUrlSegment($locale))
47
-                                                ->prepend($model->resolveUrl($locale, $model->baseUrlSegment($locale)) .'/')
48
-                                                ->name('url-slugs[' . $locale . ']')
47
+                                                ->prepend($model->resolveUrl($locale, $model->baseUrlSegment($locale)).'/')
48
+                                                ->name('url-slugs['.$locale.']')
49 49
                                                 ->label($locale);
50 50
         }
51 51
 
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
      */
59 59
     private static function fillWithExistingValues(ProvidesUrl $model, self $fields): void
60 60
     {
61
-        $records = MemoizedUrlRecord::getByModel($model)->reject(function ($record) {
61
+        $records = MemoizedUrlRecord::getByModel($model)->reject(function($record) {
62 62
             return $record->isRedirect();
63 63
         })->sortBy('locale');
64 64
 
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
             $fields['url-slugs.'.$record->locale]
71 71
                 ->setUrlRecord($record)
72 72
                 ->setBaseUrlSegment($model->baseUrlSegment($record->locale))
73
-                ->prepend($model->resolveUrl($record->locale, $model->baseUrlSegment($record->locale)) .'/');
73
+                ->prepend($model->resolveUrl($record->locale, $model->baseUrlSegment($record->locale)).'/');
74 74
         }
75 75
     }
76 76
 }
Please login to merge, or discard this patch.