jamesfredley commented on code in PR #15731:
URL: https://github.com/apache/grails-core/pull/15731#discussion_r3398876618
##########
grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/HibernateGormStaticApi.groovy:
##########
@@ -353,22 +354,42 @@ class HibernateGormStaticApi<D> extends GormStaticApi<D> {
if (!queryMap) return null
Map coercedMap = queryMap.collectEntries { k, v -> [k.toString(), v] }
String hql = buildWhereHql(coercedMap)
- doSingleInternal(hql, coercedMap, [], args, false)
+ doSingleInternal(hql, buildWhereParams(coercedMap), [],
buildFindWhereArgs(args), false)
Review Comment:
Thanks, applied in `00b1e05ce4`. The generated `findWhere`/`findAllWhere`
HQL now flows through the shared `prepareHqlQuery` path so
`HqlQueryContext.prepare(...)` remains responsible for HQL/query setup while
preserving settings, supported JPA hints, query events, and the existing `max:
1` behavior. I also added coverage for mapped-column rejection and
`AvailableHints.HINT_READ_ONLY` on `findWhere`.
--
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]
