From 460b915dca902eef1275c49d61361d3ad6a4fe24 Mon Sep 17 00:00:00 2001
From: shahrukh-compuco <shahrukh@compuco.io>
Date: Thu, 9 Nov 2023 00:03:41 +0500
Subject: [PATCH] COMCL-333: Fix bug with empty target contact for file upload
 activities

---
 .../RemoveEmptyTargetContactFromActivity.php  | 26 +++++++++++++++++++
 civicase.php                                  |  1 +
 2 files changed, 27 insertions(+)
 create mode 100644 CRM/Civicase/Hook/ValidateForm/RemoveEmptyTargetContactFromActivity.php

diff --git a/CRM/Civicase/Hook/ValidateForm/RemoveEmptyTargetContactFromActivity.php b/CRM/Civicase/Hook/ValidateForm/RemoveEmptyTargetContactFromActivity.php
new file mode 100644
index 000000000..b0f5589a8
--- /dev/null
+++ b/CRM/Civicase/Hook/ValidateForm/RemoveEmptyTargetContactFromActivity.php
@@ -0,0 +1,26 @@
+<?php
+
+/**
+ * CiviCRM errors out if an empty string is sent for target contact id and to solve this we
+ * remove the target_contact_id param for activity form if its value is empty string.
+ */
+class CRM_Civicase_Hook_ValidateForm_RemoveEmptyTargetContactFromActivity {
+
+  public function run(string $formName, array &$fields, array &$files, CRM_Core_Form $form, array &$errors) {
+    if (!$this->shouldRun($formName)) {
+      return FALSE;
+    }
+
+    $submittedData = &$form->controller->container();
+    if (is_array($submittedData)
+      && isset($submittedData['values']['Activity']['target_contact_id'])
+      && $submittedData['values']['Activity']['target_contact_id'] === ''
+    ) {
+      unset($submittedData['values']['Activity']['target_contact_id']);
+    }
+  }
+
+  private function shouldRun(string $formName) {
+    return $formName === 'CRM_Case_Form_Activity';
+  }
+}
diff --git a/civicase.php b/civicase.php
index 196181472..aa381a5b8 100644
--- a/civicase.php
+++ b/civicase.php
@@ -310,6 +310,7 @@ function civicase_civicrm_validateForm($formName, &$fields, &$files, &$form, &$e
     new CRM_Civicase_Hook_ValidateForm_SaveActivityDraft(),
     new CRM_Civicase_Hook_ValidateForm_SaveCaseTypeCategory(),
     new CRM_Civicase_Hook_ValidateForm_SendBulkEmail(),
+    new CRM_Civicase_Hook_ValidateForm_RemoveEmptyTargetContactFromActivity(),
   ];
 
   foreach ($hooks as $hook) {