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

Added es_VE provider (Venezuela) #398

Merged
merged 5 commits into from
Oct 14, 2014
Merged

Conversation

DIOHz0r
Copy link
Contributor

@DIOHz0r DIOHz0r commented Aug 22, 2014

New provider

'{{lastName}} de {{lastName}}'
);

protected static $catchPhraseWords = array(
Copy link
Owner

Choose a reason for hiding this comment

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

This one is not localized, no need to override it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't understand this note. Actually it's localized, the $companyPrefix and $companySuffix were changed and one format was removed.

Copy link
Owner

Choose a reason for hiding this comment

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

The words are in English, it's supposed to be a Venezuelian provider, its data should be in es_VE.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

If you mean the formats the words "Y" and "de" are spanish prepositions. The formats are also like others es_* providers and you can check some venezuelan formats at this list of companies http://declaraciones.seniat.gob.ve/portal/page/portal/MANEJADOR_CONTENIDO_SENIAT/05MENU_HORIZONTAL/5-2.html

The next PR won't have the methods catchPhrase and bs.

@fzaninotto
Copy link
Owner

Still missing source for datasets.

@DIOHz0r
Copy link
Contributor Author

DIOHz0r commented Aug 26, 2014

If you mean about the format for person provider, I added the link at the first method as example.
The CNE is the electoral center of the country, no wiki page is available and the only es_* provider with the link is the es_ES

@fzaninotto
Copy link
Owner

All right. Can you please copy the link before each dataset? It's a good practice that I ask to all contributors.

DIOHz0r added a commit to DIOHz0r/Faker that referenced this pull request Aug 28, 2014
@fzaninotto
Copy link
Owner

Would you mind squashing the commits to avoid the two merge commits?

@DIOHz0r
Copy link
Contributor Author

DIOHz0r commented Oct 11, 2014

Hello, Did you forget about this PR?

fzaninotto added a commit that referenced this pull request Oct 14, 2014
Added es_VE provider (Venezuela)
@fzaninotto fzaninotto merged commit 4d9244b into fzaninotto:master Oct 14, 2014
@fzaninotto
Copy link
Owner

Yes I did. Sorry for the delay and thanks for your contribution!

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.

2 participants