Quality: Fix warning error when exporting theme (#671)

WordPress · Jun 10, 2024 · 58fed31 · 58fed31
1 parent ddf7459
commit 58fed31
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 21 deletions.
diff --git a/includes/class-create-block-theme-api.php b/includes/class-create-block-theme-api.php
@@ -308,6 +308,7 @@ function rest_export_theme( $request ) {
 		header( 'Content-Length: ' . filesize( $filename ) );
 		flush();
 		echo readfile( $filename );
+		exit;
 	}
 
 	/**

diff --git a/src/utils/download-file.js b/src/utils/download-file.js
@@ -1,3 +1,8 @@
+/**
+ * WordPress dependencies
+ */
+import { downloadBlob } from '@wordpress/blob';
+
 /*
  * Download a file from in a browser.
  *
@@ -9,25 +14,5 @@ export default async function downloadFile( response ) {
 	const filename = response.headers
 		.get( 'Content-Disposition' )
 		.split( 'filename=' )[ 1 ];
-
-	// Check if the browser supports navigator.msSaveBlob or navigator.saveBlob
-	if ( window.navigator.msSaveBlob || window.navigator.saveBlob ) {
-		const saveBlob =
-			window.navigator.msSaveBlob || window.navigator.saveBlob;
-		saveBlob.call( window.navigator, blob, filename );
-	} else {
-		// Fall back to creating an object URL and triggering a download using an anchor element
-		const url = URL.createObjectURL( blob );
-
-		const a = document.createElement( 'a' );
-		a.href = url;
-		a.download = filename;
-		document.body.appendChild( a );
-		a.click();
-		document.body.removeChild( a );
-
-		setTimeout( () => {
-			URL.revokeObjectURL( url );
-		}, 100 );
-	}
+	downloadBlob( filename, blob, 'application/zip' );
 }