Fix firing onChange handler twice for equal content.

Author: m-basov

addon/components/medium-editor.js

@@ -58,6 +58,8 @@ const MediumEditorComponent = Component.extend({
    */
   onChange: null,
 
+  _prevValue: '',
+
   init() {
     this._super(...arguments);
     this._setOptions();
@@ -94,11 +96,22 @@ const MediumEditorComponent = Component.extend({
       }
     });
 
+    this._setOnChange(editor);
+  },
+
+  _setOnChange(editor) {
     let onChangeHandler = get(this, 'onChange');
     if (typeof onChangeHandler === 'function') {
-      editor.subscribe('editableInput', () => {
-        onChangeHandler(editor.getContent());
-      });
+      let handler = () => {
+        let newValue = editor.getContent();
+        let isUpdated = get(this, '_prevValue') !== newValue;
+        if (isUpdated) {
+          set(this, '_prevValue', newValue);
+          onChangeHandler(newValue);
+        }
+      };
+
+      editor.subscribe('editableInput', handler);
     }
   },
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ember-medium-editor",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "medium-editor for Ember Apps",
   "keywords": [
     "medium-editor",

tests/integration/components/medium-editor-test.js

@@ -32,11 +32,7 @@ test('it should trigger onChange action when content changed', function(assert)
   assert.expect(1);
 
   this.set('onChange', (actual) => {
-    // Action triggered twice with ember 2.4 and 2.8.
-    // Hack to use asertions only for second call.
-    if (actual !== '') {
-      assert.equal(actual, '<p>typed value</p>'); // eslint-disable-line
-    }
+    assert.equal(actual, '<p>typed value</p>');
   });
   this.render(hbs`{{medium-editor onChange=(action onChange)}}`);