Rails - NoMethodError undefined方法`> ='代表nil:NilClass

时间:2016-09-29 12:55:49

标签: ruby-on-rails ruby ruby-on-rails-4 rails-migrations

我的模型中出现上述错误。这是模型代码 -

Booking.rb

    class Booking < ActiveRecord::Base

    belongs_to :event
    belongs_to :user

    validates :quantity, presence: true, numericality: { greater_than: 0 }
    validates :total_amount, presence: true, numericality: { greater_than: 0 }
    validates :event, presence: true, numericality: {greater_than_or_equal_to: 0 }

    before_validation :set_default_values_to_greater_than_or_equal_to_zero

    def set_default_values_to_greater_than_or_equal_to_zero
        self.quantity >= 1
        self.total_amount >= 0  
        self.event.price >= 1    unless self.event.is_free
    end

    def reserve(stripe_token)
        # Don't process this booking if it isn't valid
        self.valid?

        # We can always set this, even for free events because their price will be 0.
        #self.total_amount = booking.quantity * event.price

                # Free events don't need to do anything special
                if event.is_free?
                save!

                # Paid events should charge the customer's card
                else

                    begin
                        self.total_amount = event.price * self.quantity
                        charge = Stripe::Charge.create(
                            amount: total_amount,
                            currency: "gbp",
                            source: stripe_token, 
                            description: "Booking created for amount #{total_amount}")
                        self.stripe_charge_id = charge.id
                        save!
                    rescue Stripe::CardError => e
                    errors.add(:base, e.message)
                    false
                end
            end 
        #end
    end
end

此行引发错误代码 -

self.quantity >= 1

我认为这是因为此特定列/属性的默认值(或缺少一个)。在迁移中纠正此问题的正确流程是什么?我是否进行了change_column_null迁移或change_column_default?

这是我的表 -

 create_table "bookings", force: :cascade do |t|
    t.integer  "event_id"
    t.integer  "user_id"
    t.string   "stripe_token"
    t.datetime "created_at",       null: false
    t.datetime "updated_at",       null: false
    t.integer  "quantity"
    t.integer  "total_amount"
    t.string   "stripe_charge_id"
  end

很明显,没有为数量设置(也不是total_amount)。我试图通过下面的before_validation回调和方法来解决这个问题,但它们似乎没有起作用。我相信迁移是解决这个问题的唯一方法,我是否正确?

这是我的控制器代码,我需要放在这里吗?

bookings_controller.rb

 class BookingsController < ApplicationController

before_action :authenticate_user!

def new
    # booking form
    # I need to find the event that we're making a booking on
    @event = Event.find(params[:event_id])
    # and because the event "has_many :bookings"
    @booking = @event.bookings.new(quantity: params[:quantity])
    # which person is booking the event?
    @booking.user = current_user


end

def create

    # actually process the booking
    @event = Event.find(params[:event_id])
    @booking = @event.bookings.new(booking_params)
    @booking.user = current_user

        if 
            @booking.reserve(booking_params['stripe_token'])
            flash[:success] = "Your place on our event has been booked"
            redirect_to event_path(@event)
        else
            flash[:error] = "Booking unsuccessful"
            render "new"
        end
end


private

def booking_params
    params.require(:booking).permit(:stripe_token, :quantity, :event_id, :stripe_charge_id, :total_amount)
end

1 个答案:

答案 0 :(得分:0)

如果set_default_values_to_greater_than_or_equal_to_zero尝试为这些属性设置默认值,请将>=更改为=,因为>=是一个评估运算符,它将返回truefalse。如果你走这条路,我建议把它作为私人方法。

如果你想要进行迁移,要在数据库级别设置默认值(我赞成在验证回调之前),

change_column_default(:bookings, :quantity, 1)

change_column :bookings, :quantity, :integer, :default => 1

请参阅:

change column default

change column

正如change_column_default页面注释中所述,更改列是执行此操作的首选方式