Skip to content

Commit c8c61c8

Browse files
committed
chore: removed unnecessary printlns and safely wrapped nullable value in isEnabled call
1 parent cc5c870 commit c8c61c8

2 files changed

Lines changed: 1 addition & 3 deletions

File tree

examples/spring-boot-example/src/main/java/io/getunleash/unleash/example/ProxyController.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ public ProxyController(Unleash unleash) {
2323

2424
@GetMapping("/")
2525
public Map<String, Boolean> getEnabledToggles() {
26-
System.out.println("Fetching enabled he toggles");
2726
return toggles()
2827
.filter(Map.Entry::getValue)
2928
.collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
@@ -36,7 +35,6 @@ public Map<String, Boolean> getAllToggles() {
3635

3736
@GetMapping("/toggle/{toggleName}")
3837
public Boolean getToggle(@PathVariable("toggleName") String name) {
39-
System.out.println("Hahaa" + name);
4038
return unleash.isEnabled(name);
4139
}
4240

src/main/java/io/getunleash/DefaultUnleash.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ public boolean isEnabled(
116116
Optional<FlatResponse<Boolean>> response =
117117
Optional.ofNullable(this.featureRepository.isEnabled(toggleName, enhancedContext));
118118
boolean enabled =
119-
response.map(r -> r.value)
119+
response.flatMap(r -> Optional.ofNullable(r.value))
120120
.orElseGet(() -> fallbackAction.test(toggleName, enhancedContext));
121121
eventDispatcher.dispatch(new ToggleEvaluated(toggleName, enabled));
122122
if (response.map(r -> r.impressionData).orElse(false)) {

0 commit comments

Comments
 (0)