Edgewall Software

Ticket #878 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

Field "resolution" is disabled incorrectly on Advanced Query screen

Reported by: pkou <pkou at ua.fm> Owned by: cmlenz
Priority: low Milestone: 0.8
Component: ticket system Version: devel
Severity: minor Keywords: query
Cc:

Description

Filtering tickets by field Resolution depends on the statuses that are selected by user in field Status. This is because field resolution is empty when a ticket has status new, assigned, or reopened.

Current implementation enables field Resolution if no statuses are selected or when status closed is selected. This is incorrect because when user selects closed and other status like new, the field Resolution is still enabled, and using it leads to empty screens.

Proposed implementation: Enable field Resolution if no statuses are selected or statuses new, assigned, and reopened are not selected.

Patch attached.

Attachments

patch-queryres-r1014.diff (1.0 KB) - added by pkou <pkou at ua.fm> 4 years ago.
Patch for the defect

Change History

Changed 4 years ago by pkou <pkou at ua.fm>

Patch for the defect

Changed 4 years ago by cmlenz

  • status changed from new to assigned
  • severity changed from normal to minor
  • component changed from report system to ticket system
  • priority changed from normal to low
  • owner changed from daniel to cmlenz
  • milestone set to 0.8
  • keywords query added

Good catch.

I'd like to avoid hardcoding the resolution names in the script. If we can depend on the order of the status values (which pretty much is a given in the current codebase), a simple check for selectedIndex >= 3 should be enough, and more compact.

Changed 4 years ago by cmlenz

  • status changed from assigned to closed
  • resolution set to fixed

Fixed in [1017].

Add/Change #878 (Field "resolution" is disabled incorrectly on Advanced Query screen)

Author



Change Properties
<Author field>
Action
as closed
Next status will be 'reopened'
to The owner will change from cmlenz. Next status will be 'closed'
 
Note: See TracTickets for help on using tickets.