sort version combo descending in new ticket form

[Dirk <trac@…>]

It be more natural to sort the versions combo box descending for the new ticket and ticket view?

Most bugs are filed against the jungest version of a software. If you have lots of versions you have to scroll down evertime. This change has also the side effect that the default version is set to unspecified for new issues if not specified in the trac.ini file.

A similar change would be interesting for the milestone field, as long as the milestones are related to the version number. Bugzilla has a sort order key for this to be able to sort also non numeric and non alphabetical orderd milestones correctly.

[Dirk <trac@…>]

patch for descending sort order of the version number field

[Manuzhai]

This makes sense, would be nice to have.

[deminix@…]

i agree that a DESC order is good, but it should be ordered by the date/time attribute of a version, not its string sort order.

those without times specified should sort before those that do, imo

[cboos]

Something similar to r2758 and r2760 could be done for versions, too.

(and this should take care of the version part of #1150)

[cboos]

e.g.

trac/ticket/model.py

@@ -724 +724 @@
         if not db:
             db = env.get_db_cnx()
         cursor = db.cursor()
-        cursor.execute("SELECT name,time,description FROM version "
-                       "ORDER BY COALESCE(time,0),name")
+        cursor.execute("SELECT name,time,description FROM version")
+        versions = []
         for name, time, description in cursor:
-            component = cls(env)
-            component.name = name
-            component.time = time and int(time) or None
-            component.description = description or ''
-            yield component
+            version = cls(env)
+            version.name = name
+            version.time = time and int(time) or None
+            version.description = description or ''
+            versions.append(version)
+        def version_order(v):
+            return (v.time or sys.maxint, embedded_numbers(v.name))
+        return sorted(versions, key=version_order, reverse=True)
     select = classmethod(select)

[cboos]

Implemented the above in r2767 and r2768.

[anonymous]

lll