emavgl / oinkoin

Oinkoin is a flutter app for helping you managing your expenses. No internet required.
GNU General Public License v3.0
165 stars 11 forks source link

Bug: Problem inserting record due to missing Eastern Arabic numbers support #56

Closed emavgl closed 10 months ago

emavgl commented 10 months ago

I have recevived the following email:

So my device language is Farsi but I can't change the language of the displayed keyboard and apparently the app thinks I should enter Farsi numbers and shows me invalid input error as captured below. Is there anything I can do to fix this?

Screenshot_۲۰۲۴-۰۱-۰۵-۲۰-۲۷-۵۷-۵۳۶_com github emavgl piggybankpro

At the moment, I am calling the keyboard with inputType=phone. This is because I would like the default keyboard to have symbols like +, -, * to perform simple mathematical operations.

However, here it seems that the keyboard comes with Arabic numbers, and does not give the possibility to provide the Eastern Arabic numbers instead.

I see two possibilities here:

@nizarus, do you maybe know something more about this number format?

nizarus commented 10 months ago

Hi @emavgl, Can't reproduce this issue with Arabic locales, even with countries using eastern Arabic numbers. I'm not an android developer so I don't know if there is a more suitable inputType in this case. But if I take the case of the equivalent application that I was using MoneyWallet, this application is using a calculator as input interface (screen shot attached) Screenshot_2024-01-08-13-27-02-57_c4591ddf344e7a3ac352a2553c81405a

emavgl commented 10 months ago

Addressed in https://github.com/emavgl/oinkoin/releases/tag/1.0.39

I have added a check, if the Locale does not uses https://en.wikipedia.org/wiki/Arabic_numerals then it will default to en_US locale.

I could not find a better solution at the moment, closing until a better solution is found.