better error handling

[sup] / lib / sup / person.rb

diff --git a/lib/sup/person.rb b/lib/sup/person.rb
index 9d35443e0b13cdb053ba120689e1a3d9738e5496..668f774926c06bd0dd260ba95af0da1c3668612f 100644 (file)
--- a/lib/sup/person.rb
+++ b/lib/sup/person.rb
@@ -10,11 +10,11 @@ class PersonManager
     self.class.i_am_the_instance self
   end
 
-  def name_for email; @names.member?(email) && @names[email][1]; end
+  def name_for email; @names.member?(email) ? @names[email][1] : nil; end
   def register email, name
     return unless name
 
-    name = name.gsub(/^\s+|\s+$/, "").gsub(/\s+/, " ")
+    name = name.gsub(/^\s+|\s+$/, "").gsub(/\s+/, " ").gsub(/^['"]|['"]$/, "")
 
     ## all else being equal, prefer longer names, unless the prior name
     ## doesn't contain any capitalization