butcherman / Tech_Bench

app/Console/Commands/BackupApplication.php

Spacing +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
         $i = 0;
         do
         {
-            if($i)
+            if ($i)
             {
                 $backupName = $backupBase.'('.$i.')';
             }
@@ -48,7 +48,7 @@ 
                 $backupName = $backupBase;
             }
             $i++;
-        } while(Storage::disk('backup')->exists($backupName.'.zip'));
+        } while (Storage::disk('backup')->exists($backupName.'.zip'));
         
         //  Write a file that shows the system version    
         $version = new \PragmaRX\Version\Package\Version();

Braces +1 added lines, -2 removed lines

@@ -42,8 +42,7 @@
             if($i)
             {
                 $backupName = $backupBase.'('.$i.')';
-            }
-            else
+            } else
             {
                 $backupName = $backupBase;
             }

app/Http/Controllers/UserLinksController.php

Spacing +4 added lines, -4 removed lines

@@ -26,12 +26,12 @@ 
         $details = FileLinks::where('link_hash', $id)->first();
         
         //  Verify that the link is valid
-        if(empty($details))
+        if (empty($details))
         {
             return view('links.guest.badLink');
         }
         //  Verify that the link has not expired
-        else if($details->expire <= date('Y-m-d'))
+        else if ($details->expire<=date('Y-m-d'))
         {
             return view('links.guest.expiredLink');
         }
@@ -56,7 +56,7 @@ 
         
         $filePath = config('filesystems.paths.links').DIRECTORY_SEPARATOR.$details->link_id;
 
-        foreach($request->file as $file)
+        foreach ($request->file as $file)
         {
             //  Clean the file and store it
             $fileName = Files::cleanFilename($filePath, $file->getClientOriginalName());
@@ -77,7 +77,7 @@ 
                 'upload'   => 1
             ]);
             
-            if(!empty($request->note))
+            if (!empty($request->note))
             {
                 FileLinkNotes::create([
                     'link_id' => $details->link_id,

app/Http/Controllers/DashboardController.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     public function markNotification($id)
     {
         $notification = Auth::user()->notifications()->where('id', $id)->first();
-        if($notification)
+        if ($notification)
         {
             $notification->delete();
             Log::info('Notification ID-'.$id.' deleted for User ID-'.Auth::user()->user_id);

Braces +1 added lines, -2 removed lines

@@ -44,8 +44,7 @@
         {
             $notification->delete();
             Log::info('Notification ID-'.$id.' deleted for User ID-'.Auth::user()->user_id);
-        }
-        else
+        } else
         {
             Log::notice('Notification ID-'.$id.' not found for user ID-'.Auth::user()->user_id);
         }

app/Http/Controllers/TechTipsController.php

Spacing +22 added lines, -22 removed lines

@@ -35,9 +35,9 @@ 
             ->get();
         
         $sysArr = [];
-        foreach($systems as $sys)
+        foreach ($systems as $sys)
         {
-            foreach($sys->SystemTypes as $s)
+            foreach ($sys->SystemTypes as $s)
             {
                 $sysArr[$sys->name][$s->sys_id] = $s->name;
             }
@@ -52,7 +52,7 @@ 
     public function search(Request $request)
     {
         //  Run different request based on if system field is filled out or not
-        if(!empty($request->system))
+        if (!empty($request->system))
         {
             $tipData = TechTips::where('subject', 'like', '%'.$request->tipSearch.'%')
                 ->whereHas('TechTipSystems', function($q) use($request)
@@ -81,9 +81,9 @@ 
             ->get();
         
         $sysArr = [];
-        foreach($systems as $sys)
+        foreach ($systems as $sys)
         {
-            foreach($sys->SystemTypes as $s)
+            foreach ($sys->SystemTypes as $s)
             {
                 $sysArr[$sys->name][$s->sys_id] = $s->name;
             }
@@ -112,10 +112,10 @@ 
         $tipID = $tip->tip_id;
 
         //  Enter all system tags associated with the tip
-        if(is_array($request->sysTags))
+        if (is_array($request->sysTags))
         {
             
-            foreach($request->sysTags as $tag)
+            foreach ($request->sysTags as $tag)
             {
                 TechTipSystems::create([
                     'tip_id' => $tipID,
@@ -132,10 +132,10 @@ 
         }
         
         //  If there are any files, process them
-        if(!empty($request->file))
+        if (!empty($request->file))
         {
             $filePath = config('filesystems.paths.tips').DIRECTORY_SEPARATOR.$tipID;
-            foreach($request->file as $file)
+            foreach ($request->file as $file)
             {
                 //  Clean the file and store it
                 $fileName = Files::cleanFilename($filePath, $file->getClientOriginalName());
@@ -163,7 +163,7 @@ 
         {
             Mail::to($userList)->send(new newTechtip($tipData));
         }
-        catch(Exception $e)
+        catch (Exception $e)
         {
             report($e);
         }
@@ -177,7 +177,7 @@ 
     public function details($id, $name)
     {
         $tipData = TechTips::where('tip_id', $id)->with('user')->first();
-        if(empty($tipData))
+        if (empty($tipData))
         {
             Log::warning('User ID-'.Auth::user()->user_id.' tried to access invlaid Tech Tip ID-'.$id.' Name-'.$name);
             return view('errors.tipNotFound');
@@ -226,7 +226,7 @@ 
     public function edit($id)
     {
         $tipData = TechTips::find($id);
-        if(empty($tipData))
+        if (empty($tipData))
         {
             return 'tip not found';
         }
@@ -249,9 +249,9 @@ 
             ->get();
         
         $sysArr = [];
-        foreach($systems as $sys)
+        foreach ($systems as $sys)
         {
-            foreach($sys->SystemTypes as $s)
+            foreach ($sys->SystemTypes as $s)
             {
                 $sysArr[$sys->name][$s->sys_id] = $s->name;
             }
@@ -284,9 +284,9 @@ 
         
         //  Enter all system tags associated with the tip after destroying the existing systems
         TechTipSystems::where('tip_id', $id)->delete();
-        if(is_array($request->sysTags))
+        if (is_array($request->sysTags))
         {
-            foreach($request->sysTags as $tag)
+            foreach ($request->sysTags as $tag)
             {
                 TechTipSystems::create([
                     'tip_id' => $id,
@@ -304,13 +304,13 @@ 
         
         //  Determine if any files were removed
         $tipFiles = TechTipFiles::where('tip_id', $id)->get();
-        if(!$tipFiles->isEmpty())
+        if (!$tipFiles->isEmpty())
         {
-            if(!empty($request->existingFile))
+            if (!empty($request->existingFile))
             {
-                foreach($tipFiles as $file)
+                foreach ($tipFiles as $file)
                 {
-                    if(!in_array($file->file_id, $request->existingFile))
+                    if (!in_array($file->file_id, $request->existingFile))
                     {
                         TechTipFiles::where('file_id', $file->file_id)->delete();
                         Files::deleteFile($file->file_id);
@@ -324,10 +324,10 @@ 
         }
         
         //  Process any new files
-        if(!empty($request->file))
+        if (!empty($request->file))
         {
             $filePath = config('filesystems.paths.tips').DIRECTORY_SEPARATOR.$id;
-            foreach($request->file as $file)
+            foreach ($request->file as $file)
             {
                 //  Clean the file and store it
                 $fileName = Files::cleanFilename($filePath, $file->getClientOriginalName());

app/Http/Controllers/InstallerController.php

Spacing +10 added lines, -10 removed lines

@@ -27,12 +27,12 @@ 
         $cats = SystemCategories::all();
         $sysArr = [];
         //  Populate that list with the matching systems
-        foreach($cats as $cat)
+        foreach ($cats as $cat)
         {
             $systems = SystemTypes::where('cat_id', $cat->cat_id)->get();
-            if(!$systems->isEmpty())
+            if (!$systems->isEmpty())
             {
-                foreach($systems as $sys)
+                foreach ($systems as $sys)
                 {
                     $sysArr[$cat->name][] = $sys->name;
                 }
@@ -67,7 +67,7 @@ 
         
         Log::info('Tech Bench Settings Updated', ['user_id' => Auth::user()->user_id]);
         
-        return redirect()->back()->with('success', 'Tech Bench Successfully Updated');//
+        return redirect()->back()->with('success', 'Tech Bench Successfully Updated'); //
     }
     
     //  Upload and submit a new site logo
@@ -111,7 +111,7 @@ 
             'mail.username'   => $request->username,
         ]);
         
-        if(!empty($request->password))
+        if (!empty($request->password))
         {
             config(['mail.password' => $request->password]);
         }
@@ -123,7 +123,7 @@ 
             Mail::to(Auth::user()->email)->send(new TestEmail());
             return 'success';
         }
-        catch(Exception $e)
+        catch (Exception $e)
         {
             Log::notice('Test Email Failed.  Message: '.$e);
             $msg = '['.$e->getCode().'] "'.$e->getMessage().'" on line '.$e->getTrace()[0]['line'].' of file '.$e->getTrace()[0]['file'];
@@ -146,13 +146,13 @@ 
         Settings::where('key', 'mail.port')->update(['value' => $request->port]);
         Settings::where('key', 'mail.encryption')->update(['value' => $request->encryption]);
         Settings::where('key', 'mail.username')->update(['value' => $request->username]);
-        if(!empty($request->password))
+        if (!empty($request->password))
         {
             Settings::where('key', 'mail.password')->update(['value' => $request->password]);
         }
          
         Log::info('Email Settings have been changed by User ID-'.Auth::user()->user_id);
-        return redirect()->back()->with('success', 'Tech Bench Successfully Updated');//
+        return redirect()->back()->with('success', 'Tech Bench Successfully Updated'); //
     }
     
     //  User settings form
@@ -174,14 +174,14 @@ 
         
         //  Determine if the password expires field is updated
         $oldExpire = config('users.passExpires');
-        if($request->passExpire != $oldExpire)
+        if ($request->passExpire != $oldExpire)
         {
             //  Update the setting in the database
             Settings::where('key', 'users.passExpires')->update([
                 'value' => $request->passExpire
             ]);
             //  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

Braces +3 added lines, -6 removed lines

@@ -36,8 +36,7 @@ 
                 {
                     $sysArr[$cat->name][] = $sys->name;
                 }
-            }
-            else
+            } else
             {
                 $sysArr[$cat->name] = null;
             }
@@ -122,8 +121,7 @@ 
             Log::info('Test Email Successfully Sent to '.Auth::user()->email);
             Mail::to(Auth::user()->email)->send(new TestEmail());
             return 'success';
-        }
-        catch(Exception $e)
+        } catch(Exception $e)
         {
             Log::notice('Test Email Failed.  Message: '.$e);
             $msg = '['.$e->getCode().'] "'.$e->getMessage().'" on line '.$e->getTrace()[0]['line'].' of file '.$e->getTrace()[0]['file'];
@@ -186,8 +184,7 @@ 
                 User::whereNotNull('password_expires')->update([
                     'password_expires' => null
                 ]);
-            }
-            else
+            } else
             {
                 $newExpire = Carbon::now()->addDays($request->passExpire);
                 User::whereNull('password_expires')->update([

app/Http/Controllers/SystemController.php

Spacing +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
     {
         //  Make sure that the category is valid
         $valid = SystemCategories::where('name', $cat)->get();
-        if($valid->isEmpty())
+        if ($valid->isEmpty())
         {
             Log::warning('User '.Auth::user()->user_id.' tried to visit invalid category '.$cat);
             return view('errors.badCategory');
@@ -58,7 +58,7 @@ 
         //  Make sure that the system is valid
         $sys = urldecode($sys);
         $valid = SystemTypes::where('name', $sys)->first();
-        if(empty($valid))
+        if (empty($valid))
         {
             Log::warning('User '.Auth::user()->user_id.' tried to visit invalid system - '.$sys.' for category '.$cat);
             return view('errors.badSystem');

app/Http/Controllers/CustomerFilesController.php

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
     {
         $fileTypes = CustomerFileTypes::all();
         $fTypes = [];
-        foreach($fileTypes as $type)
+        foreach ($fileTypes as $type)
         {
             $fTypes[$type->file_type_id] = $type->description;
         }
@@ -93,7 +93,7 @@ 
         $custFile  = CustomerFiles::find($id);
         $fileTypes = CustomerFileTypes::all();
         $fTypes    = [];
-        foreach($fileTypes as $type)
+        foreach ($fileTypes as $type)
         {
             $fTypes[$type->file_type_id] = $type->description;
         }

app/Http/Controllers/UserController.php

Indentation +3 added lines, -3 removed lines

@@ -147,7 +147,7 @@ 
         
         $nextChange = config('users.passExpires') != null ? Carbon::now()->addDays(config('users.passExpires')) : null;
         
-         //  Update the password
+            //  Update the password
         User::find($userData->user_id)->update(
         [
             'password'         => bcrypt($request->newPass),
@@ -188,7 +188,7 @@ 
         
         $nextChange = isset($request->force_change) && $request->force_change == 'on' ? Carbon::now()->subDay() : null;
         
-         //  Update the user data
+            //  Update the user data
         User::find($id)->update(
         [
             'password'         => bcrypt($request->password),
@@ -273,7 +273,7 @@ 
             $linkController->destroy($link->link_id);
         }
         
-         //  Update the user data
+            //  Update the user data
         User::find($id)->update(
         [
             'active'   => 0

Spacing +8 added lines, -8 removed lines

@@ -41,9 +41,9 @@ 
         $roles = Role::all();
         
         $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;
             }
@@ -96,7 +96,7 @@ 
         {
             Mail::to($request->email)->send(new InitializeUser($hash, $request->username, $request->first_name.' '.$request->last_name));
         }
-        catch(Exception $e)
+        catch (Exception $e)
         {
             report($e);
         }
@@ -114,7 +114,7 @@ 
         //  Validate the hash token
         $user = UserInitialize::where('token', $hash)->get();
         
-        if($user->isEmpty())
+        if ($user->isEmpty())
         {
             return abort(404);
         }
@@ -127,7 +127,7 @@ 
     {
         //  Verify that the link matches the assigned email address
         $valid = UserInitialize::where('token', $hash)->first();
-        if(empty($valid))
+        if (empty($valid))
         {
             Log::notice('Someone tried to access an invalid User Initialization link - '.$hash);
             return abort(404);
@@ -207,9 +207,9 @@ 
         $userRole = DB::select('SELECT `role_id` FROM `user_role` WHERE `user_id` = ?', [$id])[0]->role_id;
         
         $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;
             }
@@ -268,7 +268,7 @@ 
         $links = FileLinks::where('user_id', $id)->get();
         
         
-        foreach($links as $link)
+        foreach ($links as $link)
         {
             $linkController->destroy($link->link_id);
         }

app/Http/Controllers/CustomerDetails.php

Spacing +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
         $custDetails = Customers::find($id);
         
         //  Check for empty data set
-        if(empty($custDetails))
+        if (empty($custDetails))
         {
             return view('errors.customerNotFound');
         }
@@ -85,7 +85,7 @@ 
         Customers::find($id)->update($request->all());
 
         //  Modify to the new ID number if set
-        if(isset($request->cust_id))
+        if (isset($request->cust_id))
         {
             $id = $request->cust_id;
         }