From 3d438fc1e0f214979bb6a4e6f7af0627b525e3e4 Mon Sep 17 00:00:00 2001
From: Thijs Louisse <Thijs.Louisse@ing.com>
Date: Tue, 26 Nov 2019 18:07:43 +0100
Subject: [PATCH] fix(input-amount): handle user pasting of amounts
 heuristically

---
 packages/field/src/FormatMixin.js              |  6 +++++-
 packages/input-amount/src/LionInputAmount.js   | 18 ++++++++++++++++++
 packages/input-amount/stories/index.stories.js | 14 ++++++++++++--
 3 files changed, 35 insertions(+), 3 deletions(-)

diff --git a/packages/field/src/FormatMixin.js b/packages/field/src/FormatMixin.js
index c59bb2fa9..b532d0102 100644
--- a/packages/field/src/FormatMixin.js
+++ b/packages/field/src/FormatMixin.js
@@ -310,12 +310,16 @@ export const FormatMixin = dedupeMixin(
        *   `@user-input-changed` (this will happen later, when `formatOn` condition is met)
        */
       _reflectBackFormattedValueToUser() {
-        if (!this.__isHandlingUserInput) {
+        if (this._reflectBackOn()) {
           // Text 'undefined' should not end up in <input>
           this.value = typeof this.formattedValue !== 'undefined' ? this.formattedValue : '';
         }
       }
 
+      _reflectBackOn() {
+        return !this.__isHandlingUserInput;
+      }
+
       // This can be called whenever the view value should be updated. Dependent on component type
       // ("input" for <input> or "change" for <select>(mainly for IE)) a different event should be
       // used  as source for the "user-input-changed" event (which can be seen as an abstraction
diff --git a/packages/input-amount/src/LionInputAmount.js b/packages/input-amount/src/LionInputAmount.js
index d49c950b8..60207303b 100644
--- a/packages/input-amount/src/LionInputAmount.js
+++ b/packages/input-amount/src/LionInputAmount.js
@@ -46,10 +46,28 @@ export class LionInputAmount extends FieldCustomMixin(LocalizeMixin(LionInput))
     super();
     this.parser = parseAmount;
     this.formatter = formatAmount;
+    this.__isPasting = false;
+
+    this.addEventListener('paste', () => {
+      this.__isPasting = true;
+      this.__parserCallcountSincePaste = 0;
+    });
 
     this.defaultValidators.push(new IsNumber());
   }
 
+  __callParser(value = this.formattedValue) {
+    // TODO: input and change events both trigger parsing therefore we need to handle the second parse
+    this.__parserCallcountSincePaste += 1;
+    this.__isPasting = this.__parserCallcountSincePaste === 2;
+    this.formatOptions.mode = this.__isPasting === true ? 'pasted' : 'auto';
+    return super.__callParser(value);
+  }
+
+  _reflectBackOn() {
+    return super._reflectBackOn() || this.__isPasting;
+  }
+
   connectedCallback() {
     // eslint-disable-next-line wc/guard-super-call
     super.connectedCallback();
diff --git a/packages/input-amount/stories/index.stories.js b/packages/input-amount/stories/index.stories.js
index 079f58bec..289b8afc5 100644
--- a/packages/input-amount/stories/index.stories.js
+++ b/packages/input-amount/stories/index.stories.js
@@ -34,8 +34,12 @@ storiesOf('Forms|Input Amount', module)
   .add(
     'Force locale to nl-NL',
     () => html`
-      <lion-input-amount label="Price" currency="JOD">
-        .formatOptions="${{ locale: 'nl-NL' }}" .modelValue=${123456.78}
+      <lion-input-amount
+        label="Price"
+        currency="JOD"
+        .formatOptions="${{ locale: 'nl-NL' }}"
+        .modelValue=${123456.78}
+      >
       </lion-input-amount>
     `,
   )
@@ -79,5 +83,11 @@ storiesOf('Forms|Input Amount', module)
         Make sure to set the modelValue last as otherwise formatOptions will not be taken into
         account
       </p>
+
+      <p>
+        You can copy paste <input value="4000,0" /> and it will become 4000 independent of your
+        locale. <br />
+        If you write 4000,0 manually then it will become 4000 or 40000 dependent on your locale.
+      </p>
     `,
   );