Не удалось сохранить значения карты после изменения версии в Rails 4.2

Question

У меня есть страница регистрации пользователя, где я буду вводить личные данные вместе с информацией о кредитной карте. Аккаунты пользователя has_many и аккаунт принадлежат пользователю. Когда пользователь создается, учетная запись создается, а информация и адрес кредитной карты сохраняются в таблице подписки. После создания аккаунта он собирается использовать метод store_card в модели subscription.rb. Оно работает нормально, когда я запускаю приложение в Rails 3.2.13, и когда я перенес его на Rails 4.2.6, оно не хранит информацию о подписке и не отображает никаких ошибок. Я обновил гем activemerchant до 1.47.0 и изменил код в методе load_billing в контроллере ниже:

@creditcard = ActiveMerchant::Billing::CreditCard.new(params[:account].blank? ? nil : params[:account][:creditcard])

в

@creditcard = ActiveMerchant::Billing::CreditCard.new(params[:account].blank? ? {} : params[:account][:creditcard])

и так же @address тоже.

Нет конкретного отношения, но только эта строка добавлена ​​в account.rb:

has_subscription :user_limit => Proc.new {|a| a.users.count }

Когда я попытался запустить с консоли:

u = Account.find(1)
u.subscription 

он генерирует подписные столбцы.

class AccountsController < ApplicationController

  before_filter :build_account, :only => [:new, :create]
  before_filter :build_user, :only => [:new, :create]
  before_filter :load_billing, :only => [:new, :create, :billing]
  def create
    @address.first_name = @creditcard.first_name
    @address.last_name = @creditcard.last_name
    @account.address = @address
    @account.creditcard = @creditcard
    if @account.new_record?
      if @account.save
        flash[:notice] = 'Account was created.'
        bypass_sign_in(@user)
        redirect_to session[:previous_url] || user_reports_path(@user)
      else
        render :action => 'new'
      end
    else
      @user.account_id = @account.id
      if @user.save
        flash[:notice] = 'User was created.'
        bypass_sign_in(@user)
        redirect_to session[:previous_url] || user_reports_path(@user)
      else
        render :action => 'new'
      end
    end
  end

  def billing
    @user = current_user
    @account = Account.find(params[:id])
    if request.put?
      @address.first_name = @creditcard.first_name
      @address.last_name = @creditcard.last_name
      puts @address.first_name
      if @creditcard.valid? & @address.valid?
       if @subscription.store_card(@creditcard, :billing_address => @address.to_activemerchant, :ip => request.remote_ip)
         flash[:notice] = "Your billing information has been updated."
         redirect_to settings_path(@user)
       end
      end
    end
  end
  protected

  def resource
    @account ||= current_account
  end

  def build_account
    @account = params[:account_name].blank? ? Account.new : Account.find_by_name(params[:account_name])
  end

  def build_user
    @account.user = @user = User.new(params[:account].blank? ? nil : params[:account][:user])
  end

  def load_billing
    @creditcard = ActiveMerchant::Billing::CreditCard.new(params[:account].blank? ? {} : params[:account][:creditcard])
    @address = SubscriptionAddress.new(params[:account].blank? ? {} : params[:account][:address])
  end

end

subscription.rb:

class Subscription < ActiveRecord::Base

  attr_accessor :creditcard, :address
  def store_card(creditcard, gw_options = {})
    # Clear out payment info if switching to CC from PayPal
    destroy_gateway_record(paypal) if paypal?

    @response = if billing_id.blank?
      gateway.store(creditcard, gw_options)
    else
      gateway.update(billing_id, creditcard, gw_options)
    end

    if @response.success?
      if active_card = @response.params['active_card']
        # Stripe token-based response
        self.card_number = "XXXX-XXXX-XXXX-#{active_card['last4']}"
        self.card_expiration = "%02d-%d" % [active_card['exp_month'], active_card['exp_year']]
      else
        self.card_number = creditcard.display_number
        self.card_expiration = "%02d-%d" % [creditcard.expiry_date.month, creditcard.expiry_date.year]
      end
      set_billing
    else
      errors.add(:base, @response.message)
      false
    end
  end

  def card_storage
      self.store_card(@creditcard, :billing_address => @address.to_activemerchant) if @creditcard && @address && card_number.blank?
    end

  def set_billing
      self.billing_id = @response.token
    end

end

account.rb:

class Account < ActiveRecord::Base

validate :valid_subscription?, :on => :create

  def valid_subscription?
    puts "valid_subscription**************************"
    return if errors.any? # Don't bother with a subscription if there are errors already
    self.build_subscription(:plan => @plan, :next_renewal_at => @plan_start, :creditcard => @creditcard, :address => @address)
    if !subscription.valid?
      errors.add(:base, "Error with payment: #{subscription.errors.full_messages.to_sentence}")
      return false
    end
  end
end

Пожалуйста, помогите, нужно ли мне использовать сильные параметры и каким-либо другим способом?

Answer 1

Было бы хорошо увидеть Account модель, но, исходя из вашего кода, похоже, что вы никогда не сохраняете данные кредитной карты. метод store_card просто присваивает значения, но никогда не сохраняет их в базе данных. Самое простое решение будет просто позвонить self.save в вашем store_card метод.

 if active_card = @response.params['active_card']
    # Stripe token-based response
    self.card_number = "XXXX-XXXX-XXXX-#{active_card['last4']}"
    self.card_expiration = "%02d-%d" % [active_card['exp_month'], active_card['exp_year']]
  else
    self.card_number = creditcard.display_number
    self.card_expiration = "%02d-%d" % [creditcard.expiry_date.month, creditcard.expiry_date.year]
  end
  self.save