[shardingsphere-elasticjob-ui] 13/26: wrap the boolean response value

[zh...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

zhangliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere-elasticjob-ui.git

commit 3ef8db88aa75ba1042b0d5fcfc89c5e94b9af78c
Author: renfufei <re...@huobi.com>
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);
     }
 }