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.

Sign up for GitHub

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

Jump to bottom

with the french locale (and others), the euro symbol should be appended, not prepened #233

Closed
mastooo opened this issue Dec 4, 2012 · 3 comments · May be fixed by #1115
Closed

with the french locale (and others), the euro symbol should be appended, not prepened #233

mastooo opened this issue Dec 4, 2012 · 3 comments · May be fixed by #1115

Comments

@mastooo
Copy link

mastooo commented Dec 4, 2012

Hi,

In France, Germany, Italy and Spain and certainly other european countries, the currency symbol is displayed after the amount.

So, instead of €147,11, we should have 124,11€ for those locale.

The wikipedia page about the euro currency states that there is no standard and each country uses whatever it wants. Hence, we have countries that prepend (Ireland and the Netherlands, english speaking countries) and countries that append to the amount.

Is it something you would fix ?
@semmons99
Copy link
Member

@mastooo: This has been talked about a lot, but we've never come away with a good solution. If you have one, please let us know.

@lacco
Copy link

lacco commented Jan 9, 2013

+1 for fixing this. I am now working for the third (German) client who insists on "1,00 €".

We might need to consider the user's locale when rendering the symbol. A dirty hack would be to set this information manually (e.g. when using Rails in a before_filter):

def determine_locale
  I18n.locale = ...
  Money::Currency.table[:eur][:symbol_first] = ![:de, :es, :fr].include?(I18n.locale)
end

Better ideas?

@semmons99
Copy link
Member

That's about the best idea I've had so far too.

@TastyPi TastyPi mentioned this issue Oct 28, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Support specifying format via I18n TastyPi/money
3 participants
@lacco @semmons99 @mastooo