Edgewall Software

Ticket #3990 (closed enhancement: duplicate)

Opened 2 years ago

Last modified 2 years ago

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

Reported by: ilias@… Owned by: jonas
Priority: normal Milestone:
Component: ticket system Version: devel
Severity: normal Keywords:
Cc:

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

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

Change History

Changed 2 years ago by cmlenz

  • status changed from new to closed
  • resolution set to duplicate

This is a duplicate of #2288.

Changed 2 years ago by ilias@…

  • status changed from closed to reopened
  • resolution duplicate deleted

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.

Changed 2 years ago by ilias@…

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

Changed 2 years ago by ilias@…

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

Changed 2 years ago by ilias@…

Timestamp fields are properly formatted within the UI

Changed 2 years ago by ilias@…

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

Changed 2 years ago by ilias@…

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.

Changed 2 years ago by ilias@…

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.

Changed 2 years ago by ilias@…

  • status changed from reopened to closed
  • resolution set to wontfix

superseded by #4174

other related tickets: #4158, #4019 and others

Ticket 'wontfix' in the way suggested, closing.

Changed 2 years ago by ilias@…

  • status changed from closed to reopened
  • resolution wontfix deleted

Changed 2 years ago by ilias@…

  • status changed from reopened to closed
  • resolution set to duplicate

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

#4158, #4174

Add/Change #3990 (Enable use of 'time' and 'changetime' fields within the query module)

Author



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