Repository navigation
Consolidate unwrapped input-name collection on SettableBeanProperty - #6415
wozhenbang2004 wants to merge 1 commit into
Conversation
Code Review ✅ Approved🟡 Medium risk · Consolidating unwrapped input-name collection changes deserialization field routing; incorrect collected names could misroute or reject valid nested properties. Moves effective input-name collection for unwrapped properties onto VerificationRules No rules evaluated Functional validation Not enabledFor more verification details, comment OptionsAuto-apply is off → Gitar will not commit updates to this branch. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Powered by Gitar — free for open source |
🧪 Code Coverage Report
|
Closes #6413.
Moves effective input-name collection to
SettableBeanProperty.collectValuePropertyNamesTo(), withUnwrappedPropertyHandlerdelegating to it. This keeps the existing any-setter short circuit, per-property opaque-deserializer detection, and PROPERTY/EXISTING_PROPERTY type-id handling. Type-id names are still not transformed by unwrapping prefixes or suffixes.Adds helper contract tests and a nested-unwrapped routing test that also checks ordinary nested values stay nested. Existing alias and polymorphic-unwrapped regression tests remain unchanged.
Validation on JDK 25:
./mvnw -B -ntp clean verify: 7,811 tests, 0 failures/errors, 1 skipped../mvnw -B -ntp test -Dtest='Unwrapped*Test': 107 tests passed.git diff --check: passed.