Skip to content

Commit

Permalink
Altered JSON invocation of JMX methods to support the following chang…
Browse files Browse the repository at this point in the history
…e implemented in Jolokia Release 0.91 (http://www.jolokia.org/changes-report.html):

Fixed issue with overloaded methods where one variant takes no arguments. This no-arg variant can be specified with the signature "()" after the operation name.
  • Loading branch information
Andrew Austin authored and Johannes Stelzer committed Jan 9, 2015
1 parent 6d76fa3 commit 443e106
Showing 1 changed file with 11 additions and 11 deletions.
22 changes: 11 additions & 11 deletions spring-boot-admin-server-ui/app/js/controller/jmxCtrl.js
Expand Up @@ -102,20 +102,20 @@ module.exports = function ($scope, $modal, $log, application, ApplicationJMX) {
$scope.invocation = null;
});
} else {
var signature = '(';
for (var i in op.args) {
if (i > 0) {
signature += ',';
}
signature += op.args[i].type;
$scope.invocation.args[i] = null;
}
signature += ')';
$scope.invocation.opname = name + signature;

if (op.args.length === 0) {
$scope.invoke();
} else {
var signature = '(';
for (var i in op.args) {
if (i > 0) {
signature += ',';
}
signature += op.args[i].type;
$scope.invocation.args[i] = null;
}
signature += ')';
$scope.invocation.opname = name + signature;

$modal.open({
templateUrl: 'invocationPrepareDialog.html',
scope: $scope
Expand Down

1 comment on commit 443e106

@ajausti1
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you have a schedule for releasing updates to Maven? Any idea when this might hit public servers? Thanks!

Please sign in to comment.