Project

General

Profile

Actions

Patch #1546

closed

Associated revisions are displayed in wrong order for Git, Mercurial (SHA-1 DVCSs)

Added by Daniel Luz over 16 years ago. Updated almost 16 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
SCM
Target version:
Start date:
2008-06-29
Due date:
% Done:

0%

Estimated time:

Description

Associated revisions are currently sorted by revision name, in ascending order. (source:trunk/app/models/issue.rb@1594#L31) However, certain SCMs don't use sequential numeric values (for instance: Git, Mercurial and Monotone use SHA-1 hashes), and the commits appear possibly in the wrong order for them.

This trivial patch tries to order first by the committed_on field, leaving the revision as a secondary sorting column.


Files

order-by-committed_on.patch (768 Bytes) order-by-committed_on.patch Order primarily by the committed_on field Daniel Luz, 2008-06-29 02:50
Actions #1

Updated by Jean-Philippe Lang over 16 years ago

  • Status changed from New to Closed
  • Target version set to 0.8

You're right. This sort order wasn't updated when the revision field was changed from numeric to string.
Fixed in r1596.

has_and_belongs_to_many :changesets, :order => "#{Changeset.table_name}.committed_on ASC, #{Changeset.table_name}.id ASC"
Actions #2

Updated by Ken Sands almost 16 years ago

  • Status changed from Closed to Reopened

has_and_belongs_to_many :changesets, :order => "#{Changeset.table_name}.committed_on ASC, #{Changeset.table_name}.id ASC"

can I suggest this would be nicer (or at least optional) to be DESC rather than ASC so the latest revision is the topmost, as per activity and most other lists throughout the interface? that's how I've set it in mine.

Actions #3

Updated by Jean-Philippe Lang almost 16 years ago

  • Status changed from Reopened to Closed

Changed in r2519. Changsets are now sorted in the same order as the comments (which is based on a user preference).

Actions

Also available in: Atom PDF