Nouvelle tentative de correction des rappels d'adhésion

Damned.

diff --git a/features/send-membership-reminders.feature b/features/send-membership-reminders.feature
index 09bdce721860a0d7821587817f4c9a6b4b5aea8a..73c9dfebd3667af8b77a5bf639e187ec048ee650 100644 (file)
--- a/features/send-membership-reminders.feature
+++ b/features/send-membership-reminders.feature
@@ -35,6 +35,18 @@ Fonctionnalité: prévenir les membres qu'il faut renouveller leur cotisation
     Lorsque j'exécute send-membership-reminders
     Alors aucun email ne doit avoir été envoyé
 
     Lorsque j'exécute send-membership-reminders
     Alors aucun email ne doit avoir été envoyé
 

+  Scénario: Pas de messages entre le premier et le deuxième rappel
+    Soit une base avec Jane qui doit renouveller sa cotisation d'ici 28 jours
+    Et qui a déjà reçu un appel 2 jours plus tôt
+    Lorsque j'exécute send-membership-reminders
+    Alors aucun email ne doit avoir été envoyé
+
+  Scénario: Script pas exécuté tous les jours
+    Soit une base avec Jane qui doit renouveller sa cotisation d'ici 9 jours
+    Et qui a déjà reçu un appel 21 jours plus tôt
+    Lorsque j'exécute send-membership-reminders
+    Alors 1 email doit avoir été envoyé
+

   Scénario: Un seul envoi par membre
     Soit une base avec Jane qui doit renouveller sa cotisation d'ici 30 jours
     Lorsque j'exécute send-membership-reminders
   Scénario: Un seul envoi par membre
     Soit une base avec Jane qui doit renouveller sa cotisation d'ici 30 jours
     Lorsque j'exécute send-membership-reminders

diff --git a/lib/nos_oignons.rb b/lib/nos_oignons.rb
index 758c6752767497a01aa8fa3a3a918a1e362a6abf..4ccd04fd7b6c9dce8196046bff054ddf9a735324 100644 (file)
--- a/lib/nos_oignons.rb
+++ b/lib/nos_oignons.rb
@@ -57,11 +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
-        reminders = NosOignons::Reminder.all.sort_by(&:days).reverse
-        next if anniversary > today + reminders.first.days
+        reminders = NosOignons::Reminder.all.sort_by(&:days)

         reminders.each do |reminder|
         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
 
           member.remind(reminder)
           break