Default order by timestamp to ensure early versions are cleaned

owenr · Jan 17, 2016 · 4fdc96b · 4fdc96b
1 parent 85733fe
commit 4fdc96b
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/lib/paper_trail/cleaner.rb b/lib/paper_trail/cleaner.rb
@@ -36,6 +36,7 @@ def gather_versions(item_id = nil, date = :all)
         raise ArgumentError.new("`date` argument must receive a Timestamp or `:all`")
       end
       versions = item_id ? PaperTrail::Version.where(:item_id => item_id) : PaperTrail::Version
+      versions = versions.order(PaperTrail.timestamp_field)
       versions = versions.between(date.to_date, date.to_date + 1.day) unless date == :all
 
       # If `versions` has not been converted to an ActiveRecord::Relation yet,