Skip to content

Conversation

gavinking
Copy link
Member

@gavinking gavinking commented Aug 2, 2025

[Please describe here what your change is about]


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.


https://hibernate.atlassian.net/browse/HHH-19678
https://hibernate.atlassian.net/browse/HHH-19676

@hibernate-github-bot
Copy link

hibernate-github-bot bot commented Aug 2, 2025

Thanks for your pull request!

This pull request does not follow the contribution rules. Could you have a look?

❌ All commit messages should start with a JIRA issue key matching pattern HHH-\d+
    ↳ Offending commits: [5881ad0]

› This message was automatically generated.

}

public static void check(String fragment) {
final var parseTree = buildParseTree( fragment );

Check notice

Code scanning / CodeQL

Unread local variable Note

Variable 'OrderByFragmentContext parseTree' is never read.
@gavinking gavinking marked this pull request as ready for review August 3, 2025 06:17
@gavinking
Copy link
Member Author

I'm going to merge the work on this so far, even though OrderByFragmentTranslator can't yet do type-checking when called from the Processor. To fix this is a bigger task, probably involving rewriting ParseTreeVisitor to not depend on PluralAttributeMapping, which is not so-far available in the Processor, and looks hard to mock.

@gavinking gavinking merged commit aeaa65c into hibernate:main Aug 3, 2025
23 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant