--------------------------------------------------------------------------------
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.
<pre><code class="diff">
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
</code></pre>
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------
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.
--------------------------------------------------------------------------------
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.

--------------------------------------------------------------------------------
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.
--------------------------------------------------------------------------------

relates,Closed,6357,Show and sort by due_date in reminders