Nouvelle tentative de correction des rappels d'adhésion

[gestion-adh.git] / lib / nos_oignons.rb

diff --git a/lib/nos_oignons.rb b/lib/nos_oignons.rb
index 86be0fdc285bfad54659a709e4df18a46bf3899b..4ccd04fd7b6c9dce8196046bff054ddf9a735324 100644 (file)
--- a/lib/nos_oignons.rb
+++ b/lib/nos_oignons.rb
@@ -26,9 +26,9 @@ module NosOignons
 
   # The following class methods are all meant to be called as command-line scripts
   class << self
 
   # The following class methods are all meant to be called as command-line scripts
   class << self

-    def create_membership_fee_reciept!(member_id, amount)
+    def create_membership_fee_receipt!(member_id, amount)

       member = NosOignons::Member.new(member_id)
       member = NosOignons::Member.new(member_id)

-      member.create_reciept!(amount)
+      member.create_receipt!(amount)

     end
 
     def list_emails!
     end
 
     def list_emails!


@@ -57,9 +57,10 @@ module NosOignons
         anniversary = Time.new(today.year, member.joined_on.month,
                                member.joined_on.day).to_date
         next if member.membership_fee_paid_on >= anniversary
         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
-        NosOignons::Reminder.all.sort_by(&:days).reverse.each do |reminder|
-          next if anniversary < today + reminder.days
+        reminders = NosOignons::Reminder.all.sort_by(&:days)
+        reminders.each do |reminder|
+          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
 
           member.remind(reminder)
           break


@@ -69,14 +70,14 @@ module NosOignons
 
     def send_member_emails_to_advisors!
       uptodate_emails = NosOignons::Member.all.select(&:up_to_date?).collect(&:email)
 
     def send_member_emails_to_advisors!
       uptodate_emails = NosOignons::Member.all.select(&:up_to_date?).collect(&:email)

-      subject = 'Adresses emails des members à jour de cotisation'
+      subject = 'Adresses email des membres à jour de cotisation'

       body = <<-END_OF_BODY.gsub(/^        /, '')
         Cher comité de déontologie de Nos oignons,
 
       body = <<-END_OF_BODY.gsub(/^        /, '')
         Cher comité de déontologie de Nos oignons,
 

-       Comme le prévoit les statuts l'article 12 des statuts de l'association,
-        vous devez être en mesure de pouvoir convoquer une assemblée générale
-        extraordinaire. Pour ce faire, voici donc la liste des emails à jour
-        de cotisation à la date d'aujourd'hui :
+        Comme le prévoient les statuts l'article 12 des statuts de
+        l'association, vous devez être en mesure de pouvoir convoquer une
+        assemblée générale extraordinaire. Pour ce faire, voici donc la liste
+        des emails à jour de cotisation à la date d'aujourd'hui :

 
         #{uptodate_emails.join("\n")}
 
 
         #{uptodate_emails.join("\n")}