butcherman / Tech_Bench

app/Http/Controllers/InstallerController.php

Spacing +18 added lines, -18 removed lines

@@ -25,12 +25,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;
                 }
@@ -65,7 +65,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'); //
     }
     
     //  Email settings form
@@ -89,12 +89,12 @@ 
         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]);
         }
             
-        return redirect()->back()->with('success', 'Tech Bench Successfully Updated');//
+        return redirect()->back()->with('success', 'Tech Bench Successfully Updated'); //
     }
     
     //  Form to create a new Category
@@ -143,13 +143,13 @@ 
         $sysID = $sysData->sys_id;
         $i = 0;
         
-        foreach($request->custField as $field)
+        foreach ($request->custField as $field)
         {
-            if(!empty($field))
+            if (!empty($field))
             {
                 $id = SystemCustDataTypes::where('name', $field)->first();
 
-                if(is_null($id))
+                if (is_null($id))
                 {
                     $newField = SystemCustDataTypes::create([
                         'name' => $field
@@ -173,7 +173,7 @@ 
         
         Log::info('New System Created', ['cat_id' => $request->catName, 'sys_name' => $request->name, 'user_id' => Auth::user()->user_id]);
         
-        return redirect()->back()->with('success', 'System Successfully Added');//
+        return redirect()->back()->with('success', 'System Successfully Added'); //
     }
     
     //  Form to edit an existing system
@@ -200,7 +200,7 @@ 
         $sysName = SystemTypes::find($sysID)->name;
         
         //  Change the name of the system if it has been modified
-        if($sysName !== $request->name)
+        if ($sysName !== $request->name)
         {
             SystemTypes::find($sysID)->update([
                 'name' => $request->name
@@ -213,19 +213,19 @@ 
             ->get();
         
         $i = 0;
-        foreach($request->custField as $field)
+        foreach ($request->custField as $field)
         {
             $found = false;
-            if(!empty($field))
+            if (!empty($field))
             {
                 //  Check if the field already exists
-                foreach($dataType as $type)
+                foreach ($dataType as $type)
                 {
-                    if($type->name === $field)
+                    if ($type->name === $field)
                     {
                         $found = true;
                         //  See if the order has changed
-                        if($type->order != $i)
+                        if ($type->order != $i)
                         {
                             SystemCustDataFields::find($type->field_id)->update(
                             [
@@ -235,11 +235,11 @@ 
                     }
                 }
                 //  If the field does not exist, create it
-                if(!$found)
+                if (!$found)
                 {
                     $id = SystemCustDataTypes::where('name', $field)->first();
 
-                    if(is_null($id))
+                    if (is_null($id))
                     {
                         $newField = SystemCustDataTypes::create([
                             'name' => $field

Braces +3 added lines, -6 removed lines

@@ -34,8 +34,7 @@ 
                 {
                     $sysArr[$cat->name][] = $sys->name;
                 }
-            }
-            else
+            } else
             {
                 $sysArr[$cat->name] = null;
             }
@@ -155,8 +154,7 @@ 
                         'name' => $field
                     ]);
                     $id = $newField->data_type_id;
-                }
-                else
+                } else
                 {
                     $id = $id->data_type_id;
                 }
@@ -245,8 +243,7 @@ 
                             'name' => $field
                         ]);
                         $id = $newField->data_type_id;
-                    }
-                    else
+                    } else
                     {
                         $id = $id->data_type_id;
                     }

app/Http/Controllers/CustomerDetails.php

Spacing +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
         $custDetails = Customers::find($id);
         
         //  Check for empty data set
-        if(empty($custDetails))
+        if (empty($custDetails))
         {
             return view('err.customerNotFound');
         }
@@ -90,7 +90,7 @@ 
         $custData = 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;
         }

app/Http/Controllers/CustomerContactsController.php

Spacing +9 added lines, -9 removed lines

@@ -26,7 +26,7 @@ 
         $numberTypes = PhoneNumberType::all();
         
         $numTypes = [];
-        foreach($numberTypes as $type)
+        foreach ($numberTypes as $type)
         {
             $numTypes[$type->phone_type_id] = $type->description;
         }
@@ -49,11 +49,11 @@ 
         
         $contID = $cont->cont_id;
         
-        if(!empty(array_filter($request['phoneNumber'])))
+        if (!empty(array_filter($request['phoneNumber'])))
         {
-            for($i=0; $i<count($request['phoneNumber']); $i++)
+            for ($i = 0; $i<count($request['phoneNumber']); $i++)
             {
-                if(!empty($request['phoneNumber'][$i]))
+                if (!empty($request['phoneNumber'][$i]))
                 {
                     CustomerContactPhones::create([
                         'cont_id' => $contID,
@@ -86,7 +86,7 @@ 
         $numberTypes = PhoneNumberType::all();
         
         $numTypes = [];
-        foreach($numberTypes as $type)
+        foreach ($numberTypes as $type)
         {
             $numTypes[$type->phone_type_id] = $type->description;
         }
@@ -112,9 +112,9 @@ 
         
         //  Clear all contact phone numbers and re-enter them
         CustomerContactPhones::where('cont_id', $id)->delete();
-        if(!empty(array_filter($request['phoneNumber'])))
+        if (!empty(array_filter($request['phoneNumber'])))
         {
-            for($i=0; $i<count($request['phoneNumber']); $i++)
+            for ($i = 0; $i<count($request['phoneNumber']); $i++)
             {
                 CustomerContactPhones::create([
                     'cont_id'       => $id,
@@ -156,9 +156,9 @@ 
         $vcard->addEmail($contact->email);
         $vcard->addAddress(null, null, $custData->address, $custData->city, $custData->state, $custData->zip, null);
         
-        if(!empty($numbers))
+        if (!empty($numbers))
         {
-            foreach($numbers as $phone)
+            foreach ($numbers as $phone)
             {
                 $vcard->addPhoneNumber($phone->phone_number, $phone->description);
             }

app/Http/Controllers/CustomerController.php

Indentation +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
                 ->with('CustomerSystems.SystemTypes')
                 ->whereHas('CustomerSystems', function($q) use($request)
                 {
-                   $q->where('sys_id', $request->system);
+                    $q->where('sys_id', $request->system);
                 })
                 ->get();
         }

Spacing +3 added lines, -3 removed lines

@@ -27,9 +27,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;
             }
@@ -44,7 +44,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))
         {
             $customerData = Customers::where('name', 'like', '%'.$request->customer.'%')
                 ->where('city', 'like', '%'.$request->city.'%')

Braces +1 added lines, -2 removed lines

@@ -55,8 +55,7 @@
                    $q->where('sys_id', $request->system);
                 })
                 ->get();
-        }
-        else
+        } else
         {
             $customerData = Customers::where('name', 'like', '%'.$request->customer.'%')
                 ->where('city', 'like', '%'.$request->city.'%')

app/Http/Controllers/CustomerSystemsController.php

Spacing +6 added lines, -6 removed lines

@@ -27,9 +27,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 @@ 
         ]);
         $newSysID = $newSys->cust_sys_id;
         
-        foreach($request->field as $key => $field)
+        foreach ($request->field as $key => $field)
         {
             $fieldID = SystemCustDataFields::where('data_type_id', $key)->where('sys_id', $request->sysType)->first()->field_id;
             
@@ -81,7 +81,7 @@ 
     {
         $custSystems = CustomerSystems::where('cust_id', $id)->get();
 
-        switch($custSystems->count())
+        switch ($custSystems->count())
         {
             case 0:
                 return view('customer.system_none');
@@ -103,7 +103,7 @@ 
             default:
                 $systemData = [];
 
-                foreach($custSystems as $sys)
+                foreach ($custSystems as $sys)
                 {
                     $custSysID = $sys->cust_sys_id;
                     $sysName   = SystemTypes::find($sys->sys_id)->name;
@@ -143,7 +143,7 @@ 
         $custSys = CustomerSystems::find($id);
         $sysID   = $custSys->sys_id;
         
-        foreach($request->field as $key => $field)
+        foreach ($request->field as $key => $field)
         {
             $fieldID = SystemCustDataFields::where('data_type_id', $key)->where('sys_id', $sysID)->first()->field_id;
             

app/Http/Controllers/TechTipsController.php

Indentation +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
             $tipData = TechTips::where('subject', 'like', '%'.$request->tipSearch.'%')
                 ->whereHas('TechTipSystems', function($q) use($request)
                 {
-                   $q->where('sys_id', $request->system);
+                    $q->where('sys_id', $request->system);
                 })
                 ->get();
         }

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;
             }
@@ -108,10 +108,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,
@@ -128,10 +128,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());
@@ -159,7 +159,7 @@ 
         {
             Mail::to($userList)->send(new newTechtip($tipData));
         }
-        catch(Exception $e)
+        catch (Exception $e)
         {
             Log::error('Unable to email new Tech Tip.  Failed because of: '.$e);
         }
@@ -173,7 +173,7 @@ 
     public function details($id, $name)
     {
         $tipData = TechTips::where('tip_id', $id)->with('user')->first();
-        if(empty($tipData))
+        if (empty($tipData))
         {
             return view('err.tipNotFound');
         }
@@ -221,7 +221,7 @@ 
     public function edit($id)
     {
         $tipData = TechTips::find($id);
-        if(empty($tipData))
+        if (empty($tipData))
         {
             return 'tip not found';
         }
@@ -244,9 +244,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;
             }
@@ -275,10 +275,10 @@ 
         
         //  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,
@@ -296,13 +296,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);
@@ -317,10 +317,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());

Braces +5 added lines, -10 removed lines

@@ -60,8 +60,7 @@ 
                    $q->where('sys_id', $request->system);
                 })
                 ->get();
-        }
-        else
+        } else
         {
             $tipData = TechTips::where('subject', 'like', '%'.$request->tipSearch.'%')
                 ->get();
@@ -118,8 +117,7 @@ 
                     'sys_id' => $tag
                 ]);
             }
-        }
-        else
+        } else
         {
             TechTipSystems::create([
                 'tip_id' => $tipID,
@@ -158,8 +156,7 @@ 
         try
         {
             Mail::to($userList)->send(new newTechtip($tipData));
-        }
-        catch(Exception $e)
+        } catch(Exception $e)
         {
             Log::error('Unable to email new Tech Tip.  Failed because of: '.$e);
         }
@@ -285,8 +282,7 @@ 
                     'sys_id' => $tag
                 ]);
             }
-        }
-        else
+        } else
         {
             TechTipSystems::create([
                 'tip_id' => $id,
@@ -308,8 +304,7 @@ 
                         Files::deleteFile($file->file_id);
                     }
                 }
-            }
-            else
+            } else
             {
                 TechTipFiles::where('tip_id', $id)->delete();
             }

app/Http/Controllers/UserLinksController.php

Indentation +1 added lines, -1 removed lines

@@ -100,4 +100,4 @@
         return $request->all();
     }
 }
- 
\ No newline at end of file
+    
\ No newline at end of file

Spacing +5 added lines, -5 removed lines

@@ -25,12 +25,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');
         }
@@ -55,7 +55,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());
@@ -76,7 +76,7 @@ 
                 'upload' => 1
             ]);
             
-            if(!empty($request->note))
+            if (!empty($request->note))
             {
                 FileLinkNotes::create([
                     'link_id' => $details->link_id,
@@ -92,7 +92,7 @@ 
         {
             Mail::to($user)->send(new NewLinkFile($details));
         }
-        catch(Exception $e)
+        catch (Exception $e)
         {
             Log::error('Email not sent to user id'.$details->user_id.'.  Failed because of: '.$e);
         }

Braces +1 added lines, -2 removed lines

@@ -91,8 +91,7 @@
         try
         {
             Mail::to($user)->send(new NewLinkFile($details));
-        }
-        catch(Exception $e)
+        } catch(Exception $e)
         {
             Log::error('Email not sent to user id'.$details->user_id.'.  Failed because of: '.$e);
         }

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

@@ -143,7 +143,7 @@ 
         //  Get the users information
         $userData = User::where('username', $valid->username)->first();
         
-         //  Update the password
+            //  Update the password
         User::find($userData->user_id)->update(
         [
             'password'   => bcrypt($request->newPass)
@@ -180,7 +180,7 @@ 
             'password'   => 'required|string|min:6|confirmed'
         ]);
         
-         //  Update the user data
+            //  Update the user data
         User::find($id)->update(
         [
             'password'   => bcrypt($request->password)
@@ -252,7 +252,7 @@ 
     //  Deactivae an active user
     public function destroy($id)
     {
-         //  Update the user data
+            //  Update the user data
         User::find($id)->update(
         [
             'active'   => 0

Spacing +7 added lines, -7 removed lines

@@ -38,9 +38,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;
             }
@@ -95,7 +95,7 @@ 
         {
             Mail::to($request->email)->send(new InitializeUser($hash, $request->username, $request->first_name.' '.$request->last_name));
         }
-        catch(Exception $e)
+        catch (Exception $e)
         {
             Log::error('Email not sent to new user'.$request->email.'.  Failed because of: '.$e);
         }
@@ -113,7 +113,7 @@ 
         //  Validate the hash token
         $user = UserInitialize::where('token', $hash)->get();
         
-        if($user->isEmpty())
+        if ($user->isEmpty())
         {
             return abort(404);
         }
@@ -126,7 +126,7 @@ 
     {
         //  Verify that the link matches the assigned email address
         $valid = UserInitialize::where('token', $hash)->first();
-        if(empty($valid))
+        if (empty($valid))
         {
             return abort(404);
         }
@@ -198,9 +198,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;
             }

Braces +3 added lines, -6 removed lines

@@ -43,8 +43,7 @@ 
             if($role->role_id == 1 && Auth::user()->role_id != 1)
             {
                 continue;
-            }
-            else
+            } else
             {
                 $roleArr[$role->role_id] = $role->name;
             }
@@ -94,8 +93,7 @@ 
         try
         {
             Mail::to($request->email)->send(new InitializeUser($hash, $request->username, $request->first_name.' '.$request->last_name));
-        }
-        catch(Exception $e)
+        } catch(Exception $e)
         {
             Log::error('Email not sent to new user'.$request->email.'.  Failed because of: '.$e);
         }
@@ -203,8 +201,7 @@ 
             if($role->role_id == 1 && Auth::user()->role_id != 1)
             {
                 continue;
-            }
-            else
+            } else
             {
                 $roleArr[$role->role_id] = $role->name;
             }