[whimsy] branch master updated: Report other changes to dates

[se...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/whimsy.git

The following commit(s) were added to refs/heads/master by this push:
       new  3ae8e99   Report other changes to dates
3ae8e99 is described below

commit 3ae8e99a5993fbb223df496720cdf9ee74cb4a6a
Author: Sebb <se...@apache.org>
AuthorDate: Tue Sep 20 15:38:08 2016 +0100

    Report other changes to dates
---
 www/roster/public_committee_info.rb | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/www/roster/public_committee_info.rb b/www/roster/public_committee_info.rb
index 0a10292..b34a803 100644
--- a/www/roster/public_committee_info.rb
+++ b/www/roster/public_committee_info.rb
@@ -79,15 +79,16 @@ if changed? and @old_file
   updated_day2 = (last_updated-3600*4).strftime("%Y-%m-%d") # day of previous update
 
   info[:committees].each { |pmc, entry|
+    next if pmc == 'infrastructure' # no dates
     previouspmc = previous[pmc] # get the original details (if any)
     if previouspmc # we have an existing entry
       entry[:roster].each { |name, value|
+        newdate = value[:date]
+        if newdate == nil
+          Wunderbar.warn "Un-dated member for #{pmc}: #{name} #{value[:name]} #{newdate}"
+          next
+        end
         if !previouspmc['roster'][name] # new name, check the date is OK
-          newdate = value[:date]
-          if newdate == nil
-            Wunderbar.warn "Un-dated member for #{pmc}: #{name} #{value[:name]} #{newdate}"
-            next
-          end
           if newdate <= updated_day1 and newdate >= updated_day2 # in range
             Wunderbar.info "New member for #{pmc}: #{name} #{value[:name]} #{newdate}"
           elsif newdate > updated_day1
@@ -95,6 +96,11 @@ if changed? and @old_file
           else
             Wunderbar.warn "Past-dated member for #{pmc}: #{name} #{value[:name]} #{newdate}"
           end
+        else
+          olddate = previouspmc['roster'][name]['date']
+          if olddate != newdate
+            Wunderbar.warn "Changed date member for #{pmc}: #{name} #{value[:name]} #{olddate} => #{newdate}"
+          end
         end
       }
     else

-- 
To stop receiving notification emails like this one, please contact
['"commits@whimsical.apache.org" <co...@whimsical.apache.org>'].