ignore empty fields properly

kylemcdonald · Jun 2, 2014 · bb5945d · bb5945d
1 parent 44c4838
commit bb5945d
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/app.js b/app.js
@@ -168,7 +168,10 @@ app.post('/', function(req, res) {
 });
 
 function addField(object, field, data) {
-  if(data || data == 0) {
+  // a string with length > 0
+  // or a number that is 0 or greater
+  if((typeof data == 'string' && data.length > 0) ||
+    (typeof data == 'number' && data >= 0)) {
     object[field] = data;
   }
 }