Passed
Push — fix/long_page_title ( 03e304 )
by
unknown
11:03
created
src/Management/AbstractManager.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
             return $this->indexPagination($builder);
83 83
         }
84 84
         
85
-        return $builder->get()->map(function ($model) {
85
+        return $builder->get()->map(function($model) {
86 86
             return (new static($this->registration))->manage($model);
87 87
         });
88 88
     }
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
     {
111 111
         $paginator = $builder->paginate($this->pageCount);
112 112
 
113
-        $modifiedCollection = $builder->paginate($this->pageCount)->getCollection()->transform(function ($model) {
113
+        $modifiedCollection = $builder->paginate($this->pageCount)->getCollection()->transform(function($model) {
114 114
             return (new static($this->registration))->manage($model);
115 115
         });
116 116
 
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
 
181 181
     public function guard($verb): Manager
182 182
     {
183
-        if (! $this->can($verb)) {
183
+        if (!$this->can($verb)) {
184 184
             NotAllowedManagerRoute::notAllowedVerb($verb, $this);
185 185
         }
186 186
 
@@ -203,7 +203,7 @@  discard block
 block discarded – undo
203 203
         $fields = $this->fields();
204 204
 
205 205
         foreach ($this->assistants() as $assistant) {
206
-            if (! method_exists($assistant, 'fields')) {
206
+            if (!method_exists($assistant, 'fields')) {
207 207
                 continue;
208 208
             }
209 209
 
Please login to merge, or discard this patch.
src/Concerns/Sluggable/UniqueSlug.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@
 block discarded – undo
18 18
         $this->model = $model;
19 19
         $this->blacklist = $blacklist;
20 20
 
21
-        $this->slugResolver = function ($slug) {
21
+        $this->slugResolver = function($slug) {
22 22
             return Illuminate\Support\Str::slug($slug);
23 23
         };
24 24
     }
Please login to merge, or discard this patch.
src/Media/UploadMedia.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
         if (is_string($file)) {
112 112
             $image_name = json_decode($file)->output->name;
113 113
             $asset      = $this->addAsset(json_decode($file)->output->image, $type, $locale, $image_name, $model);
114
-        } else {
114
+        }else {
115 115
             $image_name = $file->getClientOriginalName();
116 116
             $asset      = $this->addAsset($file, $type, $locale, $image_name, $model);
117 117
         }
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
         // New files are passed with their filename (instead of their id)
120 120
         // For new files we will replace the filename with the id.
121 121
         if (false !== ($key = array_search($image_name, $files_order))) {
122
-            $files_order[$key] = (string) $asset->id;
122
+            $files_order[$key] = (string)$asset->id;
123 123
         }
124 124
     }
125 125
 
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
 
135 135
         if (is_string($file)) {
136 136
             $asset = AssetUploader::uploadFromBase64($file, $filename);
137
-        } else {
137
+        }else {
138 138
             $asset = AssetUploader::upload($file, $filename);
139 139
         }
140 140
 
@@ -151,9 +151,9 @@  discard block
 block discarded – undo
151 151
      */
152 152
     private function sluggifyFilename($filename): string
153 153
     {
154
-        $extension = substr($filename, strrpos($filename, '.') + 1);
154
+        $extension = substr($filename, strrpos($filename, '.')+1);
155 155
         $filename  = substr($filename, 0, strrpos($filename, '.'));
156
-        $filename  = Str::slug($filename) . '.' . $extension;
156
+        $filename  = Str::slug($filename).'.'.$extension;
157 157
 
158 158
         return $filename;
159 159
     }
@@ -169,9 +169,9 @@  discard block
 block discarded – undo
169 169
                 if ($file instanceof UploadedFile && !$file->isValid()) {
170 170
                     if ($file->getError() == UPLOAD_ERR_INI_SIZE) {
171 171
                         throw new FileTooBigException(
172
-                            'Cannot upload file because it exceeded the allowed upload_max_filesize: upload_max_filesize is smaller than post size. ' .
173
-                            'upload_max_filesize: ' . (int)ini_get('upload_max_filesize') . 'MB, ' .
174
-                            'post_max_size: ' . (int)(ini_get('post_max_size')) . 'MB'
172
+                            'Cannot upload file because it exceeded the allowed upload_max_filesize: upload_max_filesize is smaller than post size. '.
173
+                            'upload_max_filesize: '.(int)ini_get('upload_max_filesize').'MB, '.
174
+                            'post_max_size: '.(int)(ini_get('post_max_size')).'MB'
175 175
                         );
176 176
                     }
177 177
                 }
@@ -185,16 +185,16 @@  discard block
 block discarded – undo
185 185
         foreach ($files_by_type as $type => $files) {
186 186
             foreach ($files as $locale => $_files) {
187 187
                 if (!in_array($locale, config('translatable.locales'))) {
188
-                    throw new \InvalidArgumentException('Corrupt file payload. key is expected to be a valid locale [' . implode(',', config('translatable.locales', [])). ']. Instead [' . $locale . '] is given.');
188
+                    throw new \InvalidArgumentException('Corrupt file payload. key is expected to be a valid locale ['.implode(',', config('translatable.locales', [])).']. Instead ['.$locale.'] is given.');
189 189
                 }
190 190
 
191 191
                 if (!is_array($_files)) {
192
-                    throw new \InvalidArgumentException('A valid files entry should be an array of files, key with either [new, replace or delete]. Instead a ' . gettype($_files) . ' is given.');
192
+                    throw new \InvalidArgumentException('A valid files entry should be an array of files, key with either [new, replace or delete]. Instead a '.gettype($_files).' is given.');
193 193
                 }
194 194
 
195 195
                 foreach ($_files as $action => $file) {
196 196
                     if (!in_array($action, $actions)) {
197
-                        throw new \InvalidArgumentException('A valid files entry should have a key of either ['.implode(',', $actions).']. Instead ' . $action . ' is given.');
197
+                        throw new \InvalidArgumentException('A valid files entry should have a key of either ['.implode(',', $actions).']. Instead '.$action.' is given.');
198 198
                     }
199 199
                 }
200 200
             }
Please login to merge, or discard this patch.
src/HealthMonitor/Checks/HomepageAccessibleCheck.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\HealthMonitor\Checks;
6 6
 
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
 
17 17
     private function get_http_response_code(string $url)
18 18
     {
19
-        if ($url =='') {
19
+        if ($url == '') {
20 20
             return false;
21 21
         }
22 22
 
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
 
35 35
     public function message(): string
36 36
     {
37
-        return 'Het lijkt erop dat de homepagina niet meer bereikbaar is. <a href="'. route('chief.back.settings.edit') .'" class="text-secondary-800 underline hover:text-white">Kies een nieuwe</a>.';
37
+        return 'Het lijkt erop dat de homepagina niet meer bereikbaar is. <a href="'.route('chief.back.settings.edit').'" class="text-secondary-800 underline hover:text-white">Kies een nieuwe</a>.';
38 38
     }
39 39
 
40 40
     public function notifiers(): array
Please login to merge, or discard this patch.
src/HealthMonitor/Checks/HomepageSetCheck.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\HealthMonitor\Checks;
6 6
 
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
 
19 19
     public function message(): string
20 20
     {
21
-        return 'Het lijkt erop dat er geen homepagina ingesteld is. Stel er een in hier: <a href="'. route('chief.back.settings.edit') .'" class="text-secondary-800 underline hover:text-white">Settings</a>';
21
+        return 'Het lijkt erop dat er geen homepagina ingesteld is. Stel er een in hier: <a href="'.route('chief.back.settings.edit').'" class="text-secondary-800 underline hover:text-white">Settings</a>';
22 22
     }
23 23
 
24 24
     public function notifiers(): array
Please login to merge, or discard this patch.
src/HealthMonitor/Monitor.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\HealthMonitor;
6 6
 
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
         foreach ($this->checks as $check) {
23 23
             $checkInstance = app($check);
24 24
             
25
-            if (! $checkInstance instanceof HealthCheck) {
25
+            if (!$checkInstance instanceof HealthCheck) {
26 26
                 throw new InvalidClassException('Checks must implement Healthcheck interface.');
27 27
             }
28 28
 
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
                     app($notifier)->onFailure($checkInstance);
34 34
                 }
35 35
                 return;
36
-            } else {
36
+            }else {
37 37
                 foreach ($notifiers as $notifier) {
38 38
                     app($notifier)->onSuccess($checkInstance);
39 39
                 }
Please login to merge, or discard this patch.
src/Management/ManagesPagebuilder.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
 
50 50
         $availableChildren = AvailableChildren::forParent($model);
51 51
 
52
-        $modules = $availableChildren->onlyModules()->reject(function ($module) use ($model) {
52
+        $modules = $availableChildren->onlyModules()->reject(function($module) use ($model) {
53 53
             return $module->page_id != null && $module->page_id != $model->id;
54 54
         });
55 55
 
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
         $available_sets = FlatReferencePresenter::toGroupedSelectValues($availableChildren->onlySets())->toArray();
59 59
 
60 60
         // Current sections
61
-        $sections = $model->children()->map(function ($section, $index) {
61
+        $sections = $model->children()->map(function($section, $index) {
62 62
             if ($section instanceof TranslatableContract) {
63 63
                 $section->injectTranslationForForm();
64 64
             }
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
      */
93 93
     private function findEditUrl($model): ?string
94 94
     {
95
-        if (! $model instanceof ManagedModel) {
95
+        if (!$model instanceof ManagedModel) {
96 96
             return null;
97 97
         }
98 98
 
Please login to merge, or discard this patch.
src/Modules/Application/DeleteModule.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
 
22 22
             // Mark the slug as deleted to avoid any conflict with newly created modules with the same slug.
23 23
             $module->update([
24
-                'slug' => $module->slug . $this->appendDeleteMarker(),
24
+                'slug' => $module->slug.$this->appendDeleteMarker(),
25 25
             ]);
26 26
 
27 27
             $module->delete();
@@ -35,6 +35,6 @@  discard block
 block discarded – undo
35 35
 
36 36
     private function appendDeleteMarker(): string
37 37
     {
38
-        return '_DELETED_' . time();
38
+        return '_DELETED_'.time();
39 39
     }
40 40
 }
Please login to merge, or discard this patch.
src/Fields/Types/FieldType.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -6,19 +6,19 @@  discard block
 block discarded – undo
6 6
 
7 7
 class FieldType
8 8
 {
9
-    const INPUT = 'input';   // oneliner text (input)
10
-    const TEXT = 'text';    // Plain text (textarea)
9
+    const INPUT = 'input'; // oneliner text (input)
10
+    const TEXT = 'text'; // Plain text (textarea)
11 11
     const NUMBER = 'number'; // number
12 12
     const RANGE = 'range'; // range slider
13
-    const DATE = 'date';    // Timestamp input
14
-    const PHONENUMBER = 'phonenumber';    // Timestamp input
15
-    const HTML = 'html';    // Html text (wysiwyg)
16
-    const SELECT = 'select';  // Select options
17
-    const MEDIA = 'media';  // media file (slim uploader)
18
-    const DOCUMENT = 'document';  // documents
19
-    const RADIO = 'radio';  // radio select
20
-    const CHECKBOX = 'checkbox';  // checkbox select
21
-    const PAGEBUILDER = 'pagebuilder';  // the most special field there is...
13
+    const DATE = 'date'; // Timestamp input
14
+    const PHONENUMBER = 'phonenumber'; // Timestamp input
15
+    const HTML = 'html'; // Html text (wysiwyg)
16
+    const SELECT = 'select'; // Select options
17
+    const MEDIA = 'media'; // media file (slim uploader)
18
+    const DOCUMENT = 'document'; // documents
19
+    const RADIO = 'radio'; // radio select
20
+    const CHECKBOX = 'checkbox'; // checkbox select
21
+    const PAGEBUILDER = 'pagebuilder'; // the most special field there is...
22 22
 
23 23
     /** @var string */
24 24
     private $type;
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
 
38 38
     public static function fromString(string $type): self
39 39
     {
40
-        $class = 'Thinktomorrow\Chief\Fields\Types\\' . ucfirst($type . 'Field');
40
+        $class = 'Thinktomorrow\Chief\Fields\Types\\'.ucfirst($type.'Field');
41 41
 
42 42
         return new $class(new static($type));
43 43
     }
Please login to merge, or discard this patch.