Edgewall Software

Ticket #3968 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

TicketQueryMacro should not enforce order-id

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

Description

The TicketQueryMacro uses default-order "id"

source:trunk/trac/ticket/query.py#3882#L720

kwargs.setdefault('order', 'id')

This line does not just set an default. It overrides an existent query parameter "&order=<field>".

It would be of benefit to remove the line completely, thus the original behaviour of the ticket query becomes available within the wiki (currently: order=priority).

This would lead to a more consistent design.

You can test the behaviour by adding this code to a wiki page:

testing order=type

'''keyword-parameter'''

[[TicketQuery(status=assigned|new|reopened, order=type, format=table )]]

'''query-parameter'''

[[TicketQuery(status=assigned|new|reopened&order=type, format=table)]]

context: http://dev.lazaridis.com/base/wiki/PlanTicketQueryMacro

Attachments

Change History

follow-up: ↓ 2   Changed 2 years ago by cboos

  • owner changed from jonas to cboos
  • milestone set to 0.11

Ilias, are you color-blind? Really, please take no offense, but I can hardly stand the mix of olive green and white background you have on your Trac site ;)

Besides this little off-topic COSMETIC issue, I fully agree with you.

in reply to: ↑ 1   Changed 2 years ago by ilias@…

Replying to cboos:

Ilias, are you color-blind? Really, please take no offense, but I can hardly stand the mix of olive green and white background you have on your Trac site ;)

criticism is always welcome.

I've changed it to the value trac uses, although I'm not sure if that's better. I've not yet reached the point where I hand-over this task where it belongs too: to a graphic designer. I promise this will happen within... the next year!

follow-up: ↓ 5   Changed 2 years ago by cboos

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

Fixed in r3999, thanks for the report!

Concerning the TicketQueryin? general, I have a few more pending changes (datetime fixes and being able to select the columns in the output).

  Changed 2 years ago by ilias@…

Sounds good.

I'll continue full-time with this task during the next days, and will emmit some requirements / tickets / patches which raise from personal and from a clients use-cases.

in reply to: ↑ 3   Changed 2 years ago by ilias@…

Replying to cboos:

Fixed in r3999, thanks for the report! Concerning the TicketQueryin? general, I have a few more pending changes (datetime fixes and being able to select the columns in the output).

please look at #4005

Add/Change #3968 (TicketQueryMacro should not enforce order-id)

Author



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