You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whimsical.apache.org by Sam Ruby <ru...@apache.org> on 2016/02/22 03:00:55 UTC

[whimsy.git] [1/1] Commit ad21a28: fetch mail list once, use multiple times

Commit ad21a28488e48ce17b2682d155ecdf1fe2e2eaa6:
    fetch mail list once, use multiple times


Branch: refs/heads/master
Author: Sam Ruby <ru...@intertwingly.net>
Committer: Sam Ruby <ru...@intertwingly.net>
Pusher: rubys <ru...@apache.org>

------------------------------------------------------------
www/board/subscriptions.cgi                                  | +++ --
------------------------------------------------------------
5 changes: 3 additions, 2 deletions.
------------------------------------------------------------


diff --git a/www/board/subscriptions.cgi b/www/board/subscriptions.cgi
index a9212fa..ff63364 100755
--- a/www/board/subscriptions.cgi
+++ b/www/board/subscriptions.cgi
@@ -62,10 +62,11 @@ _html do
     end
 
     ids = []
+    maillist = ASF::Mail.list
 
     File.readlines(SUBSCRIPTIONS).each do |line|
-      person = ASF::Mail.list[line.downcase.strip]
-      person ||= ASF::Mail.list[line.downcase.strip.sub(/\+\w+@/,'@')]
+      person = maillist[line.downcase.strip]
+      person ||= maillist[line.downcase.strip.sub(/\+\w+@/,'@')]
       if person
         id = person.id
         id = '*notinavail*' if id == 'notinavail'