PersistenceExtensions: fix DateTimeException when persisting an empty TimeSeries (#4303)

jimtng · web-flow · commit ea7d61b19983 · 2024-07-08T19:53:01.000+02:00
Signed-off-by: Jimmy Tanagra &lt;jcode@tanagra.id.au&gt;
diff --git a/bundles/org.openhab.core.persistence/src/main/java/org/openhab/core/persistence/extensions/PersistenceExtensions.java b/bundles/org.openhab.core.persistence/src/main/java/org/openhab/core/persistence/extensions/PersistenceExtensions.java
@@ -211,7 +211,7 @@ public static void persist(Item item, TimeSeries timeSeries, @Nullable String se
 
     private static void internalPersist(Item item, TimeSeries timeSeries, @Nullable String serviceId) {
         String effectiveServiceId = serviceId == null ? getDefaultServiceId() : serviceId;
-        if (effectiveServiceId == null) {
+        if (effectiveServiceId == null || timeSeries.size() == 0) {
             return;
         }
         PersistenceService service = getService(effectiveServiceId);

Original file line number	Diff line number	Diff line change
`@@ -211,7 +211,7 @@ public static void persist(Item item, TimeSeries timeSeries, @Nullable String se`
`211`	`211`
`212`	`212`	`private static void internalPersist(Item item, TimeSeries timeSeries, @Nullable String serviceId) {`
`213`	`213`	`String effectiveServiceId = serviceId == null ? getDefaultServiceId() : serviceId;`
`214`		`- if (effectiveServiceId == null) {`
	`214`	`+ if (effectiveServiceId == null \|\| timeSeries.size() == 0) {`
`215`	`215`	`return;`
`216`	`216`	`}`
`217`	`217`	`PersistenceService service = getService(effectiveServiceId);`