Sort milestones by completed date and due date

[anonymous]

Currently, the milestones are sorted by due date on the roadmap and in custom queries when grouped by milestone. If some of the milestones have a due date and others do not, the sort is not as desired for completed milestones. The completed date should be used for the sort rather then the due date for completed milestones.

A workaround at the moment is to enter a due date the same as the completed date for those milestones without a due date. It is inconvenient to remember to do that, but workable.

[cboos]

See r2758 for a description of the new Milestone sort order implemented for the roadmap.

What's left to be done is something similar for the query module, when the tickets are grouped by Milestones.

[anonymous]

please ignore, testing anonymous comments...

[anonymous]

Testing

[anonymous]

Would you like some tasty viagra?

[anonymous]

http://www.bugviagra.com BUY BUY BUY!!!

[cboos]

This is not a test site. Please stop this.

[cboos]

Oops, just saw your mail on Trac Dev ;) Well simulated, jonas ;)

[cmlenz]

Ah well, postponing again.

[techtonik <techtonik@…>]

Query/filter functionality is a good idea. For possible uses see also bug #7342

[Remy Blank <remy.blank@…>]

Patch against 0.11-stable fixing sorting by milestone in queries

[Remy Blank <remy.blank@…>]

Replying to cboos:

What's left to be done is something similar for the query module, when the tickets are grouped by Milestones.

The patch above fixes sorting of query results by milestone, by applying the same sort order as used in the roadmap. The only exception is that it doesn't use the natural sort order when comparing the milestone names (if both completed and due are equal). If anybody knows how to perform a natural sort in SQL, please let me know.

The patch also refactors the query sort code to simplify the SQL construction and removing unnecessary duplication.

[Remy Blank <remy.blank@…>]

Same patch, additionally fixed test cases

[Remy Blank <remy.blank@…>]

I had forgotten to run the test suite. The second patch also fixes the test cases.

[rblank]

Patch applied in [7483].