Skip to content
This repository has been archived by the owner on May 24, 2018. It is now read-only.

Documentation for https://github.com/zendframework/zf2/pull/6431 #1398

Merged
merged 2 commits into from
Jan 22, 2015

Conversation

brettminnie
Copy link
Contributor

Documentation required for the merge method on the input filter

@Ocramius
Copy link
Member

@GeeH
Copy link
Contributor

GeeH commented Jan 1, 2015

Any comments @Ocramius? I haven't kept up with this feature.

/**
* Loads the validators
*/
protected function setValidators()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove this method: we are writing examples, keep them as small as possible.

@brettminnie
Copy link
Contributor Author

@Ocramius just got back from my break, will go through these comments now and update the pull request shortly

Only shows the use of the merge method now, has no setup code or class
variables
@Ocramius
Copy link
Member

@brettminnie missed the update. Merging, thanks!

Ocramius added a commit that referenced this pull request Jan 22, 2015
@Ocramius Ocramius merged commit f5b9573 into zendframework:develop Jan 22, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants