From 05db0f55fd56ce6ddda517607395a63d9c360fb1 Mon Sep 17 00:00:00 2001
From: Eileen McNaughton <emcnaughton@wikimedia.org>
Date: Wed, 7 Jul 2021 17:49:52 +1200
Subject: [PATCH] Further cleanup on order

Follow up to https://github.com/civicrm/civicrm-core/pull/20792
- although that can be closed if this is merged
---
 api/v3/Order.php | 28 ++++++++++++----------------
 1 file changed, 12 insertions(+), 16 deletions(-)

diff --git a/api/v3/Order.php b/api/v3/Order.php
index 5a9ac48553..1351372c54 100644
--- a/api/v3/Order.php
+++ b/api/v3/Order.php
@@ -86,12 +86,10 @@ function civicrm_api3_order_create(array $params): array {
         $order->setLineItem($lineItem, $lineIndex);
       }
 
-      $entityParams = $lineItems['params'] ?? [];
-      $entity = $order->getLineItemEntity($lineIndex);
+      $entityParams = $lineItems['params'] ?? NULL;
 
-      if ($entityParams) {
-        $supportedEntity = TRUE;
-        switch ($entity) {
+      if ($entityParams && $order->getLineItemEntity($lineIndex) !== 'contribution') {
+        switch ($order->getLineItemEntity($lineIndex)) {
           case 'participant':
             if (isset($entityParams['participant_status_id'])
               && (!CRM_Event_BAO_ParticipantStatusType::getIsValidStatusForClass($entityParams['participant_status_id'], 'Pending'))) {
@@ -99,29 +97,27 @@ function civicrm_api3_order_create(array $params): array {
             }
             $entityParams['participant_status_id'] = $entityParams['participant_status_id'] ?? 'Pending from incomplete transaction';
             $entityParams['status_id'] = $entityParams['participant_status_id'];
+            $entityParams['skipLineItem'] = TRUE;
+            $entityResult = civicrm_api3('Participant', 'create', $entityParams);
+            // @todo - once membership is cleaned up & financial validation tests are extended
+            // we can look at removing this - some weird handling in removeFinancialAccounts
             $params['contribution_mode'] = 'participant';
+            $params['participant_id'] = $entityResult['id'];
             break;
 
           case 'membership':
             $entityParams['status_id'] = 'Pending';
-            break;
-
-          default:
-            // Don't create any related entities. We might want to support eg. Pledge one day?
-            $supportedEntity = FALSE;
+            $entityParams['skipLineItem'] = TRUE;
+            $entityResult = civicrm_api3('Membership', 'create', $entityParams);
             break;
         }
-        if ($supportedEntity) {
-          $entityParams['skipLineItem'] = TRUE;
-          $entityResult = civicrm_api3($entity, 'create', $entityParams);
-          $params[$entity . '_id'] =  $entityResult['id'];
-          foreach ($lineItems['line_item'] as $innerIndex => $lineItem) {
+
+        foreach ($lineItems['line_item'] as $innerIndex => $lineItem) {
           $lineIndex = $index . '+' . $innerIndex;
           $order->setLineItemValue('entity_id', $entityResult['id'], $lineIndex);
         }
       }
     }
-    }
     $priceSetID = $order->getPriceSetID();
     $params['line_item'][$priceSetID] = $order->getLineItems();
   }
-- 
2.25.1