Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue with multi-fields with <input name="Field[]" /> #108

Open
tractorcow opened this issue Nov 28, 2019 · 2 comments
Open

Issue with multi-fields with <input name="Field[]" /> #108

tractorcow opened this issue Nov 28, 2019 · 2 comments

Comments

@tractorcow
Copy link

tractorcow commented Nov 28, 2019

This isn't a common use case, but I've had to implement displaylogic support for a module with multiple repeated fields.

A form contains multiple groups of fields with <fieldlist class="displaylogic-holder">. Additional fields are added via clientside using javascript (fieldlist is cloned).

Proof of concept is up and working at https://github.com/isobar-nz/silverstripe-display-logic/commits/fix-rebuild, although I'm not sure this should be merged upstream.

@tractorcow
Copy link
Author

I've also fixed an issue with IE11. It was erroring on the Object.entries() when looping.

@michalkleiner
Copy link
Collaborator

I'd be happy to look at a PR that adds support for array-style field names (FieldName[]), if we could also add some tests for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants