Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

fix for fprg #5497

Closed
wants to merge 2 commits into from
Closed

fix for fprg #5497

wants to merge 2 commits into from

Conversation

Kill0p
Copy link
Contributor

@Kill0p Kill0p commented Nov 18, 2013

I got problem when i make collection for my form. My collection has target_element which can be added few times. I use fileprg because this form also contains file inputs.

When we use form without fileprg plugin everything is fine and added elements to collection are validated but after we make form with fileprg plus we have new elements added to collection, those new elements are not validated at all. I dig in and realized that getInputFilter() function in handleGetRequest() doesnt add InputFilters to form for new elements.

It doesnt know about it because form only contains information from the configuration of form. If we set 'count' => 1 in options of collection it will validate only first element.

@Maks3w
Copy link
Member

Maks3w commented Nov 18, 2013

Please add a unit test case

@weierophinney weierophinney added this to the 2.2.6 milestone Mar 3, 2014
@weierophinney weierophinney self-assigned this Mar 3, 2014
weierophinney added a commit that referenced this pull request Mar 3, 2014
weierophinney added a commit that referenced this pull request Mar 3, 2014
- function declaration, braces, whitespace
weierophinney added a commit that referenced this pull request Mar 3, 2014
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants