Redmine

Currently, the order of arrangement depends on the DBMS. (In no order)

Go MAEDA wrote:

I think it is useful for most users to sort issues by the due date in descending order (overdue issues are shown first).

I made a patch, and attach it.

diff --git a/app/models/mailer.rb b/app/models/mailer.rb
index 468e6c21a..800fdde88 100644
--- a/app/models/mailer.rb
+++ b/app/models/mailer.rb
@@ -27,7 +27,7 @@ class Mailer < ActionMailer::Base
   include Roadie::Rails::Automatic

   # Overrides ActionMailer::Base#process in order to set the recipient as the current user
-  # and his language as the default locale. 
+  # and his language as the default locale.
   # The first argument of all actions of this Mailer must be a User (the recipient),
   # otherwise an ArgumentError is raised.
   def process(action, *args)
@@ -584,6 +584,7 @@ class Mailer < ActionMailer::Base
     issues_by_assignee.each do |assignee, issues|
       if assignee.is_a?(User) && assignee.active? && issues.present?
         visible_issues = issues.select {|i| i.visible?(assignee)}
+        visible_issues.sort!{|a, b| (b.due_date <=> a.due_date).nonzero? || (a.id <=> b.id)}
         reminder(assignee, visible_issues, days).deliver_later if visible_issues.present?
       end
     end

Thank you for posting the patch. The patch works fine for me.

I slightly updated the test code in Yuichi HARADA's patch to make it even easier to read. Attaching the new patch.

Yuichi HARADA wrote:

Go MAEDA wrote:

I slightly updated the test code in Yuichi HARADA's patch to make it even easier to read. Attaching the new patch.

It is easy to understand using assert_select.

I'm sorry.
The new patch was not attach.

I changed due_date in ascending order, because overdue issues are shown first.
I rebuilt a patch, and attach it.

Committed. Thank you for improving Redmine.