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/25 23:57:06 UTC
[whimsy.git] [1/1] Commit 4f8e1ad: fix regression introduced by https://github.com/apache/whimsy/commit/db10f153ddadc699d9c4950b96cccb4ef14b1ab8
Commit 4f8e1ad13d868cd1fbf0bf6545b9e3d19d6e7299:
fix regression introduced by https://github.com/apache/whimsy/commit/db10f153ddadc699d9c4950b96cccb4ef14b1ab8
instance variables don't work very as closures
Branch: refs/heads/master
Author: Sam Ruby <ru...@intertwingly.net>
Committer: Sam Ruby <ru...@intertwingly.net>
Pusher: rubys <ru...@apache.org>
------------------------------------------------------------
www/board/agenda/views/actions/send-reminders.json.rb | ++++ ---
------------------------------------------------------------
7 changes: 4 additions, 3 deletions.
------------------------------------------------------------
diff --git a/www/board/agenda/views/actions/send-reminders.json.rb b/www/board/agenda/views/actions/send-reminders.json.rb
index 35a33ae..a75c2b4 100644
--- a/www/board/agenda/views/actions/send-reminders.json.rb
+++ b/www/board/agenda/views/actions/send-reminders.json.rb
@@ -12,9 +12,10 @@
# extract values for common fields
subject = @subject.untaint
-unless @from
+from = @from
+unless from
sender = ASF::Person.find(env.user)
- @from = "#{sender.public_name} <#{...@apache.org>".untaint
+ from = "#{sender.public_name} <#{...@apache.org>".untaint
end
# iterate over the agenda
@@ -48,7 +49,7 @@
# construct email
mail = Mail.new do
- from @from
+ from from
to "#{item['owner']} <#{item['chair_email']}>".untaint
subject subject