Fixed: remove _range property from json object returned by `Attribute…

…Delta#toJSON`.
ckeditor · May 16, 2017 · 0461cb1 · 0461cb1
1 parent b1a7b72
commit 0461cb1
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/src/model/delta/attributedelta.js b/src/model/delta/attributedelta.js
@@ -87,6 +87,17 @@ export default class AttributeDelta extends Delta {
 		return AttributeDelta;
 	}
 
+	/**
+	 * @inheritDoc
+	 */
+	toJSON() {
+		const json = super.toJSON();
+
+		delete json._range;
+
+		return json;
+	}
+
 	/**
 	 * @inheritDoc
 	 */

diff --git a/tests/model/delta/attributedelta.js b/tests/model/delta/attributedelta.js
@@ -13,6 +13,7 @@ import Element from '../../../src/model/element';
 import AttributeDelta from '../../../src/model/delta/attributedelta';
 import { RootAttributeDelta } from '../../../src/model/delta/attributedelta';
 import AttributeOperation from '../../../src/model/operation/attributeoperation';
+import { jsonParseStringify } from '../../../tests/model/_utils/utils';
 
 describe( 'Batch', () => {
 	let batch, doc, root;
@@ -507,6 +508,12 @@ describe( 'AttributeDelta', () => {
 	it( 'should provide proper className', () => {
 		expect( AttributeDelta.className ).to.equal( 'engine.model.delta.AttributeDelta' );
 	} );
+
+	it( 'should not have _range property when converted to JSON', () => {
+		const json = jsonParseStringify( delta );
+
+		expect( json ).not.to.have.property( '_range' );
+	} );
 } );
 
 describe( 'RootAttributeDelta', () => {