Success! Looks like we've fixed this one. According to our records the fix was applied for EXTJS-4444 in a recent build.
  1. #1
    grgur's Avatar
    Join Date
    Aug 2007
    Location
    Split, Croatia
    Posts
    155
    Vote Rating
    24
    grgur has a spectacular aura about grgur has a spectacular aura about

      0  

    Default Ext.util.Format.currency decimals ignored when set to 0

    Ext.util.Format.currency decimals ignored when set to 0


    Ext.getVersion() => 4.0.7

    I'll try to get straight to the point. If you do:
    Code:
    Ext.util.Format.currency(100, '$', 0);
    ... the output is same as
    Code:
    Ext.util.Format.currency(100, '$', Ext.util.Format.currencyPrecision);
    ... and it reads "$100.00" instead of "$100", which is caused by this line in Ext.util.Format, currency function definition:
    Code:
     decimals = decimals || Ext.util.Format.currencyPrecision;
    Obviously, decimals should be tested with a stricter comparison operator, such as

    Code:
     decimals = Ext.isNumber(decimals) ? decimals : Ext.util.Format.currencyPrecision;

    Cheers
    Grgur Grisogono
    Ext JS in Action SE co-author
    Exercising awesomeness at Modus Create - Official Sencha Partner

    Get in touch for Sencha Touch Training
    @ggrgur

  2. #2
    Sencha - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    36,522
    Vote Rating
    814
    mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute

      0  

    Default


    Oh grgur.... reporting bugs is fun huh? Looks like it was fixed already tho!
    Mitchell Simoens @SenchaMitch
    Sencha Inc, Senior Forum Manager
    ________________
    Check out my GitHub, lots of nice things for Ext JS 4 and Sencha Touch 2
    https://github.com/mitchellsimoens

    Think my support is good? Get more personalized support via a support subscription. https://www.sencha.com/store/

    Need more help with your app? Hire Sencha Services services@sencha.com

    Want to learn Sencha Touch 2? Check out Sencha Touch in Action that is in print!

    When posting code, please use BBCode's CODE tags.