Edgewall Software
Modify

Opened 12 years ago

Closed 12 years ago

Last modified 9 years ago

#3990 closed enhancement (duplicate)

Enable use of 'time' and 'changetime' fields within the query module

Reported by: ilias@… Owned by: Jonas Borgström
Priority: normal Milestone:
Component: ticket system Version: devel
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API 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)

TicketQueryMacroTimeOrdering.diff (547 bytes ) - added by ilias@… 12 years ago.
very basic patch which enables ordering on "time" and "changetime"
TicketQueryMacroAddTimestampFields.diff (666 bytes ) - added by ilias@… 12 years ago.
(supersedes previous patch) woks fine, but misses Timestamp formatting in textfields.
TicketQueryMacroAddTimestampFormatting.diff (562 bytes ) - added by ilias@… 12 years ago.
Timestamp fields are properly formatted within the UI
TicketQueryMacroAddTimestampFormattingInt.diff (1.3 KB ) - added by ilias@… 12 years ago.
previous patch led to a crash. This one works with the UI query module, too.

Download all attachments as: .zip

Change History (11)

comment:1 by Christopher Lenz, 12 years ago

Resolution: duplicate
Status: newclosed

This is a duplicate of #2288.

comment:2 by ilias@…, 12 years ago

Resolution: duplicate
Status: closedreopened

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 ilias@…, 12 years ago

very basic patch which enables ordering on "time" and "changetime"

by ilias@…, 12 years ago

(supersedes previous patch) woks fine, but misses Timestamp formatting in textfields.

by ilias@…, 12 years ago

Timestamp fields are properly formatted within the UI

by ilias@…, 12 years ago

previous patch led to a crash. This one works with the UI query module, too.

comment:3 by ilias@…, 12 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 ilias@…, 12 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 ilias@…, 12 years ago

Resolution: wontfix
Status: reopenedclosed

superseded by #4174

other related tickets: #4158, #4019 and others

Ticket 'wontfix' in the way suggested, closing.

comment:6 by ilias@…, 12 years ago

Resolution: wontfix
Status: closedreopened

comment:7 by ilias@…, 12 years ago

Resolution: duplicate
Status: reopenedclosed

not wontfix, but superseded by refined tickets, thus this one become a duplicate of:

#4158, #4174

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jonas Borgström.
The resolution will be deleted.
to The owner will be changed from Jonas Borgström to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.