Romanian issues 259

[mariaS210]

Fixes #259 reported by @silvium

Changes proposed in this pull request:

• 2 has different gender form when in hundreds or thousands area
• currency for 100 and 1000 has una instead of o, like on banknotes
• 10.000 - 100.000 with increment by 10.000 badly represented

Status

• [X ] READY
• HOLD
• WIP (Work-In-Progress)

How to verify this change

Issues reported:
Different treatment for 2:
num2words(22000, lang='ro')
until now: douăzeci și doi mie
from now: douăzeci și două mii

Currency for 100 and 1000 has a different form than years or simply numerals:
num2words(1000, lang='ro', to='currency')
until now: o mie
from now: una mie

10.000 - 100.000 with increment by 10.000 were badly represented:
num2words(20000, lang='ro')
until now: douăzeci și mie/i
from now: douăzeci mii

Additional notes

[coveralls]

Pull Request Test Coverage Report for Build 838

• 40 of 40 (100.0%) changed or added relevant lines in 2 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.02%) to 95.156%

---

Totals
Change from base Build 837:	0.02%
Covered Lines:	10130
Relevant Lines:	10496

---

💛 - Coveralls

[mariaS210]

this puppy is sad that pull request is not reviewed

Image by Winsker from Pixabay

[mariaS210]

hi @erozqba !
i noticed that i did not implement pluralize as defined in http://docs.translatehouse.org/projects/localization-guide/en/latest/l10n/pluralforms.html and that indeed Romanian has another plural form which i ignored. i have implemented it now.
could you please take another look?
thank you, and sorry for the delay

[erozqba]

Thanks a lot for your contribution @mariaS210 and please accept my apologies for all the time it takes to review and merge this 😭

[mariaS210]

Thanks a lot for your contribution @mariaS210 and please accept my apologies for all the time it takes to review and merge this 😭

no worries @erozqba i had an excuse to search pictures of puppies on the internet 😄