butcherman / Tech_Bench

app/Http/Controllers/DashboardController.php

Spacing +6 added lines, -6 removed lines

@@ -30,9 +30,9 @@ 
         $totalLinks  = FileLinks::where('user_id', Auth::user()->user_id)->count();
 
         //  Debug Data
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
         Log::debug('Customer Favorites for '.Auth::user()->full_name.': ', $custFavs->toArray());
-        Log::debug('Tech Tip Favorites for ' . Auth::user()->full_name . ': ', $tipFavs->toArray());
+        Log::debug('Tech Tip Favorites for '.Auth::user()->full_name.': ', $tipFavs->toArray());
 
         return view('dashboard', [
             'custFavs'    => $custFavs,
@@ -47,7 +47,7 @@ 
     //  About page
     public function about()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         return view('about', [
             'branch' => 'latest'
@@ -58,7 +58,7 @@ 
     public function getNotifications()
     {
         //  Debug Data
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
         Log::debug('Notifications for '.Auth::user()->full_name.':', Auth::user()->notifications->toArray());
 
         return Auth::user()->notifications;
@@ -67,7 +67,7 @@ 
     //  Mark a notification as read
     public function markNotification($id)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $notification = Auth::user()->notifications()->where('id', $id)->where('notifiable_id', Auth::user()->user_id)->first();
         if(!$notification)
@@ -93,7 +93,7 @@ 
         }
         else
         {
-            Log::error('User ' . Auth::user()->full_name . ' tried to delete an invalid notification as read.  Notification ID: ' . $id);
+            Log::error('User '.Auth::user()->full_name.' tried to delete an invalid notification as read.  Notification ID: '.$id);
             return abort(404);
         }
 

app/Http/Controllers/Customers/CustomerContactsController.php

Spacing +1 added lines, -1 removed lines

@@ -130,7 +130,7 @@
 
         //  Update the primary contact information
         $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;
         }

app/Http/Controllers/Customers/CustomerNotesController.php

Spacing +5 added lines, -5 removed lines

@@ -31,7 +31,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;
         }
@@ -56,21 +56,21 @@ 
 
         //  Determine if there is a parent site with shared notes
         $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);
         }
 
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
         return $notes;
     }
 
     //  Update a customer note
     public function update(Request $request, $id)
     {
-        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([
             'cust_id' => 'required',
@@ -79,7 +79,7 @@ 
         ]);
 
         $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;
         }

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/Admin/AdminController.php

Spacing +2 added lines, -2 removed lines

@@ -89,7 +89,7 @@ 
         Log::notice('User '.Auth::user()->full_name.' updated User Password Policy');
 
         //  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
@@ -110,7 +110,7 @@ 
     public function roleSettings()
     {
         $this->authorize('hasAccess', 'Manage User Roles');
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
         $roles = UserRoleType::with(['UserRolePermissions' => function($query) {
             $query->join('user_role_permission_types', 'user_role_permission_types.perm_type_id', '=', 'user_role_permissions.perm_type_id');
         }])->get();

app/Http/Controllers/Admin/UserController.php

Spacing +5 added lines, -5 removed lines

@@ -182,13 +182,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 > 1)
+            else if($role->role_id == 2 && Auth::user()->role_id > 1)
             {
                 continue;
             }
@@ -285,13 +285,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.';