butcherman / Tech_Bench

app/Http/Controllers/Admin/CustomerAdminController.php

Spacing +11 added lines, -11 removed lines

@@ -16,7 +16,7 @@ 
     public function __construct()
     {
         $this->middleware('auth');
-        $this->middleware(function ($request, $next) {
+        $this->middleware(function($request, $next) {
             $this->authorize('hasAccess', 'Manage Customers');
             return $next($request);
         });
@@ -50,14 +50,14 @@ 
     //  Form to view what kind of file types can be assigned to customers
     public function fileTypes()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
         return view('admin.customerFileTypes');
     }
 
     //  Get the types of files that can be assigned to a customer file
     public function getFileTypes()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $types = new CustomerFileTypesCollection(CustomerFileTypes::all());
 
@@ -68,7 +68,7 @@ 
     //  Submit a new file type name
     public function submitNewFileType(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([
             'name' => 'required',
         ]);
@@ -77,14 +77,14 @@ 
             'description' => $request->name,
         ]);
 
-        Log::notice('New Customer File Type ' . $request->name . ' created by '. Auth::user()->full_name);
+        Log::notice('New Customer File Type '.$request->name.' created by '.Auth::user()->full_name);
         return response()->json(['success' => true]);
     }
 
     //  Update the name of a file type
     public function setFileTypes(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([
             'name' => 'required',
             'id'   => 'required|exists:customer_file_types,file_type_id',
@@ -101,14 +101,14 @@ 
     //  Try to delete a file type
     public function delFileType($id)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         try {
             //  Try to delete file type from database - will throw error if foreign key is in use
             CustomerFileTypes::find($id)->delete();
-        } catch (\Illuminate\Database\QueryException $e) {
+        } catch(\Illuminate\Database\QueryException $e) {
             //  Unable to remove file type  from the database
-            Log::warning('Attempt to delete file type ID '.$id.' by User '.Auth::user()->full_name.' failed.  Reason - ' . $e);
+            Log::warning('Attempt to delete file type ID '.$id.' by User '.Auth::user()->full_name.' failed.  Reason - '.$e);
             return response()->json(['success' => false, 'reason' => 'In Use']);
         }
 
@@ -119,7 +119,7 @@ 
     //  Show all disabled customers
     public function showDisabled()
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $custList = Customers::select('cust_id', 'name', 'deleted_at')
             ->onlyTrashed()
@@ -133,7 +133,7 @@ 
     //  Re-enable a customer
     public function enableCustomer($id)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         Customers::withTrashed()->where('cust_id', $id)->restore();
 

Braces +5 added lines, -2 removed lines

@@ -103,10 +103,13 @@
     {
         Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
 
-        try {
+        try
+        {
             //  Try to delete file type from database - will throw error if foreign key is in use
             CustomerFileTypes::find($id)->delete();
-        } catch (\Illuminate\Database\QueryException $e) {
+        }
+        catch (\Illuminate\Database\QueryException $e)
+        {
             //  Unable to remove file type  from the database
             Log::warning('Attempt to delete file type ID '.$id.' by User '.Auth::user()->full_name.' failed.  Reason - ' . $e);
             return response()->json(['success' => false, 'reason' => 'In Use']);

app/Domains/Users/GetUserStats.php

Braces +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
     public function getUserCustomerFavs()
     {
         $custFavs    = CustomerFavs::where('user_id', $this->userID)
-            ->with(array('Customers' => function($query){
+            ->with(array('Customers' => function($query) {
                 $query->select('cust_id', 'name');
             }))
             ->get();
@@ -35,7 +35,7 @@ 
     public function getUserTechTipFavs()
     {
         $tipFavs     = TechTipFavs::where('user_id', $this->userID)
-            ->with(array('TechTips' => function($query){
+            ->with(array('TechTips' => function($query) {
                 $query->select('tip_id', 'subject');
             }))
             ->get();

Spacing +5 added lines, -5 removed lines

@@ -22,8 +22,8 @@ 
 
     public function getUserCustomerFavs()
     {
-        $custFavs    = CustomerFavs::where('user_id', $this->userID)
-            ->with(array('Customers' => function($query){
+        $custFavs = CustomerFavs::where('user_id', $this->userID)
+            ->with(array('Customers' => function($query) {
                 $query->select('cust_id', 'name');
             }))
             ->get();
@@ -41,8 +41,8 @@ 
 
     public function getUserTechTipFavs()
     {
-        $tipFavs     = TechTipFavs::where('user_id', $this->userID)
-            ->with(array('TechTips' => function($query){
+        $tipFavs = TechTipFavs::where('user_id', $this->userID)
+            ->with(array('TechTips' => function($query) {
                 $query->select('tip_id', 'subject');
             }))
             ->get();
@@ -68,7 +68,7 @@ 
 
     public function getUserTotalLinks()
     {
-        $totalLinks  = FileLinks::where('user_id', $this->userID)->count();
+        $totalLinks = FileLinks::where('user_id', $this->userID)->count();
 
         Log::debug('Retrieved count of total File Links for User ID'.$this->userID.'. Data - './** @scrutinizer ignore-type */$totalLinks.' found');
         return $totalLinks;

app/Http/Controllers/FileLinks/FileLinksController.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
     public function find($id)
     {
         //  If the user is trying to access the links of another user, they must have the proper permissions permissions
-        if ($id != 0)
+        if($id != 0)
         {
             $this->authorize('hasAccess', 'manage_users');
         }

app/Domains/FileLinks/GetFileLinks.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
                     ->orderBy('expire', 'desc')->get();
 
         Log::debug('Retrieved all File Links for User ID '.$this->id.'.  Data Gathered - ', array($links));
-        if ($this->collection) {
+        if($this->collection) {
             return new FileLinksCollection($links);
         }
 

Braces +2 added lines, -1 removed lines

@@ -26,7 +26,8 @@
                     ->orderBy('expire', 'desc')->get();
 
         Log::debug('Retrieved all File Links for User ID '.$this->id.'.  Data Gathered - ', array($links));
-        if ($this->collection) {
+        if ($this->collection)
+        {
             return new FileLinksCollection($links);
         }
 

app/Domains/TechTips/GetTechTipStats.php

Spacing +2 added lines, -2 removed lines

@@ -12,14 +12,14 @@
 {
     public function getTipsInLastDays($numDays = 30)
     {
-        $tips  = TechTips::where('created_at', '>', Carbon::now()->subDays($numDays))->count();
+        $tips = TechTips::where('created_at', '>', Carbon::now()->subDays($numDays))->count();
         Log::debug('Retrieved new Tech Tip count for last '.$numDays.' days.  Count - $tips');
         return $tips;
     }
 
     public function getTotalTechTipCount()
     {
-        $tipsTotal   = TechTips::all()->count();
+        $tipsTotal = TechTips::all()->count();
         Log::debug('Retrieved total Tech Tip count.  Count - $tips');
         return $tipsTotal;
     }

app/Domains/FilesDomain.php

Spacing +6 added lines, -6 removed lines

@@ -74,13 +74,13 @@ 
     protected function isFileDup($fileName)
     {
         //  Determine if the filename already exists
-        if (Storage::exists($this->path . DIRECTORY_SEPARATOR . $fileName))
+        if(Storage::exists($this->path.DIRECTORY_SEPARATOR.$fileName))
         {
             $fileParts = pathinfo($fileName);
-            $extension = isset($fileParts['extension']) ? ('.' . $fileParts['extension']) : '';
+            $extension = isset($fileParts['extension']) ? ('.'.$fileParts['extension']) : '';
 
             //  Look to see if a number is already appended to a file.  (example - file(1).pdf)
-            if (preg_match('/(.*?)(\d+)$/', $fileParts['filename'], $match))
+            if(preg_match('/(.*?)(\d+)$/', $fileParts['filename'], $match))
             {
                 // Has a number, increment it
                 $base = $match[1];
@@ -96,8 +96,8 @@ 
             //  Increase the number until one that is not in use is found
             do
             {
-                $fileName = $base . '(' . ++$number . ')' . $extension;
-            } while (Storage::exists($this->path.DIRECTORY_SEPARATOR.$fileName));
+                $fileName = $base.'('.++$number.')'.$extension;
+            } while(Storage::exists($this->path.DIRECTORY_SEPARATOR.$fileName));
         }
 
         return $fileName;
@@ -108,7 +108,7 @@ 
     {
         $data = Files::find($fileID);
         //  Move the file to the proper folder
-        try{
+        try {
             Log::debug('Attempting to moving file '.$fileID.' to '.$newPath);
             Storage::move($data->file_link.$data->file_name, $newPath.DIRECTORY_SEPARATOR.$data->file_name);
         }

Braces +2 added lines, -1 removed lines

@@ -108,7 +108,8 @@
     {
         $data = Files::find($fileID);
         //  Move the file to the proper folder
-        try{
+        try
+        {
             Log::debug('Attempting to moving file '.$fileID.' to '.$newPath);
             Storage::move($data->file_link.$data->file_name, $newPath.DIRECTORY_SEPARATOR.$data->file_name);
         }

app/Domains/Users/SetUserDetails.php

Spacing +1 added lines, -2 removed lines

@@ -81,8 +81,7 @@
 
         //  Determine if there is a new password expire's date
         $newExpire = config('auth.passwords.settings.expire') != null ?
-            Carbon::now()->addDays(config('auth.passwords.settings.expire')) :
-            null;
+            Carbon::now()->addDays(config('auth.passwords.settings.expire')) : null;
 
         User::find($userID)->update([
             'password' => bcrypt($request->newPass),

app/Domains/Customers/CustomerSearch.php

Braces +3 added lines, -6 removed lines

@@ -47,8 +47,7 @@ 
             $searchResults = new CustomersCollection(
                 Customers::orderBy($request->sortField, $request->sortType)
                     //  Search the name, dba name, and cust id columns
-                    ->where(function($query) use ($request)
-                    {
+                    ->where(function($query) use ($request) {
                         $query->where('name', 'like', '%'.$request->name.'%')
                             ->orWhere('cust_id', 'like', '%'.$request->name.'%')
                             ->orWhere('dba_name', 'like', '%'.$request->name.'%');
@@ -58,10 +57,8 @@ 
                     //  Include the customers systems
                     ->with('CustomerSystems.SystemTypes')
                     //  If the system field is present - search for system type
-                    ->when($request->system, function($query) use ($request)
-                    {
-                        $query->whereHas('CustomerSystems.SystemTypes', function($query) use ($request)
-                        {
+                    ->when($request->system, function($query) use ($request) {
+                        $query->whereHas('CustomerSystems.SystemTypes', function($query) use ($request) {
                             $query->where('sys_id', $request->system);
                         });
                     })

app/Domains/Customers/GetCustomerContacts.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
     {
         $contact        = CustomerContacts::where('cont_id', $contID)->with('CustomerContactPhones')->first();
         $contactName    = explode(' ', $contact->name);
-        $contactDetails = collect( (object) [
+        $contactDetails = collect((object) [
             'firstName'   => $contactName[0],
             'lastName'    => isset($contactName[1]) ? $contactName[1] : '',
             'email'       => $contact->eamil,