Apply fixes from StyleCI

This commit is contained in:
Manish Verma
2020-03-17 05:14:38 +00:00
committed by StyleCI Bot
parent 9fe479b141
commit ed3bbd76d7
75 changed files with 1285 additions and 1030 deletions

View File

@@ -88,7 +88,7 @@ class FilterController extends Controller
*
* @param null
*
* @var $tickets
* @var
*
* @return builder $tickets
*/
@@ -105,7 +105,35 @@ class FilterController extends Controller
->leftJoin('ticket_thread as th', 'th.ticket_id', '=', 'tickets.id')
->leftJoin('ticket_attachment', 'ticket_attachment.thread_id', '=', 'th.id')
->select(
'tickets.id', 'th.title', 'tickets.ticket_number', 'u1.user_name as c_uname', 'u2.user_name as a_uname', \DB::raw('CONVERT_TZ(max(th.updated_at), "+00:00", "'.$this->gmt.'") as updated_at2'), \DB::raw('CONVERT_TZ(min(th.updated_at), "+00:00", "'.$this->gmt.'") as created_at2'), \DB::raw('CONVERT_TZ(max(tickets.duedate), "+00:00", "'.$this->gmt.'") as duedate'), \DB::raw('max(th.updated_at) as updated_at'), \DB::raw('min(th.updated_at) as created_at'), 'tickets.duedate as due', 'u1.id as c_uid', 'ticket_priority.priority as priority', 'u1.first_name AS c_fname', 'u1.last_name as c_lname', 'u2.id as a_uid', 'u2.first_name as a_fname', 'u2.last_name as a_lname', 'u1.active as verified', 'teams.name', 'tickets.assigned_to', 'ticket_priority.priority_color as color', 'ticket_source.css_class as css', \DB::raw('COUNT(ticket_attachment.thread_id) as countattachment'), DB::raw('COUNT(ticket_collaborator.ticket_id) as countcollaborator'), \DB::raw('COUNT(DISTINCT th.id) as countthread'), \DB::raw('substring_index(group_concat(if(`th`.`is_internal` = 0, `th`.`poster`,null)ORDER By th.id desc) , ",", 1) as last_replier'), \DB::raw('substring_index(group_concat(th.title order by th.id asc SEPARATOR "-||,||-") , "-||,||-", 1) as ticket_title'), 'ticket_source.name as source'
'tickets.id',
'th.title',
'tickets.ticket_number',
'u1.user_name as c_uname',
'u2.user_name as a_uname',
\DB::raw('CONVERT_TZ(max(th.updated_at), "+00:00", "'.$this->gmt.'") as updated_at2'),
\DB::raw('CONVERT_TZ(min(th.updated_at), "+00:00", "'.$this->gmt.'") as created_at2'),
\DB::raw('CONVERT_TZ(max(tickets.duedate), "+00:00", "'.$this->gmt.'") as duedate'),
\DB::raw('max(th.updated_at) as updated_at'),
\DB::raw('min(th.updated_at) as created_at'),
'tickets.duedate as due',
'u1.id as c_uid',
'ticket_priority.priority as priority',
'u1.first_name AS c_fname',
'u1.last_name as c_lname',
'u2.id as a_uid',
'u2.first_name as a_fname',
'u2.last_name as a_lname',
'u1.active as verified',
'teams.name',
'tickets.assigned_to',
'ticket_priority.priority_color as color',
'ticket_source.css_class as css',
\DB::raw('COUNT(ticket_attachment.thread_id) as countattachment'),
DB::raw('COUNT(ticket_collaborator.ticket_id) as countcollaborator'),
\DB::raw('COUNT(DISTINCT th.id) as countthread'),
\DB::raw('substring_index(group_concat(if(`th`.`is_internal` = 0, `th`.`poster`,null)ORDER By th.id desc) , ",", 1) as last_replier'),
\DB::raw('substring_index(group_concat(th.title order by th.id asc SEPARATOR "-||,||-") , "-||,||-", 1) as ticket_title'),
'ticket_source.name as source'
)->groupby('tickets.id');
return $tickets;
@@ -387,7 +415,7 @@ class FilterController extends Controller
*
* @param $table querybuilder
*
* @var $id, $dept
* @var, $dept
*
* @return $table
*/
@@ -410,7 +438,7 @@ class FilterController extends Controller
*
* @param array $value //requested department, $table
*
* @var array $departmentTickets
* @var array
*
* @return $table
*/
@@ -440,7 +468,7 @@ class FilterController extends Controller
*
* @param array $departments
*
* @var array $requested_dept
* @var array
*
* @return array of boolean and array values
*/
@@ -464,7 +492,7 @@ class FilterController extends Controller
*
* @param array $priority, builder $table
*
* @var array $priority_ids
* @var array
*
* @return builder
*/
@@ -483,7 +511,7 @@ class FilterController extends Controller
*
* @param string array $user_name, builder $table
*
* @var array $users
* @var array
*
* @return filtered builder
*/
@@ -508,7 +536,7 @@ class FilterController extends Controller
*
* @param array of string values $name
*
* @var $query, array $teams(all fetched teams id)
* @var, array $teams(all fetched teams id)
*
* @return array $teams
*/
@@ -525,7 +553,7 @@ class FilterController extends Controller
*
* @param string array $name, builder $table
*
* @var array $users (stores id's of agents and admin),
* @var array (stores id's of agents and admin),
* array $teams (stores ids of teams), array asssigned merged arrya of unique elements in $teams and $users
*
* @return builder $table
@@ -557,7 +585,7 @@ class FilterController extends Controller
*
* @param string $type (to check type of filter to apply on date), string $value for filters, builder $table
*
* @var array $date [start and end dates]
* @var array [start and end dates]
*
* @return builder
*/
@@ -599,7 +627,7 @@ class FilterController extends Controller
*
* @param string $value
*
* @var date string $start, date string $end
* @var date string, date string $end
*
* @return array of start and end date
*/
@@ -846,7 +874,7 @@ class FilterController extends Controller
*
* @param array $dates, builder $table, $column (type of filter based on which column is being chosen), $value
*
* @var string $check_column (name of column), array $dates
* @var string (name of column), array $dates
*
* @return builder
*/
@@ -880,7 +908,7 @@ class FilterController extends Controller
*
* @param array $name of source, builder $table
*
* @var array $sources
* @var array
*
* @return builder
*/
@@ -894,7 +922,7 @@ class FilterController extends Controller
return $table->whereIn('tickets.source', $sources);
}
/** DEPRICATED
/** DEPRICATED.
* @category function to get array of status to filter tickets
*
* @param string $status
@@ -1024,7 +1052,7 @@ class FilterController extends Controller
*
* @param null
*
* @var $system, $tz
* @var, $tz
*
* @return string GMT value of timezone
*/

View File

@@ -265,12 +265,14 @@ class FilterControllerOld extends Controller
DB::raw('COUNT(ticket_collaborator.ticket_id) as countcollaborator'),
'tickets.status',
'tickets.user_id',
'tickets.priority_id', 'tickets.assigned_to',
'tickets.priority_id',
'tickets.assigned_to',
'ticket_status.name as tickets_status',
'ticket_source.css_class as css',
DB::raw('substring_index(group_concat(ticket_thread.poster order by ticket_thread.id desc) , ",", 1) as last_replier'),
DB::raw('substring_index(group_concat(ticket_thread.title order by ticket_thread.id asc) , ",", 1) as ticket_title'),
'u.active as verified')
'u.active as verified'
)
->groupby('tickets.id');
return $tickets;

View File

@@ -412,20 +412,20 @@ class TicketController extends Controller
$link = url('check_ticket/'.$encoded_ticketid);
$this->NotificationController->create($ticket_id, Auth::user()->id, '2');
$this->PhpMailController->sendmail(
$from = $this->PhpMailController->mailfrom('0', $tickets->dept_id),
$to = ['name' => $user_name, 'email' => $email, 'cc' => $collaborators],
$message = [
'subject' => $ticket_subject.'[#'.$ticket_number.']',
'body' => $line.$request->input('reply_content'),
'scenario' => 'ticket-reply',
'attachments' => $attachment_files,
],
$template_variables = [
'ticket_number' => $ticket_number,
'user' => $username,
'agent_sign' => $agentsign,
'system_link' => $link,
]
$from = $this->PhpMailController->mailfrom('0', $tickets->dept_id),
$to = ['name' => $user_name, 'email' => $email, 'cc' => $collaborators],
$message = [
'subject' => $ticket_subject.'[#'.$ticket_number.']',
'body' => $line.$request->input('reply_content'),
'scenario' => 'ticket-reply',
'attachments' => $attachment_files,
],
$template_variables = [
'ticket_number' => $ticket_number,
'user' => $username,
'agent_sign' => $agentsign,
'system_link' => $link,
]
);
}
} catch (\Exception $e) {
@@ -733,12 +733,16 @@ class TicketController extends Controller
if ($auto_response == 0) {
$encoded_ticketid = Crypt::encrypt($ticketdata->id);
$link = url('check_ticket/'.$encoded_ticketid);
$this->PhpMailController->sendmail($from = $this->PhpMailController->mailfrom('0', $ticketdata->dept_id), $to = ['name' => $username, 'email' => $emailadd], $message = ['subject' => $updated_subject, 'scenario' => 'create-ticket-by-agent', 'body' => $body],
$template_variables = [
'agent_sign' => Auth::user()->agent_sign,
'ticket_number' => $ticket_number2,
'system_link' => $link,
]);
$this->PhpMailController->sendmail(
$from = $this->PhpMailController->mailfrom('0', $ticketdata->dept_id),
$to = ['name' => $username, 'email' => $emailadd],
$message = ['subject' => $updated_subject, 'scenario' => 'create-ticket-by-agent', 'body' => $body],
$template_variables = [
'agent_sign' => Auth::user()->agent_sign,
'ticket_number' => $ticket_number2,
'system_link' => $link,
]
);
}
} catch (\Exception $e) {
//dd($e);
@@ -748,12 +752,16 @@ class TicketController extends Controller
try {
if ($auto_response == 0) {
$this->PhpMailController->sendmail($from = $this->PhpMailController->mailfrom('0', $ticketdata->dept_id), $to = ['name' => $username, 'email' => $emailadd], $message = ['subject' => $updated_subject, 'scenario' => 'create-ticket'],
$template_variables = ['user' => $username,
'ticket_number' => $ticket_number2,
'department_sign' => '',
'system_link' => $link,
]);
$this->PhpMailController->sendmail(
$from = $this->PhpMailController->mailfrom('0', $ticketdata->dept_id),
$to = ['name' => $username, 'email' => $emailadd],
$message = ['subject' => $updated_subject, 'scenario' => 'create-ticket'],
$template_variables = ['user' => $username,
'ticket_number' => $ticket_number2,
'department_sign' => '',
'system_link' => $link,
]
);
}
} catch (\Exception $e) {
}
@@ -810,20 +818,23 @@ class TicketController extends Controller
foreach ($emails_to_be_sent as $email_data) {
try {
$this->PhpMailController->sendmail(
$from = $this->PhpMailController->mailfrom('0', $ticketdata->dept_id), $to = [
'user' => $email_data['to_user'],
'email' => $email_data['to_email'],
], $message = [
'subject' => $updated_subject,
'body' => $body, 'scenario' => $mail,
], $template_variables = [
'ticket_agent_name' => $email_data['to_user_name'],
'ticket_client_name' => $username,
'ticket_client_email' => $emailadd,
'user' => $email_data['to_user_name'],
'ticket_number' => $ticket_number2,
'email_address' => $emailadd,
'name' => $ticket_creator, ]
$from = $this->PhpMailController->mailfrom('0', $ticketdata->dept_id),
$to = [
'user' => $email_data['to_user'],
'email' => $email_data['to_email'],
],
$message = [
'subject' => $updated_subject,
'body' => $body, 'scenario' => $mail,
],
$template_variables = [
'ticket_agent_name' => $email_data['to_user_name'],
'ticket_client_name' => $username,
'ticket_client_email' => $emailadd,
'user' => $email_data['to_user_name'],
'ticket_number' => $ticket_number2,
'email_address' => $emailadd,
'name' => $ticket_creator, ]
);
} catch (\Exception $e) {
}
@@ -1153,11 +1164,11 @@ class TicketController extends Controller
return 0;
}
$data = [
'id' => $ticket_status->ticket_number,
'status' => 'Closed',
'first_name' => Auth::user()->first_name,
'last_name' => Auth::user()->last_name,
];
'id' => $ticket_status->ticket_number,
'status' => 'Closed',
'first_name' => Auth::user()->first_name,
'last_name' => Auth::user()->last_name,
];
\Event::fire('change-status', [$data]);
return 'your ticket'.$ticket_status->ticket_number.' has been closed';
@@ -1736,7 +1747,10 @@ class TicketController extends Controller
$notification = Notification::select('id')->where('model_id', '=', $ticket->id)->get();
foreach ($notification as $id) {
$user_notification = UserNotification::where(
'notification_id', '=', $id->id);
'notification_id',
'=',
$id->id
);
$user_notification->delete();
}
$notification = Notification::select('id')->where('model_id', '=', $ticket->id);
@@ -2106,9 +2120,10 @@ class TicketController extends Controller
$email = $email;
$ticket_id = $ticket_id;
$validator = \Validator::make(
['email' => $email,
'name' => $name, ], ['email' => 'required|email',
]
['email' => $email,
'name' => $name, ],
['email' => 'required|email',
]
);
$user = User::where('email', '=', $email)->first();
if ($user) {
@@ -2812,7 +2827,15 @@ class TicketController extends Controller
{
$table = \Datatable::table()
->addColumn(
'', Lang::get('lang.subject'), Lang::get('lang.ticket_id'), Lang::get('lang.priority'), Lang::get('lang.from'), Lang::get('lang.assigned_to'), Lang::get('lang.last_activity'), Lang::get('lang.created-at'))
'',
Lang::get('lang.subject'),
Lang::get('lang.ticket_id'),
Lang::get('lang.priority'),
Lang::get('lang.from'),
Lang::get('lang.assigned_to'),
Lang::get('lang.last_activity'),
Lang::get('lang.created-at')
)
->noScript();
return view('themes.default1.agent.helpdesk.ticket.inbox', compact('table'));
@@ -2827,7 +2850,15 @@ class TicketController extends Controller
{
$table = \Datatable::table()
->addColumn(
'', Lang::get('lang.subject'), Lang::get('lang.ticket_id'), Lang::get('lang.priority'), Lang::get('lang.from'), Lang::get('lang.assigned_to'), Lang::get('lang.last_activity'), Lang::get('lang.created-at'))
'',
Lang::get('lang.subject'),
Lang::get('lang.ticket_id'),
Lang::get('lang.priority'),
Lang::get('lang.from'),
Lang::get('lang.assigned_to'),
Lang::get('lang.last_activity'),
Lang::get('lang.created-at')
)
->noScript();
return view('themes.default1.agent.helpdesk.ticket.open', compact('table'));
@@ -2842,7 +2873,15 @@ class TicketController extends Controller
{
$table = \Datatable::table()
->addColumn(
'', Lang::get('lang.subject'), Lang::get('lang.ticket_id'), Lang::get('lang.priority'), Lang::get('lang.from'), Lang::get('lang.assigned_to'), Lang::get('lang.last_activity'), Lang::get('lang.created-at'))
'',
Lang::get('lang.subject'),
Lang::get('lang.ticket_id'),
Lang::get('lang.priority'),
Lang::get('lang.from'),
Lang::get('lang.assigned_to'),
Lang::get('lang.last_activity'),
Lang::get('lang.created-at')
)
->noScript();
return view('themes.default1.agent.helpdesk.ticket.answered', compact('table'));
@@ -2857,7 +2896,15 @@ class TicketController extends Controller
{
$table = \Datatable::table()
->addColumn(
'', Lang::get('lang.subject'), Lang::get('lang.ticket_id'), Lang::get('lang.priority'), Lang::get('lang.from'), Lang::get('lang.assigned_to'), Lang::get('lang.last_activity'), Lang::get('lang.created-at'))
'',
Lang::get('lang.subject'),
Lang::get('lang.ticket_id'),
Lang::get('lang.priority'),
Lang::get('lang.from'),
Lang::get('lang.assigned_to'),
Lang::get('lang.last_activity'),
Lang::get('lang.created-at')
)
->noScript();
return view('themes.default1.agent.helpdesk.ticket.myticket', compact('table'));
@@ -2872,7 +2919,15 @@ class TicketController extends Controller
{
$table = \Datatable::table()
->addColumn(
'', Lang::get('lang.subject'), Lang::get('lang.ticket_id'), Lang::get('lang.priority'), Lang::get('lang.from'), Lang::get('lang.assigned_to'), Lang::get('lang.last_activity'), Lang::get('lang.created-at'))
'',
Lang::get('lang.subject'),
Lang::get('lang.ticket_id'),
Lang::get('lang.priority'),
Lang::get('lang.from'),
Lang::get('lang.assigned_to'),
Lang::get('lang.last_activity'),
Lang::get('lang.created-at')
)
->noScript();
return view('themes.default1.agent.helpdesk.ticket.overdue', compact('table'));
@@ -2887,7 +2942,15 @@ class TicketController extends Controller
{
$table = \Datatable::table()
->addColumn(
'', Lang::get('lang.subject'), Lang::get('lang.ticket_id'), Lang::get('lang.priority'), Lang::get('lang.from'), Lang::get('lang.assigned_to'), Lang::get('lang.last_activity'), Lang::get('lang.created-at'))
'',
Lang::get('lang.subject'),
Lang::get('lang.ticket_id'),
Lang::get('lang.priority'),
Lang::get('lang.from'),
Lang::get('lang.assigned_to'),
Lang::get('lang.last_activity'),
Lang::get('lang.created-at')
)
->noScript();
return view('themes.default1.agent.helpdesk.ticket.duetodayticket', compact('table'));
@@ -2902,7 +2965,15 @@ class TicketController extends Controller
{
$table = \Datatable::table()
->addColumn(
'', Lang::get('lang.subject'), Lang::get('lang.ticket_id'), Lang::get('lang.priority'), Lang::get('lang.from'), Lang::get('lang.assigned_to'), Lang::get('lang.last_activity'), Lang::get('lang.created-at'))
'',
Lang::get('lang.subject'),
Lang::get('lang.ticket_id'),
Lang::get('lang.priority'),
Lang::get('lang.from'),
Lang::get('lang.assigned_to'),
Lang::get('lang.last_activity'),
Lang::get('lang.created-at')
)
->noScript();
return view('themes.default1.agent.helpdesk.ticket.closed', compact('table'));
@@ -2917,7 +2988,15 @@ class TicketController extends Controller
{
$table = \Datatable::table()
->addColumn(
'', Lang::get('lang.subject'), Lang::get('lang.ticket_id'), Lang::get('lang.priority'), Lang::get('lang.from'), Lang::get('lang.assigned_to'), Lang::get('lang.last_activity'), Lang::get('lang.created-at'))
'',
Lang::get('lang.subject'),
Lang::get('lang.ticket_id'),
Lang::get('lang.priority'),
Lang::get('lang.from'),
Lang::get('lang.assigned_to'),
Lang::get('lang.last_activity'),
Lang::get('lang.created-at')
)
->noScript();
return view('themes.default1.agent.helpdesk.ticket.assigned', compact('table'));
@@ -2952,7 +3031,15 @@ class TicketController extends Controller
}
$table = \Datatable::table()
->addColumn(
'', Lang::get('lang.subject'), Lang::get('lang.ticket_id'), Lang::get('lang.priority'), Lang::get('lang.from'), Lang::get('lang.assigned_to'), Lang::get('lang.last_activity'), Lang::get('lang.created-at'))
'',
Lang::get('lang.subject'),
Lang::get('lang.ticket_id'),
Lang::get('lang.priority'),
Lang::get('lang.from'),
Lang::get('lang.assigned_to'),
Lang::get('lang.last_activity'),
Lang::get('lang.created-at')
)
->noScript();
return view('themes.default1.agent.helpdesk.dept-ticket.tickets', compact('dept', 'status', 'table'));
@@ -2967,7 +3054,15 @@ class TicketController extends Controller
{
$table = \Datatable::table()
->addColumn(
'', Lang::get('lang.subject'), Lang::get('lang.ticket_id'), Lang::get('lang.priority'), Lang::get('lang.from'), Lang::get('lang.assigned_to'), Lang::get('lang.last_activity'), Lang::get('lang.created-at'))
'',
Lang::get('lang.subject'),
Lang::get('lang.ticket_id'),
Lang::get('lang.priority'),
Lang::get('lang.from'),
Lang::get('lang.assigned_to'),
Lang::get('lang.last_activity'),
Lang::get('lang.created-at')
)
->noScript();
return view('themes.default1.agent.helpdesk.ticket.trash', compact('table'));
@@ -2982,7 +3077,15 @@ class TicketController extends Controller
{
$table = \Datatable::table()
->addColumn(
'', Lang::get('lang.subject'), Lang::get('lang.ticket_id'), Lang::get('lang.priority'), Lang::get('lang.from'), Lang::get('lang.assigned_to'), Lang::get('lang.last_activity'), Lang::get('lang.created-at'))
'',
Lang::get('lang.subject'),
Lang::get('lang.ticket_id'),
Lang::get('lang.priority'),
Lang::get('lang.from'),
Lang::get('lang.assigned_to'),
Lang::get('lang.last_activity'),
Lang::get('lang.created-at')
)
->noScript();
return view('themes.default1.agent.helpdesk.ticket.unassigned', compact('table'));
@@ -2997,7 +3100,15 @@ class TicketController extends Controller
{
$table = \Datatable::table()
->addColumn(
'', Lang::get('lang.subject'), Lang::get('lang.ticket_id'), Lang::get('lang.priority'), Lang::get('lang.from'), Lang::get('lang.assigned_to'), Lang::get('lang.last_activity'), Lang::get('lang.created-at'))
'',
Lang::get('lang.subject'),
Lang::get('lang.ticket_id'),
Lang::get('lang.priority'),
Lang::get('lang.from'),
Lang::get('lang.assigned_to'),
Lang::get('lang.last_activity'),
Lang::get('lang.created-at')
)
->noScript();
return view('themes.default1.agent.helpdesk.ticket.myticket', compact('table'));
@@ -3011,7 +3122,15 @@ class TicketController extends Controller
try {
$table = \Datatable::table()
->addColumn(
'', Lang::get('lang.subject'), Lang::get('lang.ticket_id'), Lang::get('lang.priority'), Lang::get('lang.from'), Lang::get('lang.assigned_to'), Lang::get('lang.last_activity'), Lang::get('lang.created-at'))
'',
Lang::get('lang.subject'),
Lang::get('lang.ticket_id'),
Lang::get('lang.priority'),
Lang::get('lang.from'),
Lang::get('lang.assigned_to'),
Lang::get('lang.last_activity'),
Lang::get('lang.created-at')
)
->noScript();
return view('themes.default1.agent.helpdesk.followup.followup', compact('table'));

View File

@@ -82,13 +82,15 @@ class UserController extends Controller
/* get all values in Sys_user */
$table = \ Datatable::table()
->addColumn(Lang::get('lang.name'),
->addColumn(
Lang::get('lang.name'),
Lang::get('lang.email'),
Lang::get('lang.phone'),
Lang::get('lang.status'),
Lang::get('lang.last_login'),
Lang::get('lang.role'),
Lang::get('lang.action')) // these are the column headings to be shown
Lang::get('lang.action')
) // these are the column headings to be shown
->noScript();
return view('themes.default1.agent.helpdesk.user.index', compact('table'));