Moved Spree::StoreController class_eval from engine to store_controll…

…er_decorator. Part of spree#2415
chen-rubyer · Jan 15, 2013 · a10cfe1 · a10cfe1
1 parent 39323ea
commit a10cfe1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/promo/app/decorators/controllers/spree/store_controller_decorator.rb b/promo/app/decorators/controllers/spree/store_controller_decorator.rb
@@ -1,4 +1,8 @@
 Spree::StoreController.class_eval do
+  # Include list of visited paths in notification payload hash
+  def default_notification_payload
+    { :user => try_spree_current_user, :order => current_order, :visited_paths => session[:visited_paths] }
+  end
 
   protected
     def apply_coupon_code

diff --git a/promo/lib/spree/promo/engine.rb b/promo/lib/spree/promo/engine.rb
@@ -7,14 +7,6 @@ class Engine < Rails::Engine
       def self.activate
         require 'decorators'
         Decorators.register! root
-
-        # Why isn't this in a decorator?
-        Spree::StoreController.class_eval do
-          # Include list of visited paths in notification payload hash
-          def default_notification_payload
-            { :user => try_spree_current_user, :order => current_order, :visited_paths => session[:visited_paths] }
-          end
-        end
       end
 
       def self.root