diff --git a/app/server/appsmith-server/src/main/java/com/appsmith/server/controllers/ConfigController.java b/app/server/appsmith-server/src/main/java/com/appsmith/server/controllers/ConfigController.java deleted file mode 100644 index e9cb086fc46f..000000000000 --- a/app/server/appsmith-server/src/main/java/com/appsmith/server/controllers/ConfigController.java +++ /dev/null @@ -1,16 +0,0 @@ -package com.appsmith.server.controllers; - -import com.appsmith.server.constants.Url; -import com.appsmith.server.controllers.ce.ConfigControllerCE; -import com.appsmith.server.services.ConfigService; -import org.springframework.web.bind.annotation.RequestMapping; -import org.springframework.web.bind.annotation.RestController; - -@RestController -@RequestMapping(Url.CONFIG_URL) -public class ConfigController extends ConfigControllerCE { - - public ConfigController(ConfigService service) { - super(service); - } -} diff --git a/app/server/appsmith-server/src/main/java/com/appsmith/server/controllers/ce/ConfigControllerCE.java b/app/server/appsmith-server/src/main/java/com/appsmith/server/controllers/ce/ConfigControllerCE.java deleted file mode 100644 index a0580ea45b88..000000000000 --- a/app/server/appsmith-server/src/main/java/com/appsmith/server/controllers/ce/ConfigControllerCE.java +++ /dev/null @@ -1,37 +0,0 @@ -package com.appsmith.server.controllers.ce; - -import com.appsmith.external.views.Views; -import com.appsmith.server.constants.Url; -import com.appsmith.server.domains.Config; -import com.appsmith.server.dtos.ResponseDTO; -import com.appsmith.server.services.ConfigService; -import com.fasterxml.jackson.annotation.JsonView; -import org.springframework.http.HttpStatus; -import org.springframework.web.bind.annotation.GetMapping; -import org.springframework.web.bind.annotation.PathVariable; -import org.springframework.web.bind.annotation.PutMapping; -import org.springframework.web.bind.annotation.RequestBody; -import org.springframework.web.bind.annotation.RequestMapping; -import reactor.core.publisher.Mono; - -@RequestMapping(Url.CONFIG_URL) -public class ConfigControllerCE { - - private final ConfigService service; - - public ConfigControllerCE(ConfigService service) { - this.service = service; - } - - @JsonView(Views.Public.class) - @GetMapping("/name/{name}") - public Mono> getByName(@PathVariable String name) { - return service.getByName(name).map(resource -> new ResponseDTO<>(HttpStatus.OK, resource)); - } - - @JsonView(Views.Public.class) - @PutMapping("/name/{name}") - public Mono> updateByName(@PathVariable String name, @RequestBody Config config) { - return service.updateByName(config).map(resource -> new ResponseDTO<>(HttpStatus.OK, resource)); - } -} diff --git a/app/server/appsmith-server/src/main/java/com/appsmith/server/services/ce/ConfigServiceCE.java b/app/server/appsmith-server/src/main/java/com/appsmith/server/services/ce/ConfigServiceCE.java index 1f4644a5b25c..ca64d9a35880 100644 --- a/app/server/appsmith-server/src/main/java/com/appsmith/server/services/ce/ConfigServiceCE.java +++ b/app/server/appsmith-server/src/main/java/com/appsmith/server/services/ce/ConfigServiceCE.java @@ -1,8 +1,6 @@ package com.appsmith.server.services.ce; -import com.appsmith.server.acl.AclPermission; import com.appsmith.server.domains.Config; -import com.appsmith.server.domains.User; import reactor.core.publisher.Mono; import java.util.Map; @@ -11,8 +9,6 @@ public interface ConfigServiceCE { Mono getByName(String name); - Mono updateByName(Config config); - Mono save(Config config); Mono save(String name, Map config); @@ -21,10 +17,6 @@ public interface ConfigServiceCE { Mono delete(String name); - Mono getByName(String name, AclPermission permission); - - Mono getByNameAsUser(String name, User user, AclPermission permission); - /** * Get the instance variables from the instance config * @return Map containing the instance variables diff --git a/app/server/appsmith-server/src/main/java/com/appsmith/server/services/ce/ConfigServiceCEImpl.java b/app/server/appsmith-server/src/main/java/com/appsmith/server/services/ce/ConfigServiceCEImpl.java index fb544da3b901..8f44fe63ae03 100644 --- a/app/server/appsmith-server/src/main/java/com/appsmith/server/services/ce/ConfigServiceCEImpl.java +++ b/app/server/appsmith-server/src/main/java/com/appsmith/server/services/ce/ConfigServiceCEImpl.java @@ -1,9 +1,7 @@ package com.appsmith.server.services.ce; -import com.appsmith.server.acl.AclPermission; import com.appsmith.server.constants.FieldName; import com.appsmith.server.domains.Config; -import com.appsmith.server.domains.User; import com.appsmith.server.exceptions.AppsmithError; import com.appsmith.server.exceptions.AppsmithException; import com.appsmith.server.repositories.ConfigRepository; @@ -34,20 +32,6 @@ public Mono getByName(String name) { Mono.error(new AppsmithException(AppsmithError.NO_RESOURCE_FOUND, FieldName.CONFIG, name))); } - @Override - public Mono updateByName(Config config) { - final String name = config.getName(); - return repository - .findByName(name) - .switchIfEmpty( - Mono.error(new AppsmithException(AppsmithError.NO_RESOURCE_FOUND, FieldName.CONFIG, name))) - .flatMap(dbConfig -> { - log.debug("Found config with name: {} and id: {}", name, dbConfig.getId()); - dbConfig.setConfig(config.getConfig()); - return repository.save(dbConfig); - }); - } - @Override public Mono save(Config config) { return repository @@ -85,16 +69,6 @@ public Mono delete(String name) { .flatMap(repository::delete); } - @Override - public Mono getByName(String name, AclPermission permission) { - return repository.findByName(name, permission); - } - - @Override - public Mono getByNameAsUser(String name, User user, AclPermission permission) { - return repository.findByNameAsUser(name, user, permission); - } - @Override public Mono> getInstanceVariables() { return getByName(FieldName.INSTANCE_CONFIG).map(config -> {