Support for collection of java.time in query params

[indalaterre]

Hello, we need to support collection of dates in our endpoint.

This PR enable to usage of this kind of endpoints

public void myEndpoint(@RestQuery Set<LocalDate> dates)

[indalaterre]

@geoand can you please review this PR? If it's ok would it be possible to have this in the 3.14.1? Thank you very much.

[geoand]

Thanks for this.

I'm on PTO currently, I'll review when I get back

[geoand]

cc @FroMage

[FroMage]

LGTM, except the condition which should be simplified and harmonised, but great work, thanks :)

[FroMage]

I think something like this would be a better condition:

        } else if (SUPPORT_TEMPORAL_PARAMS.contains(DotName.createSimple(elementType))) {

This is what is done in handleOptionalParam already.

[indalaterre]

Now should be ok. Thanks

[FroMage]

Oh, and could you also please merge your commits?

[indalaterre]

Oh, and could you also please merge your commits?

Done

[quarkus-bot]

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit a7c18f1.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

---

Flaky tests - Develocity

⚙️ Gradle Tests - JDK 17 Windows

📦 integration-tests/gradle

✖ io.quarkus.gradle.devmode.MavenExclusionInExtensionDependencyDevModeTest.main - History

• Condition with Lambda expression in io.quarkus.test.devmode.util.DevModeClient was not fulfilled within 1 minutes 30 seconds. - org.awaitility.core.ConditionTimeoutException

org.awaitility.core.ConditionTimeoutException: Condition with Lambda expression in io.quarkus.test.devmode.util.DevModeClient was not fulfilled within 1 minutes  30 seconds.
	at app//org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at app//org.awaitility.core.CallableCondition.await(CallableCondition.java:78)
	at app//org.awaitility.core.CallableCondition.await(CallableCondition.java:26)
	at app//org.awaitility.core.ConditionFactory.until(ConditionFactory.java:1006)
	at app//org.awaitility.core.ConditionFactory.until(ConditionFactory.java:975)
	at app//io.quarkus.test.devmode.util.DevModeClient.getHttpResponse(DevModeClient.java:164)
	at app//io.quarkus.gradle.devmode.QuarkusDevGradleTestBase.getHttpResponse(QuarkusDevGradleTestBase.java:164)

[FroMage]

Great, thanks!

[indalaterre]

@FroMage is it possible to have this in a patch of 3.14?

[FroMage]

@FroMage is it possible to have this in a patch of 3.14?

@gsmet ?