butcherman / Tech_Bench

app/Http/Controllers/Customers/CustomerFilesController.php

Braces +4 added lines, -2 removed lines

@@ -100,7 +100,8 @@ 
 
         //  Determine if there is a parent site with shared files
         $parent = Customers::find($id)->parent_id;
-        if($parent) {
+        if($parent)
+        {
             $parentList = Customerfiles::where('cust_id', $parent)
                 ->where('shared', 1)
                 ->with('Files')
@@ -126,7 +127,8 @@ 
         ]);
 
         $details = Customers::find($request->cust_id);
-        if($details->parent_id && $request->shared == 1) {
+        if($details->parent_id && $request->shared == 1)
+        {
             $request->cust_id = $details->parent_id;
         }
 

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';

Doc Comments +8 added lines

@@ -204,6 +204,10 @@ 
     }
 
     //  Create the tech tip
+
+    /**
+     * @param Request $tipData
+     */
     private function createTip($tipData)
     {
         //  Remove any forward slash (/) from the Subject Field
@@ -389,6 +393,10 @@ 
     }
 
     //  Store the updated tip
+
+    /**
+     * @param Request $tipData
+     */
     public function storeUpdatedTip($tipData, $id)
     {
         $this->authorize('hasAccess', 'Edit Tech Tip');

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/Files.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
         Log::debug('Preparing to clean a file. Current Name - '.$fileName.'. Current Path - '.$path);
         //  Remove all spaces
         $fileName = str_replace(' ', '_', $fileName);
-        Log::debug('Cleaned Filename. Current Name - ' . $fileName . '. Current Path - ' . $path);
+        Log::debug('Cleaned Filename. Current Name - '.$fileName.'. Current Path - '.$path);
 
         //  Determine if the filename already exists
         if(Storage::exists($path.DIRECTORY_SEPARATOR.$fileName))
@@ -44,7 +44,7 @@ 
             //  Increase the number until one that is not in use is found
             do
             {
-                Log::debug('Filename ' . $fileName . ' already exists.  Appending name.');
+                Log::debug('Filename '.$fileName.' already exists.  Appending name.');
                 $fileName = $base.'('.++$number.')'.$extension;
             } while(Storage::exists($path.DIRECTORY_SEPARATOR.$fileName));
         }

Doc Comments +6 added lines

@@ -14,6 +14,12 @@
     protected $hidden     = ['created_at', 'updated_at', 'file_link'];
 
     //  Remove any illegal characters from filename and make sure it is unique
+
+    /**
+     * @param string $path
+     *
+     * @return string
+     */
     public static function cleanFileName($path, $fileName)
     {
         Log::debug('Preparing to clean a file. Current Name - '.$fileName.'. Current Path - '.$path);

app/Http/Controllers/DashboardController.php

Spacing +8 added lines, -8 removed lines

@@ -22,16 +22,16 @@ 
     //  Dashboard is the Logged In User home landing page
     public function index()
     {
-        Log::debug('Route "' . Route::currentRouteName() . '" visited by ' . Auth::user()->full_name);
+        Log::debug('Route "'.Route::currentRouteName().'" visited by '.Auth::user()->full_name);
 
         //  Get Dashboard data
-        $custFavs    = CustomerFavs::where('user_id', Auth::user()->user_id)
-            ->with(array('Customers' => function($query){
+        $custFavs = CustomerFavs::where('user_id', Auth::user()->user_id)
+            ->with(array('Customers' => function($query) {
                 $query->select('cust_id', 'name');
             }))
             ->get();
-        $tipFavs     = TechTipFavs::where('user_id', Auth::user()->user_id)
-            ->with(array('TechTips' => function($query){
+        $tipFavs = TechTipFavs::where('user_id', Auth::user()->user_id)
+            ->with(array('TechTips' => function($query) {
                 $query->select('tip_id', 'subject');
             }))
             ->get();
@@ -67,7 +67,7 @@ 
     //  Mark a notification as read
     public function markNotification($id)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $notification = Auth::user()->notifications()->where('id', $id)->where('notifiable_id', Auth::user()->user_id)->first();
         if(!$notification)
@@ -85,12 +85,12 @@ 
     //  Delte a user notification
     public function delNotification($id)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $notification = Auth::user()->notifications()->where('id', $id)->where('notifiable_id', Auth::user()->user_id)->first();
         if(!$notification)
         {
-            Log::error('User ' . Auth::user()->full_name . ' tried to delete an invalid notification.  Notification ID: ' . $id);
+            Log::error('User '.Auth::user()->full_name.' tried to delete an invalid notification.  Notification ID: '.$id);
             return abort(404);
         }
 

Braces +2 added lines, -2 removed lines

@@ -26,12 +26,12 @@
 
         //  Get Dashboard data
         $custFavs    = CustomerFavs::where('user_id', Auth::user()->user_id)
-            ->with(array('Customers' => function($query){
+            ->with(array('Customers' => function($query) {
                 $query->select('cust_id', 'name');
             }))
             ->get();
         $tipFavs     = TechTipFavs::where('user_id', Auth::user()->user_id)
-            ->with(array('TechTips' => function($query){
+            ->with(array('TechTips' => function($query) {
                 $query->select('tip_id', 'subject');
             }))
             ->get();

app/Http/Controllers/FileLinks/LinkFilesController.php

Spacing +2 added lines, -2 removed lines

@@ -89,7 +89,7 @@ 
     //  Show the files attached to a link
     public function show($id)
     {
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
 
         $files = new FileLinkFilesCollection(
             FileLinkFiles::where('link_id', $id)
@@ -160,7 +160,7 @@ 
     //  Delete a file attached to a link
     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);
         Log::debug('Submitted File ID to be deleted - '.$id);
 
         //  Get the necessary file information and delete it from the database