You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whimsical.apache.org by se...@apache.org on 2019/02/10 17:04:04 UTC

[whimsy] branch master updated: Fix binding issue (props greg)

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 7db94bb  Fix binding issue (props greg)
7db94bb is described below

commit 7db94bbdee33fbd59bc5bd2513f503e08fcc436b
Author: Sebb <se...@apache.org>
AuthorDate: Sun Feb 10 17:03:54 2019 +0000

    Fix binding issue (props greg)
---
 www/roster/views/actions/committee.json.rb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/www/roster/views/actions/committee.json.rb b/www/roster/views/actions/committee.json.rb
index 216bfc9..b39ff99 100644
--- a/www/roster/views/actions/committee.json.rb
+++ b/www/roster/views/actions/committee.json.rb
@@ -97,7 +97,8 @@ if env.password
   # compose E-mail
   action = (@action == 'add' ? 'added to' : 'removed from')
   if @targets.include? 'pmc'
-    list = @targets.include? 'commit' ? 'PMC and committers list' : 'PMC list'
+    # must use () to enclose method parameter below as ? binds tighter
+    list = @targets.include?('commit') ? 'PMC and committers list' : 'PMC list'
   elsif @targets.include? 'info'
     list = 'in committee-info.txt'
   else