Added new function to parse the tone options

Author: chughts

services/tone_analyzer/v3.js

@@ -102,23 +102,26 @@ module.exports = function (RED) {
     }
   };
 
+  // Function to parse and determine tone setting
+  // 'all' is the setting which needs be be blanked
+  // if not the service will throw an error
+  var parseToneOption = function (msg, config) {
+    var tones = msg.tones || config.tones;
+
+    return (tones === 'all' ? '' : tones);
+  }
+
   // function to parse through the options in preparation
   // for the sevice call.
   var parseOptions = function (msg, config) {
-    var tones = msg.tones || config.tones;
-    // var sentences = msg.sentences || config.sentences;
-    // var contentType = msg.contentType || config.contentType;
 
     var options = {
       'text': msg.payload,
       'sentences': msg.sentences || config.sentences,   
-      'isHTML': msg.contentType || config.contentType,   
-      'tones' : (tones === 'all' ? '' : tones)
+      'isHTML': msg.contentType || config.contentType    
     };
 
-    //if (tones !== 'all') {
-    // options.tones = tones;
-    //}
+    options.tones = parseToneOption(msg, config);
 
     return options;
   }