Don't allow multiple CTA / attachment block to be pasted.

assets/src/stories-editor/blocks/amp-story-page/copy-paste-handler.js

@@ -13,30 +13,27 @@ import { withDispatch, useSelect, useDispatch } from '@wordpress/data';
 /**
  * Internal dependencies
  */
-import { copyTextToClipBoard, ensureAllowedBlocksOnPaste, isPageBlock } from '../../helpers';
+import { copyTextToClipBoard, isPageBlock, useIsBlockAllowedOnPage, displayPasteError } from '../../helpers';
 
 function CopyPasteHandler( { children, onCopy, clientId, isSelected } ) {
 	const {
-		isFirstPage,
 		canUserUseUnfilteredHTML,
 		getCopiedMarkupState,
 	} = useSelect(
 		( select ) => {
-			const {
-				getBlockOrder,
-				getSettings,
-			} = select( 'core/block-editor' );
+			const { getSettings } = select( 'core/block-editor' );
 			const { __experimentalCanUserUseUnfilteredHTML } = getSettings();
 			const { getCopiedMarkup } = select( 'amp/story' );
 			return {
-				isFirstPage: getBlockOrder().indexOf( clientId ) === 0,
 				canUserUseUnfilteredHTML: __experimentalCanUserUseUnfilteredHTML,
 				getCopiedMarkupState: getCopiedMarkup,
 			};
-		}, [ clientId ]
+		}, [ ]
 	);
 
-	const { insertBlocks } = useDispatch( 'core/block-editor' );
+	const { insertBlock } = useDispatch( 'core/block-editor' );
+	const { createErrorNotice } = useDispatch( 'core/notices' );
+	const isBlockAllowedOnPage = useIsBlockAllowedOnPage();
 
 	const onPaste = ( event ) => {
 		// Ignore if the Page is not the selected page.
@@ -75,9 +72,17 @@ function CopyPasteHandler( { children, onCopy, clientId, isSelected } ) {
 			canUserUseUnfilteredHTML,
 		} );
 
-		if ( content.length > 0 ) {
-			insertBlocks( ensureAllowedBlocksOnPaste( content, clientId, isFirstPage ), null, clientId );
+		if ( ! clientId || ! content.length ) {
+			return;
 		}
+
+		content.forEach( ( pastedBlock ) => {
+			if ( isBlockAllowedOnPage( pastedBlock.name, clientId ) ) {
+				insertBlock( pastedBlock, null, clientId );
+			} else {
+				displayPasteError( pastedBlock.name, createErrorNotice );
+			}
+		} );
 	};
 
 	return (

assets/src/stories-editor/components/context-menu/index.js

@@ -24,8 +24,7 @@ import { useDispatch, useSelect } from '@wordpress/data';
  */
 import './edit.css';
 import {
-	copyTextToClipBoard,
-	ensureAllowedBlocksOnPaste,
+	copyTextToClipBoard, displayPasteError,
 	isPageBlock,
 	useIsBlockAllowedOnPage,
 	useMoveBlockToPage,
@@ -63,10 +62,11 @@ const ContextMenu = ( props ) => {
 	const {
 		removeBlock,
 		insertBlock,
-		insertBlocks,
 		updateBlockAttributes,
 	} = useDispatch( 'core/block-editor' );
 
+	const { createErrorNotice } = useDispatch( 'core/notices' );
+
 	const { setCopiedMarkup } = useDispatch( 'amp/story' );
 
 	const { __experimentalCanUserUseUnfilteredHTML: canUserUseUnfilteredHTML, isRTL } = getSettings();
@@ -109,17 +109,24 @@ const ContextMenu = ( props ) => {
 			return;
 		}
 
-		const isFirstPage = getBlockOrder().indexOf( pageClientId ) === 0;
-		insertBlocks( ensureAllowedBlocksOnPaste( content, pageClientId, isFirstPage ), null, pageClientId ).then( ( { blocks } ) => {
-			for ( const block of blocks ) {
-				if ( ALLOWED_MOVABLE_BLOCKS.includes( block.name ) ) {
-					updateBlockAttributes( block.clientId, {
-						positionTop: insidePercentageY,
-						positionLeft: insidePercentageX,
+		content.forEach( ( pastedBlock ) => {
+			if ( isBlockAllowedOnPage( pastedBlock.name, pageClientId ) ) {
+				insertBlock( pastedBlock, null, pageClientId ).then( ( { blocks } ) => {
+					blocks.forEach( ( block ) => {
+						if ( ALLOWED_MOVABLE_BLOCKS.includes( block.name ) ) {
+							updateBlockAttributes( block.clientId, {
+								positionTop: insidePercentageY,
+								positionLeft: insidePercentageX,
+							} );
+						}
 					} );
-				}
+				} ).catch( () => {
+					displayPasteError( pastedBlock.name, createErrorNotice );
+				} );
+			} else {
+				displayPasteError( pastedBlock.name, createErrorNotice );
 			}
-		} ).catch( () => {} );
+		} );
 	};
 
 	const cutBlock = ( clientId ) => {

assets/src/stories-editor/helpers/displayPasteError.js

@@ -0,0 +1,28 @@
+/**
+ * WordPress dependencies
+ */
+import { getBlockType } from '@wordpress/blocks';
+import { __, sprintf } from '@wordpress/i18n';
+
+/**
+ * Helper function to display a paste error message
+ *
+ * @param {string} name Name of block.
+ * @param {Function} createErrorNotice Bring in dispatchable createErrorNotice function.
+ */
+const displayPasteError = ( name, createErrorNotice ) => {
+	const blockType = getBlockType( name );
+	const removeMessage = sprintf(
+		// translators: %s: Type of block (i.e. Text, Image etc)
+		__( 'Unable to paste %s block.', 'amp' ),
+		blockType.title
+	);
+	createErrorNotice(
+		removeMessage,
+		{
+			type: 'snackbar',
+			isDismissible: true,
+		}
+	);
+};
+export default displayPasteError;

assets/src/stories-editor/helpers/index.js

@@ -46,7 +46,6 @@ export { default as processMedia } from './processMedia';
 export { default as addVideoAriaLabel } from './addVideoAriaLabel';
 export { default as getCallToActionBlock } from './getCallToActionBlock';
 export { default as getPageAttachmentBlock } from './getPageAttachmentBlock';
-export { default as ensureAllowedBlocksOnPaste } from './ensureAllowedBlocksOnPaste';
 export { default as isPageBlock } from './isPageBlock';
 export { default as copyTextToClipBoard } from './copyTextToClipBoard';
 export { default as getPosterImageFromFileObj } from './getPosterImageFromFileObj';
@@ -61,3 +60,4 @@ export { default as getRelativeElementPosition } from './getRelativeElementPosit
 export { default as isCTABlock } from './isCTABlock';
 export { default as useMoveBlockToPage } from './useMoveBlockToPage';
 export { default as useIsBlockAllowedOnPage } from './useIsBlockAllowedOnPage';
+export { default as displayPasteError } from './displayPasteError';