move function to a different file

Author: abcampo-iry

src/scratch.jsx

@@ -6,6 +6,7 @@ import { compose } from "redux";
 import GUI, { AppStateHOC } from "@scratch/scratch-gui";
 import ScratchIntegrationHOC from "./components/ScratchEditor/ScratchIntegrationHOC.jsx";
 import dedupeScratchWarnings from "./utils/dedupeScratchWarnings.js";
+import scratchProjectSave from "./utils/scratchProjectSave.js";
 
 import ScratchStyles from "./assets/stylesheets/Scratch.scss";
 
@@ -63,20 +64,6 @@ const handleScratchGuiAlert = (alertType) => {
 const generateNonce = () =>
   `${Math.random().toString(36).slice(2)}-${Date.now().toString(36)}`;
 
-const buildQueryString = (params = {}) => {
-  const searchParams = new URLSearchParams(
-    Object.entries({
-      original_id: params.originalId,
-      is_copy: params.isCopy,
-      is_remix: params.isRemix,
-      title: params.title,
-    }).filter(([, value]) => value !== undefined),
-  );
-
-  const queryString = searchParams.toString();
-  return queryString ? `?${queryString}` : "";
-};
-
 if (!projectId) {
   console.error("project_id is required but not set");
 } else if (!apiUrl) {
@@ -108,29 +95,13 @@ if (!projectId) {
     event.data?.nonce === nonce;
 
   const handleUpdateProjectData = async (currentProjectId, vmState, params) => {
-    const creatingProject =
-      currentProjectId === null || typeof currentProjectId === "undefined";
-    const queryString = buildQueryString(params);
-    const url = creatingProject
-      ? `${apiUrl}/api/scratch/projects/${queryString}`
-      : `${apiUrl}/api/scratch/projects/${currentProjectId}${queryString}`;
-
-    const response = await scratchFetchApi.scratchFetch(url, {
-      method: creatingProject ? "post" : "put",
-      body: vmState,
-      headers: {
-        "Content-Type": "application/json",
-      },
-      credentials: "include",
+    return scratchProjectSave({
+      scratchFetchApi,
+      apiUrl,
+      currentProjectId,
+      vmState,
+      params,
     });
-
-    if (response.status !== 200) {
-      throw response.status;
-    }
-
-    const body = await response.json();
-    body.id = creatingProject ? body["content-name"] : currentProjectId;
-    return body;
   };
 
   const mountGui = (accessToken) => {

src/utils/scratchProjectSave.js

@@ -0,0 +1,47 @@
+const buildQueryString = (params = {}) => {
+  const searchParams = new URLSearchParams(
+    Object.entries({
+      original_id: params.originalId,
+      is_copy: params.isCopy,
+      is_remix: params.isRemix,
+      title: params.title,
+    }).filter(([, value]) => value !== undefined),
+  );
+
+  const queryString = searchParams.toString();
+  return queryString ? `?${queryString}` : "";
+};
+
+const scratchProjectSave = async ({
+  scratchFetchApi,
+  apiUrl,
+  currentProjectId,
+  vmState,
+  params,
+}) => {
+  const creatingProject =
+    currentProjectId === null || typeof currentProjectId === "undefined";
+  const queryString = buildQueryString(params);
+  const url = creatingProject
+    ? `${apiUrl}/api/scratch/projects/${queryString}`
+    : `${apiUrl}/api/scratch/projects/${currentProjectId}${queryString}`;
+
+  const response = await scratchFetchApi.scratchFetch(url, {
+    method: creatingProject ? "post" : "put",
+    body: vmState,
+    headers: {
+      "Content-Type": "application/json",
+    },
+    credentials: "include",
+  });
+
+  if (response.status !== 200) {
+    throw response.status;
+  }
+
+  const body = await response.json();
+  body.id = creatingProject ? body["content-name"] : currentProjectId;
+  return body;
+};
+
+export default scratchProjectSave;