Passed
Pull Request — master (#1066)
by
unknown
03:31
created
src/Controllers/DeleteController.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
     public function getDelete()
17 17
     {
18 18
         $item_names = request('items');
19
-        $errors = [];
19
+        $errors = [ ];
20 20
 
21 21
         foreach ($item_names as $name_to_delete) {
22 22
             $file = $this->lfm->setName($name_to_delete);
@@ -35,13 +35,13 @@  discard block
 block discarded – undo
35 35
                 continue;
36 36
             }
37 37
 
38
-            if (! $this->lfm->setName($name_to_delete)->exists()) {
39
-                array_push($errors, parent::error('folder-not-found', ['folder' => $file_path]));
38
+            if (!$this->lfm->setName($name_to_delete)->exists()) {
39
+                array_push($errors, parent::error('folder-not-found', [ 'folder' => $file_path ]));
40 40
                 continue;
41 41
             }
42 42
 
43 43
             if ($this->lfm->setName($name_to_delete)->isDirectory()) {
44
-                if (! $this->lfm->setName($name_to_delete)->directoryIsEmpty()) {
44
+                if (!$this->lfm->setName($name_to_delete)->directoryIsEmpty()) {
45 45
                     array_push($errors, parent::error('delete-folder'));
46 46
                     continue;
47 47
                 }
Please login to merge, or discard this patch.
src/LfmPath.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -90,11 +90,11 @@  discard block
 block discarded – undo
90 90
 
91 91
     public function folders()
92 92
     {
93
-        $all_folders = array_map(function ($directory_path) {
93
+        $all_folders = array_map(function($directory_path) {
94 94
             return $this->pretty($directory_path, true);
95 95
         }, $this->storage->directories());
96 96
 
97
-        $folders = array_filter($all_folders, function ($directory) {
97
+        $folders = array_filter($all_folders, function($directory) {
98 98
             return $directory->name !== $this->helper->getThumbFolderName();
99 99
         });
100 100
 
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
 
104 104
     public function files()
105 105
     {
106
-        $files = array_map(function ($file_path) {
106
+        $files = array_map(function($file_path) {
107 107
             return $this->pretty($file_path);
108 108
         }, $this->storage->files());
109 109
 
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
         $working_dir = $this->path('working_dir');
149 149
         $parent_dir = substr($working_dir, 0, strrpos($working_dir, '/'));
150 150
 
151
-        $parent_directories = array_map(function ($directory_path) {
151
+        $parent_directories = array_map(function($directory_path) {
152 152
             return app(static::class)->translateToLfmPath($directory_path);
153 153
         }, app(static::class)->dir($parent_dir)->directories());
154 154
 
@@ -194,20 +194,20 @@  discard block
 block discarded – undo
194 194
     public function sortByColumn($arr_items)
195 195
     {
196 196
         $sort_by = $this->helper->input('sort_type');
197
-        if (in_array($sort_by, ['name', 'time'])) {
197
+        if (in_array($sort_by, [ 'name', 'time' ])) {
198 198
             $key_to_sort = $sort_by;
199 199
         } else {
200 200
             $key_to_sort = 'name';
201 201
         }
202 202
 
203
-        uasort($arr_items, function ($a, $b) use ($key_to_sort) {
203
+        uasort($arr_items, function($a, $b) use ($key_to_sort) {
204 204
             return strcasecmp($a->{$key_to_sort}, $b->{$key_to_sort});
205 205
         });
206 206
 
207 207
         return $arr_items;
208 208
     }
209 209
 
210
-    public function error($error_type, $variables = [])
210
+    public function error($error_type, $variables = [ ])
211 211
     {
212 212
         throw new \Exception($this->helper->error($error_type, $variables));
213 213
     }
@@ -236,10 +236,10 @@  discard block
 block discarded – undo
236 236
     {
237 237
         if (empty($file)) {
238 238
             return $this->error('file-empty');
239
-        } elseif (! $file instanceof UploadedFile) {
239
+        } elseif (!$file instanceof UploadedFile) {
240 240
             return $this->error('instance');
241 241
         } elseif ($file->getError() == UPLOAD_ERR_INI_SIZE) {
242
-            return $this->error('file-size', ['max' => ini_get('upload_max_filesize')]);
242
+            return $this->error('file-size', [ 'max' => ini_get('upload_max_filesize') ]);
243 243
         } elseif ($file->getError() != UPLOAD_ERR_OK) {
244 244
             throw new \Exception('File failed to upload. Error code: ' . $file->getError());
245 245
         }
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
252 252
 
253 253
         $mimetype = $file->getMimeType();
254 254
 
255
-        $excutable = ['text/x-php'];
255
+        $excutable = [ 'text/x-php' ];
256 256
 
257 257
         if (in_array($mimetype, $excutable)) {
258 258
             throw new \Exception('Invalid file detected');
@@ -297,11 +297,11 @@  discard block
 block discarded – undo
297 297
             $file_name_without_extentions = $new_file_name;
298 298
             while ($this->setName(($extension) ? $new_file_name_with_extention : $new_file_name)->exists()) {
299 299
                 if (config('lfm.alphanumeric_filename') === true) {
300
-                    $suffix = '_'.$counter;
300
+                    $suffix = '_' . $counter;
301 301
                 } else {
302 302
                     $suffix = " ({$counter})";
303 303
                 }
304
-                $new_file_name = $file_name_without_extentions.$suffix;
304
+                $new_file_name = $file_name_without_extentions . $suffix;
305 305
 
306 306
                 if ($extension) {
307 307
                     $new_file_name_with_extention = $new_file_name . '.' . $extension;
Please login to merge, or discard this patch.
src/Controllers/UploadController.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
     public function __construct()
15 15
     {
16 16
         parent::__construct();
17
-        $this->errors = [];
17
+        $this->errors = [ ];
18 18
     }
19 19
 
20 20
     /**
@@ -26,10 +26,10 @@  discard block
 block discarded – undo
26 26
     public function upload()
27 27
     {
28 28
         $uploaded_files = request()->file('upload');
29
-        $error_bag = [];
29
+        $error_bag = [ ];
30 30
         $new_filename = null;
31 31
 
32
-        foreach (is_array($uploaded_files) ? $uploaded_files : [$uploaded_files] as $file) {
32
+        foreach (is_array($uploaded_files) ? $uploaded_files : [ $uploaded_files ] as $file) {
33 33
             try {
34 34
                 $new_filename = $this->lfm->upload($file);
35 35
             } catch (\Exception $e) {
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
         } else { // upload via ckeditor5 expects json responses
48 48
             if (is_null($new_filename)) {
49 49
                 $response = [
50
-                    'error' => [ 'message' =>  $error_bag[0] ]
50
+                    'error' => [ 'message' =>  $error_bag[ 0 ] ]
51 51
                 ];
52 52
             } else {
53 53
                 $url = $this->lfm->setName($new_filename)->url();
Please login to merge, or discard this patch.
src/Controllers/LfmController.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -35,8 +35,8 @@  discard block
 block discarded – undo
35 35
      */
36 36
     public function show()
37 37
     {
38
-        $key_auth_token = \config('lfm')['key_auth_token'];
39
-        $no_authenticate_redirect_to = \config('lfm')['no_authenticate_token_redirect_to'];
38
+        $key_auth_token = \config('lfm')[ 'key_auth_token' ];
39
+        $no_authenticate_redirect_to = \config('lfm')[ 'no_authenticate_token_redirect_to' ];
40 40
 
41 41
         return view('laravel-filemanager::index', compact('key_auth_token', 'no_authenticate_redirect_to'))
42 42
             ->withHelper($this->helper);
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
      */
50 50
     public function getErrors()
51 51
     {
52
-        $arr_errors = [];
52
+        $arr_errors = [ ];
53 53
 
54 54
         if (!extension_loaded('gd') && !extension_loaded('imagick')) {
55 55
             array_push($arr_errors, trans('laravel-filemanager::lfm.message-extension_not_found'));
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
         return $arr_errors;
75 75
     }
76 76
 
77
-    public function error($error_type, $variables = [])
77
+    public function error($error_type, $variables = [ ])
78 78
     {
79 79
         return $this->helper->error($error_type, $variables);
80 80
     }
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
      */
87 87
     public function applyIniOverrides()
88 88
     {
89
-        $overrides = config('lfm.php_ini_overrides', []);
89
+        $overrides = config('lfm.php_ini_overrides', [ ]);
90 90
 
91 91
         if ($overrides && is_array($overrides) && count($overrides) === 0) {
92 92
             return;
@@ -115,17 +115,17 @@  discard block
 block discarded – undo
115 115
             if ($auth->check()) {
116 116
                 $response = [
117 117
                     'message' => 'Authorization',
118
-                    'errors' => [],
118
+                    'errors' => [ ],
119 119
                     'data' => [
120 120
                         'authorization' => true,
121 121
                         'redirect_to' => null,
122 122
                     ]
123 123
                 ];
124
-                $status_code  = 200;
124
+                $status_code = 200;
125 125
             } else {
126 126
                 $response = [
127 127
                     'message' => 'No authorization',
128
-                    'errors' => [],
128
+                    'errors' => [ ],
129 129
                     'data' => [
130 130
                         'authorization' => false,
131 131
                         'redirect_to' => \config('lfm.no_authenticate_token_redirect_to'),
@@ -139,9 +139,9 @@  discard block
 block discarded – undo
139 139
             return \response([
140 140
                 'message' => 'Error machine',
141 141
                 'errors' => [
142
-                    'machine' => [$e->getMessage()],
142
+                    'machine' => [ $e->getMessage() ],
143 143
                 ],
144
-                'data' => [],
144
+                'data' => [ ],
145 145
             ], 500);
146 146
         }
147 147
     }
Please login to merge, or discard this patch.
src/Lfm.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
             $path_parts = pathinfo($path);
62 62
         }
63 63
 
64
-        return substr($path_parts[$part_name], 1);
64
+        return substr($path_parts[ $part_name ], 1);
65 65
     }
66 66
 
67 67
     public function allowFolderType($type)
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
         $lfm_type = 'file';
91 91
 
92 92
         $request_type = lcfirst(Str::singular($this->input('type') ?: ''));
93
-        $available_types = array_keys($this->config->get('lfm.folder_categories') ?: []);
93
+        $available_types = array_keys($this->config->get('lfm.folder_categories') ?: [ ]);
94 94
 
95 95
         if (in_array($request_type, $available_types)) {
96 96
             $lfm_type = $request_type;
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
         $view_type = 'grid';
108 108
         $target_display_type = $this->input('show_list') ?: $startup_view;
109 109
 
110
-        if (in_array($target_display_type, ['list', 'grid'])) {
110
+        if (in_array($target_display_type, [ 'list', 'grid' ])) {
111 111
             $view_type = $target_display_type;
112 112
         }
113 113
 
@@ -248,7 +248,7 @@  discard block
 block discarded – undo
248 248
      * @param  mixed  $variables   Variables the message needs.
249 249
      * @return string
250 250
      */
251
-    public function error($error_type, $variables = [])
251
+    public function error($error_type, $variables = [ ])
252 252
     {
253 253
         throw new \Exception(trans(self::PACKAGE_NAME . '::lfm.error-' . $error_type, $variables));
254 254
     }
@@ -260,11 +260,11 @@  discard block
 block discarded – undo
260 260
      */
261 261
     public static function routes()
262 262
     {
263
-        $middleware = [CreateDefaultFolder::class, MultiUser::class];
263
+        $middleware = [ CreateDefaultFolder::class, MultiUser::class ];
264 264
         $as = 'unisharp.lfm.';
265 265
         $namespace = '\\UniSharp\\LaravelFilemanager\\Controllers\\';
266 266
 
267
-        Route::group(compact('middleware', 'as', 'namespace'), function () {
267
+        Route::group(compact('middleware', 'as', 'namespace'), function() {
268 268
 
269 269
             // display main layout
270 270
             Route::get('/', [
@@ -280,9 +280,9 @@  discard block
 block discarded – undo
280 280
         });
281 281
 
282 282
 
283
-        $config_middleware = \config('lfm.middleware') ?? [];
283
+        $config_middleware = \config('lfm.middleware') ?? [ ];
284 284
         $middleware = array_merge($middleware, $config_middleware);
285
-        Route::group(compact('middleware', 'as', 'namespace'), function () {
285
+        Route::group(compact('middleware', 'as', 'namespace'), function() {
286 286
 
287 287
             // display integration error messages
288 288
             Route::get('/errors', [
Please login to merge, or discard this patch.