chore: Replace update with simple delete for removeDeliveryOption

app/(gcforms)/[locale]/(form administration)/form-builder/actions.ts

@@ -348,20 +348,19 @@ export const sendResponsesToVault = async ({
 }: {
   id: string;
 }): Promise<{
-  formRecord: FormRecord | null;
+  success?: boolean;
   error?: string;
 }> => {
   try {
     const { ability } = await authCheckAndThrow();
 
-    const response = await removeDeliveryOption(ability, formID);
-    if (!response) {
-      throw new Error(`Template API response was null. Request information: { ${formID} }`);
-    }
+    await removeDeliveryOption(ability, formID);
 
-    return { formRecord: response };
+    return {
+      success: true,
+    };
   } catch (error) {
-    return { formRecord: null, error: (error as Error).message };
+    return { success: false, error: (error as Error).message };
   }
 };
 

app/api/templates/[formID]/route.ts

@@ -201,12 +201,6 @@ export const PUT = middleware(
         return NextResponse.json(response);
       } else if (sendResponsesToVault) {
         const response = await removeDeliveryOption(ability, formID);
-        if (!response)
-          throw new Error(
-            `Template API response was null. Request information: method = ${
-              req.method
-            } ; query = ${JSON.stringify(props.params)} ; body = ${JSON.stringify(props.body)}`
-          );
         return NextResponse.json(response);
       }
       throw new MalformedAPIRequest(

lib/templates.ts

@@ -1145,61 +1145,26 @@ export async function updateResponseDeliveryOption(
  * @param formID The unique identifier of the form you want to modify
  * @returns The updated form template or null if the record does not exist
  */
-export async function removeDeliveryOption(
-  ability: UserAbility,
-  formID: string
-): Promise<FormRecord | null> {
+export async function removeDeliveryOption(ability: UserAbility, formID: string): Promise<void> {
   try {
     await authorization.canEditForm(ability, formID);
 
-    const updatedTemplate = await prisma.template
-      .update({
+    await prisma.deliveryOption
+      .deleteMany({
         where: {
-          id: formID,
-          isPublished: false,
-          deliveryOption: {
-            isNot: null,
-          },
-        },
-        data: {
-          deliveryOption: {
-            delete: true,
-          },
-        },
-        select: {
-          id: true,
-          created_at: true,
-          updated_at: true,
-          name: true,
-          jsonConfig: true,
-          isPublished: true,
-          deliveryOption: true,
-          securityAttribute: true,
-          formPurpose: true,
-          publishReason: true,
-          publishFormType: true,
-          publishDesc: true,
+          templateId: formID,
         },
       })
       .catch((e) => {
-        if (e instanceof Prisma.PrismaClientKnownRequestError) {
-          if (e.code === "P2025") {
-            throw new TemplateAlreadyPublishedError();
-          }
-        }
         return prismaErrors(e, null);
       });
 
-    if (updatedTemplate === null) return updatedTemplate;
-
     logEvent(
       ability.userID,
       { type: "Form", id: formID },
       "ChangeDeliveryOption",
       "Delivery Option set to the Vault"
     );
-
-    return _parseTemplate(updatedTemplate);
   } catch (e) {
     if (e instanceof AccessControlError)
       logEvent(