This is an automated email from the ASF dual-hosted git repository. borinquenkid pushed a commit to branch 8.0.x-hibernate7 in repository https://gitbox.apache.org/repos/asf/grails-core.git
commit a33e502810c61c4ce9f47de47a3b220616110e73 Author: Walter Duque de Estrada <[email protected]> AuthorDate: Fri Feb 13 14:52:48 2026 -0600 refactored BasicValueIdCreator by changing its public constructor to take Table instead of RootClass as the final argument and updated the imports accordingly. I also updated SimpleIdBinder to pass entity.getTable() to the constructor. --- .../orm/hibernate/cfg/domainbinding/binder/SimpleIdBinder.java | 2 +- .../orm/hibernate/cfg/domainbinding/util/BasicValueIdCreator.java | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/cfg/domainbinding/binder/SimpleIdBinder.java b/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/cfg/domainbinding/binder/SimpleIdBinder.java index 63ec419a4a..0dba050694 100644 --- a/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/cfg/domainbinding/binder/SimpleIdBinder.java +++ b/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/cfg/domainbinding/binder/SimpleIdBinder.java @@ -51,7 +51,7 @@ public class SimpleIdBinder { boolean useSequence = result != null && result.isTablePerConcreteClass(); // create the id value - BasicValueIdCreator idCreator = this.basicValueIdCreator != null ? this.basicValueIdCreator : new BasicValueIdCreator(metadataBuildingContext, jdbcEnvironment, domainClass, entity); + BasicValueIdCreator idCreator = this.basicValueIdCreator != null ? this.basicValueIdCreator : new BasicValueIdCreator(metadataBuildingContext, jdbcEnvironment, domainClass, entity.getTable()); BasicValue id = idCreator.getBasicValueId(mappedId, useSequence); var identifier = domainClass.getIdentity(); diff --git a/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/cfg/domainbinding/util/BasicValueIdCreator.java b/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/cfg/domainbinding/util/BasicValueIdCreator.java index 862aaeda90..cd3604a7de 100644 --- a/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/cfg/domainbinding/util/BasicValueIdCreator.java +++ b/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/cfg/domainbinding/util/BasicValueIdCreator.java @@ -5,7 +5,7 @@ import org.hibernate.engine.jdbc.env.spi.JdbcEnvironment; import org.hibernate.generator.Generator; import org.hibernate.generator.GeneratorCreationContext; import org.hibernate.mapping.BasicValue; -import org.hibernate.mapping.RootClass; +import org.hibernate.mapping.Table; import org.grails.orm.hibernate.cfg.GrailsHibernatePersistentEntity; import org.grails.orm.hibernate.cfg.Identity; @@ -20,9 +20,9 @@ public class BasicValueIdCreator { private final BasicValue id; private final GrailsSequenceWrapper grailsSequenceWrapper; - public BasicValueIdCreator(MetadataBuildingContext metadataBuildingContext, JdbcEnvironment jdbcEnvironment, GrailsHibernatePersistentEntity domainClass, RootClass entity) { + public BasicValueIdCreator(MetadataBuildingContext metadataBuildingContext, JdbcEnvironment jdbcEnvironment, GrailsHibernatePersistentEntity domainClass, Table table) { // create a prototype BasicValue (table will be set per-entity when creating the actual BasicValue) - this.id = new BasicValue(metadataBuildingContext, entity.getTable()); + this.id = new BasicValue(metadataBuildingContext, table); this.jdbcEnvironment = jdbcEnvironment; this.domainClass = domainClass; this.grailsSequenceWrapper = new GrailsSequenceWrapper();
