ladybirdweb / faveo-servicedesk-community

app/Http/Controllers/Admin/helpdesk/BanlistController.php

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use App\Http\Requests\helpdesk\BanlistRequest;
 use App\Http\Requests\helpdesk\BanRequest;
 // model
-use App\Model\helpdesk\Email\Banlist;
 use App\User;
 //classes
 use Exception;

app/Http/Controllers/Admin/helpdesk/DepartmentController.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 use App\Model\helpdesk\Agent\Teams;
 use App\Model\helpdesk\Email\Emails;
 use App\Model\helpdesk\Email\Template;
-use App\Model\helpdesk\Manage\Help_topic;
 use App\Model\helpdesk\Manage\Sla_plan;
 use App\Model\helpdesk\Settings\System;
 use App\Model\helpdesk\Ticket\Tickets;

app/Http/Controllers/Admin/helpdesk/GroupController.php

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 use Exception;
 // classes
 use Illuminate\Http\Request;
-use Illuminate\Support\Facades\Input;
 use Lang;
 
 /**

app/Http/Controllers/Admin/helpdesk/LanguageController.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 // requests
 use App\Http\Controllers\Controller;
 //supports
-use App\Http\Requests;
 use Config;
 //classes
 use File;

Spacing +6 added lines, -6 removed lines

@@ -82,27 +82,27 @@ 
     public function getLanguages()
     {
         $path = base_path('resources/lang');
-        $values = scandir($path);  //Extracts names of directories present in lang directory
+        $values = scandir($path); //Extracts names of directories present in lang directory
         $values = array_slice($values, 2); // skips array element $value[0] = '.' & $value[1] = '..'
         return \Datatable::collection(new Collection($values))
-                        ->addColumn('language', function ($model) {
+                        ->addColumn('language', function($model) {
                             if ($model == Config::get('app.fallback_locale')) {
                                 return Config::get('languages.'.$model).' ('.Lang::get('lang.default').')';
                             } else {
                                 return Config::get('languages.'.$model);
                             }
                         })
-                        ->addColumn('id', function ($model) {
+                        ->addColumn('id', function($model) {
                             return $model;
                         })
-                        ->addColumn('status', function ($model) {
+                        ->addColumn('status', function($model) {
                             if (Lang::getLocale() === $model) {
                                 return "<span style='color:green'>".Lang::trans('lang.active').'</span>';
                             } else {
                                 return "<span style='color:red'>".Lang::trans('lang.inactive').'</span>';
                             }
                         })
-                        ->addColumn('Action', function ($model) {
+                        ->addColumn('Action', function($model) {
                             if (Lang::getLocale() === $model) {
                                 return "<a href='change-language/".$model."'><input type='button' class='btn btn-info btn-xs btn-flat' disabled value='".Lang::trans('lang.disable')."'/></a>  
                 <a href='change-language/".$model."' class='btn btn-danger btn-xs btn-flat' disabled><i class='fa fa-trash' style='color:black;'> </i> ".Lang::trans('lang.delete').'</a>';
@@ -227,7 +227,7 @@ 
     {
         if ($lang !== App::getLocale()) {
             if ($lang !== Config::get('app.fallback_locale')) {
-                $deletePath = base_path('resources/lang').'/'.$lang;     //define file path to delete
+                $deletePath = base_path('resources/lang').'/'.$lang; //define file path to delete
                 $success = File::deleteDirectory($deletePath); //remove extracted folder and it's subfolder from lang
                 if ($success) {
                     //sending back with success message

app/Http/Controllers/Admin/helpdesk/PriorityController.php

Unused Use Statements -3 removed lines

@@ -10,17 +10,14 @@
 use App\Http\Requests\helpdesk\PriorityRequest;
 // models
 use App\Model\helpdesk\Email\Emails;
-use App\Model\helpdesk\Manage\Help_topic;
 use App\Model\helpdesk\Settings\CommonSettings;
 use App\Model\helpdesk\Settings\Email;
 use App\Model\helpdesk\Ticket\Ticket_Priority;
-use Auth;
 use DB;
 use Exception;
 // classes
 use Illuminate\Http\Request;
 use Illuminate\support\Collection;
-use Input;
 use Lang;
 
 /**

Spacing +3 added lines, -3 removed lines

@@ -79,10 +79,10 @@ 
 
             return \Datatable::Collection($tickets)
                             ->showColumns('priority', 'priority_desc')
-                            ->addColumn('priority_color', function ($model) {
+                            ->addColumn('priority_color', function($model) {
                                 return "<button class='btn' style = 'background-color:$model->priority_color'></button>";
                             })
-                            ->addColumn('status', function ($model) {
+                            ->addColumn('status', function($model) {
                                 if ($model->status == 1) {
                                     return "<a style='color:green'>active</a>";
                                 } elseif ($model->status == 0) {
@@ -92,7 +92,7 @@ 
                                     return "<a style='color:red'>inactive</a>";
                                 }
                             })
-                            ->addColumn('action', function ($model) {
+                            ->addColumn('action', function($model) {
                                 if ($model->is_default > 0) {
                                     return '<a href='.url('ticket/priority/'.$model->priority_id.'/edit')." class='btn btn-info btn-xs' disabled='disabled'>Edit</a>&nbsp;<a href=".url('ticket/priority/'.$model->priority_id.'/destroy')." class='btn btn-warning btn-info btn-xs' disabled='disabled' > delete </a>";
                                 } else {

app/Http/Controllers/Admin/helpdesk/SecurityController.php

Unused Use Statements -2 removed lines

@@ -8,9 +8,7 @@
 use App\Http\Requests\helpdesk\SecurityRequest;
 use App\Model\helpdesk\Settings\Security;
 // Request
-use Illuminate\Http\Request;
 // Class
-use Input;
 use Lang;
 use Redirect;
 

app/Http/Controllers/Agent/helpdesk/CannedController.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 use App\Http\Requests\helpdesk\CannedUpdateRequest;
 // model
 use App\Model\helpdesk\Agent_panel\Canned;
-use App\User;
 // classes
 use Exception;
 use Lang;

app/Http/Controllers/Agent/helpdesk/DashboardController2.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 // controllers
 use App\Http\Controllers\Controller;
 // models
-use App\User;
 // classes
 use Auth;
 use DB;

app/Http/Controllers/Agent/helpdesk/Filter/FilterController.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use App\Http\Controllers\Agent\helpdesk\TicketController;
 use App\Http\Controllers\Controller;
 use App\Model\helpdesk\Filters\Filter;
-use App\Model\helpdesk\Filters\Label;
 use App\Model\helpdesk\Ticket\Tickets;
 use Auth;
 use DB;

Spacing +16 added lines, -16 removed lines

@@ -35,7 +35,7 @@ 
         $render = false;
         if (is_array($labels) && count($labels) > 0) {
             $table = $table
-                     ->leftJoin('filters as label', function ($join) {
+                     ->leftJoin('filters as label', function($join) {
                          $join->on('tickets.id', '=', 'label.ticket_id')
                                 ->where('label.key', '=', 'label');
                      })
@@ -43,7 +43,7 @@ 
         }
         if (is_array($tags) && count($tags) > 0) {
             $table = $table
-                    ->leftJoin('filters as tag', function ($join) {
+                    ->leftJoin('filters as tag', function($join) {
                         $join->on('tickets.id', '=', 'tag.ticket_id')
                                 ->where('tag.key', '=', 'tag');
                     })
@@ -60,7 +60,7 @@ 
     {
         $filter = new Filter();
         $query = $filter->where('key', $key)
-                ->where(function ($query) use ($labels) {
+                ->where(function($query) use ($labels) {
                     if (is_array($labels) && count($labels) > 0) {
                         for ($i = 0; $i < count($labels); $i++) {
                             $query->orWhere('value', 'LIKE', '%'.$labels[$i].'%');
@@ -87,7 +87,7 @@ 
                 }
 
                 return $table
-                    ->Join('ticket_status', function ($join) {
+                    ->Join('ticket_status', function($join) {
                         $join->on('ticket_status.id', '=', 'tickets.status')
                         ->whereIn('ticket_status.id', [1, 7]);
                     });
@@ -98,13 +98,13 @@ 
                 }
 
                 return $table
-                    ->Join('ticket_status', function ($join) {
+                    ->Join('ticket_status', function($join) {
                         $join->on('ticket_status.id', '=', 'tickets.status')
                                 ->whereIn('ticket_status.state', ['closed']);
                     });
             case '/ticket/myticket':
                     return $table
-                      ->leftJoin('ticket_status', function ($join) {
+                      ->leftJoin('ticket_status', function($join) {
                           $join->on('ticket_status.id', '=', 'tickets.status');
                       })
                     ->orWhere('tickets.assigned_to', '=', Auth::user()->id)
@@ -116,7 +116,7 @@ 
                 }
 
                 return $table
-                     ->leftJoin('ticket_status', function ($join) {
+                     ->leftJoin('ticket_status', function($join) {
                          $join->on('ticket_status.id', '=', 'tickets.status');
                      })
                     ->where('tickets.assigned_to', '=', null)
@@ -128,7 +128,7 @@ 
                 }
 
                   return $table
-                    ->leftJoin('ticket_status', function ($join) {
+                    ->leftJoin('ticket_status', function($join) {
                         $join->on('ticket_status.id', '=', 'tickets.status');
                     })
                     ->where('tickets.status', '=', 1)
@@ -145,7 +145,7 @@ 
                 }
 
                 return $table
-                    ->Join('ticket_status', function ($join) {
+                    ->Join('ticket_status', function($join) {
                         $join->on('ticket_status.id', '=', 'tickets.status')
                                 ->where('tickets.status', '=', 7);
                     });
@@ -157,7 +157,7 @@ 
                 }
 
                 return $table
-                    ->Join('ticket_status', function ($join) {
+                    ->Join('ticket_status', function($join) {
                         $join->on('ticket_status.id', '=', 'tickets.status')
                                 ->where('tickets.status', '=', 5);
                     });
@@ -169,7 +169,7 @@ 
                 }
 
                 return $table
-                    ->Join('ticket_status', function ($join) {
+                    ->Join('ticket_status', function($join) {
                         $join->on('ticket_status.id', '=', 'tickets.status')
                                 ->where('tickets.status', '=', 1)
                                 ->where('tickets.isanswered', '=', 1);
@@ -181,7 +181,7 @@ 
                 }
 
                 return $table
-                     ->leftJoin('ticket_status', function ($join) {
+                     ->leftJoin('ticket_status', function($join) {
                          $join->on('ticket_status.id', '=', 'tickets.status');
                      })
                     ->where('tickets.assigned_to', '>', 0)
@@ -193,7 +193,7 @@ 
                 }
 
                 return $table
-                     ->leftJoin('ticket_status', function ($join) {
+                     ->leftJoin('ticket_status', function($join) {
                          $join->on('ticket_status.id', '=', 'tickets.status');
                      })
                     ->where('isanswered', '=', 0)
@@ -205,7 +205,7 @@ 
                 }
 
                return $table
-                    ->leftJoin('ticket_status', function ($join) {
+                    ->leftJoin('ticket_status', function($join) {
                         $join->on('ticket_status.id', '=', 'tickets.status');
                     })
                     ->where('tickets.status', '=', 1)
@@ -220,7 +220,7 @@ 
                 }
 
                 return $table
-                    ->leftJoin('ticket_status', function ($join) {
+                    ->leftJoin('ticket_status', function($join) {
                         $join->on('ticket_status.id', '=', 'tickets.status');
                     })
                     ->where('tickets.status', '=', 1)
@@ -234,7 +234,7 @@ 
         // if (Auth::user()->role == 'admin') {
             $ticket = new Tickets();
         $tickets = $ticket
-                    ->leftJoin('ticket_thread', function ($join) {
+                    ->leftJoin('ticket_thread', function($join) {
                         $join->on('tickets.id', '=', 'ticket_thread.ticket_id')
                         ->whereNotNull('title')
                         ->where('ticket_thread.is_internal', '<>', 1);