Mailing List Archive

[The Trac Project] #878: Field "resolution" is disabled incorrectly on Advanced Query screen
#878: Field "resolution" is disabled incorrectly on Advanced Query screen
---------------------------+------------------------------------------------
Id: 878 | Status: new
Component: report system | Modified: Thu Nov 4 14:06:49 2004
Severity: normal | Milestone:
Priority: normal | Version: devel
Owner: daniel | Reporter: pkou <pkou at ua.fm>
---------------------------+------------------------------------------------
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.

--
Ticket URL: <http://projects.edgewall.com/trac/ticket/878>
The Trac Project <>
Re: [The Trac Project] #878: Field "resolution" is disabled incorrectly on Advanced Query screen [ In reply to ]
#878: Field "resolution" is disabled incorrectly on Advanced Query screen
---------------------------+------------------------------------------------
Id: 878 | Status: assigned
Component: ticket system | Modified: Thu Nov 4 16:05:40 2004
Severity: minor | Milestone: 0.8
Priority: low | Version: devel
Owner: cmlenz | Reporter: pkou <pkou at ua.fm>
---------------------------+------------------------------------------------
Changes (by cmlenz):

* component: report system => ticket system
* milestone: => 0.8
* severity: normal => minor
* priority: normal => low
* keywords: => query
* status: new => assigned
* owner: daniel => cmlenz

Comment:

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.

--
Ticket URL: <http://projects.edgewall.com/trac/ticket/878>
The Trac Project <>
Re: [The Trac Project] #878: Field "resolution" is disabled incorrectly on Advanced Query screen [ In reply to ]
#878: Field "resolution" is disabled incorrectly on Advanced Query screen
---------------------------+------------------------------------------------
Id: 878 | Status: closed
Component: ticket system | Modified: Thu Nov 4 16:11:00 2004
Severity: minor | Milestone: 0.8
Priority: low | Version: devel
Owner: cmlenz | Reporter: pkou <pkou at ua.fm>
---------------------------+------------------------------------------------
Changes (by cmlenz):

* resolution: => fixed
* status: assigned => closed

Comment:

Fixed in [1017].

--
Ticket URL: <http://projects.edgewall.com/trac/ticket/878>
The Trac Project <>