#3990 closed enhancement (duplicate)
Enable use of 'time' and 'changetime' fields within the query module
Reported by: | Owned by: | Jonas Borgström | |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | ticket system | Version: | devel |
Severity: | normal | Keywords: | |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
e.g.: "order=time" and "order=changetime" do not work.
ordering based on "changetime" is important, in order to identify simply active/inactive tickets.
Most possibly, the fields will become available by adding them to this function:
source:trunk/trac/ticket/api.py#3935#L88
testing order=changetime
keyword-parameter [[TicketQuery(status=assigned|new|reopened, order=changetime, format=table )]] [[TicketQuery(status=assigned|new|reopened&order=changetime, format=table )]]
context: http://dev.lazaridis.com/base/wiki/PlanTicketQueryMacro
Attachments (4)
Change History (11)
comment:1 by , 18 years ago
Resolution: | → duplicate |
---|---|
Status: | new → closed |
comment:2 by , 18 years ago
Resolution: | duplicate |
---|---|
Status: | closed → reopened |
This one is just about the core functionality (as used from trac-query macro), and does not affect the UI part.
see the following patch.
by , 18 years ago
Attachment: | TicketQueryMacroTimeOrdering.diff added |
---|
very basic patch which enables ordering on "time" and "changetime"
by , 18 years ago
Attachment: | TicketQueryMacroAddTimestampFields.diff added |
---|
(supersedes previous patch) woks fine, but misses Timestamp formatting in textfields.
by , 18 years ago
Attachment: | TicketQueryMacroAddTimestampFormatting.diff added |
---|
Timestamp fields are properly formatted within the UI
by , 18 years ago
Attachment: | TicketQueryMacroAddTimestampFormattingInt.diff added |
---|
previous patch led to a crash. This one works with the UI query module, too.
comment:3 by , 18 years ago
to avoid confusion:
those 2 patches allow sorting on the two datetime fiels (time, changetime), whilst displaying the datetime values correctly:
attachment:TicketQueryMacroAddTimestampFields.diff attachment:TicketQueryMacroAddTimestampFormattingInt.diff
It would be nice to have them applied to the trunk, before more changes are introduced during the upcoming rework of cboos.
comment:4 by , 18 years ago
In order to simplify this ticket:
I've moved the part which subjects a defect (ticket api) to #4019
The remaining enhancement patch for this ticket is this one:
attachment:TicketQueryMacroAddTimestampFormattingInt.diff
It would be possibly more elegant to declare the fields 'time' and 'changetime' as "datetime" instead of "text", and to handle formatting within the templates.
comment:5 by , 18 years ago
Resolution: | → wontfix |
---|---|
Status: | reopened → closed |
comment:6 by , 18 years ago
Resolution: | wontfix |
---|---|
Status: | closed → reopened |
comment:7 by , 18 years ago
Resolution: | → duplicate |
---|---|
Status: | reopened → closed |
This is a duplicate of #2288.