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

Fixation form collection helper lost second argument #7167

Closed
wants to merge 2 commits into from
Closed

Fixation form collection helper lost second argument #7167

wants to merge 2 commits into from

Conversation

olekhy
Copy link
Contributor

@olekhy olekhy commented Jan 29, 2015

Added second argument to helper call
Added unit test for fixation

olekhy added 2 commits January 29, 2015 13:23
@olekhy
Fixation form collection helper lost second argument
07ca640 
Added second argument to helper call
Added unit test for fixation
@olekhy
Fixation form collection helper lost second argument
e69c45e 
fixed unit test case
@Ocramius Ocramius self-assigned this Jan 29, 2015
@Ocramius Ocramius added this to the 2.3.5 milestone Jan 29, 2015
@prolic
Copy link
Contributor

prolic commented Jan 29, 2015

👍

Ocramius added a commit that referenced this pull request Feb 2, 2015
@Ocramius
#7167 - minor CS fixes (group annotations for tests, spacing)
1a62669
Ocramius added a commit that referenced this pull request Feb 2, 2015
@Ocramius
Merge branch 'hotfix/#7167-form-collection-helper-does-not-pass-unwra…
f42890e 
…p-parameter-to-fieldset-helper' into develop

Close #7167
Forward port #7167
@Ocramius Ocramius closed this in d54bbe1 Feb 2, 2015
@Ocramius
Copy link
Member

Ocramius commented Feb 2, 2015

Merged, thanks @olekhy!

master: d54bbe1
develop: f42890e

@olekhy
Copy link
Contributor Author

olekhy commented Feb 2, 2015

🙇

@olekhy olekhy deleted the FormViewHelperCollectionFix branch February 2, 2015 19:57
TomHAnderson pushed a commit to TomHAnderson/zf2 that referenced this pull request Feb 24, 2015
@Ocramius @TomHAnderson
zendframework#7167 - minor CS fixes (group annotations for tests, spa…
f5ee6c6 
…cing)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@Ocramius Ocramius

Labels
bug Form View
Projects
None yet
Milestone
2.3.5
Development

Successfully merging this pull request may close these issues.
3 participants
@olekhy @prolic @Ocramius