>
> validate :start_date_should_not_be_greater_than_current_date
>
Personally I'd rename that to:
valid :start_date_before_now
And then I'd do this as the method:
def start_date_before_now
if start_date && start_date > Date.today
errors.add("Whatever")
end
end
Just check start_date first. You don't need the .nil? check as nil
evaluates to false (and any valid date is true). Note, you don't need to do
self.start_date when reading a value, only when assigning to it (the reason
is that reading the value will default to calling the method if no local
variable exists, but writing a value will default to creating a local
variable rather than calling the value= method).
Cheers,
Andy
--
You received this message because you are subscribed to the Google Groups "Ruby
on Rails: Talk" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en.
