butcherman / Tech_Bench

app/Http/Controllers/TechTips/TechTipsController.php

Braces +17 added lines, -8 removed lines

@@ -53,14 +53,17 @@ 
         Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name.'. Submitted Data - ', $request->toArray());
 
         //  See if there are any search paramaters entered
-        if(!$request->search['searchText'] && !isset($request->search['articleType']) && !isset($request->search['systemType'])) {
+        if(!$request->search['searchText'] && !isset($request->search['articleType']) && !isset($request->search['systemType']))
+        {
             //  No search paramaters, send all tech tips
             $tips = new TechTipsCollection(
                 TechTips::orderBy('created_at', 'DESC')
                     ->with('SystemTypes')
                     ->paginate($request->pagination['perPage'])
             );
-        } else {
+        }
+        else
+        {
             $article = isset($request->search['articleType']) ? true : false;
             $system  = isset($request->search['systemType']) ? true : false;
             //  Search paramaters, filter results
@@ -151,7 +154,8 @@ 
         $save = $receiver->receive();
 
         //  See if the uploade has finished
-        if($save->isFinished()) {
+        if($save->isFinished())
+        {
             $this->saveFile($save->getFile());
 
             return 'uploaded successfully';
@@ -266,7 +270,8 @@ 
     //  Details controller - will move to the show controller with just the tech tip id
     public function details($id, $subject)
     {
-        if(session()->has('newTipFile')) {
+        if(session()->has('newTipFile'))
+        {
             session()->forget('newTipFile');
         }
 
@@ -300,7 +305,8 @@ 
     {
         Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
-        switch($action) {
+        switch($action)
+        {
             case 'add':
                 TechTipFavs::create([
                     'user_id' => Auth::user()->user_id,
@@ -329,7 +335,8 @@ 
         $this->authorize('hasAccess', 'Edit Tech Tip');
         $tipData = TechTips::where('tip_id', $id)->with('User')->with('SystemTypes')->with('TechTipTypes')->first();
 
-        if(!$tipData) {
+        if(!$tipData)
+        {
             return view('tips.tipNotFound');
         }
 
@@ -362,7 +369,8 @@ 
         $receiver = new FileReceiver('file', $request, HandlerFactory::classFromRequest($request));
 
         //  Verify if there is a file to be processed or not
-        if($receiver->isUploaded() === false || $request->_completed) {
+        if($receiver->isUploaded() === false || $request->_completed)
+        {
             $this->storeUpdatedTip($request, $id);
             Log::debug('Route '.Route::currentRouteName().' visited by User ID-'.Auth::user()->user_id);
             return response()->json(['tip_id' => $id]);
@@ -372,7 +380,8 @@ 
         $save = $receiver->receive();
 
         //  See if the uploade has finished
-        if($save->isFinished()) {
+        if($save->isFinished())
+        {
             $this->saveFile($save->getFile(), $id);
 
             return 'uploaded successfully';

app/Http/Controllers/Installer/ModuleController.php

Braces +6 added lines, -3 removed lines

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

app/Http/Controllers/Installer/SettingsController.php

Spacing +2 added lines, -2 removed lines

@@ -80,7 +80,7 @@ 
         ]);
 
         //  Update the site timezone
-        if (config('app.timezone') !== $request->timezone)
+        if(config('app.timezone') !== $request->timezone)
         {
             Settings::firstOrCreate(
                 ['key'   => 'app.timezone'],
@@ -88,7 +88,7 @@ 
             )->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'],

app/Http/Controllers/Installer/SystemsController.php

Braces +2 added lines, -1 removed lines

@@ -211,7 +211,8 @@
     public function destroy($id)
     {
         //
-        try {
+        try
+        {
             SystemTypes::find($id)->delete();
             return response()->json(['success' => true, 'reason' => 'Equipment Successfully Deleted']);
         }

app/Http/Controllers/DownloadController.php

Braces +10 added lines, -4 removed lines

@@ -59,9 +59,12 @@ 
     public function archiveFiles(Request $request)
     {
         //  Debug Data
-        if(Auth::user()) {
+        if(Auth::user())
+        {
             $user = Auth::user()->full_name;
-        } else {
+        }
+        else
+        {
             $user = \Request::ip();
         }
         Log::debug('Route '.Route::currentRouteName().' visited by '.$user.'  Request Data:', $request->toArray());
@@ -107,9 +110,12 @@ 
     public function downloadArchive($fileName)
     {
         //  Debug Data
-        if(Auth::user()) {
+        if(Auth::user())
+        {
             $user = Auth::user()->full_name;
-        } else {
+        }
+        else
+        {
             $user = \Request::ip();
         }
         Log::debug('Route '.Route::currentRouteName().' visited by '.$user);

app/Http/Controllers/Admin/AdminController.php

Spacing +2 added lines, -2 removed lines

@@ -89,7 +89,7 @@ 
         Log::notice('User '.Auth::user()->full_name.' updated User Password Policy');
 
         //  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
@@ -110,7 +110,7 @@ 
     public function roleSettings()
     {
         $this->authorize('hasAccess', 'Manage User Roles');
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
         $roles = UserRoleType::with(['UserRolePermissions' => function($query) {
             $query->join('user_role_permission_types', 'user_role_permission_types.perm_type_id', '=', 'user_role_permissions.perm_type_id');
         }])->get();

app/Http/Controllers/Admin/UserController.php

Spacing +5 added lines, -5 removed lines

@@ -182,13 +182,13 @@ 
 
         //  Good to go - get role information
         $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;
             }
-            else if ($role->role_id == 2 && Auth::user()->role_id > 1)
+            else if($role->role_id == 2 && Auth::user()->role_id > 1)
             {
                 continue;
             }
@@ -285,13 +285,13 @@ 
         $user = User::find($request->user_id);
 
         //  Verify this is a valid user ID
-        if (!$user)
+        if(!$user)
         {
             $success = false;
             $reason  = 'Cannot find user with this ID';
         }
         //  Make sure that the user is not trying to deactivate someone with more permissions
-        else if ($user->role_id < Auth::user()->role_id)
+        else if($user->role_id < Auth::user()->role_id)
         {
             $success = false;
             $reason  = 'You cannot change password for a user with higher permissions that you.  If this user has locked themselves out, have then use the reset link on the login page.';