From a7ce6ced27956b74a0d55035219801d7cdfc9404 Mon Sep 17 00:00:00 2001
From: Bruno Quaresma <bruno_nonato_quaresma@hotmail.com>
Date: Thu, 20 Apr 2023 12:06:32 -0300
Subject: [PATCH] Revert "fix(site): Fix update when missing parameters
 (#7221)"

This reverts commit 7d9a7636e94999c6e07abe78bc735daa74bfad3e.
---
 site/src/api/api.test.ts                      | 14 +-----
 site/src/api/api.ts                           | 11 +----
 .../xServices/workspace/workspaceXService.ts  | 45 +++++++++----------
 3 files changed, 25 insertions(+), 45 deletions(-)

diff --git a/site/src/api/api.test.ts b/site/src/api/api.test.ts
index 2155888534da2..70ae81b1297a7 100644
--- a/site/src/api/api.test.ts
+++ b/site/src/api/api.test.ts
@@ -173,12 +173,7 @@ describe("api.ts", () => {
       expect(api.postWorkspaceBuild).toHaveBeenCalledWith(MockWorkspace.id, {
         transition: "start",
         template_version_id: MockTemplate.active_version_id,
-        rich_parameter_values: [
-          {
-            name: "first_parameter",
-            value: "mock-abc",
-          },
-        ],
+        rich_parameter_values: [],
       })
     })
 
@@ -227,12 +222,7 @@ describe("api.ts", () => {
       expect(api.postWorkspaceBuild).toHaveBeenCalledWith(MockWorkspace.id, {
         transition: "start",
         template_version_id: MockTemplate.active_version_id,
-        rich_parameter_values: [
-          {
-            name: "first_parameter",
-            value: "mock-abc",
-          },
-        ],
+        rich_parameter_values: [],
       })
     })
   })
diff --git a/site/src/api/api.ts b/site/src/api/api.ts
index ab4b47f4820d4..73cb6de377dd0 100644
--- a/site/src/api/api.ts
+++ b/site/src/api/api.ts
@@ -1041,19 +1041,10 @@ export const updateWorkspace = async (
     throw new MissingBuildParameters(missingParameters)
   }
 
-  // Merge parameters using name as key
-  const richParameterValues = newBuildParameters
-  oldBuildParameters.forEach((oldParam) => {
-    const newParam = richParameterValues.find((p) => p.name === oldParam.name)
-    if (newParam === undefined) {
-      richParameterValues.push(oldParam)
-    }
-  })
-
   return postWorkspaceBuild(workspace.id, {
     transition: "start",
     template_version_id: activeVersionId,
-    rich_parameter_values: richParameterValues,
+    rich_parameter_values: newBuildParameters,
   })
 }
 
diff --git a/site/src/xServices/workspace/workspaceXService.ts b/site/src/xServices/workspace/workspaceXService.ts
index 2276c8034a126..2188f33d70949 100644
--- a/site/src/xServices/workspace/workspaceXService.ts
+++ b/site/src/xServices/workspace/workspaceXService.ts
@@ -117,32 +117,31 @@ export const checks = {
 const permissionsToCheck = (
   workspace: TypesGen.Workspace,
   template: TypesGen.Template,
-) =>
-  ({
-    [checks.readWorkspace]: {
-      object: {
-        resource_type: "workspace",
-        resource_id: workspace.id,
-        owner_id: workspace.owner_id,
-      },
-      action: "read",
+) => ({
+  [checks.readWorkspace]: {
+    object: {
+      resource_type: "workspace",
+      resource_id: workspace.id,
+      owner_id: workspace.owner_id,
     },
-    [checks.updateWorkspace]: {
-      object: {
-        resource_type: "workspace",
-        resource_id: workspace.id,
-        owner_id: workspace.owner_id,
-      },
-      action: "update",
+    action: "read",
+  },
+  [checks.updateWorkspace]: {
+    object: {
+      resource_type: "workspace",
+      resource_id: workspace.id,
+      owner_id: workspace.owner_id,
     },
-    [checks.updateTemplate]: {
-      object: {
-        resource_type: "template",
-        resource_id: template.id,
-      },
-      action: "update",
+    action: "update",
+  },
+  [checks.updateTemplate]: {
+    object: {
+      resource_type: "template",
+      resource_id: template.id,
     },
-  } as const)
+    action: "update",
+  },
+})
 
 export const workspaceMachine = createMachine(
   {