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

Uuid Validator #58

Merged
merged 2 commits into from
May 16, 2016
Merged

Uuid Validator #58

merged 2 commits into from
May 16, 2016

Conversation

gsomoza
Copy link
Contributor

@gsomoza gsomoza commented Feb 27, 2016

I realise there's an open road to ZF3 PR but here's what I'd consider some low-hanging fruit in case you're planning any other 2.x releases.

NOTE: I'd be happy to upgrade this to the new format once the new API for validators is stabilised in that other PR. So if you don't think you'll release this in a new 2.x release let me know and I'll rebase the work against the ZF3 WIP branch.

*/
public function validUuidProvider()
{
return array(
Copy link
Contributor

Choose a reason for hiding this comment

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

can use short array syntax []

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

@weierophinney weierophinney added this to the 2.8.0 milestone May 12, 2016
@weierophinney weierophinney self-assigned this May 12, 2016
@weierophinney weierophinney merged commit ca0f167 into zendframework:develop May 16, 2016
weierophinney added a commit that referenced this pull request May 16, 2016
weierophinney added a commit that referenced this pull request May 16, 2016
weierophinney added a commit that referenced this pull request May 16, 2016
@weierophinney
Copy link
Member

Thanks, @gsomoza !

@gsomoza
Copy link
Contributor Author

gsomoza commented May 16, 2016

It's nothing! Happy to help a little bit :)

samsonasik added a commit to samsonasik/zend-validator that referenced this pull request May 16, 2016
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.

4 participants