VideoPress: do not use JS template strings to build queryString of th…

…e chapter file to avoid concat_js=no issues (#28915)
Automattic · Feb 13, 2023 · e19693e · e19693e
1 parent 6b6631a
commit e19693e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/.../videopress/changelog/update-videopress-do-not-use-js-template-to-create-file-chapter-url b/.../videopress/changelog/update-videopress-do-not-use-js-template-to-create-file-chapter-url
@@ -0,0 +1,4 @@
+Significance: patch
+Type: changed
+
+VideoPress: do not use JS template to build queryString of the chapter file to avoid concat_js=no issues
diff --git a/projects/packages/videopress/src/client/block-editor/blocks/video/edit.tsx b/projects/packages/videopress/src/client/block-editor/blocks/video/edit.tsx
@@ -170,11 +170,13 @@ export default function VideoPressEdit( {
 			return;
 		}
 
-		const queryString = token
-			? `?${ new URLSearchParams( { metadata_token: token } ).toString() }`
-			: '';
+		let queryString = '';
+		if ( token ) {
+			queryString = '?' + new URLSearchParams( { metadata_token: token } ).toString();
+		}
 
-		const chapterUrl = `https://videos.files.wordpress.com/${ guid }/${ chapter.src }${ queryString }`;
+		const chapterUrl =
+			'https://videos.files.wordpress.com/' + guid + '/' + chapter.src + queryString;
 
 		try {
 			fetch( chapterUrl )