Optimize ZonedDateTime.now().toInstant()

[jlaur]

Creating a ZonedDateTime just to get its Instant is unnecessary.

[Copilot]

Pull request overview

Removes unnecessary ZonedDateTime allocations where only an Instant (or epoch millis) is required, and modernizes day-of-week formatting in a test.

Changes:

• Replace ZonedDateTime.now().toInstant() with Instant.now() where only epoch millis are needed.
• Replace legacy Date/SimpleDateFormat usage with DateTimeFormatter in the day-of-week test setup.
• Make locale handling for day-of-week uppercasing explicit.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
itests/org.openhab.core.automation.module.timer.tests/src/main/java/org/openhab/core/automation/module/timer/internal/DayOfWeekConditionHandlerTest.java	Switches to DateTimeFormatter for day-of-week derivation and removes legacy date APIs.
bundles/org.openhab.core/src/test/java/org/openhab/core/items/GenericItemTest.java	Uses Instant.now() for timestamp comparisons instead of creating ZonedDateTime instances.
bundles/org.openhab.core.io.rest.log/src/main/java/org/openhab/core/io/rest/log/internal/LogHandler.java	Uses Instant.now().toEpochMilli() for log timestamps instead of ZonedDateTime conversion.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[holgerfriedrich]

LGTM, thanks.
Use of default locale is gone now, too.