-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Qute message bundles - fix application class predicate #20204
Conversation
mkouba
commented
Sep 16, 2021
- resolves io.quarkus.qute.i18n.MessageBundle annotated class *__Bundle not found for multi-module gradle project in test mode #20064
@@ -1041,8 +1035,7 @@ public boolean test(String name) { | |||
} | |||
// E.g. to match the bundle class generated for a localized file; org.acme.Foo_en -> org.acme.Foo | |||
className = additionalClassNameSanitizer.apply(className); | |||
return applicationArchives.containingArchive(className) != null | |||
|| GeneratedClassGizmoAdaptor.isApplicationClass(name); | |||
return GeneratedClassGizmoAdaptor.isApplicationClass(className); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The main part of the fix is GeneratedClassGizmoAdaptor.isApplicationClass(name)
-> GeneratedClassGizmoAdaptor.isApplicationClass(className)
!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I assume this should be backported, right? |