From 3bec42a9b7eb6d874b717a328de752da4f8629ad Mon Sep 17 00:00:00 2001 From: Jared Hancock <jared@osticket.com> Date: Mon, 8 Sep 2014 17:02:24 -0500 Subject: [PATCH] search: Fix incorrect search hits for assignee --- include/ajax.tickets.php | 6 +++--- include/staff/tickets.inc.php | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/include/ajax.tickets.php b/include/ajax.tickets.php index cdb1a71ef..d481e1c7c 100644 --- a/include/ajax.tickets.php +++ b/include/ajax.tickets.php @@ -159,7 +159,7 @@ class TicketsAjaxAPI extends AjaxController { } //Assignee - if(isset($req['assignee']) && strcasecmp($req['status'], 'closed')) { + if($req['assignee'] && strcasecmp($req['status'], 'closed')) { # assigned-to $id=preg_replace("/[^0-9]/", "", $req['assignee']); $assignee = $req['assignee']; $where.= ' AND ( ( status.state="open" '; @@ -179,11 +179,11 @@ class TicketsAjaxAPI extends AjaxController { if($req['staffId'] && !$req['status']) //Assigned TO + Closed By $where.= ' OR (ticket.staff_id='.db_input($req['staffId']). ' AND status.state IN("resolved", "closed")) '; - elseif(isset($req['staffId'])) // closed by any + elseif($req['staffId']) // closed by any $where.= ' OR status.state IN("resolved", "closed") '; $where.= ' ) '; - } elseif($req['staffId']) { + } elseif($req['staffId']) { # closed-by $where.=' AND (ticket.staff_id='.db_input($req['staffId']).' AND status.state IN("resolved", "closed")) '; } diff --git a/include/staff/tickets.inc.php b/include/staff/tickets.inc.php index 671600b78..a5ad1dcdd 100644 --- a/include/staff/tickets.inc.php +++ b/include/staff/tickets.inc.php @@ -591,7 +591,7 @@ if ($results) { <select id="assignee" name="assignee"> <option value="">— <?php echo __('Anyone');?> —</option> <option value="0">— <?php echo __('Unassigned');?> —</option> - <option value="<?php echo $thisstaff->getId(); ?>"><?php echo __('Me');?></option> + <option value="s<?php echo $thisstaff->getId(); ?>"><?php echo __('Me');?></option> <?php if(($users=Staff::getStaffMembers())) { echo '<OPTGROUP label="'.sprintf(__('Agents (%d)'),count($users)).'">'; -- GitLab