Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some Arabic ordinals are not correct #166

Closed
abdelrhmanghreeb opened this issue Apr 12, 2018 · 0 comments
Closed

Some Arabic ordinals are not correct #166

abdelrhmanghreeb opened this issue Apr 12, 2018 · 0 comments
Projects

Comments

@abdelrhmanghreeb
Copy link

Expected Behaviour

>import pyarabic.number as number
>nums = [6, 7, 10, 11,]
>for num in range(0, 20):
>    printnum2words(num, lang='ar', to='ordinal'))
أول
ثاني
ثالث
رابع
خامس
سادس
سابع
ثامن
تاسع
عاشر
حادي عشر
ثاني عشر
ثالث عشر
رابع عشر
خامس عشر
سادس عشر
سابع عشر
ثامن عشر
تاسع عشر

Actual Behaviour

Ordinals of 6, 7, 10, 11, 13-19 are wrong! This is what previous code snippet actually produces:

أول
ثاني
ثالث
رابع
خامس
*ستة*
*سبعة*
ثامن
تاسع
*عشرة*
أحد عشر
*ثاني عشر*
*ثلاثة عشر*
*أربعة عشر*
*خمسة عشر*
*ستة عشر*
*سبعة عشر*
*ثمانية عشر*
*تسعة عشر*

Steps to reproduce

Type the code I pasted

@ventilooo ventilooo added this to Needs triage in Support Jun 15, 2018
alhazmy13 added a commit to alhazmy13/num2words that referenced this issue Jul 3, 2018
ventilooo pushed a commit that referenced this issue Sep 11, 2018
* Change Arabic Lang implementation
* Improve Arabic class, * update Arabic test cases
* Support EGP and KWD currency
* Add test cases
* Remove .cache and .pytest_cache folder
* Increase coverage from 84 to 90 #122
* Increase coverage from 90 to 92 #122
* Fix ordinal issue #166
Support automation moved this from Needs triage to Closed Sep 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Support
  
Closed
Development

No branches or pull requests

2 participants