From 7b8455d973d5b32d19cf356bea0de0a2e1d2b88b Mon Sep 17 00:00:00 2001
From: Ivo Georgiev <ivo@linvo.org>
Date: Mon, 10 Sep 2012 12:24:08 +0300
Subject: [PATCH] Update lib/introspect.js

1. The "add method" procedure does not belong in the loop that builds the signature. It is just re-defined for each new signature part.

2. Made callback optional for cases in which we do not care about the method result. E.g.
player.playPause(false);
---
 lib/introspect.js | 39 ++++++++++++++++++++-------------------
 1 file changed, 20 insertions(+), 19 deletions(-)

diff --git a/lib/introspect.js b/lib/introspect.js
index a43fe406..2c407c5d 100644
--- a/lib/introspect.js
+++ b/lib/introspect.js
@@ -51,26 +51,27 @@ module.exports = function(obj, callback) {
                       console.log(arg);
                       if (arg.direction === 'in')
                           signature += arg.type;
-                      
-                      // add method
-                      currentIface[name] = (function(ifName, methodName, signature) {
-                          return function() {
-                              var args = Array.prototype.slice.apply(arguments);
-                              var callback = args[args.length - 1];
-                              var msg = {
-                                  destination: obj.service.name,
-                                  path: obj.name,
-                                  'interface': ifName,
-                                  member: methodName
-                              };
-                              if (signature !== '') {
-                                  msg.signature = signature;
-                                  msg.body = args.slice(0,-1);
-                              }
-                              bus.invoke(msg, callback);
-                         };
-                      })(ifaceName, name, signature);
                   }
+                      
+    			 // add method
+				  currentIface[name] = (function(ifName, methodName, signature) {
+					  return function() {
+						  var args = Array.prototype.slice.apply(arguments);
+    					  var callback = (typeof(args[args.length - 1]) == "function") ? args.pop() : function() {};
+						  var msg = {
+							  destination: obj.service.name,
+							  path: obj.name,
+							  'interface': ifName,
+							  member: methodName
+						  };
+						  if (signature !== '') {
+							  msg.signature = signature;
+							  msg.body = args;
+						  }
+						  bus.invoke(msg, callback);
+					 };
+				  })(ifaceName, name, signature);
+
                   console.log('    method: ', name, signature);
               }
               for (p=0; iface.property && p < iface.property.length; ++p)