Changed: model.Document#getAppliedDeltas should return empty string…

… if no deltas were applied.
ckeditor · Apr 4, 2017 · dd3015b · dd3015b
1 parent e0145f2
commit dd3015b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/src/dev-utils/enableenginedebug.js b/src/dev-utils/enableenginedebug.js
@@ -462,16 +462,21 @@ function enableReplayerTools() {
 	ModelDocument.prototype.applyOperation = function( operation ) {
 		if ( !this._lastDelta ) {
 			this._appliedDeltas = [];
-			this._lastDelta = operation.delta;
 		} else if ( this._lastDelta !== operation.delta ) {
 			this._appliedDeltas.push( this._lastDelta.toJSON() );
-			this._lastDelta = operation.delta;
 		}
 
+		this._lastDelta = operation.delta;
+
 		_modelDocumentApplyOperation.call( this, operation );
 	};
 
 	ModelDocument.prototype.getAppliedDeltas = function() {
+		// No deltas has been applied yet, return empty string.
+		if ( !this._lastDelta ) {
+			return '';
+		}
+
 		const appliedDeltas = this._appliedDeltas.concat( this._lastDelta.toJSON() );
 
 		return appliedDeltas.map( JSON.stringify ).join( LOG_SEPARATOR );

diff --git a/tests/dev-utils/enableenginedebug.js b/tests/dev-utils/enableenginedebug.js
@@ -717,6 +717,8 @@ describe( 'debug tools', () => {
 		it( 'getAppliedDeltas()', () => {
 			const modelDoc = new ModelDocument();
 
+			expect( modelDoc.getAppliedDeltas() ).to.equal( '' );
+
 			const otherRoot = modelDoc.createRoot( '$root', 'otherRoot' );
 			const firstEle = new ModelElement( 'paragraph' );
 			const removedEle = new ModelElement( 'paragraph', null, [ new ModelText( 'foo' ) ] );