Only store user location if the user has to log in. Was causing us to…

… require log in for controllers that should not, like resetting passwords

app/controllers/application_controller.rb

@@ -3,23 +3,6 @@
 class ApplicationController < ActionController::Base
   before_action :set_paper_trail_whodunnit
   before_action :configure_permitted_parameters, if: :devise_controller?
-  before_action :store_user_location!, if: :storable_location?
-
-  private
-
-  def storable_location?
-    if devise_controller? && action_name == "edit" && current_user.nil?
-      redirect_to new_user_session_path
-      return true
-    end
-
-    request.get? && is_navigational_format? && !devise_controller? && !request.xhr?
-  end
-
-  def store_user_location!
-    # :user is the scope we are authenticating
-    store_location_for(:user, request.fullpath)
-  end
 
   protected
 

app/controllers/base_controllers/authenticate_controller.rb

@@ -3,6 +3,23 @@
 module BaseControllers
   class AuthenticateController < ApplicationController
     before_action :authenticate_user!
+    before_action :store_user_location!, if: :storable_location?
+
+    private
+
+    def storable_location?
+      if devise_controller? && action_name == "edit" && current_user.nil?
+        redirect_to new_user_session_path
+        return true
+      end
+
+      request.get? && is_navigational_format? && !devise_controller? && !request.xhr?
+    end
+
+    def store_user_location!
+      # :user is the scope we are authenticating
+      store_location_for(:user, request.fullpath)
+    end
 
     protected