Closed (fixed)
Project:
Commerce Core
Version:
7.x-1.8
Component:
Commerce
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
26 Aug 2013 at 11:24 UTC
Updated:
30 Jan 2015 at 15:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
sepehr.sadatifar commentedComment #3
rszrama commentedLooks like the patch failed b/c it uses a . instead of a ,. Can you point me to any Iranian eCommerce sites for currency formatting reference?
Comment #4
sepehr.sadatifar commentedoops! sry.
I think this is a good reference (see "E" column in "Active codes" table).
Comment #5
sepehr.sadatifar commentedComment #6
sepehr.sadatifar commenteda patch which covers both this issue and another issue with Iranian rial symbol character.
Comment #7
sepehr.sadatifar commented@rszrama will this patch commited any soon?
Comment #8
rszrama commentedWait, I don't understand - that Wikipedia page says the Iranian rial uses two decimal places. Any ideas why?
Comment #9
sepehr.sadatifar commented@rszrama at the time of writing the comment it was 0 decimal points in wikipedia article (see here). after some digging it turned out that officially Iranian rial uses two decimal points which means it divides to a minor currency unit (dinar) which because rial itself has a very low value we don't even have a official paper money or coin for dinar in iran but we have an unofficial major unit (toman).
so if we want to be conformed with ISO 4217 we should have these corrections on current code:
1- use rial for major unit instead of toman which is an unoffical unit.
2- use dinar for minor unit.
I strongly suggest we don't use minor unit of `dinar` because believe me, even iranian people don't know it's the official minor unit of iran currency!
my preference is to use unofficial major unit of `toman` and minor unit of `rial` and have 0 for decimal points.
Comment #11
bojanz commentedValidated #6 against CLDR, and committed. Thanks!