Fixed bug: send notifications even if title not specified

kamranayub · Dec 17, 2011 · 972b208 · 972b208
1 parent f6e97be
commit 972b208
Showing 1 changed file with 15 additions and 10 deletions.
diff --git a/lib/growl.js b/lib/growl.js
@@ -150,20 +150,25 @@ exports.notify = function(msg, options, fn) {
       args.push('--name', options.name);
     }
 
-    // title
-    if (options.title) {
-      switch(cmd.type) {
-        case 'Darwin':
-          args.push(cmd.msg);
-          args.push('"' + msg + '"');
+    switch(cmd.type) {
+      case 'Darwin':
+        args.push(cmd.msg);
+        args.push('"' + msg + '"');
+        // title
+        if (options.title) {
           args.push(options.title);
-          break;
-        case 'Linux':
+        }
+        break;
+      case 'Linux':
+        // title
+        if (options.title) {
           args.push("'" + options.title + "'");
           args.push(cmd.msg);
           args.push("'" + msg + "'");
-          break;
-      }
+        } else {
+          args.push("'" + msg + "'");
+        }
+        break;
     }
 
     // execute