TicketQuery "format=table" generates bad table header links

[Jason Winnebeck <jpwasp@…>]

[[TicketQuery(status!=closed,summary^=PPR,max=0,order=id,format=table)]] generates bad links on the table header. It generates a link (for example on the "summary" header) to https://<MyTracSite>/query?status=%21closed&max=0&summary=%5EPPR&order=summary&page=0. When clicking the link, up comes a Trac error "Query page 0 is invalid." If I manually change the link in my browser to page=1, it works.

My opinion is that either having it link to page=1 (so that clicking goes to a query page), or not having any links at all (because it's a "static" query in the page) would be good possible solutions. I didn't suggest that the query macro could update in the page itself (i.e. the actual wiki page refreshes with the new order), as I figure that would be a very difficult task (esp with multiple TicketQuery per page) with not a lot of benefit.

(0.11-stable r7295)

[Christian Boos]

Thanks for the report, I'll look into this one as well.

[Christian Boos]

Fix the page=0 issue + some cleanups

[Christian Boos]

Please try out the above patch.

[Jason Winnebeck <jpwasp@…>]

It appears to be working, actually in the generated URL I don't see a page parameter at all anymore.

[Christian Boos]

Thanks for the feedback Jason.

Fix committed as [7305].