butcherman / Tech_Bench

app/Http/Controllers/Auth/InitializeUserController.php

Braces +4 added lines, -2 removed lines

@@ -25,7 +25,8 @@ 
         //  Validate the hash token
         $user = UserInitialize::where('token', $hash)->get();
 
-        if($user->isEmpty()) {
+        if($user->isEmpty())
+        {
             Log::warning('Visitor at IP Address '.\Request::ip().' tried to access invalid initialize hash - '.$hash);
             return abort(404);
         }
@@ -43,7 +44,8 @@ 
 
         //  Verify that the link matches the assigned email address
         $valid = UserInitialize::where('token', $hash)->first();
-        if(empty($valid)) {
+        if(empty($valid))
+        {
             Log::warning('Visitor at IP Address '.\Request::ip().' tried to submit an invalid User Initialization link - '.$hash);
             return abort(404);
         }

app/Http/Controllers/Installer/ModuleController.php

Braces +6 added lines, -3 removed lines

@@ -41,7 +41,8 @@ 
             ];
         }
         $mods = Module::allDisabled();
-        foreach($mods as $name => $mod) {
+        foreach($mods as $name => $mod)
+        {
             $moduleList[] = [
                 'name' => $name,
                 'status' => 'Disabled',
@@ -78,7 +79,8 @@ 
         $receiver = new FileReceiver('file', $request, HandlerFactory::classFromRequest($request));
 
         //  Verify that the upload is valid and being processed
-        if($receiver->isUploaded() === false) {
+        if($receiver->isUploaded() === false)
+        {
             Log::error('Upload File Missing - '.
                 /** @scrutinizer ignore-type */
                 $request->toArray());
@@ -89,7 +91,8 @@ 
         $save = $receiver->receive();
 
         //  See if the uploade has finished
-        if($save->isFinished()) {
+        if($save->isFinished())
+        {
             $this->saveFile($save->getFile());
 
             return 'uploaded successfully';

app/Http/Controllers/Installer/SettingsController.php

Spacing +2 added lines, -2 removed lines

@@ -80,7 +80,7 @@ 
         ]);
 
         //  Update the site timezone
-        if (config('app.timezone') !== $request->timezone)
+        if(config('app.timezone') !== $request->timezone)
         {
             Settings::firstOrCreate(
                 ['key'   => 'app.timezone'],
@@ -88,7 +88,7 @@ 
             )->update(['value' => $request->timezone]);
         }
         //  Update the maximum file upload size
-        if (config('filesystems.paths.max_size') !== $request->filesize)
+        if(config('filesystems.paths.max_size') !== $request->filesize)
         {
             Settings::firstOrCreate(
                 ['key'   => 'filesystems.paths.max_size'],

app/Http/Controllers/Installer/SystemsController.php

Braces +2 added lines, -1 removed lines

@@ -211,7 +211,8 @@
     public function destroy($id)
     {
         //
-        try {
+        try
+        {
             SystemTypes::find($id)->delete();
             return response()->json(['success' => true, 'reason' => 'Equipment Successfully Deleted']);
         }

app/Http/Controllers/DownloadController.php

Braces +10 added lines, -4 removed lines

@@ -59,9 +59,12 @@ 
     public function archiveFiles(Request $request)
     {
         //  Debug Data
-        if(Auth::user()) {
+        if(Auth::user())
+        {
             $user = Auth::user()->full_name;
-        } else {
+        }
+        else
+        {
             $user = \Request::ip();
         }
         Log::debug('Route '.Route::currentRouteName().' visited by '.$user.'  Request Data:', $request->toArray());
@@ -107,9 +110,12 @@ 
     public function downloadArchive($fileName)
     {
         //  Debug Data
-        if(Auth::user()) {
+        if(Auth::user())
+        {
             $user = Auth::user()->full_name;
-        } else {
+        }
+        else
+        {
             $user = \Request::ip();
         }
         Log::debug('Route '.Route::currentRouteName().' visited by '.$user);

app/Files.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
         Log::debug('Preparing to clean a file. Current Name - '.$fileName.'. Current Path - '.$path);
         //  Remove all spaces
         $fileName = str_replace(' ', '_', $fileName);
-        Log::debug('Cleaned Filename. Current Name - ' . $fileName . '. Current Path - ' . $path);
+        Log::debug('Cleaned Filename. Current Name - '.$fileName.'. Current Path - '.$path);
 
         //  Determine if the filename already exists
         if(Storage::exists($path.DIRECTORY_SEPARATOR.$fileName))
@@ -44,7 +44,7 @@ 
             //  Increase the number until one that is not in use is found
             do
             {
-                Log::debug('Filename ' . $fileName . ' already exists.  Appending name.');
+                Log::debug('Filename '.$fileName.' already exists.  Appending name.');
                 $fileName = $base.'('.++$number.')'.$extension;
             } while(Storage::exists($path.DIRECTORY_SEPARATOR.$fileName));
         }

app/Http/Controllers/Admin/AdminController.php

Spacing +3 added lines, -3 removed lines

@@ -34,7 +34,7 @@ 
     //  Display all file links
     public function userLinks()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $userLinks = new UserCollection(
             User::withCount([
@@ -94,7 +94,7 @@ 
         Log::notice('User '.Auth::user()->full_name.' updated User Password Policy requiring resets every '.$request->passExpire.' days');
 
         //  If the setting is changing from never to xx days, update all users
-        if ($request->passExpire == 0)
+        if($request->passExpire == 0)
         {
             User::whereNotNull('password_expires')->update([
                 'password_expires' => null
@@ -116,7 +116,7 @@ 
     //  View the current roles that can be assigned to users
     public function roleSettings()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
         $this->authorize('hasAccess', 'Manage User Roles');
 
         $roles = UserRoleType::with(['UserRolePermissions' => function($query) {

app/Http/Controllers/Admin/UserController.php

Spacing +14 added lines, -14 removed lines

@@ -34,7 +34,7 @@ 
     //  Show the list of current users to edit
     public function index()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $userList = User::with('LastUserLogin')->get()->makeVisible('user_id');
         $route    = 'admin.user.edit';
@@ -71,7 +71,7 @@ 
     //  Show the Add User form
     public function create()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $roles = UserRoleType::all();
         $roleArr = [];
@@ -154,7 +154,7 @@ 
     //  List all inactive users
     public function show($type)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
         $route = '';
 
         if($type !== 'inactive')
@@ -177,10 +177,10 @@ 
     //  Reactivate a disabled user
     public function reactivateUser($id)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
         User::withTrashed()->where('user_id', $id)->restore();
 
-        Log::notice('User ID ' . $id . ' reactivated by ' . Auth::user()->full_name);
+        Log::notice('User ID '.$id.' reactivated by '.Auth::user()->full_name);
         return response()->json([
             'success' => true,
         ]);
@@ -189,13 +189,13 @@ 
     //  Open the edit user form
     public function edit($id)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $roles = UserRoleType::all();
         $user  = new UserResource(User::findOrFail($id));
 
         //  Make sure that the user is not trying to edit someone with more permissions
-        if(($user->role_id == 1 || $user->role_id == 2) && Auth::user()->role_id <=2)
+        if(($user->role_id == 1 || $user->role_id == 2) && Auth::user()->role_id <= 2)
         {
             Log::warning('User '.Auth::user()->full_name.' tried to update user ID '.$id.' that has more permissions than they do.  This request was denied.');
             return abort(403);
@@ -205,13 +205,13 @@ 
 
         //  Good to go - get role information
         $roleArr = [];
-        foreach ($roles as $role)
+        foreach($roles as $role)
         {
-            if ($role->role_id == 1 && Auth::user()->role_id != 1)
+            if($role->role_id == 1 && Auth::user()->role_id != 1)
             {
                 continue;
             }
-            else if ($role->role_id == 2 && Auth::user()->role_id > 2)
+            else if($role->role_id == 2 && Auth::user()->role_id > 2)
             {
                 continue;
             }
@@ -257,7 +257,7 @@ 
         $user = User::findOrFail($id);
         if($user->role_id < Auth::user()->role_id)
         {
-            Log::warning('User ' . Auth::user()->full_name . ' tried to update user ID ' . $id . ' that has more permissions than they do.  This request was denied.');
+            Log::warning('User '.Auth::user()->full_name.' tried to update user ID '.$id.' that has more permissions than they do.  This request was denied.');
             return abort(403);
         }
 
@@ -297,13 +297,13 @@ 
         $user = User::find($request->user_id);
 
         //  Verify this is a valid user ID
-        if (!$user)
+        if(!$user)
         {
             $success = false;
             $reason  = 'Cannot find user with this ID';
         }
         //  Make sure that the user is not trying to deactivate someone with more permissions
-        else if ($user->role_id < Auth::user()->role_id)
+        else if($user->role_id < Auth::user()->role_id)
         {
             $success = false;
             $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.';
@@ -335,7 +335,7 @@ 
     //  Disable the user
     public function destroy($id)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $user = User::find($id);
 

app/Http/Controllers/Admin/CustomerAdminController.php

Spacing +11 added lines, -11 removed lines

@@ -16,7 +16,7 @@ 
     public function __construct()
     {
         $this->middleware('auth');
-        $this->middleware(function ($request, $next) {
+        $this->middleware(function($request, $next) {
             $this->authorize('hasAccess', 'Manage Customers');
             return $next($request);
         });
@@ -50,14 +50,14 @@ 
     //  Form to view what kind of file types can be assigned to customers
     public function fileTypes()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
         return view('admin.customerFileTypes');
     }
 
     //  Get the types of files that can be assigned to a customer file
     public function getFileTypes()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $types = new CustomerFileTypesCollection(CustomerFileTypes::all());
 
@@ -68,7 +68,7 @@ 
     //  Submit a new file type name
     public function submitNewFileType(Request $request)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name . '. Submitted Data - ', $request->toArray());
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name.'. Submitted Data - ', $request->toArray());
         $request->validate([
             'name' => 'required',
         ]);
@@ -77,14 +77,14 @@ 
             'description' => $request->name,
         ]);
 
-        Log::notice('New Customer File Type ' . $request->name . ' created by '. Auth::user()->full_name);
+        Log::notice('New Customer File Type '.$request->name.' created by '.Auth::user()->full_name);
         return response()->json(['success' => true]);
     }
 
     //  Update the name of a file type
     public function setFileTypes(Request $request)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name . '. Submitted Data - ', $request->toArray());
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name.'. Submitted Data - ', $request->toArray());
         $request->validate([
             'name' => 'required',
             'id'   => 'required|exists:customer_file_types,file_type_id',
@@ -101,14 +101,14 @@ 
     //  Try to delete a file type
     public function delFileType($id)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         try {
             //  Try to delete file type from database - will throw error if foreign key is in use
             CustomerFileTypes::find($id)->delete();
-        } catch (\Illuminate\Database\QueryException $e) {
+        } catch(\Illuminate\Database\QueryException $e) {
             //  Unable to remove file type  from the database
-            Log::warning('Attempt to delete file type ID '.$id.' by User '.Auth::user()->full_name.' failed.  Reason - ' . $e);
+            Log::warning('Attempt to delete file type ID '.$id.' by User '.Auth::user()->full_name.' failed.  Reason - '.$e);
             return response()->json(['success' => false, 'reason' => 'In Use']);
         }
 
@@ -119,7 +119,7 @@ 
     //  Show all disabled customers
     public function showDisabled()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $custList = Customers::select('cust_id', 'name', 'deleted_at')
             ->onlyTrashed()
@@ -133,7 +133,7 @@ 
     //  Re-enable a customer
     public function enableCustomer($id)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         Customers::withTrashed()->where('cust_id', $id)->restore();
 

Braces +5 added lines, -2 removed lines

@@ -103,10 +103,13 @@
     {
         Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
 
-        try {
+        try
+        {
             //  Try to delete file type from database - will throw error if foreign key is in use
             CustomerFileTypes::find($id)->delete();
-        } catch (\Illuminate\Database\QueryException $e) {
+        }
+        catch (\Illuminate\Database\QueryException $e)
+        {
             //  Unable to remove file type  from the database
             Log::warning('Attempt to delete file type ID '.$id.' by User '.Auth::user()->full_name.' failed.  Reason - ' . $e);
             return response()->json(['success' => false, 'reason' => 'In Use']);