Add fluidity to the prepare() function for a form #2644

Closed
wants to merge 1 commit into from

2 participants

@Mezzle

When dealing with some of the code, I found that there was some fluidity missing ... I've added this.

@weierophinney weierophinney commented on the diff Oct 2, 2012
library/Zend/Form/Form.php
@@ -159,7 +159,7 @@ public function add($elementOrFieldset, array $flags = array())
* available, and prepares any elements and/or fieldsets that require
* preparation.
*
- * @return void
+ * @return \Zend\Form\Form
@weierophinney
Zend Framework member

This can be simply "Form"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@weierophinney weierophinney was assigned Oct 2, 2012
@weierophinney weierophinney added a commit that referenced this pull request Oct 2, 2012
@weierophinney weierophinney [#2644] CS/logic fixes
- Updated @return annotation
- Moved logic out of conditional
9548341
@weierophinney weierophinney added a commit that referenced this pull request Oct 2, 2012
@weierophinney weierophinney Merge branch 'hotfix/2644' into develop
Forward port #2644
39e4e1d
@weierophinney weierophinney added a commit that closed this pull request Oct 2, 2012
@weierophinney weierophinney Merge branch 'hotfix/2644'
Close #2644
a5d4f8d
@weierophinney
Zend Framework member

I made the change on merge. Thanks for the pull request!

@ghost Unknown pushed a commit that referenced this pull request Jul 14, 2013
@weierophinney weierophinney [#2644] CS/logic fixes
- Updated @return annotation
- Moved logic out of conditional
b74a2b0
@ghost Unknown pushed a commit that referenced this pull request Jul 14, 2013
@weierophinney weierophinney Merge branch 'hotfix/2644'
Close #2644
860b4ee
@ghost Unknown pushed a commit that referenced this pull request Jul 14, 2013
@weierophinney weierophinney Merge branch 'hotfix/2644' into develop
Forward port #2644
84d6f9a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment