Fix optional

mansoor-sajjad · mansoor-sajjad · commit 8e95a50831b3 · 2024-12-18T12:28:19.000+01:00
diff --git a/src/main/java/no/entur/antu/netexdata/collectors/activedatecollector/calender/ActiveDatesBuilder.java b/src/main/java/no/entur/antu/netexdata/collectors/activedatecollector/calender/ActiveDatesBuilder.java
@@ -177,7 +177,7 @@ private void activeDatesForDates(
       if (activeDatesForDayTypeRef.containsKey(dayTypeId)) {
         Optional
           .ofNullable(includedAndExcludedDates.get(Boolean.TRUE))
-          .map(activeDatesForDayTypeRef.get(dayTypeId).dates()::addAll);
+          .ifPresent(activeDatesForDayTypeRef.get(dayTypeId).dates()::addAll);
       }
     }
   }
@@ -208,7 +208,7 @@ private void activeDatesForOperatingDays(
       if (activeDatesForDayTypeRef.containsKey(dayTypeId)) {
         Optional
           .ofNullable(includedAndExcludedDates.get(Boolean.TRUE))
-          .map(activeDatesForDayTypeRef.get(dayTypeId).dates()::addAll);
+          .ifPresent(activeDatesForDayTypeRef.get(dayTypeId).dates()::addAll);
       }
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -177,7 +177,7 @@ private void activeDatesForDates(`
`177`	`177`	`if (activeDatesForDayTypeRef.containsKey(dayTypeId)) {`
`178`	`178`	`Optional`
`179`	`179`	`.ofNullable(includedAndExcludedDates.get(Boolean.TRUE))`
`180`		`- .map(activeDatesForDayTypeRef.get(dayTypeId).dates()::addAll);`
	`180`	`+ .ifPresent(activeDatesForDayTypeRef.get(dayTypeId).dates()::addAll);`
`181`	`181`	`}`
`182`	`182`	`}`
`183`	`183`	`}`
`@@ -208,7 +208,7 @@ private void activeDatesForOperatingDays(`
`208`	`208`	`if (activeDatesForDayTypeRef.containsKey(dayTypeId)) {`
`209`	`209`	`Optional`
`210`	`210`	`.ofNullable(includedAndExcludedDates.get(Boolean.TRUE))`
`211`		`- .map(activeDatesForDayTypeRef.get(dayTypeId).dates()::addAll);`
	`211`	`+ .ifPresent(activeDatesForDayTypeRef.get(dayTypeId).dates()::addAll);`
`212`	`212`	`}`
`213`	`213`	`}`
`214`	`214`	`}`