[whimsy] branch master updated: only set ICLAs to 1 if there is data to fill in the form

[ru...@apache.org]

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

rubys 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  ee637d9   only set ICLAs to 1 if there is data to fill in the form
ee637d9 is described below

commit ee637d936cc74c5fabe13836ed80759115e5f925
Author: Sam Ruby <ru...@intertwingly.net>
AuthorDate: Thu Aug 11 15:56:34 2016 -0400

    only set ICLAs to 1 if there is data to fill in the form
---
 www/secretary/workbench/worklist.cgi | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/www/secretary/workbench/worklist.cgi b/www/secretary/workbench/worklist.cgi
index a0e12e4..d873935 100755
--- a/www/secretary/workbench/worklist.cgi
+++ b/www/secretary/workbench/worklist.cgi
@@ -472,11 +472,12 @@ _html do
 
           if File.exist? COMPLETED_YML
             last = YAML.load(File.read COMPLETED_YML).last
-            params = {iclas: '1'}
+            params = {}
             %w{email user pmc podling votelink}.each do |name|
               params[name] = last[name] if last[name]
             end
             unless params.empty?
+              params['iclas'] = '1'
               query = '?' + params.
                 map {|name,value| "#{name}=#{CGI.escape value}"}.join('&')
             end

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