LearnNewsExamplesServices
Frontmatter
id4893
titleform.Container: adjustTreeLeaves() => sharper separation of the key & value realms
stateClosed
labels
bug
assigneestobiu
createdAtSep 11, 2023, 11:03 PM
updatedAtSep 12, 2023, 3:29 PM
githubUrlhttps://github.com/neomjs/neo/issues/4893
authortobiu
commentsCount0
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtSep 12, 2023, 3:29 PM

form.Container: adjustTreeLeaves() => sharper separation of the key & value realms

Closed v8.1.0 bug
tobiu
tobiu commented on Sep 11, 2023, 11:03 PM

when using a deeply nested form field names structure including objects & arrays, we need a smart way to check when we are leaving field paths and getting into the value realm.

the current version will break for checkbox groups, containing an array of items.

we should check for objects & arrays if a path matches a form field (then entering the value realm) or if it is not (staying inside the key realm).

i will dive into this tomorrow.

tobiu added the bug label on Sep 11, 2023, 11:03 PM
tobiu assigned to @tobiu on Sep 11, 2023, 11:03 PM
tobiu referenced in commit 19bd2f8 - "#4893 form.Container: adjustTreeLeaves() => passing all field paths to the method" on Sep 12, 2023, 8:12 AM
tobiu referenced in commit 6200d85 - "form.Container: adjustTreeLeaves() => sharper separation of the key & value realms #4893" on Sep 12, 2023, 3:26 PM
tobiu referenced in commit 41db2d6 - "#4893 removed a testing log" on Sep 12, 2023, 3:27 PM
tobiu closed this issue on Sep 12, 2023, 3:29 PM
tobiu referenced in commit c76eccb - "v6.5.0 (#4898) on Sep 12, 2023, 3:37 PM