From 4fdc96b4c43c5088838f52cd523a011f2bdc05f8 Mon Sep 17 00:00:00 2001 From: Owen Rodda Date: Sat, 16 Jan 2016 21:48:48 -0500 Subject: [PATCH] Default order by timestamp to ensure early versions are cleaned --- lib/paper_trail/cleaner.rb | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/paper_trail/cleaner.rb b/lib/paper_trail/cleaner.rb index d4c593165..d3261ac01 100644 --- 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,