def send_membership_reminders!
today = Time.now.to_date
NosOignons::Member.all.select(&:up_to_date?).each do |member|
+ reminders = NosOignons::Reminder.all.sort_by(&:days)
anniversary = Time.new(today.year, member.joined_on.month,
member.joined_on.day).to_date
- next if member.membership_fee_paid_on >= anniversary
- next if member.reminded_on && member.reminded_on >= today
- reminders = NosOignons::Reminder.all.sort_by(&:days).reverse
- next if anniversary > today + reminders.first.days
+ next if member.membership_fee_paid_on >= anniversary - reminders.last.days
reminders.each do |reminder|
- next if anniversary < today + reminder.days
+ next if (anniversary - today).to_i > reminder.days
+ next if member.reminded_on && (anniversary - member.reminded_on).to_i <= reminder.days
member.remind(reminder)
break