From 0cee8e7c53e6df73144130e0a3b21510f77453b9 Mon Sep 17 00:00:00 2001 From: swcraig Date: Tue, 12 Sep 2017 09:53:31 -0700 Subject: [PATCH] Stop raising exception for undefined callback The specs against Solidus master are failing because `set_current_order` is not defined as a process_action callback. This used to be the case but Solidus has removed this and slated it for v2.4. For compatibility with older versions of Solidus, we should not raise an ArgumentError if this callback is not defined but still skip it if it is. PR removing this callback: https://github.com/solidusio/solidus/pull/2185 --- lib/controllers/frontend/spree/users_controller.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/controllers/frontend/spree/users_controller.rb b/lib/controllers/frontend/spree/users_controller.rb index 6ebe7f52..8c0fc1ae 100644 --- a/lib/controllers/frontend/spree/users_controller.rb +++ b/lib/controllers/frontend/spree/users_controller.rb @@ -1,5 +1,5 @@ class Spree::UsersController < Spree::StoreController - skip_before_action :set_current_order, only: :show + skip_before_action :set_current_order, only: :show, raise: false prepend_before_action :load_object, only: [:show, :edit, :update] prepend_before_action :authorize_actions, only: :new