Sort time entries by name, not ID

[kfischer-okarin]

Redmine Issue: https://www.redmine.org/issues/2848

Scrapbox Memo (Japanese): https://scrapbox.io/redmine-patch/%E7%AC%AC2%E5%9B%9E%2FSort(%232848)

Written in cooperation with Goh Matsumoto, Akiko Takano and akipii

[kfischer-okarin]

ここはソートが解除されますが、@scopeにクエリーで指定したのソート順が入っているので、それを適用すればいいのでは？

[kfischer-okarin]

もう一度考えてみれば、ここは本当はこういう感じでやりたい

results = @scope.includes(:activity).
                 reorder(nil).
                 group(...).
                 joins(...)

@criteria.collect{|criteria| @available_criteria[criteria][:order]}.compact.each do |order_by_criteria|
  results = order_by_criteria(results)
end

result.sum(:hours).each do |hash, hours|
  ...

orderの項目を変換するブロックにしたらいい

@available_criteria = {
          'project' => {:sql => "#{TimeEntry.table_name}.project_id",
                        ...
                        order: ->(results) { results.order("#{Project.table_name}.name") }

これで、joinsとか必要だった場合に、その「メソッド」の任せたらいい。

[github-actions]

Patch can be downloaded here

[sanak]

@kfischer-okarin こちら、本家のIssueのその後の経緯を確認したところ、Yuichi HARADAさんが6ヶ月前から別のパッチを作成され、1ヶ月ほど前にtrunkにマージされ、次期メジャーバージョンでリリースされるようでした。
https://www.redmine.org/issues/2848#note-9

問題なさそうでしたら、こちらはクローズしようかと思いますので、よろしくお願いします。