Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MessageBundle naming issue #39960

Closed
ercalamar opened this issue Apr 9, 2024 · 3 comments · Fixed by #39990
Closed

MessageBundle naming issue #39960

ercalamar opened this issue Apr 9, 2024 · 3 comments · Fixed by #39990
Labels
area/qute The template engine env/windows Impacts Windows machines kind/bug Something isn't working
Milestone

Comments

@ercalamar
Copy link

ercalamar commented Apr 9, 2024

Describe the bug

In this kind of situation:

public class EmailBundles {
    @MessageBundle
    interface started {
        @Message
        String started(String id, String filename);
        @Message
        String documentAccessUrl(String url);
...
    @MessageBundle
    interface startedValidator {
        @Message
        String started(String id, String filename);
        @Message
        String turnEmailWillBeSent();
...

The following error appears:

io.quarkus.qute.deployment.MessageBundleException: Message bundle method turnEmailWillBeSent() not found on: com.iarchiva.EmailBundles$started

If I rename "interface started {" to "interface rstarted {" the initial part of the name does not match and there is no issue.

Expected behavior

I expect no name collision between "started" and "startedValidator".

Actual behavior

Names collide being different.

How to Reproduce?

Having two interfaces that start with the same string.

Output of uname -a or ver

Microsoft Windows [Versión 10.0.22631.3296]

Output of java -version

openjdk version "21" 2023-09-19 OpenJDK Runtime Environment GraalVM CE 21+35.1 (build 21+35-jvmci-23.1-b15) OpenJDK 64-Bit Server VM GraalVM CE 21+35.1 (build 21+35-jvmci-23.1-b15, mixed mode, sharing)

Quarkus version or git rev

3.9.2

Build tool (ie. output of mvnw --version or gradlew --version)

Gradle 8.6

Additional information

No response

@ercalamar ercalamar added the kind/bug Something isn't working label Apr 9, 2024
@quarkus-bot quarkus-bot bot added env/windows Impacts Windows machines triage/needs-triage labels Apr 9, 2024
@gsmet gsmet added area/qute The template engine and removed triage/needs-triage labels Apr 9, 2024
Copy link

quarkus-bot bot commented Apr 9, 2024

/cc @mkouba (qute)

@gastaldi
Copy link
Contributor

gastaldi commented Apr 9, 2024

Can you attach a sample reproducer project please?

@ercalamar
Copy link
Author

qute-locale.zip

As requested :)

Another issue is moving the 3 classes in the project to the default package.

@quarkus-bot quarkus-bot bot added this to the 3.10 - main milestone Apr 12, 2024
@gsmet gsmet modified the milestones: 3.10.0.CR1, 3.9.4 Apr 17, 2024
@gsmet gsmet modified the milestones: 3.9.4, 3.8.5 May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/qute The template engine env/windows Impacts Windows machines kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants