Also catch Psych specific exception to identify wrong YAML

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

diff --git a/lib/nos_oignons.rb b/lib/nos_oignons.rb
index 694a87b9649af1b1153d5b700be9618043832e2c..2a5d4143464cc3c1e8b5919465f7dc30644999bf 100644 (file)
--- a/lib/nos_oignons.rb
+++ b/lib/nos_oignons.rb
@@ -3,8 +3,13 @@
 require 'nos_oignons/git'
 require 'nos_oignons/mailman'
 require 'nos_oignons/member'
+require 'nos_oignons/reminder'
+require 'nos_oignons/reminder_db'
 
 module NosOignons
+  BOARD_EMAIL = 'ca@nos-oignons.net'
+  MEMBER_MAILING_LIST = 'ag'
+
   # The following class methods are all meant to be called as command-line scripts
   class << self
     def list_emails!
@@ -15,13 +20,34 @@ module NosOignons
       end
     end
 
-    MEMBER_MAILING_LIST = 'ag'
     def update_ag_subscribers!
-      current_emails = NosOignons::Mailman.list_members(MEMBER_MAILING_LIST)
+      list = NosOignons::MEMBER_MAILING_LIST
+
+      current_emails = NosOignons::Mailman.list_members(list)
       uptodate_emails = NosOignons::Member.all.select(&:up_to_date?).collect(&:email)
 
-      NosOignons::Mailman.add_members(MEMBER_MAILING_LIST, uptodate_emails - current_emails)
-      NosOignons::Mailman.remove_members(MEMBER_MAILING_LIST, current_emails - uptodate_emails)
+      emails_to_add = uptodate_emails - current_emails
+      NosOignons::Mailman.add_members(list, emails_to_add) unless emails_to_add.empty?
+      emails_to_remove = current_emails - uptodate_emails
+      NosOignons::Mailman.remove_members(list, emails_to_remove) unless emails_to_remove.empty?
+    end
+
+    def send_membership_reminders!
+      today = Time.now.to_date
+      NosOignons::Member.all.select(&:up_to_date?).each do |member|
+        NosOignons::Reminder.all.sort_by(&:days).reverse.each do |reminder|
+          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.membership_fee_paid_on > today - reminder.days
+          next if anniversary > today - reminder.days
+          next if member.reminded_on && member.reminded_on >= today
+
+          member.remind(reminder)
+          break
+        end
+      end
     end
 
     def pre_commit_hook!
@@ -34,11 +60,11 @@ module NosOignons
 
       IO.popen(['git', 'diff-index', '--cached', '--name-status', against]) do |io|
         NosOignons::Git.handle_modified_files(io) do |file|
-          next unless file.start_with?("#{NosOignons::MEMBERS_ROOT}/")
+          next unless file.start_with?("#{NosOignons::MEMBERS_DB_DIR}/")
           begin
             # Use empty ref to get the index
             NosOignons::Member.read_from_git('', file)
-          rescue ArgumentError
+          rescue ArgumentError, Psych::SyntaxError
             $stderr.puts "Désolé : #{file} n'a pas le bon format !"
             exit 1
           end
@@ -51,10 +77,10 @@ module NosOignons
         old_value, new_value, ref_name = ref_line.rstrip.split(' ', 3)
         IO.popen(['git', 'diff', '--name-status', "#{old_value}..#{new_value}"]) do |io|
           NosOignons::Git.handle_modified_files(io) do |file|
-            next unless file.start_with?("#{NosOignons::MEMBERS_ROOT}/")
+            next unless file.start_with?("#{NosOignons::MEMBERS_DB_DIR}/")
             begin
               NosOignons::Member.read_from_git(new_value, file)
-            rescue ArgumentError
+            rescue ArgumentError, Psych::SyntaxError
               $stderr.puts "Désolé : #{file} n'a pas le bon format !"
               exit 1
             end