diff options
| author | Azul <azul@leap.se> | 2013-08-30 16:48:12 +0200 |
|---|---|---|
| committer | Azul <azul@leap.se> | 2013-08-30 16:48:12 +0200 |
| commit | 77af7ac953fb22c181056e8c40c8749d12a63922 (patch) | |
| tree | 4393c3e8fcb2969b0bd21800bff7dd5a1fc9c3d7 | |
| parent | 7bb1a16d4eee3eb3ef36c6f5b4fc7968e3da7dd6 (diff) | |
also replace find_by_param in billing engine
| -rw-r--r-- | billing/app/controllers/billing_base_controller.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/billing/app/controllers/billing_base_controller.rb b/billing/app/controllers/billing_base_controller.rb index 06820a6..c250831 100644 --- a/billing/app/controllers/billing_base_controller.rb +++ b/billing/app/controllers/billing_base_controller.rb @@ -6,12 +6,12 @@ class BillingBaseController < ApplicationController # required for navigation to work. def assign_user if params[:user_id] - @user = User.find_by_param(params[:user_id]) + @user = User.find(params[:user_id]) elsif params[:action] == "confirm" or params[:action] == "destroy" # confirms and subscription deletes will come back with different ID set, so check for this first # This is only for cases where an admin cannot apply action for customer, but should be all confirms @user = current_user elsif params[:id] - @user = User.find_by_param(params[:id]) + @user = User.find(params[:id]) else # TODO # hacky, what are cases where @user hasn't yet been set? certainly some cases with subscriptions and payments |