jdaugherty commented on code in PR #15587:
URL: https://github.com/apache/grails-core/pull/15587#discussion_r3150873376
##########
grails-data-graphql/core/src/test/groovy/org/grails/gorm/graphql/entity/property/impl/HibernatePersistentGraphQLPropertySpec.groovy:
##########
@@ -359,8 +359,11 @@ class HibernatePersistentGraphQLPropertySpec extends
HibernateSpec {
'order8' | 8 //specified via mapping
'order0' | 0 //specified as 0
'orderNeg' | -21 //specified as -10
- 'orderNullc' | 6 //not specified, gorm supplied
- 'orderNulld' | 7 //not specified, gorm supplied
+ // Grails 7: GORM no longer auto-supplies a constraint order for
properties
Review Comment:
I'm assuming it's related to the groovy property handling changes. It's
probably an ok change, but it would be nice if @kirpi4ik could weigh in to see
if they think this is a possible problem. I'm inclined to take it and let it
release with 7.2 unless objections require us to dig further on why it changed.
--
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]
