[whimsy] branch master updated: Need to pass the hash, not the value

[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://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new 8945578  Need to pass the hash, not the value
8945578 is described below

commit 894557810ac8125eb97d7fc91977bd8a42a0e497
Author: Sebb <se...@apache.org>
AuthorDate: Mon Jul 22 14:20:29 2019 +0100

    Need to pass the hash, not the value
---
 www/officers/list-traffic.cgi | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/www/officers/list-traffic.cgi b/www/officers/list-traffic.cgi
index 0d27e86..6470e89 100755
--- a/www/officers/list-traffic.cgi
+++ b/www/officers/list-traffic.cgi
@@ -50,7 +50,7 @@ def get_mails_month(yearmonth:, nondiscuss:)
       data = {}
       data[DATE] = DateTime.parse(message[/^Date: (.*)/, 1]).iso8601
       data[FROM] = message[/^From: (.*)/, 1]
-      data[WHO], data[MailUtils::COMMITTER] = MailUtils.find_who_from(data[FROM])
+      data[WHO], data[MailUtils::COMMITTER] = MailUtils.find_who_from(data)
       data[SUBJECT] = message[/^Subject: (.*)/, 1]
       if nondiscuss
         nondiscuss.each do |typ, rx|
@@ -96,6 +96,9 @@ def display_monthly(months:, nondiscuss:)
   months.sort.reverse.each do |month|
     data = get_mails_month(yearmonth: month, nondiscuss: nondiscuss)
     next if data.empty?
+    $stderr.puts data.inspect
+    data[MAILS] ||= []
+    data[TOOLS] ||= []
     _h1 "board@ statistics for #{month} (total mails: #{data[MAILS].length + data[TOOLS].length})", id: "#{month}"
     _div.row do
       _div.col_sm_6 do
@@ -205,6 +208,7 @@ _html do
       }
     ) do
       months = Dir["#{SRV_MAIL}/*"].map {|path| File.basename(path).untaint}.grep(/^\d+$/)
+      $stderr.puts months.inspect
       if ENV['QUERY_STRING'].include? 'week'
         display_weekly(months: months, nondiscuss: MailUtils::NONDISCUSSION_SUBJECTS['<board.apache.org>'])
       else