diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt index 3626da3c10a..7a45d9b032f 100644 --- a/solr/CHANGES.txt +++ b/solr/CHANGES.txt @@ -171,6 +171,8 @@ Bug Fixes * SOLR-9405: ConcurrentModificationException in ZkStateReader.getStateWatchers. (Alan Woodward, Edward Ribeiro, shalin) +* SOLR-9232: Admin UI now fully implements Swap Cores interface (Alexandre Rafalovitch) + Optimizations ---------------------- diff --git a/solr/webapp/web/js/angular/controllers/cores.js b/solr/webapp/web/js/angular/controllers/cores.js index 347dbf4c1f8..41863f2b591 100644 --- a/solr/webapp/web/js/angular/controllers/cores.js +++ b/solr/webapp/web/js/angular/controllers/cores.js @@ -129,15 +129,15 @@ solrAdminApp.controller('CoreAdminController', }; $scope.swapCores = function() { - if ($scope.swapOther) { - $swapMessage = "Please select a core to swap with"; + if (!$scope.swapOther) { + $scope.swapMessage = "Please select a core to swap with"; } else if ($scope.swapOther == $scope.selectedCore) { - $swapMessage = "Cannot swap with the same core"; + $scope.swapMessage = "Cannot swap with the same core"; } else { Cores.swap({core: $scope.selectedCore, other: $scope.swapOther}, function(data) { $location.path("/~cores/" + $scope.swapOther); delete $scope.swapOther; - $scope.cancelSwap(); + $scope.cancelSwapCores(); }); } }; diff --git a/solr/webapp/web/js/angular/services.js b/solr/webapp/web/js/angular/services.js index 014d36ba525..f050c9b7253 100644 --- a/solr/webapp/web/js/angular/services.js +++ b/solr/webapp/web/js/angular/services.js @@ -47,7 +47,7 @@ solrAdminServices.factory('System', "add": {params:{action: "CREATE"}}, "unload": {params:{action: "UNLOAD", core: "@core"}}, "rename": {params:{action: "RENAME"}}, - "swap": {params:{}}, + "swap": {params:{action: "SWAP"}}, "reload": {method: "GET", params:{action:"RELOAD", core: "@core"}, headers:{doNotIntercept: "true"}}, "optimize": {params:{}} });