Fix: UnlinkCommand should update its state upon editor load. Closes #93.

ckeditor · May 12, 2017 · 6b015e2 · 6b015e2
1 parent 6d18c55
commit 6b015e2
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/unlinkcommand.js b/src/unlinkcommand.js
@@ -25,6 +25,10 @@ export default class UnlinkCommand extends Command {
 
 		// Checks when command should be enabled or disabled.
 		this.listenTo( editor.document.selection, 'change:attribute', () => this.refreshState() );
+
+		// Determine the initial state of the command.
+		// https://github.com/ckeditor/ckeditor5-link/issues/93
+		this.refreshState();
 	}
 
 	/**

diff --git a/tests/unlinkcommand.js b/tests/unlinkcommand.js
@@ -41,6 +41,15 @@ describe( 'UnlinkCommand', () => {
 
 			expect( refreshStateSpy.calledOnce ).to.true;
 		} );
+
+		// https://github.com/ckeditor/ckeditor5-link/issues/93
+		it( 'should determine the initial state of the command', () => {
+			const refreshStateSpy = testUtils.sinon.spy( UnlinkCommand.prototype, 'refreshState' );
+
+			new UnlinkCommand( editor );
+
+			expect( refreshStateSpy.calledOnce ).to.true;
+		} );
 	} );
 
 	describe( '_doExecute', () => {