Decimal inserted despite mantissaLength: 0

[ericbenwa]

Trying to display 400000 as $400,000.

However, the following,
print(400000.toCurrencyString(mantissaLength: 0, leadingSymbol: MoneySymbols.DOLLAR_SIGN, trailingSymbol: ''));
displays as $400,000., displaying a period after the value.

In a prior release, this wasn't happening (I just updated from 2.0.2)
How do I avoid the . from being displayed?
Or is this the expected behavior?

Flutter (Channel stable, 3.3.8)

Thank you!

[gomezjc]

Hi Everyone, any solution for this? I have the same issue :(

[caseyryan]

Hi!
I just checked it and it works correctly. Can you confirm you have this problem with the latest version 2.9.6? I fixed this the other day #105 and I believe it should also fix the problem you're talking about because I had completely rewritten the logic of that function from scratch

Now my check was like this:

print(
      400000.toCurrencyString(
        mantissaLength: 0,
        leadingSymbol: MoneySymbols.DOLLAR_SIGN,
        trailingSymbol: '',
      ),
    );

And the display is:

flutter: $400,000

[ericbenwa]

When I run flutter pub upgrade it displays flutter_multi_formatter 2.9.6.
So I presume I've got latest... ?

My pubspec.yaml displays the package as flutter_multi_formatter: ^2.9.6.

Thoughts on what I'm screwing up?

[caseyryan]

When I run flutter pub upgrade it displays flutter_multi_formatter 2.9.6. So I presume I've got latest... ?

My pubspec.yaml displays the package as flutter_multi_formatter: ^2.9.6.

Thoughts on what I'm screwing up?

Have you tried it again after you did it? I'm sure there was a bug you described but not anymore

[ericbenwa]

Yes. Was trying a few times not sure what I'm missing.
I may switch to the Intl package to resolve this.

I'm happy to continue troubleshooting this issue if you have other ideas, but also fine if closed as I think I'll be switching to the Intl package.

[caseyryan]

I must have misunderstood the question when I first read it. You said you only have period at the end. But I thought you also have a zero after it. I'll check it again. I think I know what may cause it

[caseyryan]

You're right. The problem was in this function

  ThousandSeparator thousandSeparator,
  int mantissaLength,
) {
  if (mantissaLength < 1) {
    return '';
  }
  if (thousandSeparator == ThousandSeparator.Comma) {
    return '.';
  }
  if (thousandSeparator == ThousandSeparator.Period ||
      thousandSeparator == ThousandSeparator.SpaceAndCommaMantissa) {
    return ',';
  }
  return '.';
}

It paid no mind to the mantissa length. Fixed in 2.9.7

[ericbenwa]

Ah, thank you, thank you!