Skip to content

Commit c3fa4d4

Browse files
author
iliax
committed
ISSUE-80: Fixing ActionDTO parsing from config
1 parent 42c6a43 commit c3fa4d4

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

api/src/main/java/io/kafbat/ui/controller/ApplicationConfigController.java

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55

66
import io.kafbat.ui.api.ApplicationConfigApi;
77
import io.kafbat.ui.config.ClustersProperties;
8+
import io.kafbat.ui.model.ActionDTO;
89
import io.kafbat.ui.model.ApplicationConfigDTO;
910
import io.kafbat.ui.model.ApplicationConfigPropertiesDTO;
1011
import io.kafbat.ui.model.ApplicationConfigValidationDTO;
@@ -18,6 +19,7 @@
1819
import io.kafbat.ui.util.ApplicationRestarter;
1920
import io.kafbat.ui.util.DynamicConfigOperations;
2021
import java.util.Map;
22+
import java.util.Optional;
2123
import javax.annotation.Nullable;
2224
import lombok.RequiredArgsConstructor;
2325
import lombok.extern.slf4j.Slf4j;
@@ -46,6 +48,12 @@ interface PropertiesMapper {
4648
DynamicConfigOperations.PropertiesStructure fromDto(ApplicationConfigPropertiesDTO dto);
4749

4850
ApplicationConfigPropertiesDTO toDto(DynamicConfigOperations.PropertiesStructure propertiesStructure);
51+
52+
default ActionDTO stringToActionDTO(String str) {
53+
return Optional.ofNullable(str)
54+
.map(s -> Enum.valueOf(ActionDTO.class, s.toUpperCase()))
55+
.orElseThrow();
56+
}
4957
}
5058

5159
private final DynamicConfigOperations dynamicConfigOperations;
@@ -75,7 +83,7 @@ public Mono<ResponseEntity<ApplicationConfigDTO>> getCurrentConfig(ServerWebExch
7583
@Override
7684
public Mono<ResponseEntity<Void>> restartWithConfig(Mono<RestartRequestDTO> restartRequestDto,
7785
ServerWebExchange exchange) {
78-
var context = AccessContext.builder()
86+
var context = AccessContext.builder()
7987
.applicationConfigActions(EDIT)
8088
.operationName("restartWithConfig")
8189
.build();

0 commit comments

Comments
 (0)