borinquenkid opened a new issue, #15764:
URL: https://github.com/apache/grails-core/issues/15764
## Summary
Two classes in `grails-data-hibernate7` were ported from H5 code and carry
style issues that were intentionally deferred from PR #15568 to keep the review
surface focused on the H7 integration work.
## Files
-
`grails-data-hibernate7/core/src/main/groovy/grails/orm/HibernateCriteriaBuilder.java`
- Many fully-qualified class names used inline (e.g.
`org.hibernate.Session`, `grails.gorm.DetachedCriteria`) instead of imports
- Local variable types that could use `var`
-
`grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/cfg/domainbinding/binder/CollectionBinder.java`
- Local variable declarations that could use `var` (e.g. line 95)
## Work
1. Replace inline FQCNs with import statements in
`HibernateCriteriaBuilder.java`
2. Replace eligible local variable declarations with `var` in both files
3. Run `./gradlew :grails-data-hibernate7-core:test` to verify no regressions
## References
- PR #15568 (H7 Step 2) — review comments from @matrei
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
