Extended allowedActions property to allow registering non-copyable ma…

…rkers.

packages/ckeditor5-clipboard/src/clipboardmarkersutils.ts

@@ -34,7 +34,7 @@ export default class ClipboardMarkersUtils extends Plugin {
 	 *
 	 * @internal
 	 */
-	private _markersToCopy: Map<string, ClipboardMarkerConfiguration> = new Map();
+	private _markersToCopy: Map<string, ClipboardMarkerConfiguration | null> = new Map();
 
 	/**
 	 * @inheritDoc
@@ -228,11 +228,7 @@ export default class ClipboardMarkersUtils extends Plugin {
 	): void {
 		const before = this._markersToCopy.get( markerName );
 
-		if ( config ) {
-			this._markersToCopy.set( markerName, config );
-		} else {
-			this._markersToCopy.delete( markerName );
-		}
+		this._markersToCopy.set( markerName, config );
 
 		executor();
 
@@ -257,13 +253,18 @@ export default class ClipboardMarkersUtils extends Plugin {
 			return false;
 		}
 
-		// If there is no action provided then only presence of marker is checked.
+		const { allowedActions } = config;
+
+		// If `allowedActions` is set to `null` then markers should not be copied.
+		if ( !allowedActions ) {
+			return false;
+		}
+
+		// If there is no action provided then presence of marker is checked.
 		if ( !action ) {
 			return true;
 		}
 
-		const { allowedActions } = config;
-
 		return allowedActions === 'all' || allowedActions.includes( action );
 	}
 
@@ -649,7 +650,7 @@ export type ClipboardMarkerRestrictedAction = 'copy' | 'cut' | 'dragstart';
  * @internal
  */
 export type ClipboardMarkerConfiguration = {
-	allowedActions: NonEmptyArray<ClipboardMarkerRestrictedAction> | 'all';
+	allowedActions: NonEmptyArray<ClipboardMarkerRestrictedAction> | 'all' | null;
 
 	// If `false`, do not copy marker when only part of its content is selected.
 	copyPartiallySelected?: boolean;

packages/ckeditor5-clipboard/tests/clipboardmarkersutils.js

@@ -765,11 +765,27 @@ describe( 'Clipboard Markers Utils', () => {
 	} );
 
 	describe( '_isMarkerCopyable', () => {
-		it( 'returns false on non existing clipboard markers', () => {
+		it( 'returns `false` on non existing clipboard markers', () => {
 			const result = clipboardMarkersUtils._isMarkerCopyable( 'Hello', 'cut' );
 
 			expect( result ).to.false;
 		} );
+
+		it( 'returns `true` when action is not defined', () => {
+			clipboardMarkersUtils._registerMarkerToCopy( 'comment', { allowedActions: [ 'copy' ] } );
+
+			const result = clipboardMarkersUtils._isMarkerCopyable( 'comment' );
+
+			expect( result ).to.true;
+		} );
+
+		it( 'returns `false` when action is not defined and allowed actions are set to `null`', () => {
+			clipboardMarkersUtils._registerMarkerToCopy( 'comment', { allowedActions: null } );
+
+			const result = clipboardMarkersUtils._isMarkerCopyable( 'comment' );
+
+			expect( result ).to.false;
+		} );
 	} );
 
 	describe( '_getCopyableMarkersFromSelection', () => {