ladybirdweb / faveo-servicedesk-community

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

Doc Comments +4 added lines, -5 removed lines

@@ -81,7 +81,7 @@ 
      * @param type Template        $template
      * @param type TemplateRequest $request
      *
-     * @return type Response
+     * @return \Illuminate\Http\RedirectResponse Response
      */
     public function store(Template $template, TemplateRequest $request)
     {
@@ -115,7 +115,6 @@ 
     /**
      * Show the form for editing the specified resource.
      *
-     * @param type           $id
      * @param type Template  $template
      * @param type Languages $language
      *
@@ -243,7 +242,7 @@ 
      * @param type Template      $template
      * @param type TemplateUdate $request
      *
-     * @return type Response
+     * @return \Illuminate\Http\RedirectResponse Response
      */
     public function update($id, Template $template, TemplateUdate $request)
     {
@@ -270,7 +269,7 @@ 
      * @param type int      $id
      * @param type Template $template
      *
-     * @return type Response
+     * @return \Illuminate\Http\RedirectResponse Response
      */
     public function destroy($id, Template $template)
     {
@@ -313,7 +312,7 @@ 
      *
      * @param type Request $request
      *
-     * @return type
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function postDiagno(DiagnosRequest $request)
     {

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

Doc Comments +9 added lines

@@ -97,6 +97,9 @@ 
         return $string;
     }
 
+    /**
+     * @param string $string
+     */
     public function writeHtaccess($string)
     {
         //dd(public_path('.htaccess'),base_path('.htaccess'));
@@ -144,6 +147,9 @@ 
         return $check;
     }
 
+    /**
+     * @param string $url
+     */
     public function checkWWW($url)
     {
         $check = $this->checkWwwInUrl($url);
@@ -157,6 +163,9 @@ 
         return $array;
     }
 
+    /**
+     * @param string $url
+     */
     public function checkHTTP($url)
     {
         $check = $this->checkHttpsInUrl($url);

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

Doc Comments +10 added lines, -11 removed lines

@@ -161,7 +161,7 @@ 
      *
      * @param \App\Http\Requests\helpdesk\WorkflowCreateRequest $request
      *
-     * @return type view
+     * @return \Illuminate\Http\RedirectResponse view
      */
     public function store(WorkflowCreateRequest $request)
     {
@@ -205,7 +205,6 @@ 
      * Editing the details of the banned users.
      *
      * @param type $id
-     * @param User $ban
      *
      * @return type Response
      */
@@ -229,7 +228,7 @@ 
      * @param type                                              $id
      * @param \App\Http\Requests\helpdesk\WorkflowUpdateRequest $request
      *
-     * @return type view
+     * @return \Illuminate\Http\RedirectResponse view
      */
     public function update($id, WorkflowUpdateRequest $request)
     {
@@ -297,7 +296,7 @@ 
      * @param type                     $id
      * @param \Illuminate\Http\Request $request
      *
-     * @return type void
+     * @return string|null void
      */
     public function selectAction($id, Request $request)
     {
@@ -335,7 +334,7 @@ 
     /**
      * function to return deprtment select option.
      *
-     * @return type string
+     * @return string string
      */
     public function department($id)
     {
@@ -352,7 +351,7 @@ 
     /**
      * function to return the priority select option.
      *
-     * @return type string
+     * @return string string
      */
     public function priority($id)
     {
@@ -369,7 +368,7 @@ 
     /**
      * function to return the slaplan select option.
      *
-     * @return type string
+     * @return string string
      */
     public function slaPlan($id)
     {
@@ -386,7 +385,7 @@ 
     /**
      * function to get system team select option.
      *
-     * @return type string
+     * @return string string
      */
     public function assignTeam($id)
     {
@@ -403,7 +402,7 @@ 
     /**
      * function to get system agents select option.
      *
-     * @return type string
+     * @return string string
      */
     public function assignAgent($id)
     {
@@ -420,7 +419,7 @@ 
     /**
      * function to get the helptopic select option.
      *
-     * @return type string
+     * @return string string
      */
     public function helptopic($id)
     {
@@ -437,7 +436,7 @@ 
     /**
      * function to get the select option to choose the ticket status.
      *
-     * @return type string
+     * @return string string
      */
     public function ticketStatus($id)
     {

Spacing +8 added lines, -8 removed lines

@@ -79,11 +79,11 @@ 
                         /* order column name and description */
                         ->orderColumns('name')
                         /* add column name */
-                        ->addColumn('name', function ($model) {
+                        ->addColumn('name', function($model) {
                             return $model->name;
                         })
                         /* add column status */
-                        ->addColumn('status', function ($model) {
+                        ->addColumn('status', function($model) {
                             if ($model->status == 1) {
                                 return 'Active';
                             } elseif ($model->status == 0) {
@@ -91,17 +91,17 @@ 
                             }
                         })
                         /* add column order */
-                        ->addColumn('order', function ($model) {
+                        ->addColumn('order', function($model) {
                             return $model->order;
                         })
                         /* add column rules */
-                        ->addColumn('rules', function ($model) {
+                        ->addColumn('rules', function($model) {
                             $rules = WorkflowRules::where('workflow_id', '=', $model->id)->count();
 
                             return $rules;
                         })
                         /* add column target */
-                        ->addColumn('target', function ($model) {
+                        ->addColumn('target', function($model) {
                             $target = $model->target;
                             $target1 = explode('-', $target);
                             if ($target1[0] == 'A') {
@@ -121,15 +121,15 @@ 
                             }
                         })
                         /* add column created */
-                        ->addColumn('Created', function ($model) {
+                        ->addColumn('Created', function($model) {
                             return TicketController::usertimezone($model->created_at);
                         })
                         /* add column updated */
-                        ->addColumn('Updated', function ($model) {
+                        ->addColumn('Updated', function($model) {
                             return TicketController::usertimezone($model->updated_at);
                         })
                         /* add column action */
-                        ->addColumn('Actions', function ($model) {
+                        ->addColumn('Actions', function($model) {
                             $confirmation = 'Are you sure?';
 
                             return "<a class='btn btn-info btn-xs btn-flat' href='".route('workflow.edit', $model->id)."'><i class='fa fa-edit text-black'></i> Edit</a>  <a class='btn btn-danger btn-xs btn-flat' href='".route('workflow.delete', $model->id)."'><i class='fa fa-trash text-black'></i> Delete</a>";

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

Doc Comments +4 added lines, -4 removed lines

@@ -74,7 +74,7 @@ 
      * @param type CannedRequest $request
      * @param type Canned        $canned
      *
-     * @return type Redirect
+     * @return \Illuminate\Http\RedirectResponse Redirect
      */
     public function store(CannedRequest $request, Canned $canned)
     {
@@ -119,7 +119,7 @@ 
      * @param type CannedUpdateRequest $request
      * @param type Canned              $canned
      *
-     * @return type Redirect
+     * @return \Illuminate\Http\RedirectResponse Redirect
      */
     public function update($id, CannedUpdateRequest $request, Canned $canned)
     {
@@ -145,7 +145,7 @@ 
      * @param type        $id
      * @param type Canned $canned
      *
-     * @return type Redirect
+     * @return \Illuminate\Http\RedirectResponse Redirect
      */
     public function destroy($id, Canned $canned)
     {
@@ -168,7 +168,7 @@ 
      *
      * @param type $id
      *
-     * @return type json
+     * @return \Illuminate\Http\JsonResponse json
      */
     public function get_canned($id)
     {

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

Doc Comments +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
     /**
      * Fetching dashboard graph data to implement graph.
      *
-     * @return type Json
+     * @return string Json
      */
     public function ChartData($date111 = '', $date122 = '')
     {

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

Doc Comments +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
     /**
      * Fetching dashboard graph data to implement graph.
      *
-     * @return type Json
+     * @return string Json
      */
     public function ChartData($date111 = '', $date122 = '')
     {

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;

Indentation +19 added lines, -19 removed lines

@@ -35,10 +35,10 @@ 
         $render = false;
         if (is_array($labels) && count($labels) > 0) {
             $table = $table
-                     ->leftJoin('filters as label', function ($join) {
-                         $join->on('tickets.id', '=', 'label.ticket_id')
+                        ->leftJoin('filters as label', function ($join) {
+                            $join->on('tickets.id', '=', 'label.ticket_id')
                                 ->where('label.key', '=', 'label');
-                     })
+                        })
                     ->whereIn('label.value', $labels);
         }
         if (is_array($tags) && count($tags) > 0) {
@@ -104,9 +104,9 @@ 
                     });
             case '/ticket/myticket':
                     return $table
-                      ->leftJoin('ticket_status', function ($join) {
-                          $join->on('ticket_status.id', '=', 'tickets.status');
-                      })
+                        ->leftJoin('ticket_status', function ($join) {
+                            $join->on('ticket_status.id', '=', 'tickets.status');
+                        })
                     ->orWhere('tickets.assigned_to', '=', Auth::user()->id)
                     ->where('tickets.status', '=', 1);
             case '/unassigned':
@@ -116,9 +116,9 @@ 
                 }
 
                 return $table
-                     ->leftJoin('ticket_status', function ($join) {
-                         $join->on('ticket_status.id', '=', 'tickets.status');
-                     })
+                        ->leftJoin('ticket_status', function ($join) {
+                            $join->on('ticket_status.id', '=', 'tickets.status');
+                        })
                     ->where('tickets.assigned_to', '=', null)
                     ->where('tickets.status', '=', 1);
             case '/ticket/overdue':
@@ -127,7 +127,7 @@ 
                     $table = $table->where('tickets.dept_id', '=', $id);
                 }
 
-                  return $table
+                    return $table
                     ->leftJoin('ticket_status', function ($join) {
                         $join->on('ticket_status.id', '=', 'tickets.status');
                     })
@@ -181,9 +181,9 @@ 
                 }
 
                 return $table
-                     ->leftJoin('ticket_status', function ($join) {
-                         $join->on('ticket_status.id', '=', 'tickets.status');
-                     })
+                        ->leftJoin('ticket_status', function ($join) {
+                            $join->on('ticket_status.id', '=', 'tickets.status');
+                        })
                     ->where('tickets.assigned_to', '>', 0)
                     ->where('tickets.status', '=', 1);
             case '/ticket/open':
@@ -193,9 +193,9 @@ 
                 }
 
                 return $table
-                     ->leftJoin('ticket_status', function ($join) {
-                         $join->on('ticket_status.id', '=', 'tickets.status');
-                     })
+                        ->leftJoin('ticket_status', function ($join) {
+                            $join->on('ticket_status.id', '=', 'tickets.status');
+                        })
                     ->where('isanswered', '=', 0)
                     ->where('tickets.status', '=', 1);
             case '/duetoday':
@@ -204,7 +204,7 @@ 
                     $table = $table->where('tickets.dept_id', '=', $id);
                 }
 
-               return $table
+                return $table
                     ->leftJoin('ticket_status', function ($join) {
                         $join->on('ticket_status.id', '=', 'tickets.status');
                     })
@@ -326,8 +326,8 @@ 
         $table = $this->table();
         if ($user->role == 'user') {
             $table = $table->leftJoin('ticket_status', 'tickets.status', '=', 'ticket_status.id')
-                     ->where('tickets.user_id', '=', $user->id)
-                     ->where('ticket_status.name', $convert_to_array[3]);
+                        ->where('tickets.user_id', '=', $user->id)
+                        ->where('ticket_status.name', $convert_to_array[3]);
         } else {
             $table = $table->leftJoin('ticket_status', 'tickets.status', '=', 'ticket_status.id')
                     ->where('tickets.assigned_to', '=', $user->id)

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

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

Doc Comments +6 added lines

@@ -270,6 +270,9 @@ 
         }
     }
 
+    /**
+     * @param string $filename
+     */
     public function manageAttachment($data, $filename, $type, $size, $disposition, $thread_id)
     {
         $upload = new Ticket_attachments();
@@ -284,6 +287,9 @@ 
         }
     }
 
+    /**
+     * @param string $filename
+     */
     public function updateBody($attachment, $thread_id, $filename)
     {
         $structure = $attachment->getStructure();

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

Doc Comments +5 added lines, -5 removed lines

@@ -144,7 +144,7 @@ 
      * @param type Organization        $org
      * @param type OrganizationRequest $request
      *
-     * @return type Redirect
+     * @return \Illuminate\Http\RedirectResponse Redirect
      */
     public function store(Organization $org, OrganizationRequest $request)
     {
@@ -211,7 +211,7 @@ 
      * @param type Organization       $org
      * @param type OrganizationUpdate $request
      *
-     * @return type Redirect
+     * @return \Illuminate\Http\RedirectResponse Redirect
      */
     public function update($id, Organization $org, OrganizationUpdate $request)
     {
@@ -239,7 +239,7 @@ 
      *
      * @param type int $id
      *
-     * @return type Redirect
+     * @return \Illuminate\Http\RedirectResponse Redirect
      */
     public function destroy($id, Organization $org, User_org $user_org)
     {
@@ -266,7 +266,7 @@ 
      *
      * @param type $id
      *
-     * @return type boolean
+     * @return integer boolean
      */
     public function Head_Org($id)
     {
@@ -288,7 +288,7 @@ 
      * @param type $date111
      * @param type $date122
      *
-     * @return type array
+     * @return string array
      */
     public function orgChartData($id, $date111 = '', $date122 = '')
     {

Spacing +4 added lines, -4 removed lines

@@ -75,7 +75,7 @@ 
                         /* order by name and website */
                         ->orderColumns('name', 'website')
                         /* column name */
-                        ->addColumn('name', function ($model) {
+                        ->addColumn('name', function($model) {
                             // return $model->name;
                             if (strlen($model->name) > 20) {
                                 $orgname = substr($model->name, 0, 25);
@@ -87,19 +87,19 @@ 
                             return $orgname;
                         })
                         /* column website */
-                        ->addColumn('website', function ($model) {
+                        ->addColumn('website', function($model) {
                             $website = $model->website;
 
                             return $website;
                         })
                         /* column phone number */
-                        ->addColumn('phone', function ($model) {
+                        ->addColumn('phone', function($model) {
                             $phone = $model->phone;
 
                             return $phone;
                         })
                         /* column action buttons */
-                        ->addColumn('Actions', function ($model) {
+                        ->addColumn('Actions', function($model) {
                             // displaying action buttons
                             // modal popup to delete data
                             return '<span  data-toggle="modal" data-target="#deletearticle'.$model->id.'"><a href="#" ><button class="btn btn-danger btn-xs"></a> '.\Lang::get('lang.delete').' </button></span>&nbsp;<a href="'.route('organizations.edit', $model->id).'" class="btn btn-warning btn-xs">'.\Lang::get('lang.edit').'</a>&nbsp;<a href="'.route('organizations.show', $model->id).'" class="btn btn-primary btn-xs">'.\Lang::get('lang.view').'</a>