Merge pull request #6 from joutvhu/develop

Fix group of negative number.

CHANGELOG.md

@@ -1,3 +1,7 @@
+## 1.0.0+5
+
+- Fix group of negative numbers.
+
 ## 1.0.0+4
 
 - Allow disable fix number.

lib/src/filter.dart

@@ -34,6 +34,7 @@ class TextNumberFilter {
   bool limitedNumber = false;
   bool limitedInteger = false;
   bool? limitedDecimal;
+  bool isNegative = false;
   bool numberStarted = false;
   bool hasNumber = false;
   bool foundNumbers = false;
@@ -81,6 +82,7 @@ class TextNumberFilter {
   bool filterNext(int value, int index, LookupTextValueEditor state) {
     bool allow = false;
     if (!numberStarted && options.allowNegative && value == _negative) {
+      isNegative = true;
       allow = true;
     } else if (decimalPoint == null) {
       if (_number_0 <= value && value <= _number_9) {
@@ -315,9 +317,14 @@ class TextNumberFilter {
   void groupDigits() {
     if (options.groupDigits != null) {
       var index = integerDigits;
+      var startPoint = 0;
+      if (isNegative) {
+        index += 1;
+        startPoint = 1;
+      }
       while (true) {
         index -= options.groupDigits!;
-        if (index < 1) {
+        if (index <= startPoint) {
           break;
         }
         editor.insert(index, groupSeparator);

pubspec.yaml

@@ -1,6 +1,6 @@
 name: number_text_input_formatter
 description: Number Text Input Formatter for Flutter
-version: 1.0.0+4
+version: 1.0.0+5
 
 homepage: https://github.com/joutvhu/number_text_input_formatter
 repository: https://github.com/joutvhu/number_text_input_formatter.git

test/number_text_input_formatter_test.dart

@@ -410,6 +410,56 @@ void main() {
     expect(result.text, '-2454.41');
   });
 
+  test('allow_negative_4', () {
+    var result = NumberTextInputFormatter(
+      integerDigits: 13,
+      decimalDigits: 2,
+      groupDigits: 3,
+      allowNegative: true,
+    ).formatEditUpdate(
+      const TextEditingValue(
+        text: '',
+      ),
+      const TextEditingValue(
+        text: '-245.41',
+      ),
+    );
+    expect(result.text, '-245.41');
+  });
+
+  test('allow_negative_5', () {
+    var result = NumberTextInputFormatter(
+      integerDigits: 13,
+      decimalDigits: 2,
+      groupDigits: 3,
+      allowNegative: true,
+    ).formatEditUpdate(
+      const TextEditingValue(
+        text: '',
+      ),
+      const TextEditingValue(
+        text: '-2405.41',
+      ),
+    );
+    expect(result.text, '-2,405.41');
+  });
+
+  test('allow_negative_6', () {
+    var result = NumberTextInputFormatter(
+      integerDigits: 13,
+      decimalDigits: 2,
+      allowNegative: true,
+    ).formatEditUpdate(
+      const TextEditingValue(
+        text: '',
+      ),
+      const TextEditingValue(
+        text: '--245.41',
+      ),
+    );
+    expect(result.text, '-245.41');
+  });
+
   test('insert_decimal_point_1', () {
     var result = NumberTextInputFormatter(
       integerDigits: 13,