@@ -37,8 +37,7 @@ discard block |
||
37 | 37 | $userLinks = new UserCollection( |
38 | 38 | User::withCount([ |
39 | 39 | 'FileLinks', |
40 | - 'FileLinks as expired_file_links_count' => function($query) |
|
41 | - { |
|
40 | + 'FileLinks as expired_file_links_count' => function($query) { |
|
42 | 41 | $query->where('expire', '<', Carbon::now()); |
43 | 42 | } |
44 | 43 | ]) |
@@ -84,13 +83,15 @@ discard block |
||
84 | 83 | |
85 | 84 | // Determine if the password expires field is updated |
86 | 85 | $oldExpire = config('users.passExpires'); |
87 | - if ($request->passExpire != $oldExpire) { |
|
86 | + if ($request->passExpire != $oldExpire) |
|
87 | + { |
|
88 | 88 | // Update the setting in the database |
89 | 89 | Settings::where('key', 'users.passExpires')->update([ |
90 | 90 | 'value' => $request->passExpire |
91 | 91 | ]); |
92 | 92 | // If the setting is changing from never to xx days, update all users |
93 | - if ($request->passExpire == 0) { |
|
93 | + if ($request->passExpire == 0) |
|
94 | + { |
|
94 | 95 | User::whereNotNull('password_expires')->update([ |
95 | 96 | 'password_expires' => null |
96 | 97 | ]); |
@@ -113,8 +114,7 @@ discard block |
||
113 | 114 | public function roleSettings() |
114 | 115 | { |
115 | 116 | $this->authorize('hasAccess', 'Manage User Roles'); |
116 | - $roles = UserRoleType::with(['UserRolePermissions' => function($query) |
|
117 | - { |
|
117 | + $roles = UserRoleType::with(['UserRolePermissions' => function($query) { |
|
118 | 118 | $query->join('user_role_permission_types', 'user_role_permission_types.perm_type_id', '=', 'user_role_permissions.perm_type_id'); |
119 | 119 | }])->get(); |
120 | 120 | $perms = UserRolePermissionTypes::all(); |
@@ -181,12 +181,18 @@ discard block |
||
181 | 181 | |
182 | 182 | // Good to go - update user password |
183 | 183 | $roleArr = []; |
184 | - foreach ($roles as $role) { |
|
185 | - if ($role->role_id == 1 && Auth::user()->role_id != 1) { |
|
184 | + foreach ($roles as $role) |
|
185 | + { |
|
186 | + if ($role->role_id == 1 && Auth::user()->role_id != 1) |
|
187 | + { |
|
186 | 188 | continue; |
187 | - } else if ($role->role_id == 2 && Auth::user()->role_id > 1) { |
|
189 | + } |
|
190 | + else if ($role->role_id == 2 && Auth::user()->role_id > 1) |
|
191 | + { |
|
188 | 192 | continue; |
189 | - } else { |
|
193 | + } |
|
194 | + else |
|
195 | + { |
|
190 | 196 | // $roleArr[$role->role_id] = $role->name; |
191 | 197 | $roleArr[] = [ |
192 | 198 | 'value' => $role->role_id, |
@@ -275,17 +281,20 @@ discard block |
||
275 | 281 | $user = User::find($request->user_id); |
276 | 282 | |
277 | 283 | // Verify this is a valid user ID |
278 | - if (!$user) { |
|
284 | + if (!$user) |
|
285 | + { |
|
279 | 286 | $success = false; |
280 | 287 | $reason = 'Cannot find user with this ID'; |
281 | 288 | } |
282 | 289 | // Make sure that the user is not trying to deactivate someone with more permissions |
283 | - else if ($user->role_id < Auth::user()->role_id) { |
|
290 | + else if ($user->role_id < Auth::user()->role_id) |
|
291 | + { |
|
284 | 292 | $success = false; |
285 | 293 | $reason = 'You cannot change password for a user with higher permissions that you. If this user has locked themselves out, have then use the reset link on the login page.'; |
286 | 294 | } |
287 | 295 | // Good to go - update user password |
288 | - else { |
|
296 | + else |
|
297 | + { |
|
289 | 298 | // Update the user data |
290 | 299 | $user->update( |
291 | 300 | [ |
@@ -117,7 +117,8 @@ |
||
117 | 117 | */ |
118 | 118 | public function destroy($id) |
119 | 119 | { |
120 | - try { |
|
120 | + try |
|
121 | + { |
|
121 | 122 | SystemCategories::find($id)->delete(); |
122 | 123 | return response()->json(['success' => true, 'reason' => 'Category Successfully Deleted']); |
123 | 124 | } |
@@ -51,14 +51,17 @@ discard block |
||
51 | 51 | Log::debug('request Data -> ', $request->toArray()); |
52 | 52 | |
53 | 53 | // See if there are any search paramaters entered |
54 | - if (!$request->search['searchText'] && !isset($request->search['articleType']) && !isset($request->search['systemType'])) { |
|
54 | + if (!$request->search['searchText'] && !isset($request->search['articleType']) && !isset($request->search['systemType'])) |
|
55 | + { |
|
55 | 56 | // No search paramaters, send all tech tips |
56 | 57 | $tips = new TechTipsCollection( |
57 | 58 | TechTips::orderBy('created_at', 'DESC') |
58 | 59 | ->with('SystemTypes') |
59 | 60 | ->paginate($request->pagination['perPage']) |
60 | 61 | ); |
61 | - } else { |
|
62 | + } |
|
63 | + else |
|
64 | + { |
|
62 | 65 | $article = isset($request->search['articleType']) ? true : false; |
63 | 66 | $system = isset($request->search['systemType']) ? true : false; |
64 | 67 | // Search paramaters, filter results |
@@ -146,7 +149,8 @@ discard block |
||
146 | 149 | $save = $receiver->receive(); |
147 | 150 | |
148 | 151 | // See if the uploade has finished |
149 | - if ($save->isFinished()) { |
|
152 | + if ($save->isFinished()) |
|
153 | + { |
|
150 | 154 | $this->saveFile($save->getFile()); |
151 | 155 | |
152 | 156 | return 'uploaded successfully'; |
@@ -248,8 +252,7 @@ discard block |
||
248 | 252 | if(!$tipData->supressEmail) |
249 | 253 | { |
250 | 254 | $details = TechTips::find($tipID); |
251 | - $users = User::whereHas('UserSettings', function($query) |
|
252 | - { |
|
255 | + $users = User::whereHas('UserSettings', function($query) { |
|
253 | 256 | $query->where('em_tech_tip', 1); |
254 | 257 | })->get(); |
255 | 258 | |
@@ -263,7 +266,8 @@ discard block |
||
263 | 266 | // Details controller - will move to the show controller with just the tech tip id |
264 | 267 | public function details($id, $subject) |
265 | 268 | { |
266 | - if (session()->has('newTipFile')) { |
|
269 | + if (session()->has('newTipFile')) |
|
270 | + { |
|
267 | 271 | session()->forget('newTipFile'); |
268 | 272 | } |
269 | 273 | |
@@ -293,7 +297,8 @@ discard block |
||
293 | 297 | // Add or remove this tip as a favorite of the user |
294 | 298 | public function toggleFav($action, $id) |
295 | 299 | { |
296 | - switch ($action) { |
|
300 | + switch ($action) |
|
301 | + { |
|
297 | 302 | case 'add': |
298 | 303 | TechTipFavs::create([ |
299 | 304 | 'user_id' => Auth::user()->user_id, |
@@ -321,7 +326,8 @@ discard block |
||
321 | 326 | $this->authorize('hasAccess', 'Edit Tech Tip'); |
322 | 327 | $tipData = TechTips::where('tip_id', $id)->with('User')->with('SystemTypes')->with('TechTipTypes')->first(); |
323 | 328 | |
324 | - if (!$tipData) { |
|
329 | + if (!$tipData) |
|
330 | + { |
|
325 | 331 | return view('tips.tipNotFound'); |
326 | 332 | } |
327 | 333 | |
@@ -353,7 +359,8 @@ discard block |
||
353 | 359 | $receiver = new FileReceiver('file', $request, HandlerFactory::classFromRequest($request)); |
354 | 360 | |
355 | 361 | // Verify if there is a file to be processed or not |
356 | - if ($receiver->isUploaded() === false || $request->_completed) { |
|
362 | + if ($receiver->isUploaded() === false || $request->_completed) |
|
363 | + { |
|
357 | 364 | $this->storeUpdatedTip($request, $id); |
358 | 365 | Log::debug('Route ' . Route::currentRouteName() . ' visited by User ID-' . Auth::user()->user_id); |
359 | 366 | return response()->json(['tip_id' => $id]); |
@@ -363,7 +370,8 @@ discard block |
||
363 | 370 | $save = $receiver->receive(); |
364 | 371 | |
365 | 372 | // See if the uploade has finished |
366 | - if ($save->isFinished()) { |
|
373 | + if ($save->isFinished()) |
|
374 | + { |
|
367 | 375 | $this->saveFile($save->getFile(), $id); |
368 | 376 | |
369 | 377 | return 'uploaded successfully'; |
@@ -82,7 +82,8 @@ discard block |
||
82 | 82 | // Added the ability to set an expiration date for user passwords - will force them to change after this expires |
83 | 83 | private function addPasswordExpiresColumn() |
84 | 84 | { |
85 | - if(!Schema::hasColumn('users', 'password_expires')) { |
|
85 | + if(!Schema::hasColumn('users', 'password_expires')) |
|
86 | + { |
|
86 | 87 | Schema::table('users', function(Blueprint $table) { |
87 | 88 | $table->timestamp('password_expires') |
88 | 89 | ->nullable() |
@@ -118,7 +119,8 @@ discard block |
||
118 | 119 | // Added a 'hidden' attribute to system customer data types to allow passwords to not be viewed unless clicked or focus |
119 | 120 | private function addHiddenColumn() |
120 | 121 | { |
121 | - if(!Schema::hasColumn('system_data_field_types', 'hidden')) { |
|
122 | + if(!Schema::hasColumn('system_data_field_types', 'hidden')) |
|
123 | + { |
|
122 | 124 | Schema::table('system_cust_data_types', function(Blueprint $table) { |
123 | 125 | $table->boolean('hidden') |
124 | 126 | ->default(0) |
@@ -130,7 +132,8 @@ discard block |
||
130 | 132 | // Update the File links table - add cust_id and note column |
131 | 133 | private function addColumnsToFileLinksTable() |
132 | 134 | { |
133 | - if(!Schema::hasColumn('file_links', 'cust_id')) { |
|
135 | + if(!Schema::hasColumn('file_links', 'cust_id')) |
|
136 | + { |
|
134 | 137 | Schema::table('file_links', function(Blueprint $table) { |
135 | 138 | $table->integer('cust_id') |
136 | 139 | ->unsigned() |
@@ -139,7 +142,8 @@ discard block |
||
139 | 142 | $table->foreign('cust_id')->references('cust_id')->on('customers')->onUpdate('cascade')->onDelete('cascade'); |
140 | 143 | }); |
141 | 144 | } |
142 | - if(!Schema::hasColumn('file_links', 'note')) { |
|
145 | + if(!Schema::hasColumn('file_links', 'note')) |
|
146 | + { |
|
143 | 147 | Schema::table('file_links', function(Blueprint $table) { |
144 | 148 | $table->longText('note') |
145 | 149 | ->nullable() |
@@ -147,7 +151,8 @@ discard block |
||
147 | 151 | }); |
148 | 152 | // Migrate the instructions from the old table to the new column |
149 | 153 | $instructions = DB::select('SELECT * FROM `file_link_instructions`'); |
150 | - foreach($instructions as $ins) { |
|
154 | + foreach($instructions as $ins) |
|
155 | + { |
|
151 | 156 | FileLinks::find($ins->link_id)->update([ |
152 | 157 | 'note' => $ins->instruction |
153 | 158 | ]); |
@@ -158,7 +163,8 @@ discard block |
||
158 | 163 | // Add Notes column to the file links files table |
159 | 164 | private function addNotesColumnToFileLinkFiles() |
160 | 165 | { |
161 | - if(!Schema::hasColumn('file_link_files', 'note')) { |
|
166 | + if(!Schema::hasColumn('file_link_files', 'note')) |
|
167 | + { |
|
162 | 168 | Schema::table('file_link_files', function(Blueprint $table) { |
163 | 169 | $table->longText('note') |
164 | 170 | ->nullable() |
@@ -166,7 +172,8 @@ discard block |
||
166 | 172 | }); |
167 | 173 | // Migrate the existing notes to the new table |
168 | 174 | $notes = DB::select('SELECT * FROM `file_link_notes`'); |
169 | - foreach($notes as $note) { |
|
175 | + foreach($notes as $note) |
|
176 | + { |
|
170 | 177 | FileLinkFiles::where('file_id', $note->file_id)->update([ |
171 | 178 | 'note' => $note->note |
172 | 179 | ]); |
@@ -177,7 +184,8 @@ discard block |
||
177 | 184 | // Add the documentation column to the tech tips table |
178 | 185 | private function migrateSystemDocumentation() |
179 | 186 | { |
180 | - if(!Schema::hasColumn('tech_tips', 'tip_type_id')) { |
|
187 | + if(!Schema::hasColumn('tech_tips', 'tip_type_id')) |
|
188 | + { |
|
181 | 189 | Schema::table('tech_tips', function(Blueprint $table) { |
182 | 190 | $table->bigInteger('tip_type_id')->unsigned()->after('public')->default(1); |
183 | 191 | $table->foreign('tip_type_id')->references('tip_type_id')->on('tech_tip_types')->onUpdate('cascade'); |
@@ -185,7 +193,8 @@ discard block |
||
185 | 193 | |
186 | 194 | // Move all of the system files over to the tech tips table |
187 | 195 | $sysFiles = DB::select('SELECT * FROM `system_files`'); |
188 | - foreach($sysFiles as $sysFile) { |
|
196 | + foreach($sysFiles as $sysFile) |
|
197 | + { |
|
189 | 198 | $newTip = TechTips::create([ |
190 | 199 | 'user_id' => $sysFile->user_id, |
191 | 200 | 'public' => 0, |
@@ -279,7 +288,8 @@ discard block |
||
279 | 288 | // Remove the system files and system file types table |
280 | 289 | private function removeSystemFilesTables() |
281 | 290 | { |
282 | - if(Schema::hasTable('system_files')) { |
|
291 | + if(Schema::hasTable('system_files')) |
|
292 | + { |
|
283 | 293 | Schema::table('system_files', function(Blueprint $table) { |
284 | 294 | $table->dropForeign(['sys_id']); |
285 | 295 | $table->dropForeign(['type_id']); |
@@ -325,7 +335,8 @@ discard block |
||
325 | 335 | // Add soft deletes to tech tips table to prevent accidental deletes |
326 | 336 | private function addSoftDeleteToTechTips() |
327 | 337 | { |
328 | - if (!Schema::hasColumn('tech_tips', 'deleted_at')) { |
|
338 | + if (!Schema::hasColumn('tech_tips', 'deleted_at')) |
|
339 | + { |
|
329 | 340 | Schema::table('tech_tips', function (Blueprint $table) { |
330 | 341 | $table->softDeletes()->after('description'); |
331 | 342 | }); |
@@ -337,8 +348,7 @@ discard block |
||
337 | 348 | { |
338 | 349 | if(!Schema::hasColumn('users', 'deleted_at')) |
339 | 350 | { |
340 | - Schema::table('users', function(Blueprint $table) |
|
341 | - { |
|
351 | + Schema::table('users', function(Blueprint $table) { |
|
342 | 352 | $table->softDeletes()->after('password_expires'); |
343 | 353 | }); |
344 | 354 | // Migrate over all deactivated users |