Qute: fix template variants for templates with dot in the name

[mkouba]

• fixes Error with Qute templates with dot in the names #46254

[mkouba]

This should fix the problem but keep in mind that the template foo.1.html cannot be injected directly in a bean unless the @Location is used (or 1.html is registered as a supported suffix).

[ia3andy]

any reason not to use lastIndexOf? I guess it's probably better to use the config like you do..?

[quarkus-bot]

Status for workflow Quarkus CI

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

✅ 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.

[gsmet]

This should fix the problem but keep in mind that the template foo.1.html cannot be injected directly in a bean unless the @location is used (or 1.html is registered as a supported suffix).

Not sure about lastIndexOf as you might have composite extensions but couldn't you try with 1.html and if not found and contains a dot, try with .html?

[mkouba]

This should fix the problem but keep in mind that the template foo.1.html cannot be injected directly in a bean unless the @location is used (or 1.html is registered as a supported suffix).

Not sure about lastIndexOf as you might have composite extensions but couldn't you try with 1.html and if not found and contains a dot, try with .html?

I'm not sure I completely understand 🤔. In any case, this would not work if you have @Inject Template foo and multiple matching templates, e.g. foo.1.html and foo.2.html.

I'm using the quarkus.qute.suffixes here, because they're used when performing the lookup. For example, if you do @Inject Template foo then we attempt to find the templates matching the name foo, foo.qute.html, foo.qute.txt, foo.html and foo.txt by default. Afterwards, we can determine a specific template based on selected variant or simply take the first found (according to the ordering in quarkus.qute.suffixes).

[ia3andy]

I will give it a try today.. thanks @mkouba

[mkouba]

I will give it a try today.. thanks @mkouba

Thanks, let me know... I tested this fix with your reproducer so it should be fine 😉.

[ia3andy]

Thanks!!