LearnNewsExamplesServices
Frontmatter
id4515
titleform.Container: setValues() => no support for nested structures
stateClosed
labels
bug
assigneestobiu
createdAtJun 27, 2023, 7:21 PM
updatedAtJun 27, 2023, 7:35 PM
githubUrlhttps://github.com/neomjs/neo/issues/4515
authortobiu
commentsCount0
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtJun 27, 2023, 7:35 PM

form.Container: setValues() => no support for nested structures

Closed v8.1.0 bug
tobiu
tobiu commented on Jun 27, 2023, 7:21 PM

This bug sneaked into the framework when adding the setConfigs() method.

The current logic:

    async setValues(values={}, suspendEvents=false) {
        Object.entries(values).forEach(([key, value]) => {
            values[key] = {value}
        })

        await this.setConfigs(values, suspendEvents)
    }

is fine for simple forms, but not sufficient for nested structures.

All top level (root) items will get prefixed with value. However, we want to do this for all leave items instead.

This one will need testing!

@subramaniyamP: thanks for the heads up!

FYI: @dztoprak @Dinkh

tobiu added the bug label on Jun 27, 2023, 7:21 PM
tobiu assigned to @tobiu on Jun 27, 2023, 7:21 PM
tobiu referenced in commit 52792c1 - "form.Container: setValues() => no support for nested structures #4515" on Jun 27, 2023, 7:30 PM
tobiu closed this issue on Jun 27, 2023, 7:35 PM