Passed
Pull Request — master (#1102)
by Iman
06:57
created
src/controllers/FormValidator.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
 
20 20
         $validator = Validator::make(request()->all(), $rules);
21 21
 
22
-        if (! $validator->fails()) {
22
+        if (!$validator->fails()) {
23 23
             return null;
24 24
         }
25 25
 
@@ -37,12 +37,12 @@  discard block
 block discarded – undo
37 37
         $rules = [];
38 38
         foreach ($form as $formInput) {
39 39
             $name = $formInput['name'];
40
-            if (! $name) {
40
+            if (!$name) {
41 41
                 continue;
42 42
             }
43 43
 
44 44
             $ai = [];
45
-            if ($formInput['required'] && ! Request::hasFile($name)) {
45
+            if ($formInput['required'] && !Request::hasFile($name)) {
46 46
                 $ai[] = 'required';
47 47
             }
48 48
 
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
     {
72 72
         $exp = explode('|', $formInput['validation']);
73 73
 
74
-        if (! count($exp)) {
74
+        if (!count($exp)) {
75 75
             return '';
76 76
         }
77 77
 
Please login to merge, or discard this patch.
src/CRUDBoosterServiceProvider.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -53,16 +53,16 @@  discard block
 block discarded – undo
53 53
             __DIR__.'/userfiles/views/vendor/crudbooster/type_components/readme.txt' => resource_path('views/vendor/crudbooster/type_components/readme.txt'),
54 54
         ], 'cb_type_components');
55 55
 
56
-        if (! file_exists(app_path('Http/Controllers/CBHook.php'))) {
56
+        if (!file_exists(app_path('Http/Controllers/CBHook.php'))) {
57 57
             $this->publishes([__DIR__.'/userfiles/controllers/CBHook.php' => app_path('Http/Controllers/CBHook.php')], 'CBHook');
58 58
         }
59 59
 
60
-        if (! file_exists(app_path('Http/Controllers/AdminUsersController.php'))) {
60
+        if (!file_exists(app_path('Http/Controllers/AdminUsersController.php'))) {
61 61
             $this->publishes([__DIR__.'/userfiles/controllers/AdminUsersController.php' => app_path('Http/Controllers/AdminUsersController.php')], 'cb_user_controller');
62 62
         }
63 63
 
64 64
         $this->defineValidationRules();
65
-        $this->loadRoutesFrom( __DIR__.'/routes.php');
65
+        $this->loadRoutesFrom(__DIR__.'/routes.php');
66 66
     }
67 67
 
68 68
     /**
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
     {
75 75
         $this->mergeConfigFrom(__DIR__.'/configs/crudbooster.php', 'crudbooster');
76 76
 
77
-        $this->app->singleton('crudbooster', function () {
77
+        $this->app->singleton('crudbooster', function() {
78 78
             return true;
79 79
         });
80 80
 
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
 
116 116
     private function registerCrudboosterCommand()
117 117
     {
118
-        $this->app->singleton('crudboosterinstall', function () {
118
+        $this->app->singleton('crudboosterinstall', function() {
119 119
             return new CrudboosterInstallationCommand;
120 120
         });
121 121
 
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
 
125 125
     private function defineValidationRules()
126 126
     {
127
-        \Validator::extend('alpha_spaces', function ($attribute, $value) {
127
+        \Validator::extend('alpha_spaces', function($attribute, $value) {
128 128
             // This will only accept alpha and spaces.
129 129
             // If you want to accept hyphens use: /^[\pL\s-]+$/u.
130 130
             return preg_match('/^[\pL\s]+$/u', $value);
Please login to merge, or discard this patch.
src/helpers/Time.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@
 block discarded – undo
31 31
             }
32 32
         }
33 33
 
34
-        if (! $full) {
34
+        if (!$full) {
35 35
             $this->string = array_slice($this->string, 0, 1);
36 36
         }
37 37
 
Please login to merge, or discard this patch.
src/helpers/CRUDBooster.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
     public static function first($table, $id)
175 175
     {
176 176
         $table = self::parseSqlTable($table)['table'];
177
-        if (! is_array($id)) {
177
+        if (!is_array($id)) {
178 178
             $pk = self::pk($table);
179 179
 
180 180
             return DB::table($table)->where($pk, $id)->first();
@@ -302,7 +302,7 @@  discard block
 block discarded – undo
302 302
         $controllerName = basename($controllerName);
303 303
         $route_url = route($controllerName.'GetIndex');
304 304
 
305
-        if (! $path) {
305
+        if (!$path) {
306 306
             return trim($route_url, '/');
307 307
         }
308 308
 
@@ -362,7 +362,7 @@  discard block
 block discarded – undo
362 362
         @$get = $_GET;
363 363
         $inputhtml = '';
364 364
 
365
-        if (! $get) {
365
+        if (!$get) {
366 366
             return $inputhtml;
367 367
         }
368 368
         if (is_array($exception)) {
Please login to merge, or discard this patch.
src/Modules/ModuleGenerator/Step2Handler.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@  discard block
 block discarded – undo
8 8
 class Step2Handler
9 9
 {
10 10
     private $hooks = ['hook_query_index', 'hook_row_index', 'hook_before_add', 'hook_after_add',
11
-        'hook_before_edit', 'hook_after_edit', 'hook_before_delete', 'hook_after_delete',];
11
+        'hook_before_edit', 'hook_after_edit', 'hook_before_delete', 'hook_after_delete', ];
12 12
 
13 13
     public function showForm($id)
14 14
     {
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
         $data['cols'] = ScaffoldingParser::parse($controllerCode, 'col');
26 26
 
27 27
 
28
-        foreach($this->hooks as $hook){
28
+        foreach ($this->hooks as $hook) {
29 29
             $data[$hook] = FileManipulator::readMethodContent($controllerCode, $hook);
30 30
         }
31 31
 
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
         $code = FileManipulator::readCtrlContent($controller);
42 42
         $fileResult = FileManipulator::replaceBetweenMark($code, 'COLUMNS', $newCode);
43 43
 
44
-        foreach($this->hooks as $hook){
44
+        foreach ($this->hooks as $hook) {
45 45
             $fileResult = FileManipulator::writeMethodContent($fileResult, $hook, request($hook));
46 46
         }
47 47
 
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
         $columnScript[] = '            $this->col[] = [];';
66 66
         foreach ($labels as $i => $label) {
67 67
 
68
-            if (! $name[$i]) {
68
+            if (!$name[$i]) {
69 69
                 continue;
70 70
             }
71 71
 
Please login to merge, or discard this patch.
src/Modules/PrivilegeModule/PrivilegeHelpers.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
     public static function myPrivilege()
75 75
     {
76 76
         $roles = session('admin_privileges_roles');
77
-        if (! $roles) {
77
+        if (!$roles) {
78 78
             return;
79 79
         }
80 80
         foreach ($roles as $role) {
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
 
87 87
     private static function getModulePath()
88 88
     {
89
-        $adminPathSegments = count(explode('/',cbConfig('ADMIN_PATH')));
89
+        $adminPathSegments = count(explode('/', cbConfig('ADMIN_PATH')));
90 90
         return Request::segment(1 + $adminPathSegments);
91 91
     }
92 92
 
Please login to merge, or discard this patch.
src/controllers/CBController/Deleter.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -23,12 +23,12 @@  discard block
 block discarded – undo
23 23
         $this->cbLoader();
24 24
         $selectedIds = request('checkbox');
25 25
         $btnName = request('button_name');
26
-        if (! $selectedIds) {
26
+        if (!$selectedIds) {
27 27
             CB::redirect($_SERVER['HTTP_REFERER'], 'Please select at least one row!', 'warning');
28 28
         }
29 29
         if ($btnName == 'delete') {
30 30
             (new DataRemover($this))->doDeleteWithHook($selectedIds);
31
-            $this->insertLog('log_delete', implode(',', $selectedIds) . ' - '. $this->table);
31
+            $this->insertLog('log_delete', implode(',', $selectedIds).' - '.$this->table);
32 32
             backWithMsg(cbTrans('alert_delete_selected_success'));
33 33
         }
34 34
 
@@ -47,8 +47,8 @@  discard block
 block discarded – undo
47 47
         $message = cbTrans('alert_action', ['action' => $action]);
48 48
 
49 49
         if ($this->actionButtonSelected($id_selected, $button_name) === false) {
50
-            $message = ! empty($this->alert['message']) ? $this->alert['message'] : 'Error';
51
-            $type = ! empty($this->alert['type']) ? $this->alert['type'] : 'danger';
50
+            $message = !empty($this->alert['message']) ? $this->alert['message'] : 'Error';
51
+            $type = !empty($this->alert['type']) ? $this->alert['type'] : 'danger';
52 52
         }
53 53
 
54 54
         backWithMsg($message, $type);
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
 
68 68
         $this->findRow($id)->update([$column => null]);
69 69
 
70
-        $this->insertLog('log_delete_image', $id . ' - '. $this->table);
70
+        $this->insertLog('log_delete_image', $id.' - '.$this->table);
71 71
 
72 72
         CB::redirect(Request::server('HTTP_REFERER'), cbTrans('alert_delete_data_success'), 'success');
73 73
     }
Please login to merge, or discard this patch.
src/helpers/cb_helpers.php 1 patch
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -8,28 +8,28 @@  discard block
 block discarded – undo
8 8
 | ---------------------------------------------------------------------------------------------------------------
9 9
 |
10 10
 */
11
-if (! function_exists('cbModulesNS')) {
11
+if (!function_exists('cbModulesNS')) {
12 12
     function cbModulesNS($path = '')
13 13
     {
14 14
         return '\crocodicstudio\crudbooster\Modules\\'.$path;
15 15
     }
16 16
 }
17 17
 
18
-if (! function_exists('cbAdminPath')) {
18
+if (!function_exists('cbAdminPath')) {
19 19
     function cbAdminPath()
20 20
     {
21 21
         return cbConfig('ADMIN_PATH');
22 22
     }
23 23
 }
24 24
 
25
-if (! function_exists('ctrlNamespace')) {
25
+if (!function_exists('ctrlNamespace')) {
26 26
     function ctrlNamespace()
27 27
     {
28 28
         return 'App\Http\Controllers';
29 29
     }
30 30
 }
31 31
 
32
-if (! function_exists('is_checked')) {
32
+if (!function_exists('is_checked')) {
33 33
     /**
34 34
      * @param $format
35 35
      * @param $value
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
     }
54 54
 }
55 55
 
56
-if (! function_exists('controllers_dir')) {
56
+if (!function_exists('controllers_dir')) {
57 57
     function controllers_dir()
58 58
     {
59 59
         $_ = DIRECTORY_SEPARATOR;
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
     }
62 62
 }
63 63
 
64
-if (! function_exists('controller_path')) {
64
+if (!function_exists('controller_path')) {
65 65
     function controller_path($controller)
66 66
     {
67 67
         $_ = DIRECTORY_SEPARATOR;
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
     }
70 70
 }
71 71
 
72
-if (! function_exists('extract_unit')) {
72
+if (!function_exists('extract_unit')) {
73 73
     /*
74 74
     Credits: Bit Repository
75 75
     URL: http://www.bitrepository.com/extract-content-between-two-delimiters-with-php.html
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
     }
88 88
 }
89 89
 
90
-if (! function_exists('now')) {
90
+if (!function_exists('now')) {
91 91
     function now()
92 92
     {
93 93
         return date('Y-m-d H:i:s');
@@ -101,17 +101,17 @@  discard block
 block discarded – undo
101 101
 | $name = name of input
102 102
 |
103 103
 */
104
-if (! function_exists('g')) {
104
+if (!function_exists('g')) {
105 105
     function g($name)
106 106
     {
107 107
         return Request::get($name);
108 108
     }
109 109
 }
110 110
 
111
-if (! function_exists('min_var_export')) {
111
+if (!function_exists('min_var_export')) {
112 112
     function min_var_export($input, $indent = "")
113 113
     {
114
-        if (! is_array($input)) {
114
+        if (!is_array($input)) {
115 115
             return var_export($input, true);
116 116
         }
117 117
         $buffer = [];
@@ -126,41 +126,41 @@  discard block
 block discarded – undo
126 126
     }
127 127
 }
128 128
 
129
-if (! function_exists('cbTrans')) {
129
+if (!function_exists('cbTrans')) {
130 130
     function cbTrans($key, $params = [])
131 131
     {
132 132
         return trans('crudbooster.'.$key, $params);
133 133
     }
134 134
 }
135 135
 
136
-if (! function_exists('cbAsset')) {
136
+if (!function_exists('cbAsset')) {
137 137
     function cbAsset($key)
138 138
     {
139 139
         return asset('vendor/crudbooster/assets/'.$key);
140 140
     }
141 141
 }
142 142
 
143
-if (! function_exists('cbScript')) {
143
+if (!function_exists('cbScript')) {
144 144
     function cbScript($key)
145 145
     {
146 146
         return '<script src="'.cbAsset($key).'" type="text/javascript"></script>';
147 147
     }
148 148
 }
149 149
 
150
-if (! function_exists('cbStyleSheet')) {
150
+if (!function_exists('cbStyleSheet')) {
151 151
     function cbStyleSheet($key)
152 152
     {
153 153
         return '<link rel="stylesheet" type="text/css" href="'.cbAsset($key).'"/>';
154 154
     }
155 155
 }
156 156
 
157
-if (! function_exists('cbConfig')) {
157
+if (!function_exists('cbConfig')) {
158 158
     function cbConfig($key, $default = null)
159 159
     {
160 160
         return config('crudbooster.'.$key, $default);
161 161
     }
162 162
 }
163
-if (! function_exists('makeValidationForHTML')) {
163
+if (!function_exists('makeValidationForHTML')) {
164 164
     function makeValidationForHTML($rules)
165 165
     {
166 166
         $validation = [];
@@ -178,7 +178,7 @@  discard block
 block discarded – undo
178 178
     }
179 179
 }
180 180
 
181
-if (! function_exists('findSelected')) {
181
+if (!function_exists('findSelected')) {
182 182
     /**
183 183
      * @param $rawvalue
184 184
      * @param $form
@@ -187,7 +187,7 @@  discard block
 block discarded – undo
187 187
      */
188 188
     function findSelected($rawvalue, $form, $optionValue)
189 189
     {
190
-        if (! $rawvalue) {
190
+        if (!$rawvalue) {
191 191
             return '';
192 192
         }
193 193
         $value = $rawvalue;
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
         return $selected;
210 210
     }
211 211
 }
212
-if (! function_exists('array_get_keys')) {
212
+if (!function_exists('array_get_keys')) {
213 213
 
214 214
     /**
215 215
      * @param array $_array
@@ -225,24 +225,24 @@  discard block
 block discarded – undo
225 225
     }
226 226
 }
227 227
 
228
-if (! function_exists('cbGetSetting')) {
228
+if (!function_exists('cbGetSetting')) {
229 229
     function cbGetSetting($name)
230 230
     {
231 231
         return \crocodicstudio\crudbooster\Modules\SettingModule\SettingRepo::getSetting($name);
232 232
     }
233 233
 }
234 234
 
235
-if (! function_exists('backWithMsg')) {
235
+if (!function_exists('backWithMsg')) {
236 236
     function backWithMsg($msg, $type = 'success')
237 237
     {
238 238
         sendAndTerminate(redirect()->back()->with(['message_type' => $type, 'message' => $msg]));
239 239
     }
240 240
 }
241 241
 
242
-if (! function_exists('underField')) {
242
+if (!function_exists('underField')) {
243 243
     function underField($help, $error)
244 244
     {
245
-        $error = $error ? "<i class='fa fa-info-circle'></i> $error":'' ;
245
+        $error = $error ? "<i class='fa fa-info-circle'></i> $error" : '';
246 246
         return "<div class='text-danger'>$error</div><p class='help-block'>$help</p>";
247 247
     }
248 248
 }
Please login to merge, or discard this patch.
src/helpers/DbInspector.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
      */
17 17
     public static function findPK($table)
18 18
     {
19
-        if (! $table) {
19
+        if (!$table) {
20 20
             return 'id';
21 21
         }
22 22
 
@@ -25,13 +25,13 @@  discard block
 block discarded – undo
25 25
         }
26 26
         $table = CRUDBooster::parseSqlTable($table);
27 27
 
28
-        if (! $table['table']) {
28
+        if (!$table['table']) {
29 29
             throw new \Exception("parseSqlTable can't determine the table");
30 30
         }
31 31
 
32 32
         $primary_key = self::findPKname($table);
33 33
 
34
-        if (! $primary_key) {
34
+        if (!$primary_key) {
35 35
             return 'id';
36 36
         }
37 37
         CbCache::put('table_'.$table, 'primary_key', $primary_key);
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
         $cols = collect(DB::select('SELECT * FROM information_schema.COLUMNS WHERE TABLE_SCHEMA = :database AND TABLE_NAME = :table', [
102 102
             'database' => $table['database'],
103 103
             'table' => $table['table'],
104
-        ]))->map(function ($x) {
104
+        ]))->map(function($x) {
105 105
             return (array) $x;
106 106
         })->toArray();
107 107
 
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
     {
118 118
         $field = 'field_type_'.$table.'_'.$field;
119 119
 
120
-        return Cache::rememberForever($field, function () use ($table, $field) {
120
+        return Cache::rememberForever($field, function() use ($table, $field) {
121 121
             try {
122 122
                 //MySQL & SQL Server
123 123
                 $typedata = DB::select(DB::raw("select DATA_TYPE from INFORMATION_SCHEMA.COLUMNS where TABLE_NAME='$table' and COLUMN_NAME = '$field'"))[0]->DATA_TYPE;
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
         }
143 143
 
144 144
         $table = CRUDBooster::getTableForeignKey($fieldName);
145
-        if (! $table) {
145
+        if (!$table) {
146 146
             return false;
147 147
         }
148 148
 
Please login to merge, or discard this patch.