butcherman / Tech_Bench

app/Http/Controllers/Installer/CategoriesController.php

Spacing +5 added lines, -5 removed lines

@@ -15,7 +15,7 @@ 
     public function __construct()
     {
         $this->middleware('auth');
-        $this->middleware(function ($request, $next) {
+        $this->middleware(function($request, $next) {
             $this->authorize('hasAccess', 'Manage Equipment');
             return $next($request);
         });
@@ -35,7 +35,7 @@ 
     //  Store the new category form
     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([
             'name' => 'required|string|unique:system_categories|regex:/^[a-zA-Z0-9_ ]*$/'
@@ -53,7 +53,7 @@ 
     //  Submit the Edit Category form
     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([
             'name' => [
@@ -75,14 +75,14 @@ 
     //  Delete an existing category - note this will fail if the category has systems assigned to it
     public function destroy($id)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         try {
             SystemCategories::find($id)->delete();
             Log::notice('Category ID '.$id.' deleted by '.Auth::user()->full_name);
             return response()->json(['success' => true, 'reason' => 'Category Successfully Deleted']);
         }
-        catch (\Illuminate\Database\QueryException $e)
+        catch(\Illuminate\Database\QueryException $e)
         {
             Log::warning('User '.Auth::user()->full_name.' tried to delete category ID '.$id.' but was unable to since it is still in use.');
             return response()->json(['success' => false, 'reason' => 'Category still in use.  You must delete all systems attached to this category first.']);

Braces +2 added lines, -1 removed lines

@@ -77,7 +77,8 @@
     {
         Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
 
-        try {
+        try
+        {
             SystemCategories::find($id)->delete();
             Log::notice('Category ID '.$id.' deleted by '.Auth::user()->full_name);
             return response()->json(['success' => true, 'reason' => 'Category Successfully Deleted']);

app/Policies/GatePolicy.php

Spacing +7 added lines, -7 removed lines

@@ -37,13 +37,13 @@ 
     //  Determine if a user can see the Administration Nav Link
     public function seeAdminLink(User $user)
     {
-        if ($this->isInstaller($user))
+        if($this->isInstaller($user))
         {
             return true;
         }
 
         $data = UserRolePermissions::with('UserRolePermissionTypes')
-            ->whereHas('UserRolePermissionTypes', function ($query) {
+            ->whereHas('UserRolePermissionTypes', function($query) {
                 $query->where('description', 'Manage Users')
                     ->orWhere('description', 'Manage User Roles')
                     ->orWhere('description', 'Manage Customers')
@@ -54,7 +54,7 @@ 
             ->get();
 
         $allow = $data->isEmpty() ? 'Denied' : 'Allowed';
-        Log::debug('User ' . $user->full_name . ' is trying to see admin link.  Result - ' . $allow);
+        Log::debug('User '.$user->full_name.' is trying to see admin link.  Result - '.$allow);
 
         return  $data->isEmpty() ? false : true;
     }
@@ -69,7 +69,7 @@ 
         }
 
         $data = UserRolePermissions::with('UserRolePermissionTypes')
-            ->whereHas('UserRolePermissionTypes', function ($query) use ($task) {
+            ->whereHas('UserRolePermissionTypes', function($query) use ($task) {
                 $query->where('description', $task);
             })
             ->where('role_id', $user->role_id)
@@ -77,10 +77,10 @@ 
             ->get();
 
         $allow = $data->isEmpty() ? 'false' : 'true';
-        Log::debug('User '.$user->full_name.' is trying to access '.$task.'.  Result - ' . $allow);
-        if ($allow === 'Denied')
+        Log::debug('User '.$user->full_name.' is trying to access '.$task.'.  Result - '.$allow);
+        if($allow === 'Denied')
         {
-            Log::alert('User ' . $user->full_name . ' was denied from accessing '.$task.' link.');
+            Log::alert('User '.$user->full_name.' was denied from accessing '.$task.' link.');
         }
 
         return  $data->isEmpty() ? false : true;