kirpi4ik commented on code in PR #15587:
URL: https://github.com/apache/grails-core/pull/15587#discussion_r3181498564
##########
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:
Default order for unspecified properties is assigned sequentially by
DefaultConstraintEvaluator(PersistentEntity.getPersistentProperties()
iteration) which under Groovy 4's MetaClass returns properties in source
declaration order (vs Groovy 3's alphabetical order).
Source order in `Ordering.groovy` : `orderNullc, orderNulld, order8` — so
the two(order8 has 8 via mapping override) without an explicit
graphql/constraint mapping land at constraintCount+1 and +2.
--
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]
