@wordpress/editor 14.50.0 → 14.51.1-next.v.202607070741.0

package/src/utils/media-sideload-from-url/index.js

@@ -0,0 +1,54 @@
+/**
+ * WordPress dependencies
+ */
+import apiFetch from '@wordpress/api-fetch';
+import { select } from '@wordpress/data';
+import { transformAttachment } from '@wordpress/media-utils';
+
+/**
+ * Internal dependencies
+ */
+import { store as editorStore } from '../../store';
+
+const noop = () => {};
+
+/**
+ * Sideloads an external image into the media library from its URL.
+ *
+ * The server downloads the remote file, which avoids a cross-origin browser
+ * fetch that fails when the editor is cross-origin isolated. Since the server
+ * owns the upload it also generates sub-sizes, as with a regular upload.
+ *
+ * @param {Object}   $0           Parameters object.
+ * @param {string}   $0.url       URL of the external image to sideload.
+ * @param {Function} $0.onSuccess Function called with the new attachment on success.
+ * @param {Function} $0.onError   Function called with an error message on failure.
+ */
+export default function mediaSideloadFromUrl( {
+	url,
+	onSuccess,
+	onError = noop,
+} ) {
+	const currentPost = select( editorStore ).getCurrentPost();
+	// Templates and template parts store their numerical ID in `wp_id`.
+	const currentPostId =
+		typeof currentPost?.id === 'number'
+			? currentPost.id
+			: currentPost?.wp_id;
+	const postData = currentPostId ? { post: currentPostId } : {};
+
+	apiFetch( {
+		path: '/wp/v2/media',
+		method: 'POST',
+		data: {
+			url,
+			...postData,
+		},
+	} )
+		.then( ( attachment ) => {
+			onSuccess?.( transformAttachment( attachment ) );
+		} )
+		.catch( ( error ) => {
+			onError( error?.message || error );
+		} );
+}

package/src/utils/media-sideload-from-url/test/index.js

@@ -0,0 +1,162 @@
+/**
+ * WordPress dependencies
+ */
+import apiFetch from '@wordpress/api-fetch';
+import { select } from '@wordpress/data';
+import { transformAttachment } from '@wordpress/media-utils';
+
+/**
+ * Internal dependencies
+ */
+import mediaSideloadFromUrl from '..';
+
+jest.mock( '@wordpress/api-fetch', () => jest.fn() );
+jest.mock( '@wordpress/data', () => ( {
+	select: jest.fn(),
+} ) );
+/*
+ * The store and transformAttachment are mocked so this unit test does not pull
+ * in the full @wordpress/media-utils / core-data import chain (which needs the
+ * real @wordpress/data). transformAttachment's mapping is covered in
+ * @wordpress/media-utils; here we only verify mediaSideloadFromUrl forwards
+ * whatever it returns.
+ */
+jest.mock( '../../../store', () => ( { store: 'core/editor' } ) );
+jest.mock( '@wordpress/media-utils', () => ( {
+	transformAttachment: jest.fn( ( attachment ) => ( {
+		id: attachment.id,
+		url: attachment.source_url,
+		alt: attachment.alt_text,
+	} ) ),
+} ) );
+
+const mockRestAttachment = {
+	id: 123,
+	alt_text: 'An external photo.',
+	caption: { raw: '' },
+	title: { raw: 'photo' },
+	source_url: 'https://example.com/wp-content/uploads/photo.jpg',
+};
+
+/**
+ * Sets the post returned by `getCurrentPost()` for a test.
+ *
+ * @param {Object|undefined} post The post object, or undefined for no post.
+ */
+function mockCurrentPost( post ) {
+	select.mockReturnValue( {
+		getCurrentPost: () => post,
+	} );
+}
+
+describe( 'mediaSideloadFromUrl', () => {
+	beforeEach( () => {
+		jest.clearAllMocks();
+		mockCurrentPost( { id: 42 } );
+	} );
+
+	it( 'should POST to /wp/v2/media with the url, leaving sub-size generation to the server', async () => {
+		apiFetch.mockResolvedValue( mockRestAttachment );
+
+		await new Promise( ( resolve ) => {
+			mediaSideloadFromUrl( {
+				url: 'https://example.com/photo.jpg',
+				onSuccess: resolve,
+			} );
+		} );
+
+		expect( apiFetch ).toHaveBeenCalledWith( {
+			path: '/wp/v2/media',
+			method: 'POST',
+			data: {
+				url: 'https://example.com/photo.jpg',
+				post: 42,
+			},
+		} );
+	} );
+
+	it( 'should pass the transformed attachment to onSuccess', async () => {
+		apiFetch.mockResolvedValue( mockRestAttachment );
+
+		const media = await new Promise( ( resolve ) => {
+			mediaSideloadFromUrl( {
+				url: 'https://example.com/photo.jpg',
+				onSuccess: resolve,
+			} );
+		} );
+
+		/*
+		 * The raw REST attachment is run through transformAttachment, which maps
+		 * it to the editor shape the image block expects (url, alt).
+		 */
+		expect( transformAttachment ).toHaveBeenCalledWith(
+			mockRestAttachment
+		);
+		expect( media ).toMatchObject( {
+			id: 123,
+			url: 'https://example.com/wp-content/uploads/photo.jpg',
+			alt: 'An external photo.',
+		} );
+	} );
+
+	it( 'should fall back to wp_id when the post id is not numeric', async () => {
+		apiFetch.mockResolvedValue( mockRestAttachment );
+		// Templates and template parts expose their numeric id as `wp_id`.
+		mockCurrentPost( { id: 'my-theme//header', wp_id: 99 } );
+
+		await new Promise( ( resolve ) => {
+			mediaSideloadFromUrl( {
+				url: 'https://example.com/photo.jpg',
+				onSuccess: resolve,
+			} );
+		} );
+
+		expect( apiFetch ).toHaveBeenCalledWith(
+			expect.objectContaining( {
+				data: expect.objectContaining( { post: 99 } ),
+			} )
+		);
+	} );
+
+	it( 'should omit the post param when there is no current post', async () => {
+		apiFetch.mockResolvedValue( mockRestAttachment );
+		mockCurrentPost( undefined );
+
+		await new Promise( ( resolve ) => {
+			mediaSideloadFromUrl( {
+				url: 'https://example.com/photo.jpg',
+				onSuccess: resolve,
+			} );
+		} );
+
+		const { data } = apiFetch.mock.calls[ 0 ][ 0 ];
+		expect( data ).not.toHaveProperty( 'post' );
+	} );
+
+	it( 'should pass the error message to onError on failure', async () => {
+		apiFetch.mockRejectedValue(
+			new Error( 'Could not download the image.' )
+		);
+
+		const message = await new Promise( ( resolve ) => {
+			mediaSideloadFromUrl( {
+				url: 'https://example.com/photo.jpg',
+				onSuccess: () => {},
+				onError: resolve,
+			} );
+		} );
+
+		expect( message ).toBe( 'Could not download the image.' );
+	} );
+
+	it( 'should not throw when onError is omitted and the request fails', async () => {
+		apiFetch.mockRejectedValue( new Error( 'Network error' ) );
+
+		expect( () =>
+			mediaSideloadFromUrl( { url: 'https://example.com/photo.jpg' } )
+		).not.toThrow();
+
+		// Let the rejected promise settle so the default noop onError runs.
+		await Promise.resolve();
+	} );
+} );

package/build/components/post-publish-panel/media-util.cjs

@@ -1,72 +0,0 @@
-"use strict";
-var __defProp = Object.defineProperty;
-var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
-var __getOwnPropNames = Object.getOwnPropertyNames;
-var __hasOwnProp = Object.prototype.hasOwnProperty;
-var __export = (target, all) => {
-  for (var name in all)
-    __defProp(target, name, { get: all[name], enumerable: true });
-};
-var __copyProps = (to, from, except, desc) => {
-  if (from && typeof from === "object" || typeof from === "function") {
-    for (let key of __getOwnPropNames(from))
-      if (!__hasOwnProp.call(to, key) && key !== except)
-        __defProp(to, key, { get: () => from[key], enumerable: !(desc = __getOwnPropDesc(from, key)) || desc.enumerable });
-  }
-  return to;
-};
-var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
-
-// packages/editor/src/components/post-publish-panel/media-util.js
-var media_util_exports = {};
-__export(media_util_exports, {
-  fetchMedia: () => fetchMedia,
-  generateUniqueBasenames: () => generateUniqueBasenames
-});
-module.exports = __toCommonJS(media_util_exports);
-var import_uuid = require("uuid");
-var import_url = require("@wordpress/url");
-function generateUniqueBasenames(urls) {
-  const basenames = /* @__PURE__ */ new Set();
-  return Object.fromEntries(
-    urls.map((url) => {
-      const filename = (0, import_url.getFilename)(url);
-      let basename = "";
-      if (filename) {
-        const parts = filename.split(".");
-        if (parts.length > 1) {
-          parts.pop();
-        }
-        basename = parts.join(".");
-      }
-      if (!basename) {
-        basename = (0, import_uuid.v4)();
-      }
-      if (basenames.has(basename)) {
-        basename = `${basename}-${(0, import_uuid.v4)()}`;
-      }
-      basenames.add(basename);
-      return [url, basename];
-    })
-  );
-}
-function fetchMedia(urls) {
-  return Object.fromEntries(
-    Object.entries(generateUniqueBasenames(urls)).map(
-      ([url, basename]) => {
-        const filePromise = window.fetch(url.includes("?") ? url : url + "?").then((response) => response.blob()).then((blob) => {
-          return new File([blob], `${basename}.png`, {
-            type: blob.type
-          });
-        });
-        return [url, filePromise];
-      }
-    )
-  );
-}
-// Annotate the CommonJS export names for ESM import in node:
-0 && (module.exports = {
-  fetchMedia,
-  generateUniqueBasenames
-});
-//# sourceMappingURL=media-util.cjs.map

package/build/components/post-publish-panel/media-util.cjs.map

@@ -1,7 +0,0 @@
-{
-  "version": 3,
-  "sources": ["../../../src/components/post-publish-panel/media-util.js"],
-  "sourcesContent": ["/**\n * External dependencies\n */\nimport { v4 as uuid } from 'uuid';\n\n/**\n * WordPress dependencies\n */\nimport { getFilename } from '@wordpress/url';\n\n/**\n * Generate a list of unique basenames given a list of URLs.\n *\n * We want all basenames to be unique, since sometimes the extension\n * doesn't reflect the mime type, and may end up getting changed by\n * the server, on upload.\n *\n * @param {string[]} urls The list of URLs\n * @return {Record< string, string >} A URL => basename record.\n */\nexport function generateUniqueBasenames( urls ) {\n\tconst basenames = new Set();\n\n\treturn Object.fromEntries(\n\t\turls.map( ( url ) => {\n\t\t\t// We prefer to match the remote filename, if possible.\n\t\t\tconst filename = getFilename( url );\n\t\t\tlet basename = '';\n\n\t\t\tif ( filename ) {\n\t\t\t\tconst parts = filename.split( '.' );\n\t\t\t\tif ( parts.length > 1 ) {\n\t\t\t\t\t// Assume the last part is the extension.\n\t\t\t\t\tparts.pop();\n\t\t\t\t}\n\t\t\t\tbasename = parts.join( '.' );\n\t\t\t}\n\n\t\t\tif ( ! basename ) {\n\t\t\t\t// It looks like we don't have a basename, so let's use a UUID.\n\t\t\t\tbasename = uuid();\n\t\t\t}\n\n\t\t\tif ( basenames.has( basename ) ) {\n\t\t\t\t// Append a UUID to deduplicate the basename.\n\t\t\t\t// The server will try to deduplicate on its own if we don't do this,\n\t\t\t\t// but it may run into a race condition\n\t\t\t\t// (see https://github.com/WordPress/gutenberg/issues/64899).\n\t\t\t\t// Deduplicating the filenames before uploading is safer.\n\t\t\t\tbasename = `${ basename }-${ uuid() }`;\n\t\t\t}\n\n\t\t\tbasenames.add( basename );\n\n\t\t\treturn [ url, basename ];\n\t\t} )\n\t);\n}\n\n/**\n * Fetch a list of URLs, turning those into promises for files with\n * unique filenames.\n *\n * @param {string[]} urls The list of URLs\n * @return {Record< string, Promise< File > >} A URL => File promise record.\n */\nexport function fetchMedia( urls ) {\n\treturn Object.fromEntries(\n\t\tObject.entries( generateUniqueBasenames( urls ) ).map(\n\t\t\t( [ url, basename ] ) => {\n\t\t\t\tconst filePromise = window\n\t\t\t\t\t.fetch( url.includes( '?' ) ? url : url + '?' )\n\t\t\t\t\t.then( ( response ) => response.blob() )\n\t\t\t\t\t.then( ( blob ) => {\n\t\t\t\t\t\t// The server will reject the upload if it doesn't have an extension,\n\t\t\t\t\t\t// even though it'll rewrite the file name to match the mime type.\n\t\t\t\t\t\t// Here we provide it with a safe extension to get it past that check.\n\t\t\t\t\t\treturn new File( [ blob ], `${ basename }.png`, {\n\t\t\t\t\t\t\ttype: blob.type,\n\t\t\t\t\t\t} );\n\t\t\t\t\t} );\n\n\t\t\t\treturn [ url, filePromise ];\n\t\t\t}\n\t\t)\n\t);\n}\n"],
-  "mappings": ";;;;;;;;;;;;;;;;;;;;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAGA,kBAA2B;AAK3B,iBAA4B;AAYrB,SAAS,wBAAyB,MAAO;AAC/C,QAAM,YAAY,oBAAI,IAAI;AAE1B,SAAO,OAAO;AAAA,IACb,KAAK,IAAK,CAAE,QAAS;AAEpB,YAAM,eAAW,wBAAa,GAAI;AAClC,UAAI,WAAW;AAEf,UAAK,UAAW;AACf,cAAM,QAAQ,SAAS,MAAO,GAAI;AAClC,YAAK,MAAM,SAAS,GAAI;AAEvB,gBAAM,IAAI;AAAA,QACX;AACA,mBAAW,MAAM,KAAM,GAAI;AAAA,MAC5B;AAEA,UAAK,CAAE,UAAW;AAEjB,uBAAW,YAAAA,IAAK;AAAA,MACjB;AAEA,UAAK,UAAU,IAAK,QAAS,GAAI;AAMhC,mBAAW,GAAI,QAAS,QAAK,YAAAA,IAAK,CAAE;AAAA,MACrC;AAEA,gBAAU,IAAK,QAAS;AAExB,aAAO,CAAE,KAAK,QAAS;AAAA,IACxB,CAAE;AAAA,EACH;AACD;AASO,SAAS,WAAY,MAAO;AAClC,SAAO,OAAO;AAAA,IACb,OAAO,QAAS,wBAAyB,IAAK,CAAE,EAAE;AAAA,MACjD,CAAE,CAAE,KAAK,QAAS,MAAO;AACxB,cAAM,cAAc,OAClB,MAAO,IAAI,SAAU,GAAI,IAAI,MAAM,MAAM,GAAI,EAC7C,KAAM,CAAE,aAAc,SAAS,KAAK,CAAE,EACtC,KAAM,CAAE,SAAU;AAIlB,iBAAO,IAAI,KAAM,CAAE,IAAK,GAAG,GAAI,QAAS,QAAQ;AAAA,YAC/C,MAAM,KAAK;AAAA,UACZ,CAAE;AAAA,QACH,CAAE;AAEH,eAAO,CAAE,KAAK,WAAY;AAAA,MAC3B;AAAA,IACD;AAAA,EACD;AACD;",
-  "names": ["uuid"]
-}

package/build-module/components/post-publish-panel/media-util.mjs

@@ -1,46 +0,0 @@
-// packages/editor/src/components/post-publish-panel/media-util.js
-import { v4 as uuid } from "uuid";
-import { getFilename } from "@wordpress/url";
-function generateUniqueBasenames(urls) {
-  const basenames = /* @__PURE__ */ new Set();
-  return Object.fromEntries(
-    urls.map((url) => {
-      const filename = getFilename(url);
-      let basename = "";
-      if (filename) {
-        const parts = filename.split(".");
-        if (parts.length > 1) {
-          parts.pop();
-        }
-        basename = parts.join(".");
-      }
-      if (!basename) {
-        basename = uuid();
-      }
-      if (basenames.has(basename)) {
-        basename = `${basename}-${uuid()}`;
-      }
-      basenames.add(basename);
-      return [url, basename];
-    })
-  );
-}
-function fetchMedia(urls) {
-  return Object.fromEntries(
-    Object.entries(generateUniqueBasenames(urls)).map(
-      ([url, basename]) => {
-        const filePromise = window.fetch(url.includes("?") ? url : url + "?").then((response) => response.blob()).then((blob) => {
-          return new File([blob], `${basename}.png`, {
-            type: blob.type
-          });
-        });
-        return [url, filePromise];
-      }
-    )
-  );
-}
-export {
-  fetchMedia,
-  generateUniqueBasenames
-};
-//# sourceMappingURL=media-util.mjs.map

package/build-module/components/post-publish-panel/media-util.mjs.map

@@ -1,7 +0,0 @@
-{
-  "version": 3,
-  "sources": ["../../../src/components/post-publish-panel/media-util.js"],
-  "sourcesContent": ["/**\n * External dependencies\n */\nimport { v4 as uuid } from 'uuid';\n\n/**\n * WordPress dependencies\n */\nimport { getFilename } from '@wordpress/url';\n\n/**\n * Generate a list of unique basenames given a list of URLs.\n *\n * We want all basenames to be unique, since sometimes the extension\n * doesn't reflect the mime type, and may end up getting changed by\n * the server, on upload.\n *\n * @param {string[]} urls The list of URLs\n * @return {Record< string, string >} A URL => basename record.\n */\nexport function generateUniqueBasenames( urls ) {\n\tconst basenames = new Set();\n\n\treturn Object.fromEntries(\n\t\turls.map( ( url ) => {\n\t\t\t// We prefer to match the remote filename, if possible.\n\t\t\tconst filename = getFilename( url );\n\t\t\tlet basename = '';\n\n\t\t\tif ( filename ) {\n\t\t\t\tconst parts = filename.split( '.' );\n\t\t\t\tif ( parts.length > 1 ) {\n\t\t\t\t\t// Assume the last part is the extension.\n\t\t\t\t\tparts.pop();\n\t\t\t\t}\n\t\t\t\tbasename = parts.join( '.' );\n\t\t\t}\n\n\t\t\tif ( ! basename ) {\n\t\t\t\t// It looks like we don't have a basename, so let's use a UUID.\n\t\t\t\tbasename = uuid();\n\t\t\t}\n\n\t\t\tif ( basenames.has( basename ) ) {\n\t\t\t\t// Append a UUID to deduplicate the basename.\n\t\t\t\t// The server will try to deduplicate on its own if we don't do this,\n\t\t\t\t// but it may run into a race condition\n\t\t\t\t// (see https://github.com/WordPress/gutenberg/issues/64899).\n\t\t\t\t// Deduplicating the filenames before uploading is safer.\n\t\t\t\tbasename = `${ basename }-${ uuid() }`;\n\t\t\t}\n\n\t\t\tbasenames.add( basename );\n\n\t\t\treturn [ url, basename ];\n\t\t} )\n\t);\n}\n\n/**\n * Fetch a list of URLs, turning those into promises for files with\n * unique filenames.\n *\n * @param {string[]} urls The list of URLs\n * @return {Record< string, Promise< File > >} A URL => File promise record.\n */\nexport function fetchMedia( urls ) {\n\treturn Object.fromEntries(\n\t\tObject.entries( generateUniqueBasenames( urls ) ).map(\n\t\t\t( [ url, basename ] ) => {\n\t\t\t\tconst filePromise = window\n\t\t\t\t\t.fetch( url.includes( '?' ) ? url : url + '?' )\n\t\t\t\t\t.then( ( response ) => response.blob() )\n\t\t\t\t\t.then( ( blob ) => {\n\t\t\t\t\t\t// The server will reject the upload if it doesn't have an extension,\n\t\t\t\t\t\t// even though it'll rewrite the file name to match the mime type.\n\t\t\t\t\t\t// Here we provide it with a safe extension to get it past that check.\n\t\t\t\t\t\treturn new File( [ blob ], `${ basename }.png`, {\n\t\t\t\t\t\t\ttype: blob.type,\n\t\t\t\t\t\t} );\n\t\t\t\t\t} );\n\n\t\t\t\treturn [ url, filePromise ];\n\t\t\t}\n\t\t)\n\t);\n}\n"],
-  "mappings": ";AAGA,SAAS,MAAM,YAAY;AAK3B,SAAS,mBAAmB;AAYrB,SAAS,wBAAyB,MAAO;AAC/C,QAAM,YAAY,oBAAI,IAAI;AAE1B,SAAO,OAAO;AAAA,IACb,KAAK,IAAK,CAAE,QAAS;AAEpB,YAAM,WAAW,YAAa,GAAI;AAClC,UAAI,WAAW;AAEf,UAAK,UAAW;AACf,cAAM,QAAQ,SAAS,MAAO,GAAI;AAClC,YAAK,MAAM,SAAS,GAAI;AAEvB,gBAAM,IAAI;AAAA,QACX;AACA,mBAAW,MAAM,KAAM,GAAI;AAAA,MAC5B;AAEA,UAAK,CAAE,UAAW;AAEjB,mBAAW,KAAK;AAAA,MACjB;AAEA,UAAK,UAAU,IAAK,QAAS,GAAI;AAMhC,mBAAW,GAAI,QAAS,IAAK,KAAK,CAAE;AAAA,MACrC;AAEA,gBAAU,IAAK,QAAS;AAExB,aAAO,CAAE,KAAK,QAAS;AAAA,IACxB,CAAE;AAAA,EACH;AACD;AASO,SAAS,WAAY,MAAO;AAClC,SAAO,OAAO;AAAA,IACb,OAAO,QAAS,wBAAyB,IAAK,CAAE,EAAE;AAAA,MACjD,CAAE,CAAE,KAAK,QAAS,MAAO;AACxB,cAAM,cAAc,OAClB,MAAO,IAAI,SAAU,GAAI,IAAI,MAAM,MAAM,GAAI,EAC7C,KAAM,CAAE,aAAc,SAAS,KAAK,CAAE,EACtC,KAAM,CAAE,SAAU;AAIlB,iBAAO,IAAI,KAAM,CAAE,IAAK,GAAG,GAAI,QAAS,QAAQ;AAAA,YAC/C,MAAM,KAAK;AAAA,UACZ,CAAE;AAAA,QACH,CAAE;AAEH,eAAO,CAAE,KAAK,WAAY;AAAA,MAC3B;AAAA,IACD;AAAA,EACD;AACD;",
-  "names": []
-}

package/build-types/components/post-publish-panel/media-util.d.ts

@@ -1,20 +0,0 @@
-/**
- * Generate a list of unique basenames given a list of URLs.
- *
- * We want all basenames to be unique, since sometimes the extension
- * doesn't reflect the mime type, and may end up getting changed by
- * the server, on upload.
- *
- * @param {string[]} urls The list of URLs
- * @return {Record< string, string >} A URL => basename record.
- */
-export declare function generateUniqueBasenames(urls: string[]): Record<string, string>;
-/**
- * Fetch a list of URLs, turning those into promises for files with
- * unique filenames.
- *
- * @param {string[]} urls The list of URLs
- * @return {Record< string, Promise< File > >} A URL => File promise record.
- */
-export declare function fetchMedia(urls: string[]): Record<string, Promise<File>>;
-//# sourceMappingURL=media-util.d.ts.map

package/build-types/components/post-publish-panel/media-util.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"media-util.d.ts","sourceRoot":"","sources":["../../../src/components/post-publish-panel/media-util.js"],"names":[],"mappings":"AAUA;;;;;;;;;GASG;AACH,wBAAgB,uBAAuB,CAAE,IAAI,EAHlC,MAAM,EAG4B,0BAqC5C;AAED;;;;;;GAMG;AACH,wBAAgB,UAAU,CAAE,IAAI,EAHrB,MAAM,EAGe,iCAoB/B"}

package/src/components/post-publish-panel/media-util.js

@@ -1,87 +0,0 @@
-/**
- * External dependencies
- */
-import { v4 as uuid } from 'uuid';
-
-/**
- * WordPress dependencies
- */
-import { getFilename } from '@wordpress/url';
-
-/**
- * Generate a list of unique basenames given a list of URLs.
- *
- * We want all basenames to be unique, since sometimes the extension
- * doesn't reflect the mime type, and may end up getting changed by
- * the server, on upload.
- *
- * @param {string[]} urls The list of URLs
- * @return {Record< string, string >} A URL => basename record.
- */
-export function generateUniqueBasenames( urls ) {
-	const basenames = new Set();
-
-	return Object.fromEntries(
-		urls.map( ( url ) => {
-			// We prefer to match the remote filename, if possible.
-			const filename = getFilename( url );
-			let basename = '';
-
-			if ( filename ) {
-				const parts = filename.split( '.' );
-				if ( parts.length > 1 ) {
-					// Assume the last part is the extension.
-					parts.pop();
-				}
-				basename = parts.join( '.' );
-			}
-
-			if ( ! basename ) {
-				// It looks like we don't have a basename, so let's use a UUID.
-				basename = uuid();
-			}
-
-			if ( basenames.has( basename ) ) {
-				// Append a UUID to deduplicate the basename.
-				// The server will try to deduplicate on its own if we don't do this,
-				// but it may run into a race condition
-				// (see https://github.com/WordPress/gutenberg/issues/64899).
-				// Deduplicating the filenames before uploading is safer.
-				basename = `${ basename }-${ uuid() }`;
-			}
-
-			basenames.add( basename );
-
-			return [ url, basename ];
-		} )
-	);
-}
-
-/**
- * Fetch a list of URLs, turning those into promises for files with
- * unique filenames.
- *
- * @param {string[]} urls The list of URLs
- * @return {Record< string, Promise< File > >} A URL => File promise record.
- */
-export function fetchMedia( urls ) {
-	return Object.fromEntries(
-		Object.entries( generateUniqueBasenames( urls ) ).map(
-			( [ url, basename ] ) => {
-				const filePromise = window
-					.fetch( url.includes( '?' ) ? url : url + '?' )
-					.then( ( response ) => response.blob() )
-					.then( ( blob ) => {
-						// The server will reject the upload if it doesn't have an extension,
-						// even though it'll rewrite the file name to match the mime type.
-						// Here we provide it with a safe extension to get it past that check.
-						return new File( [ blob ], `${ basename }.png`, {
-							type: blob.type,
-						} );
-					} );
-
-				return [ url, filePromise ];
-			}
-		)
-	);
-}

package/src/components/post-publish-panel/test/media-util.js

@@ -1,118 +0,0 @@
-/**
- * Internal dependencies
- */
-import { generateUniqueBasenames } from '../media-util';
-
-describe( 'generateUniqueBasenames', () => {
-	it( 'should prefer the original basenames', () => {
-		const urls = [
-			'https://example.com/images/image1.jpg',
-			'https://example.com/images/image2.jpg',
-			'https://example.com/images/image3.jpg',
-			'https://example.com/images/image4.jpg',
-		];
-
-		expect( generateUniqueBasenames( urls ) ).toEqual( {
-			'https://example.com/images/image1.jpg': 'image1',
-			'https://example.com/images/image2.jpg': 'image2',
-			'https://example.com/images/image3.jpg': 'image3',
-			'https://example.com/images/image4.jpg': 'image4',
-		} );
-	} );
-
-	it( 'should handle filenames with no extensions', () => {
-		const urls = [
-			'https://example.com/images/image1',
-			'https://example.com/images/image2',
-			'https://example.com/images/image3',
-			'https://example.com/images/image4',
-		];
-
-		expect( generateUniqueBasenames( urls ) ).toEqual( {
-			'https://example.com/images/image1': 'image1',
-			'https://example.com/images/image2': 'image2',
-			'https://example.com/images/image3': 'image3',
-			'https://example.com/images/image4': 'image4',
-		} );
-	} );
-
-	it( 'should handle query parameters correctly', () => {
-		const urls = [
-			'https://example.com/images/image1.jpg?a=notafile.npg',
-			'https://example.com/images/image2.jpg?a=notafile.npg',
-			'https://example.com/images/image3.jpg?a=notafile.npg',
-			'https://example.com/images/image4.jpg?a=notafile.npg',
-		];
-
-		expect( generateUniqueBasenames( urls ) ).toEqual( {
-			'https://example.com/images/image1.jpg?a=notafile.npg': 'image1',
-			'https://example.com/images/image2.jpg?a=notafile.npg': 'image2',
-			'https://example.com/images/image3.jpg?a=notafile.npg': 'image3',
-			'https://example.com/images/image4.jpg?a=notafile.npg': 'image4',
-		} );
-	} );
-
-	it( 'should deduplicate identical filenames', () => {
-		const urls = [
-			'https://example.com/image1/image.jpg',
-			'https://example.com/image2/image.jpg',
-			'https://example.com/image3/image.jpg',
-			'https://example.com/image4/image.jpg',
-		];
-
-		const results = generateUniqueBasenames( urls );
-		const resultLength = Object.entries( results ).length;
-		expect( resultLength ).toBe( urls.length );
-
-		const basenames = new Set( Object.values( results ) );
-		expect( basenames.size ).toBe( resultLength );
-	} );
-
-	it( 'should deduplicate identical basenames', () => {
-		const urls = [
-			'https://example.com/images/image.jpg',
-			'https://example.com/images/image.png',
-			'https://example.com/images/image.webp',
-			'https://example.com/images/image.avif',
-		];
-
-		const results = generateUniqueBasenames( urls );
-		const resultLength = Object.entries( results ).length;
-		expect( resultLength ).toBe( urls.length );
-
-		const basenames = new Set( Object.values( results ) );
-		expect( basenames.size ).toBe( resultLength );
-	} );
-
-	it( 'should deduplicate filenames without extensions', () => {
-		const urls = [
-			'https://example.com/image1/image',
-			'https://example.com/image2/image',
-			'https://example.com/image3/image',
-			'https://example.com/image4/image',
-		];
-
-		const results = generateUniqueBasenames( urls );
-		const resultLength = Object.entries( results ).length;
-		expect( resultLength ).toBe( urls.length );
-
-		const basenames = new Set( Object.values( results ) );
-		expect( basenames.size ).toBe( resultLength );
-	} );
-
-	it( 'should deduplicate paths with no filename', () => {
-		const urls = [
-			'https://example.com/image1/dir/',
-			'https://example.com/image2/dir/',
-			'https://example.com/image3/dir/',
-			'https://example.com/image4/dir/',
-		];
-
-		const results = generateUniqueBasenames( urls );
-		const resultLength = Object.entries( results ).length;
-		expect( resultLength ).toBe( urls.length );
-
-		const basenames = new Set( Object.values( results ) );
-		expect( basenames.size ).toBe( resultLength );
-	} );
-} );