class Client < ActiveRecord::Base
  # TODO: figure out why application_date validation is fubar
  # validates :application_date, presence: true
  has_one :bike
  belongs_to :agency

  def self.waiting_list
    clients = Client.all
    non_voided_clients = clients.select{|client| !client.application_voided}
    waiting_list_with_null_application_dates = non_voided_clients.select{|client| !client.pickup_date}
    clients_with_bikes = Client.all.select{|c| !c.bike_id.nil?}
    waiting_list = waiting_list_with_null_application_dates.select{|client| !client.application_date.nil?} - clients_with_bikes
    waiting_list.sort_by!{|client| client.application_date}
  end

  def name
    self.first_name + ' ' + self.last_name
  end

  def self.closed_applications
    Client.all.select{|client| client.application_voided || client.pickup_date}
  end

  def bike
    Bike.find(self.bike_id)
  end

end