Default to not requiring the presence of the deleter attribute

Fixes delynn#29.
richwallett · Oct 18, 2017 · 752f1e3 · 752f1e3
1 parent e3c808e
commit 752f1e3
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/lib/active_record/userstamp/stampable.rb b/lib/active_record/userstamp/stampable.rb
@@ -84,8 +84,11 @@ def add_userstamp_associations(options)
         associations.first
       belongs_to :updater, relation_options.reverse_merge(foreign_key: config.updater_attribute) if
         associations.second
-      belongs_to :deleter, relation_options.reverse_merge(foreign_key: config.deleter_attribute) if
-        associations.third
+      if associations.third
+        relation_options.reverse_merge!(required: false) if ActiveRecord::VERSION::MAJOR >= 5 ||
+          (ActiveRecord::VERSION::MAJOR == 4 && ActiveRecord::VERSION::MINOR >= 2)
+        belongs_to :deleter, relation_options.reverse_merge(foreign_key: config.deleter_attribute)
+      end
     end
   end