From 008c84bfd3f6b0fb72b8f5d55a466212385b5298 Mon Sep 17 00:00:00 2001 From: Andy Stewart Date: Thu, 28 Oct 2010 11:14:19 +0100 Subject: [PATCH] Make private methods that should be private. --- lib/paper_trail/has_paper_trail.rb | 44 +++++++++++++++--------------- 1 file changed, 22 insertions(+), 22 deletions(-) diff --git a/lib/paper_trail/has_paper_trail.rb b/lib/paper_trail/has_paper_trail.rb index f133f774..8eba64b4 100644 --- a/lib/paper_trail/has_paper_trail.rb +++ b/lib/paper_trail/has_paper_trail.rb @@ -56,28 +56,6 @@ module PaperTrail # Wrap the following methods in a module so we can include them only in the # ActiveRecord models that declare `has_paper_trail`. module InstanceMethods - def record_create - if switched_on? - versions.create merge_metadata(:event => 'create', :whodunnit => PaperTrail.whodunnit) - end - end - - def record_update - if switched_on? && changed_and_we_care? - versions.build merge_metadata(:event => 'update', - :object => object_to_string(item_before_change), - :whodunnit => PaperTrail.whodunnit) - end - end - - def record_destroy - if switched_on? - versions.create merge_metadata(:event => 'destroy', - :object => object_to_string(item_before_change), - :whodunnit => PaperTrail.whodunnit) - end - end - # Returns true if this instance is the current, live one; # returns false if this instance came from a previous version. def live? @@ -113,6 +91,28 @@ module PaperTrail private + def record_create + if switched_on? + versions.create merge_metadata(:event => 'create', :whodunnit => PaperTrail.whodunnit) + end + end + + def record_update + if switched_on? && changed_and_we_care? + versions.build merge_metadata(:event => 'update', + :object => object_to_string(item_before_change), + :whodunnit => PaperTrail.whodunnit) + end + end + + def record_destroy + if switched_on? + versions.create merge_metadata(:event => 'destroy', + :object => object_to_string(item_before_change), + :whodunnit => PaperTrail.whodunnit) + end + end + def merge_metadata(data) # First we merge the model-level metadata in `meta`. meta.each do |k,v|