javaJmx#setAttribute passing className bug parameter fixed

zuazo · May 13, 2013 · cf41408 · cf41408
1 parent 488be29
commit cf41408
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/javaJmx.js b/lib/javaJmx.js
@@ -136,9 +136,13 @@ JavaJmx.prototype.setAttribute = function(mbean, attributeName, value, className
   var self = this;
 
   if (typeof classNameOrCallback === "string" && !conversions.isJavaPrimitiveClass(classNameOrCallback)) {
-    java.newInstance(classNameOrCallback, value, function(obj) {
+    var newInstanceArgs = [ classNameOrCallback ];
+    newInstanceArgs = newInstanceArgs.concat(value);
+    newInstanceArgs.push(function(err, obj) {
+      if (checkError(err, self)) return;
       setAttribute.call(self, mbean, attributeName, obj, callback);
     });
+    java.newInstance.apply(java, newInstanceArgs);
   } else {
     if (typeof callback === "undefined") {
       callback = classNameOrCallback;