Passed
Pull Request — master (#1092)
by Iman
07:45 queued 01:54
created
src/views/default/type_components/select/component.blade.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,4 +1,4 @@  discard block
 block discarded – undo
1
-<?php $default = ! empty($formInput['default']) ? $formInput['default'] : trans('crudbooster.text_prefix_option')." ".$label;?>
1
+<?php $default = !empty($formInput['default']) ? $formInput['default'] : trans('crudbooster.text_prefix_option')." ".$label; ?>
2 2
 @if($formInput['parent_select'])
3 3
 
4 4
     @push('bottom')
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
         <select class='form-control' id="{{$name}}" data-value='{{$value}}' {{$required}} {!!$placeholder!!} {{$readonly}} {{$disabled}} name="{{$name}}">
47 47
             <option value=''>{{$default}}</option>
48 48
             <?php
49
-            if (! $formInput['parent_select']) {
49
+            if (!$formInput['parent_select']) {
50 50
                 if (@$formInput['dataquery']):
51 51
 
52 52
                     $query = DB::select(DB::raw($formInput['dataquery']));
Please login to merge, or discard this patch.
views/default/type_components/checkbox_dataenum/component_detail.blade.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@
 block discarded – undo
1 1
 <?php
2
-if ($formInput['options']['value']){
2
+if ($formInput['options']['value']) {
3 3
     $dataEnum = explode(';', $formInput['options']['enum']);
4 4
     $result = [];
5 5
     foreach (explode(';', $formInput['options']['value']) as $i => $v) {
Please login to merge, or discard this patch.
src/views/default/type_components/checkbox_dataenum/component.blade.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -10,9 +10,9 @@
 block discarded – undo
10 10
             foreach ($formInput['options']['enum'] as $i => $d) {
11 11
                 $options[$i]['label'] = $d;
12 12
                 if ($selects_value) {
13
-                     $options[$i]['value'] = $selects_value[$i];
13
+                        $options[$i]['value'] = $selects_value[$i];
14 14
                 } else {
15
-                     $options[$i]['value'] = $d;
15
+                        $options[$i]['value'] = $d;
16 16
                 }
17 17
             }
18 18
         endif;
Please login to merge, or discard this patch.
src/laravel-filemanager/src/middleware/MultiUser.php 2 patches
Indentation   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -9,31 +9,31 @@
 block discarded – undo
9 9
 {
10 10
     public function handle($request, Closure $next)
11 11
     {
12
-    	if (\Config::get('lfm.allow_multi_user') === true) {
13
-    		$slug = \Config::get('lfm.user_field');
12
+        if (\Config::get('lfm.allow_multi_user') === true) {
13
+            $slug = \Config::get('lfm.user_field');
14 14
 
15 15
             $new_working_dir = '/' . CRUDBooster::myId();
16 16
 
17
-	        $previous_dir = $request->input('working_dir');
17
+            $previous_dir = $request->input('working_dir');
18 18
 
19
-	        if ($previous_dir == null) {
20
-	            $request->merge(['working_dir' => $new_working_dir]);
21
-	        } elseif (! $this->validDir($previous_dir)) {
22
-	            $request->replace(['working_dir' => $new_working_dir]);
23
-	        }
24
-	    }
19
+            if ($previous_dir == null) {
20
+                $request->merge(['working_dir' => $new_working_dir]);
21
+            } elseif (! $this->validDir($previous_dir)) {
22
+                $request->replace(['working_dir' => $new_working_dir]);
23
+            }
24
+        }
25 25
 
26 26
         return $next($request);
27 27
     }
28 28
 
29 29
     private function validDir($previous_dir)
30 30
     {
31
-    	if (starts_with($previous_dir, '/' . \Config::get('lfm.shared_folder_name'))) {
32
-    		return true;
31
+        if (starts_with($previous_dir, '/' . \Config::get('lfm.shared_folder_name'))) {
32
+            return true;
33 33
         }
34 34
 
35 35
         if (starts_with($previous_dir, '/' . CRUDBooster::myId() )) {
36
-        	return true;
36
+            return true;
37 37
         }
38 38
 
39 39
         return false;
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -12,13 +12,13 @@  discard block
 block discarded – undo
12 12
     	if (\Config::get('lfm.allow_multi_user') === true) {
13 13
     		$slug = \Config::get('lfm.user_field');
14 14
 
15
-            $new_working_dir = '/' . CRUDBooster::myId();
15
+            $new_working_dir = '/'.CRUDBooster::myId();
16 16
 
17 17
 	        $previous_dir = $request->input('working_dir');
18 18
 
19 19
 	        if ($previous_dir == null) {
20 20
 	            $request->merge(['working_dir' => $new_working_dir]);
21
-	        } elseif (! $this->validDir($previous_dir)) {
21
+	        } elseif (!$this->validDir($previous_dir)) {
22 22
 	            $request->replace(['working_dir' => $new_working_dir]);
23 23
 	        }
24 24
 	    }
@@ -28,11 +28,11 @@  discard block
 block discarded – undo
28 28
 
29 29
     private function validDir($previous_dir)
30 30
     {
31
-    	if (starts_with($previous_dir, '/' . \Config::get('lfm.shared_folder_name'))) {
31
+    	if (starts_with($previous_dir, '/'.\Config::get('lfm.shared_folder_name'))) {
32 32
     		return true;
33 33
         }
34 34
 
35
-        if (starts_with($previous_dir, '/' . CRUDBooster::myId() )) {
35
+        if (starts_with($previous_dir, '/'.CRUDBooster::myId())) {
36 36
         	return true;
37 37
         }
38 38
 
Please login to merge, or discard this patch.
src/laravel-filemanager/src/config/lfm.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@
 block discarded – undo
10 10
     'use_package_routes'    => true,
11 11
 
12 12
     // For laravel 5.2, please set to ['web', 'auth']
13
-    'middlewares'           => ['web','\crocodicstudio\crudbooster\middlewares\CBBackend'],  
13
+    'middlewares'           => ['web', '\crocodicstudio\crudbooster\middlewares\CBBackend'],  
14 14
 
15 15
     // Add prefix for routes
16 16
     'prefix'                => 'laravel-filemanager',  
Please login to merge, or discard this patch.
src/laravel-filemanager/src/controllers/LfmController.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
         $working_dir = '/';
55 55
         $working_dir .= (Config::get('lfm.allow_multi_user')) ? $this->getUserSlug() : Config::get('lfm.shared_folder_name');
56 56
 
57
-        $extension_not_found = ! extension_loaded('gd') && ! extension_loaded('imagick');
57
+        $extension_not_found = !extension_loaded('gd') && !extension_loaded('imagick');
58 58
 
59 59
         return view('laravel-filemanager::index')
60 60
             ->with('working_dir', $working_dir)
@@ -85,9 +85,9 @@  discard block
 block discarded – undo
85 85
     private function formatLocation($location, $type = null, $get_thumb = false)
86 86
     {
87 87
         if ($type === 'share') {
88
-            return $location . Config::get('lfm.shared_folder_name');
88
+            return $location.Config::get('lfm.shared_folder_name');
89 89
         } elseif ($type === 'user') {
90
-            return $location . $this->getUserSlug();
90
+            return $location.$this->getUserSlug();
91 91
         }
92 92
 
93 93
         $working_dir = Input::get('working_dir');
@@ -106,10 +106,10 @@  discard block
 block discarded – undo
106 106
 
107 107
         //if user is inside thumbs folder there is no need
108 108
         // to add thumbs substring to the end of $location
109
-        $in_thumb_folder = preg_match('/'.Config::get('lfm.thumb_folder_name').'$/i',$working_dir);
109
+        $in_thumb_folder = preg_match('/'.Config::get('lfm.thumb_folder_name').'$/i', $working_dir);
110 110
 
111 111
         if ($type === 'thumb' && !$in_thumb_folder) {
112
-            $location .= Config::get('lfm.thumb_folder_name') . '/';
112
+            $location .= Config::get('lfm.thumb_folder_name').'/';
113 113
         }
114 114
 
115 115
         return $location;
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
 
130 130
     public function getPath($type = null, $get_thumb = false)
131 131
     {
132
-        $path = base_path() . '/' . $this->file_location;
132
+        $path = base_path().'/'.$this->file_location;
133 133
 
134 134
         $path = $this->formatLocation($path, $type);
135 135
 
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
 
144 144
         $url = $this->formatLocation($url, $type);
145 145
 
146
-        $url = str_replace('\\','/',$url);
146
+        $url = str_replace('\\', '/', $url);
147 147
 
148 148
         return $url;
149 149
     }
@@ -176,7 +176,7 @@  discard block
 block discarded – undo
176 176
 
177 177
         $arr_dir = explode('/', $lfm_file_path);
178 178
         $arr_filename['short'] = end($arr_dir);
179
-        $arr_filename['long'] = '/' . $lfm_file_path;
179
+        $arr_filename['long'] = '/'.$lfm_file_path;
180 180
 
181 181
         return $arr_filename;
182 182
     }
Please login to merge, or discard this patch.
src/CRUDBoosterServiceProvider.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -54,16 +54,16 @@  discard block
 block discarded – undo
54 54
             __DIR__.'/userfiles/views/vendor/crudbooster/type_components/readme.txt' => resource_path('views/vendor/crudbooster/type_components/readme.txt'),
55 55
         ], 'cb_type_components');
56 56
 
57
-        if (! file_exists(app_path('Http/Controllers/CBHook.php'))) {
57
+        if (!file_exists(app_path('Http/Controllers/CBHook.php'))) {
58 58
             $this->publishes([__DIR__.'/userfiles/controllers/CBHook.php' => app_path('Http/Controllers/CBHook.php')], 'CBHook');
59 59
         }
60 60
 
61
-        if (! file_exists(app_path('Http/Controllers/AdminUsersController.php'))) {
61
+        if (!file_exists(app_path('Http/Controllers/AdminUsersController.php'))) {
62 62
             $this->publishes([__DIR__.'/userfiles/controllers/AdminUsersController.php' => app_path('Http/Controllers/AdminUsersController.php')], 'cb_user_controller');
63 63
         }
64 64
 
65 65
         $this->defineValidationRules();
66
-        $this->loadRoutesFrom( __DIR__.'/routes.php');
66
+        $this->loadRoutesFrom(__DIR__.'/routes.php');
67 67
     }
68 68
 
69 69
     /**
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 
78 78
         $this->mergeConfigFrom(__DIR__.'/configs/crudbooster.php', 'crudbooster');
79 79
 
80
-        $this->app->singleton('crudbooster', function () {
80
+        $this->app->singleton('crudbooster', function() {
81 81
             return true;
82 82
         });
83 83
 
@@ -118,18 +118,18 @@  discard block
 block discarded – undo
118 118
 
119 119
     private function registerCrudboosterCommand()
120 120
     {
121
-        $this->app->singleton('crudboosterinstall', function () {
121
+        $this->app->singleton('crudboosterinstall', function() {
122 122
             return new CrudboosterInstallationCommand;
123 123
         });
124 124
 
125
-        $this->app->singleton('crudboosterupdate', function () {
125
+        $this->app->singleton('crudboosterupdate', function() {
126 126
             return new CrudboosterUpdateCommand;
127 127
         });
128 128
     }
129 129
 
130 130
     private function defineValidationRules()
131 131
     {
132
-        \Validator::extend('alpha_spaces', function ($attribute, $value) {
132
+        \Validator::extend('alpha_spaces', function($attribute, $value) {
133 133
             // This will only accept alpha and spaces.
134 134
             // If you want to accept hyphens use: /^[\pL\s-]+$/u.
135 135
             return preg_match('/^[\pL\s]+$/u', $value);
Please login to merge, or discard this patch.
src/routes.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
 
5 5
 $namespace = '\crocodicstudio\crudbooster\controllers';
6 6
 /* ROUTER FOR UPLOADS */
7
-Route::group(['middleware' => ['web'], 'namespace' => $namespace], function () {
7
+Route::group(['middleware' => ['web'], 'namespace' => $namespace], function() {
8 8
     Route::get('uploads/{one?}/{two?}/{three?}/{four?}/{five?}', ['uses' => 'FileController@getPreview', 'as' => 'fileControllerPreview']);
9 9
 });
10 10
 
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
13 13
     'middleware' => ['web', CBBackend::class],
14 14
     'prefix' => cbAdminPath(),
15 15
     'namespace' => ctrlNamespace(),
16
-], function () {
16
+], function() {
17 17
     CRUDBooster::routeController('users', 'AdminUsersController');
18 18
     try {
19 19
         $modules = DB::table('cms_moduls')->where('path', '!=', '')->where('controller', '!=', '')->where('is_protected', 0)->get();
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
 
27 27
 /* ROUTER FOR BACKEND CRUDBOOSTER */
28 28
 Route::group([
29
-    'middleware' => ['web', CBBackend::class], 'prefix' => cbAdminPath(), 'namespace' => $namespace,], function () use ($namespace){
29
+    'middleware' => ['web', CBBackend::class], 'prefix' => cbAdminPath(), 'namespace' => $namespace, ], function() use ($namespace){
30 30
     CRUDBooster::routeController('notifications', 'AdminNotificationsController', $namespace);
31 31
 });
32 32
 
@@ -34,10 +34,10 @@  discard block
 block discarded – undo
34 34
     'middleware' => ['web', \crocodicstudio\crudbooster\middlewares\CBSuperadmin::class],
35 35
     'prefix' => cbAdminPath(),
36 36
     'namespace' => $namespace,
37
-], function () {
38
-    Route::post('{module}/do-upload-import-data', ['uses' => 'FileController@uploadImportData', 'as' => 'UploadImportData',]);
39
-    Route::post('{module}/upload-summernote', ['uses' => 'FileController@uploadSummernote', 'as' => 'UploadImportData',]);
40
-    Route::post('{module}/upload-file', ['uses' => 'FileController@uploadFile', 'as' => 'UploadImportData',]);
41
-    Route::post('{module}/done-import', ['uses' => 'FileController@doneImport', 'as' => 'doneImportData',]);
37
+], function() {
38
+    Route::post('{module}/do-upload-import-data', ['uses' => 'FileController@uploadImportData', 'as' => 'UploadImportData', ]);
39
+    Route::post('{module}/upload-summernote', ['uses' => 'FileController@uploadSummernote', 'as' => 'UploadImportData', ]);
40
+    Route::post('{module}/upload-file', ['uses' => 'FileController@uploadFile', 'as' => 'UploadImportData', ]);
41
+    Route::post('{module}/done-import', ['uses' => 'FileController@doneImport', 'as' => 'doneImportData', ]);
42 42
 });
43 43
 
Please login to merge, or discard this patch.
src/controllers/ApiController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@
 block discarded – undo
48 48
      */
49 49
     private function handleFile($name, $row_assign)
50 50
     {
51
-        if (! Request::hasFile($name)) {
51
+        if (!Request::hasFile($name)) {
52 52
             return;
53 53
         }
54 54
         $file = Request::file($name);
Please login to merge, or discard this patch.