fix for issue "Error Range start 5 is out of text of length 1 #136"

[gokberkbar]

The solution was already implemented in your code but it was commented out.

I tested every scenario could not find any bug.

What was the issue that you fixed by commenting this out? @caseyryan

[gokberkbar]

I noticed that you changed the plugins version to 2.11.2 to 2.11.4 and in your CHANGELOG.md you have a description says "- Fixed a bug with incorrect decimal point detection when mantissa length is 0".
However because of this String? fractionalSeparator = _detectFractionSeparator(value); you have a bug with mantissa length being 0.

After changes the issue with mantissa length being zero and also Issue 134 is fixed.

It is an issue for production code. That would be great for us if you can take a look and release this version. @caseyryan

Thanks

[caseyryan]

I noticed that you changed the plugins version to 2.11.2 to 2.11.4 and in your CHANGELOG.md you have a description says "- Fixed a bug with incorrect decimal point detection when mantissa length is 0". However because of this String? fractionalSeparator = _detectFractionSeparator(value); you have a bug with mantissa length being 0.

After changes the issue with mantissa length being zero and also Issue 134 is fixed.

It is an issue for production code. That would be great for us if you can take a look and release this version. @caseyryan

Thanks

The issue in this case was because of a period used as a thousand separator. double.tryParse() accepted it as a fractional part.
The ones you mentioned above are different