BTHAB-185: Update invoicing/payment statuses when creating a payment

compucorp · Sep 20, 2023 · 167e6fa · 167e6fa
1 parent b20b4a6
commit 167e6fa
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/CRM/Civicase/Hook/Post/CreateSalesOrderContribution.php b/CRM/Civicase/Hook/Post/CreateSalesOrderContribution.php
@@ -21,12 +21,16 @@ class CRM_Civicase_Hook_Post_CreateSalesOrderContribution {
    *   Object reference.
    */
   public function run($op, $objectName, $objectId, &$objectRef) {
+    if (!$this->shouldRun($op, $objectName)) {
+      return;
+    }
+
     $salesOrderId = CRM_Utils_Request::retrieve('sales_order', 'Integer');
     if (empty($salesOrderId)) {
       $salesOrderId = $this->getQuotationID($objectId)['Opportunity_Details.Quotation'];
     }
 
-    if (!$this->shouldRun($op, $objectName, $salesOrderId)) {
+    if (empty($salesOrderId)) {
       return;
     }
 
@@ -65,14 +69,12 @@ public function run($op, $objectName, $objectId, &$objectRef) {
    *   The operation being performed.
    * @param string $objectName
    *   Object name.
-   * @param string $salesOrderId
-   *   The sales order that triggered the contribution (if any).
    *
    * @return bool
    *   returns a boolean to determine if hook will run or not.
    */
-  private function shouldRun($op, $objectName, $salesOrderId) {
-    return strtolower($objectName) == 'contribution' && !empty($salesOrderId) && $op == 'create';
+  private function shouldRun($op, $objectName) {
+    return strtolower($objectName) == 'contribution' && $op == 'create';
   }
 
   /**

diff --git a/civicase.php b/civicase.php
@@ -245,6 +245,7 @@ function civicase_civicrm_validateForm($formName, &$fields, &$files, &$form, &$e
  */
 function civicase_civicrm_post($op, $objectName, $objectId, &$objectRef) {
   $hooks = [
+    new CRM_Civicase_Hook_Post_CaseSalesOrderPayment(),
     new CRM_Civicase_Hook_Post_CreateSalesOrderContribution(),
     new CRM_Civicase_Hook_Post_PopulateCaseCategoryForCaseType(),
     new CRM_Civicase_Hook_Post_CaseCategoryCustomGroupSaver(),