From cf4582c08e3609cd760251c07fe2ecb98302d8f5 Mon Sep 17 00:00:00 2001 From: David Cook Date: Wed, 12 Jun 2013 11:24:49 +1000 Subject: [PATCH] Create respond_override to return user back to the referring page. (But it breaks payments#create respond_with..) --- .../spree/admin/payments_controller_decorator.rb | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/app/controllers/spree/admin/payments_controller_decorator.rb b/app/controllers/spree/admin/payments_controller_decorator.rb index 11894d5fe0..ff3da521b1 100644 --- a/app/controllers/spree/admin/payments_controller_decorator.rb +++ b/app/controllers/spree/admin/payments_controller_decorator.rb @@ -1,9 +1,16 @@ # If a user fires an event on a payment from the orders page, set the responder to takes you back to the orders page (not payments page) #Responder: http://guides.spreecommerce.com/developer/logic.html#overriding-controller-action-responses +#TODO: for some reason this :fire responder kills the payments#create respond_with.. (core/app/controllers/spree/admin/payments_controller.rb:37) + Spree::Admin::PaymentsController.class_eval do respond_override :fire => { :html => { :success => lambda { - #if referrer == orders#index - # redirect_to orders#index - } } } + redirect_to request.referer #keeps any filter and sort prefs + + #if request.referrer.path_parameters['controller'] == 'spree/admin/orders' + # redirect_to admin_orders_path + #else + # redirect_to admin_order_payments_path(@order) #default action + #end + } } } end