Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

[PATCH, TESTED] Static backend removes unit in number_to_currency

#1

ZenCocoon says: Try it from a new rails app, display number_to_currency(10) ... and boom.

Reported by Sven Fuchs · January 15th, 2009 @ 02:40 AM

State: new
Milestone: none
Assigned to: Sven Fuchs Sven Fuchs

Activity

  1. Sébastien Grosjean - ZenCocoon
    Sébastien Grosjean - ZenCocoon
    • Tag set to number_to_currency, patch, tests

    I think this tests should pass once the issue with number_to_currency solved.

    January 16th, 2009 @ 10:23 PM

  2. Sébastien Grosjean - ZenCocoon
    Sébastien Grosjean - ZenCocoon
    • Tag changed from number_to_currency, patch, tests to number_to_currency, patched, tests

    Ok, I've made the patch and sent the pull request to Josh.

    For commit details please look at : http://github.com/ZenCocoon/globalize2/commit/b5fd59ed7b2268370955d...

    The patch is tested as well.

    January 20th, 2009 @ 05:23 PM

  3. Sébastien Grosjean - ZenCocoon
    Sébastien Grosjean - ZenCocoon
    • Title changed from Globalize backend removes unit in number_to_currency to [PATCH, TESTED] Static backend removes unit in number_to_currency

    January 20th, 2009 @ 07:19 PM

  4. Sven Fuchs
    Sven Fuchs
    • Tag changed from number_to_currency, patched, tests to backend, number_to_currency, numeric, patched, static, tested, translation
    • Assigned user changed from Sven Fuchs to Josh H

    Looks good to me. Josh, can we apply this?

    January 22nd, 2009 @ 12:55 AM

  5. Josh H
    Josh H
    • State changed from new to resolved

    Applied.

    January 22nd, 2009 @ 03:03 AM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.