Nested Fieldset value can be a Traversable #4927

Merged
merged 5 commits into from Aug 19, 2013

Conversation

Projects
None yet
3 participants
Contributor

jaapio commented Aug 5, 2013

Since the populateValues method of Fieldset can be a Traversable it should be possible to add a traversable to a nested Fieldset. Which was impossible due the check on line 400.

@samsonasik samsonasik commented on an outdated diff Aug 5, 2013

tests/ZendTest/Form/TestAsset/ValueStoringFieldset.php
@@ -0,0 +1,29 @@
+<?php
+/**
+ * Zend Framework (http://framework.zend.com/)
+ *
+ * @link http://github.com/zendframework/zf2 for the canonical source repository
+ * @copyright Copyright (c) 2005-2013 Zend Technologies USA Inc. (http://www.zend.com)
+ * @license http://framework.zend.com/license/new-bsd New BSD License
+ * @package Zend_Form

@weierophinney weierophinney added a commit that referenced this pull request Aug 19, 2013

@weierophinney weierophinney Merge pull request #4927 from youngguns-nl/fieldset_traversable
Nested Fieldset value can be a Traversable
de01969

@weierophinney weierophinney added a commit that referenced this pull request Aug 19, 2013

@weierophinney weierophinney Merge branch 'feature/4927' into develop
Close #4927
7f7b14e

@weierophinney weierophinney merged commit a371885 into zendframework:develop Aug 19, 2013

1 check passed

default The Travis CI build passed
Details

jaapio deleted the youngguns-nl:fieldset_traversable branch Aug 20, 2013

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