butcherman / Tech_Bench

app/Providers/FormGroupServiceProvider.php

Spacing +5 added lines, -5 removed lines

@@ -25,14 +25,14 @@
     public function boot()
     {
         //  Service provider to customize form inputs
-        Form::component('bsText',     'components.form.text',     ['name', 'label', 'value' => null, 'attributes' => []]);
+        Form::component('bsText', 'components.form.text', ['name', 'label', 'value' => null, 'attributes' => []]);
         Form::component('bsPassword', 'components.form.password', ['name', 'label', 'value' => null, 'attributes' => []]);
-        Form::component('bsNumber',   'components.form.number',   ['name', 'label', 'value' => null, 'attributes' => []]);
-        Form::component('bsEmail',    'components.form.email',    ['name', 'label', 'value' => null, 'attributes' => []]);
-        Form::component('bsDate',     'components.form.date',     ['name', 'label', 'value' => null, 'attributes' => []]);
+        Form::component('bsNumber', 'components.form.number', ['name', 'label', 'value' => null, 'attributes' => []]);
+        Form::component('bsEmail', 'components.form.email', ['name', 'label', 'value' => null, 'attributes' => []]);
+        Form::component('bsDate', 'components.form.date', ['name', 'label', 'value' => null, 'attributes' => []]);
         Form::component('bsTextarea', 'components.form.textarea', ['name', 'label', 'value' => null, 'attributes' => []]);
         Form::component('bsCheckbox', 'components.form.checkbox', ['name', 'label', 'value' => 'on', 'checked' => false, 'attributes' => []]);
-        Form::component('bsSelect',   'components.form.select',   ['name', 'label', 'list', 'selected' => null, 'attributes' => []]);
+        Form::component('bsSelect', 'components.form.select', ['name', 'label', 'list', 'selected' => null, 'attributes' => []]);
         //  Custom Submit button
         Form::component('bsSubmit', 'components.form.submit', ['name']);
         //  List of all states in a dropdown box

app/Http/Controllers/Customers/CustomerSystemsController.php

Spacing +6 added lines, -6 removed lines

@@ -24,7 +24,7 @@ 
     //  Get the possible system types that can be assigned to the customer
     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);
         $sysList = new CategoriesCollection(SystemCategories::with('SystemTypes')->with('SystemTypes.SystemDataFields.SystemDataFieldTypes')->get());
 
         return $sysList;
@@ -33,7 +33,7 @@ 
     //  Store a new system for the customer
     public function store(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([
             'cust_id' => 'required',
@@ -76,7 +76,7 @@ 
     //  Get the list of systems attached to the customer
     public function show($id)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $sysList = CustomerSystems::where('cust_id', $id)
                     ->with('SystemTypes')
@@ -105,7 +105,7 @@ 
     // Update the customers system data
     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',
@@ -124,7 +124,7 @@ 
 
         foreach($fields as $data)
         {
-            $fieldName = 'field_' . $data->field_id;
+            $fieldName = 'field_'.$data->field_id;
             if(isset($request->$fieldName))
             {
                 Log::debug($request->$fieldName);
@@ -142,7 +142,7 @@ 
     //  Delete a system attached to a customer
     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);
 
         // return response('deleted '.$id);
         $system = CustomerSystems::find($id);

app/Policies/GatePolicy.php

Spacing +7 added lines, -7 removed lines

@@ -37,13 +37,13 @@ 
     //  Determine if a user can see the Administration Nav Link
     public function seeAdminLink(User $user)
     {
-        if ($this->isInstaller($user))
+        if($this->isInstaller($user))
         {
             return true;
         }
 
         $data = UserRolePermissions::with('UserRolePermissionTypes')
-            ->whereHas('UserRolePermissionTypes', function ($query) {
+            ->whereHas('UserRolePermissionTypes', function($query) {
                 $query->where('description', 'Manage Users')
                     ->orWhere('description', 'Manage User Roles')
                     ->orWhere('description', 'Manage Customers')
@@ -54,7 +54,7 @@ 
             ->get();
 
         $allow = $data->isEmpty() ? 'Denied' : 'Allowed';
-        Log::debug('User ' . $user->full_name . ' is trying to see admin link.  Result - ' . $allow);
+        Log::debug('User '.$user->full_name.' is trying to see admin link.  Result - '.$allow);
 
         return  $data->isEmpty() ? false : true;
     }
@@ -69,7 +69,7 @@ 
         }
 
         $data = UserRolePermissions::with('UserRolePermissionTypes')
-            ->whereHas('UserRolePermissionTypes', function ($query) use ($task) {
+            ->whereHas('UserRolePermissionTypes', function($query) use ($task) {
                 $query->where('description', $task);
             })
             ->where('role_id', $user->role_id)
@@ -77,10 +77,10 @@ 
             ->get();
 
         $allow = $data->isEmpty() ? 'false' : 'true';
-        Log::debug('User '.$user->full_name.' is trying to access '.$task.'.  Result - ' . $allow);
-        if ($allow === 'Denied')
+        Log::debug('User '.$user->full_name.' is trying to access '.$task.'.  Result - '.$allow);
+        if($allow === 'Denied')
         {
-            Log::alert('User ' . $user->full_name . ' was denied from accessing '.$task.' link.');
+            Log::alert('User '.$user->full_name.' was denied from accessing '.$task.' link.');
         }
 
         return  $data->isEmpty() ? false : true;

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/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);