This is an automated email from the ASF dual-hosted git repository. zhangyonglun pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/shardingsphere-elasticjob-ui.git
commit 3ef8db88aa75ba1042b0d5fcfc89c5e94b9af78c Author: renfufei <[email protected]> AuthorDate: Mon Jul 20 20:40:44 2020 +0800 wrap the boolean response value --- .../ui/web/controller/JobConfigController.java | 6 ++++-- .../ui/web/controller/JobOperationController.java | 18 ++++++++++++------ 2 files changed, 16 insertions(+), 8 deletions(-) diff --git a/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobConfigController.java b/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobConfigController.java index 37c0ca8..ad8b76e 100644 --- a/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobConfigController.java +++ b/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobConfigController.java @@ -64,8 +64,9 @@ public final class JobConfigController { * @param jobConfiguration job configuration */ @PutMapping(consumes = MediaType.APPLICATION_JSON) - public void updateJobConfig(@RequestBody final JobConfigurationPOJO jobConfiguration) { + public ResponseResult<Boolean> updateJobConfig(@RequestBody final JobConfigurationPOJO jobConfiguration) { jobAPIService.getJobConfigurationAPI().updateJobConfiguration(jobConfiguration); + return ResponseResultUtil.build(Boolean.TRUE); } /** @@ -74,7 +75,8 @@ public final class JobConfigController { * @param jobName job name */ @DeleteMapping("/{jobName}") - public void removeJob(@PathVariable("jobName") final String jobName) { + public ResponseResult<Boolean> removeJob(@PathVariable("jobName") final String jobName) { jobAPIService.getJobConfigurationAPI().removeJobConfiguration(jobName); + return ResponseResultUtil.build(Boolean.TRUE); } } diff --git a/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobOperationController.java b/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobOperationController.java index 32fde53..6c79d9a 100644 --- a/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobOperationController.java +++ b/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobOperationController.java @@ -73,8 +73,9 @@ public final class JobOperationController { * @param jobName job name */ @PostMapping("/{jobName}/trigger") - public void triggerJob(@PathVariable("jobName") final String jobName) { + public ResponseResult<Boolean> triggerJob(@PathVariable("jobName") final String jobName) { jobAPIService.getJobOperatorAPI().trigger(jobName); + return ResponseResultUtil.build(Boolean.TRUE); } /** @@ -83,8 +84,9 @@ public final class JobOperationController { * @param jobName job name */ @PostMapping(value = "/{jobName}/disable") - public void disableJob(@PathVariable("jobName") final String jobName) { + public ResponseResult<Boolean> disableJob(@PathVariable("jobName") final String jobName) { jobAPIService.getJobOperatorAPI().disable(jobName, null); + return ResponseResultUtil.build(Boolean.TRUE); } /** @@ -93,8 +95,9 @@ public final class JobOperationController { * @param jobName job name */ @PostMapping(value = "/{jobName}/enable") - public void enableJob(@PathVariable("jobName") final String jobName) { + public ResponseResult<Boolean> enableJob(@PathVariable("jobName") final String jobName) { jobAPIService.getJobOperatorAPI().enable(jobName, null); + return ResponseResultUtil.build(Boolean.TRUE); } /** @@ -103,8 +106,9 @@ public final class JobOperationController { * @param jobName job name */ @PostMapping(value = "/{jobName}/shutdown") - public void shutdownJob(@PathVariable("jobName") final String jobName) { + public ResponseResult<Boolean> shutdownJob(@PathVariable("jobName") final String jobName) { jobAPIService.getJobOperatorAPI().shutdown(jobName, null); + return ResponseResultUtil.build(Boolean.TRUE); } /** @@ -126,8 +130,9 @@ public final class JobOperationController { * @param item sharding item */ @PostMapping(value = "/{jobName}/sharding/{item}/disable") - public void disableSharding(@PathVariable("jobName") final String jobName, @PathVariable("item") final String item) { + public ResponseResult<Boolean> disableSharding(@PathVariable("jobName") final String jobName, @PathVariable("item") final String item) { jobAPIService.getShardingOperateAPI().disable(jobName, item); + return ResponseResultUtil.build(Boolean.TRUE); } /** @@ -137,7 +142,8 @@ public final class JobOperationController { * @param item sharding item */ @PostMapping(value = "/{jobName}/sharding/{item}/enable") - public void enableSharding(@PathVariable("jobName") final String jobName, @PathVariable("item") final String item) { + public ResponseResult<Boolean> enableSharding(@PathVariable("jobName") final String jobName, @PathVariable("item") final String item) { jobAPIService.getShardingOperateAPI().enable(jobName, item); + return ResponseResultUtil.build(Boolean.TRUE); } }
