This is an automated email from the ASF dual-hosted git repository. matrei pushed a commit to branch fix/issue-16133 in repository https://gitbox.apache.org/repos/asf/grails-core.git
commit 22633948995761dd1ac09e6a7ef72e74fc47ddaf Author: Mattias Reichel <[email protected]> AuthorDate: Thu Aug 13 17:17:59 2026 +0200 fix: initialize `defaultNullable` to `true` in `AbstractGormMappingFactory` --- .../org/grails/datastore/mapping/config/AbstractGormMappingFactory.java | 2 +- .../org/grails/datastore/mapping/model/AbstractMappingContext.java | 1 + .../mapping/keyvalue/mapping/KeyValueMappingFactoryTests.groovy | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/grails-datastore-core/src/main/groovy/org/grails/datastore/mapping/config/AbstractGormMappingFactory.java b/grails-datastore-core/src/main/groovy/org/grails/datastore/mapping/config/AbstractGormMappingFactory.java index d7a33ed134..9e3f66a649 100644 --- a/grails-datastore-core/src/main/groovy/org/grails/datastore/mapping/config/AbstractGormMappingFactory.java +++ b/grails-datastore-core/src/main/groovy/org/grails/datastore/mapping/config/AbstractGormMappingFactory.java @@ -54,7 +54,7 @@ public abstract class AbstractGormMappingFactory<R extends Entity, T extends Pro private Object contextObject; protected Closure defaultConstraints; protected boolean versionByDefault = true; - protected boolean defaultNullable = false; + protected boolean defaultNullable = true; /** * @param contextObject Context object to be passed to mapping blocks diff --git a/grails-datastore-core/src/main/groovy/org/grails/datastore/mapping/model/AbstractMappingContext.java b/grails-datastore-core/src/main/groovy/org/grails/datastore/mapping/model/AbstractMappingContext.java index 8afab3ed45..e8e139b619 100644 --- a/grails-datastore-core/src/main/groovy/org/grails/datastore/mapping/model/AbstractMappingContext.java +++ b/grails-datastore-core/src/main/groovy/org/grails/datastore/mapping/model/AbstractMappingContext.java @@ -109,6 +109,7 @@ public abstract class AbstractMappingContext implements MappingContext, Initiali AbstractGormMappingFactory gormMappingFactory = (AbstractGormMappingFactory) mappingFactory; gormMappingFactory.setDefaultConstraints(settings.getDefault().getConstraints()); gormMappingFactory.setDefaultMapping(settings.getDefault().getMapping()); + gormMappingFactory.setDefaultNullable(settings.getDefault().isNullable()); } } diff --git a/grails-datastore-core/src/test/groovy/org/grails/datastore/mapping/keyvalue/mapping/KeyValueMappingFactoryTests.groovy b/grails-datastore-core/src/test/groovy/org/grails/datastore/mapping/keyvalue/mapping/KeyValueMappingFactoryTests.groovy index 681898b1b9..ad013b9f94 100644 --- a/grails-datastore-core/src/test/groovy/org/grails/datastore/mapping/keyvalue/mapping/KeyValueMappingFactoryTests.groovy +++ b/grails-datastore-core/src/test/groovy/org/grails/datastore/mapping/keyvalue/mapping/KeyValueMappingFactoryTests.groovy @@ -67,7 +67,7 @@ class KeyValueMappingFactoryTests { PersistentProperty prop = entity.getPropertyByName('nonFormulaProperty') assert !prop.mapping.mappedForm.derived - assert !prop.mapping.mappedForm.nullable + assert prop.mapping.mappedForm.nullable // Formula properties should be flagged as derived prop = entity.getPropertyByName('formulaProperty')
