butcherman / Tech_Bench

app/Http/Middleware/CheckBrowser.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
         if(!empty($_SERVER['HTTP_USER_AGENT']))
         {
             //  Check for Internet Explorer 11
-            if (preg_match("/Trident\/7.0;(.*)rv:11.0/", $_SERVER["HTTP_USER_AGENT"], $match) != 0)
+            if(preg_match("/Trident\/7.0;(.*)rv:11.0/", $_SERVER["HTTP_USER_AGENT"], $match) != 0)
             {
                 return response()->make(view('error.426'), 426);
             }

app/Http/Controllers/Admin/UserController.php

Spacing +13 added lines, -13 removed lines

@@ -29,7 +29,7 @@ 
     public function __construct()
     {
         $this->middleware('auth')->except('initializeUser', 'submitInitializeUser');
-        $this->middleware(function ($request, $next) {
+        $this->middleware(function($request, $next) {
             $this->authorize('hasAccess', 'Manage Users');
             return $next($request);
         });
@@ -144,7 +144,7 @@ 
     //  List all inactive users
     public function show($type)
     {
-        $route    = '';
+        $route = '';
 
         if($type !== 'inactive')
         {
@@ -168,17 +168,17 @@ 
         $user  = new UserResource(User::findOrFail($id));
 
         //  Make sure that the user is not trying to deactivate someone with more permissions
-        if ($user->role_id < Auth::user()->role_id)
+        if($user->role_id < Auth::user()->role_id)
         {
             return abort(403);
         }
 
         //  Good to go - update user password
         $roleArr = [];
-        foreach ($roles as $role) {
-            if ($role->role_id == 1 && Auth::user()->role_id != 1) {
+        foreach($roles as $role) {
+            if($role->role_id == 1 && Auth::user()->role_id != 1) {
                 continue;
-            } else if ($role->role_id == 2 && Auth::user()->role_id > 1) {
+            } else if($role->role_id == 2 && Auth::user()->role_id > 1) {
                 continue;
             } else {
                 // $roleArr[$role->role_id] = $role->name;
@@ -189,7 +189,7 @@ 
             }
         }
 
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by User ID-' . Auth::user()->user_id);
+        Log::debug('Route '.Route::currentRouteName().' visited by User ID-'.Auth::user()->user_id);
         return view('admin.userEdit', [
             'roles' => $roleArr,
             'user'  => $user->
@@ -228,7 +228,7 @@ 
         //  Update the user data
         $user = User::findOrFail($id);
 
-        if ($user->role_id < Auth::user()->role_id)
+        if($user->role_id < Auth::user()->role_id)
         {
             return abort(403);
         }
@@ -271,12 +271,12 @@ 
         $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.';
         }
@@ -289,11 +289,11 @@ 
                 'password_expires' => $nextChange
             ]);
             $success = true;
-            $reason  = 'Password for ' . $user->full_name . ' successfully reset.';
+            $reason  = 'Password for '.$user->full_name.' successfully reset.';
         }
 
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by User ID-' . Auth::user()->user_id);
-        Log::notice('User ID-' . $request->user_id . ' password chagned by ' . Auth::user()->user_id, [
+        Log::debug('Route '.Route::currentRouteName().' visited by User ID-'.Auth::user()->user_id);
+        Log::notice('User ID-'.$request->user_id.' password chagned by '.Auth::user()->user_id, [
             'success' => $success,
             'reason'  => $reason,
         ]);

app/Http/Controllers/Customers/CustomerFilesController.php

Spacing +3 added lines, -3 removed lines

@@ -37,7 +37,7 @@ 
         //  Verify that the upload is valid and being processed
         if($receiver->isUploaded() === false)
         {
-            Log::error('Upload File Missing - ' .
+            Log::error('Upload File Missing - '.
             /** @scrutinizer ignore-type */
             $request->toArray());
             throw new UploadMissingFileException();
@@ -51,7 +51,7 @@ 
         {
             //  Determine if the note should go to the customer, or its parent
             $details = Customers::find($request->cust_id);
-            if ($details->parent_id && $request->shared == 'true')
+            if($details->parent_id && $request->shared == 'true')
             {
                 $request->cust_id = $details->parent_id;
             }
@@ -111,7 +111,7 @@ 
 
         //  Determine if there is a parent site with shared files
         $parent = Customers::find($id)->parent_id;
-        if ($parent) {
+        if($parent) {
             $parentList = Customerfiles::where('cust_id', $parent)
                 ->where('shared', 1)
                 ->with('Files')

Braces +2 added lines, -1 removed lines

@@ -111,7 +111,8 @@
 
         //  Determine if there is a parent site with shared files
         $parent = Customers::find($id)->parent_id;
-        if ($parent) {
+        if ($parent)
+        {
             $parentList = Customerfiles::where('cust_id', $parent)
                 ->where('shared', 1)
                 ->with('Files')

app/Http/Controllers/Customers/CustomerNotesController.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
 
         //  Determine if the note should go to the customer, or its parent
         $details = Customers::find($request->cust_id);
-        if ($details->parent_id && $request->shared == 'true') {
+        if($details->parent_id && $request->shared == 'true') {
             $request->cust_id = $details->parent_id;
         }
 
@@ -55,7 +55,7 @@ 
 
         //  Determine if there is a parent site with shared contacts
         $parent = Customers::find($id)->parent_id;
-        if ($parent) {
+        if($parent) {
             $parentList = CustomerNotes::where('cust_id', $parent)->where('shared', 1)->orderBy('urgent', 'desc')->get();
 
             $notes = $notes->merge($parentList);

Braces +4 added lines, -2 removed lines

@@ -29,7 +29,8 @@ 
 
         //  Determine if the note should go to the customer, or its parent
         $details = Customers::find($request->cust_id);
-        if ($details->parent_id && $request->shared == 'true') {
+        if ($details->parent_id && $request->shared == 'true')
+        {
             $request->cust_id = $details->parent_id;
         }
 
@@ -55,7 +56,8 @@ 
 
         //  Determine if there is a parent site with shared contacts
         $parent = Customers::find($id)->parent_id;
-        if ($parent) {
+        if ($parent)
+        {
             $parentList = CustomerNotes::where('cust_id', $parent)->where('shared', 1)->orderBy('urgent', 'desc')->get();
 
             $notes = $notes->merge($parentList);