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.';

routes/web.php

Spacing +76 added lines, -76 removed lines

@@ -8,148 +8,148 @@
 /*
 *   Non user Routes
 */
-Route::get('/',                    'Auth\LoginController@showLoginForm')                ->name('index');
-Route::get('logout',               'Auth\LoginController@logout')                       ->name('logout');
-Route::get('no-script',            'Controller@noScript')                               ->name('noscript');
-Route::get('finish-setup/{hash}',  'Auth\InitializeUserController@initializeUser')      ->name('initialize');
+Route::get('/', 'Auth\LoginController@showLoginForm')                ->name('index');
+Route::get('logout', 'Auth\LoginController@logout')                       ->name('logout');
+Route::get('no-script', 'Controller@noScript')                               ->name('noscript');
+Route::get('finish-setup/{hash}', 'Auth\InitializeUserController@initializeUser')      ->name('initialize');
 Route::post('finish-setup/{hash}', 'Auth\InitializeUserController@submitInitializeUser')->name('initialize');
 
 /*
 *   Download File Routes
 */
-Route::get('download/{id}/{filename}',    'DownloadController@index')          ->name('download');
+Route::get('download/{id}/{filename}', 'DownloadController@index')          ->name('download');
 Route::get('download-archive/{filename}', 'DownloadController@downloadArchive')->name('downloadArchive');
-Route::put('download-archive',            'DownloadController@archiveFiles')   ->name('archiveFiles');
+Route::put('download-archive', 'DownloadController@archiveFiles')   ->name('archiveFiles');
 
 /*
 *   User Settings Routes
 */
-Route::get('account',                   'AccountController@index')         ->name('account');
-Route::post('account',                  'AccountController@submit')        ->name('account');
-Route::put('account',                   'AccountController@notifications') ->name('account');
-Route::get('/account/change-password',  'AccountController@changePassword')->name('changePassword');
+Route::get('account', 'AccountController@index')         ->name('account');
+Route::post('account', 'AccountController@submit')        ->name('account');
+Route::put('account', 'AccountController@notifications') ->name('account');
+Route::get('/account/change-password', 'AccountController@changePassword')->name('changePassword');
 Route::post('/account/change-password', 'AccountController@submitPassword')->name('changePassword');
 
 /*
 *   Basic Logged In Routes
 */
-Route::get('about',                  'DashboardController@about')           ->name('about');
-Route::get('dashboard',              'DashboardController@index')           ->name('dashboard');
-Route::get('notifications',          'DashboardController@getNotifications')->name('getNotifications');
+Route::get('about', 'DashboardController@about')           ->name('about');
+Route::get('dashboard', 'DashboardController@index')           ->name('dashboard');
+Route::get('notifications', 'DashboardController@getNotifications')->name('getNotifications');
 Route::get('mark-notification/{id}', 'DashboardController@markNotification')->name('markNotification');
-Route::delete('notifications/{id}',  'DashboardController@delNotification') ->name('delNotification');
+Route::delete('notifications/{id}', 'DashboardController@delNotification') ->name('delNotification');
 
 /*
 *   File Link Routes
 */
-Route::prefix('links')->name('links.')->group(function () {
+Route::prefix('links')->name('links.')->group(function() {
     //  Resource controllers for base access
-    Route::resource('data',           'FileLinks\FileLinksController');
-    Route::get('new',                 'FileLinks\FileLinksController@create')     ->name('new');
-    Route::get('find/{id}',           'FileLinks\FileLinksController@find')       ->name('user');
+    Route::resource('data', 'FileLinks\FileLinksController');
+    Route::get('new', 'FileLinks\FileLinksController@create')     ->name('new');
+    Route::get('find/{id}', 'FileLinks\FileLinksController@find')       ->name('user');
     Route::get('details/{id}/{name}', 'FileLinks\FileLinksController@details')    ->name('details');
-    Route::get('disable/{id}',        'FileLinks\FileLinksController@disableLink')->name('disable');
+    Route::get('disable/{id}', 'FileLinks\FileLinksController@disableLink')->name('disable');
     //  File Link Files
-    Route::resource('files',          'FileLinks\LinkFilesController');
+    Route::resource('files', 'FileLinks\LinkFilesController');
     //  Index landing page
-    Route::get('/',                   'FileLinks\FileLinksController@index')      ->name('index');
+    Route::get('/', 'FileLinks\FileLinksController@index')      ->name('index');
 });
 
 /*
 *   Guest File Link Routes
 */
 Route::get('file-links/{id}/get-files', 'FileLinks\GuestLinksController@getFiles')->name('file-links.getFiles');
-Route::put('file-links/{id}',           'FileLinks\GuestLinksController@notify')  ->name('file-links.show');
-Route::post('file-links/{id}',          'FileLinks\GuestLinksController@update')  ->name('file-links.show');
-Route::get('file-links/{id}',           'FileLinks\GuestLinksController@show')    ->name('file-links.show');
-Route::get('file-links',                'FileLinks\GuestLinksController@index')   ->name('file-links.index');
+Route::put('file-links/{id}', 'FileLinks\GuestLinksController@notify')  ->name('file-links.show');
+Route::post('file-links/{id}', 'FileLinks\GuestLinksController@update')  ->name('file-links.show');
+Route::get('file-links/{id}', 'FileLinks\GuestLinksController@show')    ->name('file-links.show');
+Route::get('file-links', 'FileLinks\GuestLinksController@index')   ->name('file-links.index');
 
 /*
 *   Customer Routes
 */
 Route::prefix('customer')->name('customer.')->group(function() {
     //  Customer Files
-    Route::resource('files',                   'Customers\CustomerFilesController');
+    Route::resource('files', 'Customers\CustomerFilesController');
     //  Custome Notes
-    Route::get('download-note/{id}',           'DownloadController@downloadCustNote')             ->name('download-note');
-    Route::resource('notes',                   'Customers\CustomerNotesController');
+    Route::get('download-note/{id}', 'DownloadController@downloadCustNote')             ->name('download-note');
+    Route::resource('notes', 'Customers\CustomerNotesController');
     //  Customer Contacts
-    Route::resource('contacts',                'Customers\CustomerContactsController');
+    Route::resource('contacts', 'Customers\CustomerContactsController');
     //  Customer Systems
-    Route::resource('systems',                 'Customers\CustomerSystemsController');
+    Route::resource('systems', 'Customers\CustomerSystemsController');
     //  Customer Details
-    Route::get('link-parent/{id}',             'Customers\CustomerDetailsController@removeParent')->name('removeParent');
-    Route::post('link-parent',                 'Customers\CustomerDetailsController@linkParent')  ->name('linkParent');
-    Route::get('id/{id}/{name}',               'Customers\CustomerDetailsController@details')     ->name('details');
-    Route::resource('id',                      'Customers\CustomerDetailsController');
+    Route::get('link-parent/{id}', 'Customers\CustomerDetailsController@removeParent')->name('removeParent');
+    Route::post('link-parent', 'Customers\CustomerDetailsController@linkParent')  ->name('linkParent');
+    Route::get('id/{id}/{name}', 'Customers\CustomerDetailsController@details')     ->name('details');
+    Route::resource('id', 'Customers\CustomerDetailsController');
     //  check Id and bookmark customer
     Route::get('toggle-fav/{action}/{custID}', 'Customers\CustomerController@toggleFav')          ->name('toggle-fav');
-    Route::get('check-id/{id}',                'Customers\CustomerController@checkID')            ->name('check-id');
+    Route::get('check-id/{id}', 'Customers\CustomerController@checkID')            ->name('check-id');
     //  Index landing/search page
-    Route::get('search',                       'Customers\CustomerController@search')             ->name('search');
-    Route::get('/',                            'Customers\CustomerController@index')              ->name('index');
+    Route::get('search', 'Customers\CustomerController@search')             ->name('search');
+    Route::get('/', 'Customers\CustomerController@index')              ->name('index');
 });
 
 /*
 *   Tech Tip Routes
 */
-Route::resource('tips',                       'TechTips\TechTipsController');
-Route::post('submit-edit/{id}',               'TechTips\TechTipsController@update')      ->name('tips.submit-edit');
+Route::resource('tips', 'TechTips\TechTipsController');
+Route::post('submit-edit/{id}', 'TechTips\TechTipsController@update')      ->name('tips.submit-edit');
 Route::prefix('tip')->name('tip.')->group(function() {
-    Route::resource('comments',               'TechTips\TechTipCommentsController');
-    Route::get('search',                      'TechTips\TechTipsController@search')      ->name('search');
-    Route::get('details/{id}/{name}',         'TechTips\TechTipsController@details')     ->name('details');
-    Route::post('process-image',              'TechTips\TechTipsController@processImage')->name('processImage');
+    Route::resource('comments', 'TechTips\TechTipCommentsController');
+    Route::get('search', 'TechTips\TechTipsController@search')      ->name('search');
+    Route::get('details/{id}/{name}', 'TechTips\TechTipsController@details')     ->name('details');
+    Route::post('process-image', 'TechTips\TechTipsController@processImage')->name('processImage');
     Route::get('toggle-fav/{action}/{tipID}', 'TechTips\TechTipsController@toggleFav')   ->name('toggle-fav');
-    Route::get('download-tip/{id}',           'DownloadController@downloadTechTip')      ->name('downloadTip');
+    Route::get('download-tip/{id}', 'DownloadController@downloadTechTip')      ->name('downloadTip');
 });
 
 /*
 *   Administration Routes
 */
-Route::prefix('admin')->name('admin.')->group(function () {
+Route::prefix('admin')->name('admin.')->group(function() {
     //  Routes for Tech Bench Add-ons
     Route::prefix('modules')->name('module.')->group(function() {
         Route::delete('delete-staged/{name}', 'Installer\ModuleController@delStaged') ->name('deleteStaged');
-        Route::post('upload',                 'Installer\ModuleController@upload')    ->name('upload');
-        Route::get('activate/{name}',         'Installer\ModuleController@activate')  ->name('activate');
-        Route::get('disable/{name}',          'Installer\ModuleController@disable')   ->name('disable');
-        Route::get('enable/{name}',           'Installer\ModuleController@enable')    ->name('enable');
-        Route::get('get-active',              'Installer\ModuleController@getEnabled')->name('getEnabled');
-        Route::get('get-staged',              'Installer\ModuleController@getStaged') ->name('getStaged');
-        Route::get('/',                       'Installer\ModuleController@index')     ->name('index');
+        Route::post('upload', 'Installer\ModuleController@upload')    ->name('upload');
+        Route::get('activate/{name}', 'Installer\ModuleController@activate')  ->name('activate');
+        Route::get('disable/{name}', 'Installer\ModuleController@disable')   ->name('disable');
+        Route::get('enable/{name}', 'Installer\ModuleController@enable')    ->name('enable');
+        Route::get('get-active', 'Installer\ModuleController@getEnabled')->name('getEnabled');
+        Route::get('get-staged', 'Installer\ModuleController@getStaged') ->name('getStaged');
+        Route::get('/', 'Installer\ModuleController@index')     ->name('index');
     });
     //  Administrative routes for equipment and equipment categories
-    Route::resource('categories',          'Installer\CategoriesController');
-    Route::resource('systems',             'Installer\SystemsController');
+    Route::resource('categories', 'Installer\CategoriesController');
+    Route::resource('systems', 'Installer\SystemsController');
     //  Administrative routes for users
-    Route::resource('user',                'Admin\UserController');
-    Route::get('user/enable/{id}',         'Admin\UserController@reactivateUser')             ->name('user.reactivate');
-    Route::post('user/change-password',    'Admin\UserController@submitPassword')             ->name('user.changePassword');
+    Route::resource('user', 'Admin\UserController');
+    Route::get('user/enable/{id}', 'Admin\UserController@reactivateUser')             ->name('user.reactivate');
+    Route::post('user/change-password', 'Admin\UserController@submitPassword')             ->name('user.changePassword');
     Route::get('check-user/{name}/{type}', 'Admin\UserController@checkUser')                  ->name('checkUser');
-    Route::get('links/show/{id}',          'Admin\AdminController@showLinks')                 ->name('user.showLinks');
-    Route::get('links',                    'Admin\AdminController@userLinks')                 ->name('user.links');
-    Route::post('password-policy',         'Admin\AdminController@submitPolicy')              ->name('passwordPolicy');
-    Route::get('password-policy',          'Admin\AdminController@passwordPolicy')            ->name('passwordPolicy');
-    Route::post('role-policy',             'Admin\AdminController@submitRoleSettings')        ->name('roleSettings');
-    Route::get('role-policy',              'Admin\AdminController@roleSettings')              ->name('roleSettings');
+    Route::get('links/show/{id}', 'Admin\AdminController@showLinks')                 ->name('user.showLinks');
+    Route::get('links', 'Admin\AdminController@userLinks')                 ->name('user.links');
+    Route::post('password-policy', 'Admin\AdminController@submitPolicy')              ->name('passwordPolicy');
+    Route::get('password-policy', 'Admin\AdminController@passwordPolicy')            ->name('passwordPolicy');
+    Route::post('role-policy', 'Admin\AdminController@submitRoleSettings')        ->name('roleSettings');
+    Route::get('role-policy', 'Admin\AdminController@roleSettings')              ->name('roleSettings');
     //  Tech Bench Settings Routes
-    Route::post('logo',                    'Installer\SettingsController@submitLogo')         ->name('submitLogo');
-    Route::get('logo',                     'Installer\SettingsController@logoSettings')       ->name('logoSettings');
-    Route::post('configuratin',            'Installer\SettingsController@submitConfiguration')->name('submitConfig');
-    Route::get('configuration',            'Installer\SettingsController@configuration')      ->name('config');
-    Route::post('email-settings',          'Installer\SettingsController@submitEmailSettings')->name('emailSettings');
-    Route::put('email-settings',           'Installer\SettingsController@sendTestEmail')      ->name('emailSettings');
-    Route::get('email-settings',           'Installer\SettingsController@emailSettings')      ->name('emailSettings');
+    Route::post('logo', 'Installer\SettingsController@submitLogo')         ->name('submitLogo');
+    Route::get('logo', 'Installer\SettingsController@logoSettings')       ->name('logoSettings');
+    Route::post('configuratin', 'Installer\SettingsController@submitConfiguration')->name('submitConfig');
+    Route::get('configuration', 'Installer\SettingsController@configuration')      ->name('config');
+    Route::post('email-settings', 'Installer\SettingsController@submitEmailSettings')->name('emailSettings');
+    Route::put('email-settings', 'Installer\SettingsController@sendTestEmail')      ->name('emailSettings');
+    Route::get('email-settings', 'Installer\SettingsController@emailSettings')      ->name('emailSettings');
     //  Tech Bench Backup Routes
-    Route::get('backups/download/{name}',  'Installer\SettingsController@downloadBackup')     ->name('downloadBackup');
-    Route::get('backups/delete/{name}',    'Installer\SettingsController@delBackup')          ->name('delBackup');
-    Route::get('backups/run',              'Installer\SettingsController@runBackup')          ->name('runBackup');
-    Route::get('backups/get',              'Installer\SettingsController@getBackups')         ->name('getBackups');
-    Route::get('backups',                  'Installer\SettingsController@backupsIndex')       ->name('backups');
+    Route::get('backups/download/{name}', 'Installer\SettingsController@downloadBackup')     ->name('downloadBackup');
+    Route::get('backups/delete/{name}', 'Installer\SettingsController@delBackup')          ->name('delBackup');
+    Route::get('backups/run', 'Installer\SettingsController@runBackup')          ->name('runBackup');
+    Route::get('backups/get', 'Installer\SettingsController@getBackups')         ->name('getBackups');
+    Route::get('backups', 'Installer\SettingsController@backupsIndex')       ->name('backups');
     //  Tech Bench Updates Routs
 
     Route::get('updates', 'Installer\UpdateController@index')->name('updates');
     //  Admin index route
-    Route::get('/',                        'Admin\AdminController@index')                     ->name('index');
+    Route::get('/', 'Admin\AdminController@index')                     ->name('index');
 });