jamesfredley commented on code in PR #16733:
URL: https://github.com/apache/grails-core/pull/16733#discussion_r4355313996


##########
grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/query/JpaQueryContext.java:
##########
@@ -173,6 +175,7 @@ public Expression<?> getAliasedExpression(String alias) {
         return expr;
     }
 
+    @Override
     public Expression<?> getFullyQualifiedExpression(String path) {

Review Comment:
   `getFullyQualifiedExpression('total')` still asks only the property 
resolver. The `sqlProjection` alias is recorded with `registerSelectionAlias`, 
so this call returns null, or an entity property named `total`. The context 
javadoc says this method accepts the alias of a projection.
   
   Return the recorded selection when the path is that alias. Please order 
through `getFullyQualifiedExpression('total')`, not only `getSelectionAlias`.



-- 
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]

Reply via email to