Thanks to visit codestin.com
Credit goes to github.com

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

Validator Translations#2312

Merged
weierophinney merged 3 commits into
zendframework:masterfrom
krmarien:master
Sep 13, 2012
Merged

Validator Translations#2312
weierophinney merged 3 commits into
zendframework:masterfrom
krmarien:master

Conversation

@krmarien
Copy link
Copy Markdown
Contributor

@krmarien krmarien commented Sep 7, 2012

Updated the English and Dutch validation message translations (http://framework.zend.com/issues/browse/ZF2-491)
French validation messages are also updated (@bakura10)

Comment thread library/Zend/Validator/Identical.php Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Our coding standards suggest to use single quotes where possible Can you revert this changes?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

My changes in the validators are reverted

@bakura10
Copy link
Copy Markdown
Contributor

bakura10 commented Sep 9, 2012

Hi krmarien,

Based on your updated English validation messages, I've also updated the French one. Please could you add it to your PR (and update the description) ?

https://github.com/bakura10/zf2/blob/8b1f93333ff5753fa6494d06c735d7a40f16181e/resources/languages/fr/Zend_Validate.php

Thanks :)

@weierophinney weierophinney merged commit dde1c76 into zendframework:master Sep 13, 2012
@weierophinney
Copy link
Copy Markdown
Member

Merged to release and master branches.

@ghost ghost assigned weierophinney Sep 13, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants