Ruby on Rails

It might help to have the models for auction and user... also, I like to add the annotate gem which gives some header comments in all my models to show what fields etc are there.  Very helpful info and references.

you already know that the validate needs to be validates...

is this an active system or SUD?  do you have a solid development database that is well populated to test/develop against?

this link may help some:  http://guides.rubyonrails.org/active_record_validations.html#custom-methods

other than that I'd have to see it in action to debug further. 

Good luck

Max

On Monday, March 26, 2018 at 6:17:56 PM UTC-6, fugee ohu wrote:

This model has some errors that I wasn't able to get sorted out the two lines that begin with validate should be validates but after that I don't understand what the author's trying to do If I change them to validates I get the error Unknown validator: 'MessageValidator' for   validate :auction_active?, message: :auction_active

class Bid < ActiveRecord::Base

  belongs_to :auction

  belongs_to :user

  validates :auction, :user, presence: true

  validate :auction_active?, message: :auction_active

  validate :last_user_different?

  scope :sorted, -> { order(:created_at) }

  after_create :update_auction

  private

  def auction_active?

    if auction && !auction.active?

      if auction.finished?

        errors.add(:auction, :finished)

      else

        errors.add(:auction, :not_started)

      end

    end

  end

  def last_user_different?

    if auction && user

      errors.add(:user, :same_user_twice) if auction.last_user == user

    end

  end

  def update_auction

    auction.increase_price_and_time

    auction.publish_updates

  end

end