butcherman / Tech_Bench

database/migrations/2019_11_30_215206_updates_for_version_5_0.php

Spacing +13 added lines, -13 removed lines

@@ -66,8 +66,8 @@ 
     private function updatePhoneIcons()
     {
         $newIcons = [
-            ['description' => 'Home',   'icon_class' => 'fas fa-home'],
-            ['description' => 'Work',   'icon_class' => 'fas fa-briefcase'],
+            ['description' => 'Home', 'icon_class' => 'fas fa-home'],
+            ['description' => 'Work', 'icon_class' => 'fas fa-briefcase'],
             ['description' => 'Mobile', 'icon_class' => 'fas fa-mobile-alt'],
         ];
 
@@ -99,7 +99,7 @@ 
         {
             if(!Schema::hasColumn('users', 'role_id'))
             {
-                Schema::table('users', function (Blueprint $table) {
+                Schema::table('users', function(Blueprint $table) {
                     $table->integer('role_id')->after('user_id')->unsigned()->default(4);
                     $table->foreign('role_id')->references('role_id')->on('user_role_descriptions')->onUpdate('cascade');
                 });
@@ -326,8 +326,8 @@ 
     //  Add soft deletes to tech tips table to prevent accidental deletes
     private function addSoftDeleteToTechTips()
     {
-        if (!Schema::hasColumn('tech_tips', 'deleted_at')) {
-            Schema::table('tech_tips', function (Blueprint $table) {
+        if(!Schema::hasColumn('tech_tips', 'deleted_at')) {
+            Schema::table('tech_tips', function(Blueprint $table) {
                 $table->softDeletes()->after('description');
             });
         }
@@ -345,7 +345,7 @@ 
             //  Migrate over all deactivated users
             DB::update('UPDATE `users` SET `deleted_at` = "'.Carbon::now().'" WHERE `active` = 0');
             //  Remove the Active column
-            Schema::table('users', function (Blueprint $table) {
+            Schema::table('users', function(Blueprint $table) {
                 $table->dropColumn('active');
             });
         }
@@ -364,27 +364,27 @@ 
         }
         if(!Schema::hasColumn('customer_systems', 'shared'))
         {
-            Schema::table('customer_systems', function (Blueprint $table)
+            Schema::table('customer_systems', function(Blueprint $table)
             {
                 $table->boolean('shared')->default(0)->after('sys_id');
             });
         }
-        if (!Schema::hasColumn('customer_contacts', 'shared'))
+        if(!Schema::hasColumn('customer_contacts', 'shared'))
         {
-            Schema::table('customer_contacts', function (Blueprint $table)
+            Schema::table('customer_contacts', function(Blueprint $table)
             {
                 $table->boolean('shared')->default(0)->after('cust_id');
             });
         }
-        if (!Schema::hasColumn('customer_notes', 'shared'))
+        if(!Schema::hasColumn('customer_notes', 'shared'))
         {
-            Schema::table('customer_notes', function (Blueprint $table)
+            Schema::table('customer_notes', function(Blueprint $table)
             {
                 $table->boolean('shared')->default(0)->after('user_id');
             });
         }
-        if (!Schema::hasColumn('customer_files', 'shared')) {
-            Schema::table('customer_files', function (Blueprint $table) {
+        if(!Schema::hasColumn('customer_files', 'shared')) {
+            Schema::table('customer_files', function(Blueprint $table) {
                 $table->boolean('shared')->default(0)->after('user_id');
             });
         }

Braces +29 added lines, -22 removed lines

@@ -83,7 +83,8 @@ 
     //  Added the ability to set an expiration date for user passwords - will force them to change after this expires
     private function addPasswordExpiresColumn()
     {
-        if(!Schema::hasColumn('users', 'password_expires')) {
+        if(!Schema::hasColumn('users', 'password_expires'))
+        {
             Schema::table('users', function(Blueprint $table) {
                 $table->timestamp('password_expires')
                     ->nullable()
@@ -119,7 +120,8 @@ 
     //  Added a 'hidden' attribute to system customer data types to allow passwords to not be viewed unless clicked or focus
     private function addHiddenColumn()
     {
-        if(!Schema::hasColumn('system_data_field_types', 'hidden')) {
+        if(!Schema::hasColumn('system_data_field_types', 'hidden'))
+        {
             Schema::table('system_cust_data_types', function(Blueprint $table) {
                 $table->boolean('hidden')
                     ->default(0)
@@ -131,7 +133,8 @@ 
     //  Update the File links table - add cust_id and note column
     private function addColumnsToFileLinksTable()
     {
-        if(!Schema::hasColumn('file_links', 'cust_id')) {
+        if(!Schema::hasColumn('file_links', 'cust_id'))
+        {
             Schema::table('file_links', function(Blueprint $table) {
                 $table->integer('cust_id')
                     ->unsigned()
@@ -140,7 +143,8 @@ 
                 $table->foreign('cust_id')->references('cust_id')->on('customers')->onUpdate('cascade')->onDelete('cascade');
             });
         }
-        if(!Schema::hasColumn('file_links', 'note')) {
+        if(!Schema::hasColumn('file_links', 'note'))
+        {
             Schema::table('file_links', function(Blueprint $table) {
                 $table->longText('note')
                     ->nullable()
@@ -148,7 +152,8 @@ 
             });
             //  Migrate the instructions from the old table to the new column
             $instructions = DB::select('SELECT * FROM `file_link_instructions`');
-            foreach($instructions as $ins) {
+            foreach($instructions as $ins)
+            {
                 FileLinks::find($ins->link_id)->update([
                     'note' => $ins->instruction
                 ]);
@@ -159,7 +164,8 @@ 
     //  Add Notes column to the file links files table
     private function addNotesColumnToFileLinkFiles()
     {
-        if(!Schema::hasColumn('file_link_files', 'note')) {
+        if(!Schema::hasColumn('file_link_files', 'note'))
+        {
             Schema::table('file_link_files', function(Blueprint $table) {
                 $table->longText('note')
                     ->nullable()
@@ -167,7 +173,8 @@ 
             });
             //  Migrate the existing notes to the new table
             $notes = DB::select('SELECT * FROM `file_link_notes`');
-            foreach($notes as $note) {
+            foreach($notes as $note)
+            {
                 FileLinkFiles::where('file_id', $note->file_id)->update([
                     'note' => $note->note
                 ]);
@@ -178,7 +185,8 @@ 
     //  Add the documentation column to the tech tips table
     private function migrateSystemDocumentation()
     {
-        if(!Schema::hasColumn('tech_tips', 'tip_type_id')) {
+        if(!Schema::hasColumn('tech_tips', 'tip_type_id'))
+        {
             Schema::table('tech_tips', function(Blueprint $table) {
                 $table->bigInteger('tip_type_id')->unsigned()->after('public')->default(1);
                 $table->foreign('tip_type_id')->references('tip_type_id')->on('tech_tip_types')->onUpdate('cascade');
@@ -186,7 +194,8 @@ 
 
             //  Move all of the system files over to the tech tips table
             $sysFiles = DB::select('SELECT * FROM `system_files`');
-            foreach($sysFiles as $sysFile) {
+            foreach($sysFiles as $sysFile)
+            {
                 $newTip = TechTips::create([
                     'user_id'       => $sysFile->user_id,
                     'public'        => 0,
@@ -280,7 +289,8 @@ 
     //  Remove the system files and system file types table
     private function removeSystemFilesTables()
     {
-        if(Schema::hasTable('system_files')) {
+        if(Schema::hasTable('system_files'))
+        {
             Schema::table('system_files', function(Blueprint $table) {
                 $table->dropForeign(['sys_id']);
                 $table->dropForeign(['type_id']);
@@ -326,7 +336,8 @@ 
     //  Add soft deletes to tech tips table to prevent accidental deletes
     private function addSoftDeleteToTechTips()
     {
-        if (!Schema::hasColumn('tech_tips', 'deleted_at')) {
+        if (!Schema::hasColumn('tech_tips', 'deleted_at'))
+        {
             Schema::table('tech_tips', function (Blueprint $table) {
                 $table->softDeletes()->after('description');
             });
@@ -338,8 +349,7 @@ 
     {
         if(!Schema::hasColumn('users', 'deleted_at'))
         {
-            Schema::table('users', function(Blueprint $table)
-            {
+            Schema::table('users', function(Blueprint $table) {
                 $table->softDeletes()->after('password_expires');
             });
             //  Migrate over all deactivated users
@@ -356,34 +366,31 @@ 
     {
         if(!Schema::hasColumn('customers', 'parent_id'))
         {
-            Schema::table('customers', function(Blueprint $table)
-            {
+            Schema::table('customers', function(Blueprint $table) {
                 $table->integer('parent_id')->after('cust_id')->nullable()->unsigned();
                 $table->foreign('parent_id')->references('cust_id')->on('customers')->onUpdate('cascade');
             });
         }
         if(!Schema::hasColumn('customer_systems', 'shared'))
         {
-            Schema::table('customer_systems', function (Blueprint $table)
-            {
+            Schema::table('customer_systems', function (Blueprint $table) {
                 $table->boolean('shared')->default(0)->after('sys_id');
             });
         }
         if (!Schema::hasColumn('customer_contacts', 'shared'))
         {
-            Schema::table('customer_contacts', function (Blueprint $table)
-            {
+            Schema::table('customer_contacts', function (Blueprint $table) {
                 $table->boolean('shared')->default(0)->after('cust_id');
             });
         }
         if (!Schema::hasColumn('customer_notes', 'shared'))
         {
-            Schema::table('customer_notes', function (Blueprint $table)
-            {
+            Schema::table('customer_notes', function (Blueprint $table) {
                 $table->boolean('shared')->default(0)->after('user_id');
             });
         }
-        if (!Schema::hasColumn('customer_files', 'shared')) {
+        if (!Schema::hasColumn('customer_files', 'shared'))
+        {
             Schema::table('customer_files', function (Blueprint $table) {
                 $table->boolean('shared')->default(0)->after('user_id');
             });

app/Http/Controllers/Installer/ModuleController.php

Spacing +8 added lines, -8 removed lines

@@ -43,7 +43,7 @@ 
             ];
         }
         $mods = Module::allDisabled();
-        foreach ($mods as $name => $mod) {
+        foreach($mods as $name => $mod) {
             $moduleList[] = [
                 'name' => $name,
                 'status' => 'Disabled',
@@ -80,8 +80,8 @@ 
         $receiver = new FileReceiver('file', $request, HandlerFactory::classFromRequest($request));
 
         //  Verify that the upload is valid and being processed
-        if ($receiver->isUploaded() === false) {
-            Log::error('Upload File Missing - ' .
+        if($receiver->isUploaded() === false) {
+            Log::error('Upload File Missing - '.
                 /** @scrutinizer ignore-type */
                 $request->toArray());
             throw new UploadMissingFileException();
@@ -91,7 +91,7 @@ 
         $save = $receiver->receive();
 
         //  See if the uploade has finished
-        if ($save->isFinished()) {
+        if($save->isFinished()) {
             $this->saveFile($save->getFile());
 
             return 'uploaded successfully';
@@ -100,8 +100,8 @@ 
         //  Get the current progress
         $handler = $save->handler();
 
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by User ID-' . Auth::user()->user_id);
-        Log::debug('File being uploaded.  Percentage done - ' . $handler->getPercentageDone());
+        Log::debug('Route '.Route::currentRouteName().' visited by User ID-'.Auth::user()->user_id);
+        Log::debug('File being uploaded.  Percentage done - '.$handler->getPercentageDone());
         return response()->json([
             'done'   => $handler->getPercentageDone(),
             'status' => true
@@ -112,7 +112,7 @@ 
     //  Save a file attached to the link
     private function saveFile(UploadedFile $file)
     {
-        $filePath = config('filesystems.disks.staging.root') . DIRECTORY_SEPARATOR . 'modules'.DIRECTORY_SEPARATOR;
+        $filePath = config('filesystems.disks.staging.root').DIRECTORY_SEPARATOR.'modules'.DIRECTORY_SEPARATOR;
 
         //  Clean the file and store it
         $fileName = Files::cleanFilename($filePath, $file->getClientOriginalName());
@@ -135,7 +135,7 @@ 
         $module->extract(config('modules.paths.modules').DIRECTORY_SEPARATOR.$modName);
         $module->close();
 
-        Artisan::call('module:enable',  ['module' => $modName]);
+        Artisan::call('module:enable', ['module' => $modName]);
         Artisan::call('module:migrate', ['module' => $modName]);
         $this->delStaged($name);
 

Braces +6 added lines, -3 removed lines

@@ -43,7 +43,8 @@ 
             ];
         }
         $mods = Module::allDisabled();
-        foreach ($mods as $name => $mod) {
+        foreach ($mods as $name => $mod)
+        {
             $moduleList[] = [
                 'name' => $name,
                 'status' => 'Disabled',
@@ -80,7 +81,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());
@@ -91,7 +93,8 @@ 
         $save = $receiver->receive();
 
         //  See if the uploade has finished
-        if ($save->isFinished()) {
+        if ($save->isFinished())
+        {
             $this->saveFile($save->getFile());
 
             return 'uploaded successfully';

database/seeds/EquipmentSeeder.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
                 SystemDataFields::create([
                     'sys_id' => $sys->sys_id,
                     'data_type_id' => $i,
-                    'order' => $i-1,
+                    'order' => $i - 1,
                 ]);
             }
         }

database/factories/Customers.php

Spacing +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
 use App\Customers;
 use Faker\Generator as Faker;
 
-$factory->define(Customers::class, function (Faker $faker) {
+$factory->define(Customers::class, function(Faker $faker) {
     return [
         'cust_id'   => $faker->unique()->numberBetween(50, 10000),
         'parent_id' => null,

database/factories/UserFactory.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
         'first_name'       => $faker->firstName,
         'last_name'        => $faker->lastName,
         'email'            => $faker->unique()->safeEmail,
-        'password'         => bcrypt('password'),  //  All test users will have the password of 'password' to allow testing access
+        'password'         => bcrypt('password'), //  All test users will have the password of 'password' to allow testing access
         'remember_token'   => Str::random(10),
         'password_expires' => null,
     ];

routes/web.php

Braces +5 added lines, -10 removed lines

@@ -42,8 +42,7 @@ 
 /*
 *   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');
@@ -68,8 +67,7 @@ 
 /*
 *   Customer Routes
 */
-Route::prefix('customer')->name('customer.')->group(function()
-{
+Route::prefix('customer')->name('customer.')->group(function() {
     //  Customer Files
     Route::resource('files',                   'Customers\CustomerFilesController');
     //  Custome Notes
@@ -97,8 +95,7 @@ 
 */
 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::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');
@@ -110,11 +107,9 @@ 
 /*
 *   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::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');

Spacing +76 added lines, -76 removed lines

@@ -8,62 +8,62 @@ 
 /*
 *   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
@@ -71,90 +71,90 @@ 
 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');
 });

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/Installer/SettingsController.php

Braces +4 added lines, -2 removed lines

@@ -74,14 +74,16 @@
         ]);
 
         //  Update the site timezone
-        if (config('app.timezone') !== $request->timezone) {
+        if (config('app.timezone') !== $request->timezone)
+        {
             Settings::firstOrCreate(
                 ['key'   => 'app.timezone'],
                 ['key'   => 'app.timezone', 'value' => $request->timezone]
             )->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'],
                 ['key'   => 'filesystems.paths.max_size', 'value' => $request->filesize]

Spacing +18 added lines, -18 removed lines

@@ -26,14 +26,14 @@ 
     //  Bring up the change logo form
     public function logoSettings()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         return view('installer.logoSettings');
     }
     //  Submit the new company logo
     public function submitLogo(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([
             'file' => 'mimes:jpeg,bmp,png,jpg,gif'
@@ -45,17 +45,17 @@ 
 
         Settings::firstOrCreate(
             ['key'   => 'app.logo'],
-            ['key'   => 'app.logo', 'value' => '/storage/img/' . $fileName]
-        )->update(['value' => '/storage/img/' . $fileName]);
+            ['key'   => 'app.logo', 'value' => '/storage/img/'.$fileName]
+        )->update(['value' => '/storage/img/'.$fileName]);
 
-        Log::notice('A new company logo has been uploaded by ' . Auth::user()->full_name);
+        Log::notice('A new company logo has been uploaded by '.Auth::user()->full_name);
 
-        return response()->json(['url' => '/storage/img/' . $fileName]);
+        return response()->json(['url' => '/storage/img/'.$fileName]);
     }
 
     public function configuration()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $settings = collect([
             // 'url'      => config('app.url'),
@@ -71,7 +71,7 @@ 
 
     public function submitConfiguration(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([
             'timezone' => 'required',
@@ -80,14 +80,14 @@ 
         ]);
 
         //  Update the site timezone
-        if (config('app.timezone') !== $request->timezone) {
+        if(config('app.timezone') !== $request->timezone) {
             Settings::firstOrCreate(
                 ['key'   => 'app.timezone'],
                 ['key'   => 'app.timezone', 'value' => $request->timezone]
             )->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'],
                 ['key'   => 'filesystems.paths.max_size', 'value' => $request->filesize]
@@ -101,7 +101,7 @@ 
     //  System backups index page
     public function backupsIndex()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         return view('installer.backupsIndex');
     }
@@ -109,7 +109,7 @@ 
     //  Retrieve the list of backups
     public function getBackups()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $backups = Storage::disk('backup')->files();
         $buArray = [];
@@ -133,7 +133,7 @@ 
     //  Delete a backup
     public function delBackup($name)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         Storage::disk('backup')->delete($name);
 
@@ -144,7 +144,7 @@ 
     //  Download a backup
     public function downloadBackup($name)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         if(Storage::disk('backup')->exists($name))
         {
@@ -159,7 +159,7 @@ 
     //  Create a new backup
     public function runBackup()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         Artisan::call('tb-backup:run');
 
@@ -175,7 +175,7 @@ 
     //  Email Settings form
     public function emailSettings()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $settings = collect([
             'host'       => config('mail.host'),
@@ -194,7 +194,7 @@ 
     //  Send a test email
     public function sendTestEmail(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());
 
         //  TODO - fix this and make it work
         return response()->json(['success' => false]);
@@ -257,7 +257,7 @@ 
     //  Submit the test email form
     public function submitEmailSettings(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([
             'host'       => 'required',

app/Http/Controllers/Customers/CustomerContactsController.php

Braces +2 added lines, -1 removed lines

@@ -128,7 +128,8 @@
 
         //  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;
         }
 

Spacing +4 added lines, -4 removed lines

@@ -23,7 +23,7 @@ 
     //  Store a new customer contact
     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',
@@ -70,7 +70,7 @@ 
 
         //  Determine if there is a parent site with shared contacts
         $parent = Customers::find($id)->parent_id;
-        if ($parent)
+        if($parent)
         {
             $parentList = CustomerContacts::where('cust_id', $parent)
                 ->where('shared', 1)
@@ -121,7 +121,7 @@ 
     //  Update an existing Customer Contact
     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',
@@ -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;
         }