From 68ea36a7fe56fad2b2a080537a14d67ed0c717c1 Mon Sep 17 00:00:00 2001 From: "Peter M. Goldstein" Date: Sun, 22 Sep 2013 11:43:59 -0700 Subject: [PATCH] Make associative_create_statement protected rather than private, so it can be accessed by subclasses. This makes it easy for subclasses to add custom fields to the impression. --- app/controllers/impressionist_controller.rb | 28 +++++++++++---------- 1 file changed, 15 insertions(+), 13 deletions(-) diff --git a/app/controllers/impressionist_controller.rb b/app/controllers/impressionist_controller.rb index 05a3976..275ed23 100644 --- a/app/controllers/impressionist_controller.rb +++ b/app/controllers/impressionist_controller.rb @@ -38,6 +38,21 @@ module ImpressionistController end end + protected + + # creates a statment hash that contains default values for creating an impression via an AR relation. + def associative_create_statement(query_params={}) + query_params.reverse_merge!( + :controller_name => controller_name, + :action_name => action_name, + :user_id => user_id, + :request_hash => @impressionist_hash, + :session_hash => session_hash, + :ip_address => request.remote_ip, + :referrer => request.referer + ) + end + private def bypass @@ -62,19 +77,6 @@ module ImpressionistController end end - # creates a statment hash that contains default values for creating an impression via an AR relation. - def associative_create_statement(query_params={}) - query_params.reverse_merge!( - :controller_name => controller_name, - :action_name => action_name, - :user_id => user_id, - :request_hash => @impressionist_hash, - :session_hash => session_hash, - :ip_address => request.remote_ip, - :referrer => request.referer - ) - end - # creates a statment hash that contains default values for creating an impression. def direct_create_statement(query_params={}) query_params.reverse_merge!(