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

personalIdentityNumber on es_ES Person #408

Closed
wants to merge 2 commits into from

Conversation

mimopo
Copy link

@mimopo mimopo commented Aug 29, 2014

Hi! Can we have the personalIdentityNumber method on es_ES Person like in sv_SE Person?

Thanks!

*/
public function personalIdentityNumber()
{
$number = rand(10000000, 99999999);
Copy link
Owner

Choose a reason for hiding this comment

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

using rand() in Faker is prohibited; it kills seeding. Use mt_rand() instead.

Generate numbers from 100 to 99999999.
Doc added.
@mimopo
Copy link
Author

mimopo commented Sep 1, 2014

Hi!

I've done another pull request, I've added three urls with documentation about the number generation. The function returns a DNI with letter (NIF).

Thanks!

@fzaninotto
Copy link
Owner

Could you add a unit test?

@mimopo
Copy link
Author

mimopo commented Sep 1, 2014

Ok, I will add the unit test.

@fzaninotto
Copy link
Owner

Bump! Could you add a unit test?

@mimopo
Copy link
Author

mimopo commented Jan 8, 2015

Sorry, I had completely forgotten. I'will try to add the unit test asap (may be this weekend).

@fzaninotto
Copy link
Owner

You should add a unit test and a link in the README (in the locale section). I'll close this PR within a few weeks otherwise.

@fzaninotto
Copy link
Owner

Superseded by #763

@fzaninotto fzaninotto closed this Feb 23, 2016
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