Skip to content

Commit c5ce718

Browse files
committed
Fix PMD
1 parent 2f951d0 commit c5ce718

File tree

2 files changed

+6
-16
lines changed

2 files changed

+6
-16
lines changed

vaadin-date-range-picker/src/main/java/software/xdev/vaadin/daterange_picker/business/AbstractDateRange.java

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -34,14 +34,8 @@ public abstract class AbstractDateRange<SELF extends AbstractDateRange<SELF>> im
3434
private String defaultDesc;
3535
private Function<LocalDate, Optional<DateRangeResult>> calcForFunc;
3636
private BiFunction<LocalDate, Integer, Optional<DateRangeResult>> moveFunc = (date, count) ->
37-
{
38-
if(this.optMovePeriod.isEmpty())
39-
{
40-
return Optional.empty();
41-
}
42-
43-
return this.calcForFunc.apply(count != 0 ? date.plus(this.optMovePeriod.get().multipliedBy(count)) : date);
44-
};
37+
this.optMovePeriod.flatMap(movePeriod ->
38+
this.calcForFunc.apply(count != 0 ? date.plus(movePeriod.multipliedBy(count)) : date));
4539
private boolean movable = true;
4640
private boolean calcable = true;
4741
private boolean setable = true;

vaadin-date-range-picker/src/main/java/software/xdev/vaadin/daterange_picker/ui/DateRangePickerOverlay.java

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -174,14 +174,10 @@ protected void moveRange(final int dif)
174174

175175
protected void calcModel(final Optional<DateRangeResult> optResult, final DateRangeModel<D> model)
176176
{
177-
if(optResult.isEmpty())
178-
{
179-
return;
180-
}
181-
182-
final DateRangeResult result = optResult.get();
183-
model.setStart(result.getStart());
184-
model.setEnd(result.getEnd());
177+
optResult.ifPresent(result -> {
178+
model.setStart(result.getStart());
179+
model.setEnd(result.getEnd());
180+
});
185181
}
186182

187183
protected void onValueChange(final Function<DateRangeModel<D>, Optional<DateRangeResult>> calcFunc)

0 commit comments

Comments
 (0)