jamesfredley commented on code in PR #15950:
URL: https://github.com/apache/grails-core/pull/15950#discussion_r3726585335
##########
grails-web-databinding/src/main/groovy/grails/web/databinding/DataBindingUtils.java:
##########
@@ -576,6 +605,619 @@ else if (include.isEmpty()) {
return bindingResult;
}
+ private static void assignNullToMissingIncludedProperties(Object object,
DataBindingSource bindingSource, List include, List exclude, String filter) {
Review Comment:
Addressed in 69ed46edca. The second-pass path grammar and clearing logic are
extracted from `DataBindingUtils` into the focused `NullMissingPropertyClearer`
collaborator. Public `bindData` regressions cover nested and indexed collection
paths, map paths, prefixes, exclusions, blank values, framework properties,
bindability, primitives, and failure paths, without testing package-private
implementation details directly.
--
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]
