From ae09cf4853e3b0cb6e3ae58d6fcc3bbe7a8727de Mon Sep 17 00:00:00 2001
From: Robert Anderson <robert@noisysocks.com>
Date: Fri, 20 Jul 2018 14:03:22 +1000
Subject: [PATCH] File block: Handle drag-and-drop errors

Handle any errors that occur when a File block is created via a files
transform. That is, when a file is drag-and-dropped into the editor.
---
 core-blocks/file/edit.js | 21 ++++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/core-blocks/file/edit.js b/core-blocks/file/edit.js
index 0f9bf176675b9..621ef31c30e0c 100644
--- a/core-blocks/file/edit.js
+++ b/core-blocks/file/edit.js
@@ -44,12 +44,14 @@ class FileEdit extends Component {
 		this.changeShowDownloadButton = this.changeShowDownloadButton.bind( this );
 
 		this.state = {
+			hasError: false,
 			showCopyConfirmation: false,
 		};
 	}
 
 	componentDidMount() {
-		const { href } = this.props.attributes;
+		const { attributes, noticeOperations } = this.props;
+		const { href } = attributes;
 
 		// Upload a file drag-and-dropped into the editor
 		if ( this.isBlobURL( href ) ) {
@@ -59,6 +61,10 @@ class FileEdit extends Component {
 				allowedType: '*',
 				filesList: [ file ],
 				onFileChange: ( [ media ] ) => this.onSelectFile( media ),
+				onError: ( message ) => {
+					this.setState( { hasError: true } );
+					noticeOperations.createErrorNotice( message );
+				},
 			} );
 
 			revokeBlobURL( href );
@@ -74,6 +80,7 @@ class FileEdit extends Component {
 
 	onSelectFile( media ) {
 		if ( media && media.url ) {
+			this.setState( { hasError: false } );
 			this.props.setAttributes( {
 				href: media.url,
 				fileName: media.title,
@@ -129,14 +136,10 @@ class FileEdit extends Component {
 			downloadButtonText,
 			id,
 		} = attributes;
-		const { showCopyConfirmation } = this.state;
+		const { hasError, showCopyConfirmation } = this.state;
 		const attachmentPage = media && media.link;
 
-		const classes = classnames( className, {
-			'is-transient': this.isBlobURL( href ),
-		} );
-
-		if ( ! href ) {
+		if ( ! href || hasError ) {
 			return (
 				<MediaPlaceholder
 					icon="media-default"
@@ -153,6 +156,10 @@ class FileEdit extends Component {
 			);
 		}
 
+		const classes = classnames( className, {
+			'is-transient': this.isBlobURL( href ),
+		} );
+
 		return (
 			<Fragment>
 				<FileBlockInspector