From eeabce64098388e465c90c039e89687dae4ca1fd Mon Sep 17 00:00:00 2001 From: Frederic PAUT Date: Wed, 10 Jul 2013 14:42:11 +0200 Subject: [PATCH] simplifying a small part of code --- cloudeebus/cloudeebus.js | 23 +++++++++++------------ 1 file changed, 11 insertions(+), 12 deletions(-) diff --git a/cloudeebus/cloudeebus.js b/cloudeebus/cloudeebus.js index 4cc6f53..a14a87d 100644 --- a/cloudeebus/cloudeebus.js +++ b/cloudeebus/cloudeebus.js @@ -438,10 +438,18 @@ cloudeebus.Service.prototype.removeAgent = function(rmAgent) { var self = this; var promise = new cloudeebus.Promise(function (resolver) { - function ServiceRemoveAgentSuccessCB(agent) { + function ServiceRemoveAgentSuccessCB(objectPath) { + // Searching agent in list + var idx; + for (idx in self.agents) + if (self.agents[idx].objectPath == objectPath) { + agent = self.agents[idx]; + break; + } + try { // calling dbus hook object function for un-translated types - self.agents.pop(agent); - agent.registered = false; + self.agents.splice(idx, 1); + self._deleteWrapper(agent); resolver.fulfill(agent, true); } catch (e) { @@ -457,15 +465,6 @@ cloudeebus.Service.prototype.removeAgent = function(rmAgent) { self.promise.resolver.reject(errorStr, true); } - try { - self._deleteWrapper(rmAgent); - } - catch (e) { - var errorStr = cloudeebus.getError(e); - cloudeebus.log("Exception removing wrapper of agent " + rmAgent.objectPath + " : " + errorStr); - errorCB(errorStr); - } - var arglist = [ rmAgent.objectPath ]; -- 2.7.4