whimsical-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From s...@apache.org
Subject [whimsy] branch master updated: Clarify errors
Date Tue, 23 Jan 2018 18:21:52 GMT
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 b82cac1  Clarify errors
b82cac1 is described below

commit b82cac1eaabfde8240697f4c829c87f8a6559a4f
Author: Sebb <sebb@apache.org>
AuthorDate: Tue Jan 23 18:21:51 2018 +0000

    Clarify errors
---
 www/roster/views/actions/committee.json.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/www/roster/views/actions/committee.json.rb b/www/roster/views/actions/committee.json.rb
index f93ff3e..e7ff97d 100644
--- a/www/roster/views/actions/committee.json.rb
+++ b/www/roster/views/actions/committee.json.rb
@@ -7,11 +7,11 @@ if env.password
     people = @ids.split(',').map {|id| ASF::Person.find(id)}
   else
     people = @ids.split(',').map {|id| ASF::Person[id]}
-    raise ArgumentError.new("ids=#{@ids}") if people.any? {|person| person.nil?}
+    raise ArgumentError.new("One or more null entries found: '#{@ids}'") if people.any? {|person|
person.nil?}
   end
 
   # Don't allow empty list
-  raise ArgumentError.new("ids='#{@ids}'") unless people.length > 0
+  raise ArgumentError.new("No valid entries found: '#{@ids}'") unless people.length >
0
 
   raise ArgumentError.new("project=#{@project}") unless pmc
 

-- 
To stop receiving notification emails like this one, please contact
sebb@apache.org.

Mime
View raw message