Rails - NoMethodError méthode non définie `> = 'pour nil: NilClass

Question

Je reçois l'erreur ci-dessus dans mon modèle. Voici le code de modèle -

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 

Le code d'erreur est jeté sur cette ligne -

self.quantity >= 1 

Je crois que cela est dû à la valeur par défaut (ou l'absence de un) sur cette colonne/attribut particulier. Quel est le processus correct pour corriger cela dans les migrations? Dois-je effectuer une migration change_column_null ou change_column_default?

C'est ma table -

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 

Comme on le voit, il n'y a rien prévu pour la quantité (ni TOTAL_AMOUNT). J'ai essayé de corriger cela avec le rappel et la méthode before_validation ci-dessous, mais ils ne semblent pas fonctionner. Je crois que la migration est le seul moyen de résoudre cela, ai-je raison?

Ceci est mon code de contrôleur, y a-t-il quelque chose que je dois mettre ici?

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 

fin

Answer 1

Si set_default_values_to_greater_than_or_equal_to_zero était une tentative de définir une valeur par défaut pour ces attributs changent >=-= parce >= est un opérateur d'évaluation qui retournera true ou false. Si vous allez dans cette voie, je suggère d'en faire une méthode privée.

Si vous souhaitez effectuer une migration à la place, pour définir les valeurs par défaut au niveau de la base de données, (que je serais en faveur de plus d'un avant le rappel de validation),

change_column_default(:bookings, :quantity, 1)

ou

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

Voir:

change column default

change column

Comme il est indiqué dans les commentaires de page change_column_default, colonne de changement est la meilleure façon de le faire