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 9320869b16d178b2c5df910b41b75aa66257f3f9 Author: Walter Duque de Estrada <[email protected]> AuthorDate: Wed Feb 4 16:35:12 2026 -0600 GrailsPropertyBinder --- .../cfg/domainbinding/GrailsPropertyBinder.java | 3 +-- .../domainbinding/GrailsPropertyBinderSpec.groovy | 29 ++++++++++++++++------ 2 files changed, 22 insertions(+), 10 deletions(-) diff --git a/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/cfg/domainbinding/GrailsPropertyBinder.java b/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/cfg/domainbinding/GrailsPropertyBinder.java index 7256d4addd..eccb2b7916 100644 --- a/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/cfg/domainbinding/GrailsPropertyBinder.java +++ b/grails-data-hibernate7/core/src/main/groovy/org/grails/orm/hibernate/cfg/domainbinding/GrailsPropertyBinder.java @@ -69,10 +69,9 @@ public class GrailsPropertyBinder { // see if it's a collection type CollectionType collectionType = collectionHolder.get(currentGrailsProp.getType()); - Class<?> userType = currentGrailsProp.getUserType(); // 1. Create Value and apply binders (consolidated block) - if (userType != null && !UserCollectionType.class.isAssignableFrom(userType)) { + if (currentGrailsProp.isUserButNotCollectionType()) { value = new BasicValue(metadataBuildingContext, table); // No specific binder call needed for this case per original logic new SimpleValueBinder(namingStrategy).bindSimpleValue(currentGrailsProp, null,(SimpleValue) value, EMPTY_PATH); diff --git a/grails-data-hibernate7/core/src/test/groovy/org/grails/orm/hibernate/cfg/domainbinding/GrailsPropertyBinderSpec.groovy b/grails-data-hibernate7/core/src/test/groovy/org/grails/orm/hibernate/cfg/domainbinding/GrailsPropertyBinderSpec.groovy index 9c99dda646..4fa0d1f806 100644 --- a/grails-data-hibernate7/core/src/test/groovy/org/grails/orm/hibernate/cfg/domainbinding/GrailsPropertyBinderSpec.groovy +++ b/grails-data-hibernate7/core/src/test/groovy/org/grails/orm/hibernate/cfg/domainbinding/GrailsPropertyBinderSpec.groovy @@ -13,14 +13,14 @@ import org.hibernate.mapping.SimpleValue class GrailsPropertyBinderSpec extends HibernateGormDatastoreSpec { - void setupSpec() { - manager.addAllDomainClasses([ - org.apache.grails.data.testing.tck.domains.Pet, - org.apache.grails.data.testing.tck.domains.Person, - org.apache.grails.data.testing.tck.domains.PetType, - org.apache.grails.data.testing.tck.domains.PersonWithCompositeKey - ]) - } +// void setupSpec() { +// manager.addAllDomainClasses([ +// org.apache.grails.data.testing.tck.domains.Pet, +// org.apache.grails.data.testing.tck.domains.Person, +// org.apache.grails.data.testing.tck.domains.PetType, +// org.apache.grails.data.testing.tck.domains.PersonWithCompositeKey +// ]) +// } void "Test bind simple property"() { given: @@ -301,6 +301,19 @@ class Address { String zip } +@Entity +class TestEntityWithSerializableCollection { + Long id + List<SerializableObject> serializableObjects + static mapping = { + serializableObjects type: 'serializable' + } +} + +class SerializableObject { + String data +} + @Entity class ListAuthor { Long id
