dev/financial#152 simplify passed parameters
authoreileen <emcnaughton@wikimedia.org>
Sun, 11 Oct 2020 21:07:40 +0000 (10:07 +1300)
committereileen <emcnaughton@wikimedia.org>
Sun, 11 Oct 2020 21:07:40 +0000 (10:07 +1300)
the  function now only requires  so this simplifies so it only receives that

CRM/Core/Payment/PayPalIPN.php

index 6f600ccd1dbb84bb83815db053dcc0d6d4dc4757..7c05eb696f2dd3fb35943efbe25a94a1104c581c 100644 (file)
@@ -210,21 +210,20 @@ class CRM_Core_Payment_PayPalIPN extends CRM_Core_Payment_BaseIPN {
       'related_contact' => $ids['related_contact'] ?? NULL,
       'participant' => !empty($objects['participant']) ? $objects['participant']->id : NULL,
       'contributionRecur' => !empty($objects['contributionRecur']) ? $objects['contributionRecur']->id : NULL,
-    ], $objects, TRUE);
+    ], $objects['contribution'], TRUE);
   }
 
   /**
    * @param array $input
    * @param array $ids
-   * @param array $objects
+   * @param \CRM_Contribute_BAO_Contribution $contribution
    * @param bool $recur
    *
    * @return void
    * @throws \CRM_Core_Exception
    * @throws \CiviCRM_API3_Exception
    */
-  public function single($input, $ids, $objects, $recur = FALSE) {
-    $contribution = &$objects['contribution'];
+  public function single($input, $ids, $contribution, $recur = FALSE) {
 
     // make sure the invoice is valid and matches what we have in the contribution record
     if ($contribution->invoice_id != $input['invoice']) {
@@ -252,7 +251,7 @@ class CRM_Core_Payment_PayPalIPN extends CRM_Core_Payment_BaseIPN {
       return;
     }
 
-    CRM_Contribute_BAO_Contribution::completeOrder($input, $ids, $objects['contribution']);
+    CRM_Contribute_BAO_Contribution::completeOrder($input, $ids, $contribution);
   }
 
   /**
@@ -363,7 +362,7 @@ class CRM_Core_Payment_PayPalIPN extends CRM_Core_Payment_BaseIPN {
         'related_contact' => $ids['related_contact'] ?? NULL,
         'participant' => !empty($objects['participant']) ? $objects['participant']->id : NULL,
         'contributionRecur' => !empty($objects['contributionRecur']) ? $objects['contributionRecur']->id : NULL,
-      ], $objects);
+      ], $objects['contribution']);
     }
     catch (CRM_Core_Exception $e) {
       Civi::log()->debug($e->getMessage());