Merge pull request #6 from integrallis/fix-redirect-without-scope

fix: issue #5 - redirect_to_sign_up fails with missing param (scope)
integrallis · Nov 14, 2017 · 130acf9 · 130acf9
2 parents 328d916 + c78eff7
commit 130acf9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/app/controllers/stripe_saas/subscriptions_controller.rb b/app/controllers/stripe_saas/subscriptions_controller.rb
@@ -84,15 +84,15 @@ def redirect_to_sign_up
         plan = ::Plan.by_stripe_id(params[:plan]).try(:first)
       end
 
+      devise_scope = (StripeSaas.devise_scope || StripeSaas.subscriptions_owned_by).to_s
+
       if plan
         unless plan.free?
           session["user_return_to"] = new_subscription_path(plan: plan)
         end
-
-        devise_scope = (StripeSaas.devise_scope || StripeSaas.subscriptions_owned_by).to_s
         redirect_to new_registration_path(devise_scope, plan: plan)
       else
-        redirect_to new_registration_path
+        redirect_to new_registration_path(devise_scope)
       end
     end
 

diff --git a/lib/stripe_saas/version.rb b/lib/stripe_saas/version.rb
@@ -1,3 +1,3 @@
 module StripeSaas
-  VERSION = "0.0.8"
+  VERSION = "0.0.9"
 end