Simplify function signature as parameter is not used
authorMatthew Wire (MJW Consulting) <mjw@mjwconsult.co.uk>
Tue, 8 Oct 2019 08:24:52 +0000 (10:24 +0200)
committereileen <emcnaughton@wikimedia.org>
Wed, 9 Oct 2019 08:20:27 +0000 (10:20 +0200)
CRM/Event/Form/Registration.php
CRM/Event/Form/Registration/Confirm.php

index 2ac411173084408c1538eda1749c9c219b116bc4..1054b83b49c970dc27042c8fa05ae2cc1d6c6cdb 100644 (file)
@@ -697,10 +697,9 @@ class CRM_Event_Form_Registration extends CRM_Core_Form {
    * Handle process after the confirmation of payment by User.
    *
    * @param int $contactID
-   * @param null $contribution
-   * @param null $payment
+   * @param \CRM_Contribute_BAO_Contribution $contribution
    */
-  public function confirmPostProcess($contactID = NULL, $contribution = NULL, $payment = NULL) {
+  public function confirmPostProcess($contactID = NULL, $contribution = NULL) {
     // add/update contact information
     $fields = array();
     unset($this->_params['note']);
@@ -1598,7 +1597,7 @@ class CRM_Event_Form_Registration extends CRM_Core_Form {
         }
 
         $this->set('value', $value);
-        $this->confirmPostProcess($contactID, NULL, NULL);
+        $this->confirmPostProcess($contactID, NULL);
 
         //lets get additional participant id to cancel.
         if ($this->_allowConfirmation && is_array($cancelledIds)) {
index 05b5aa02aedeadfe16401a36c249921abd97dce3..6abb9318836678ab18a379455c9771dce0f17eb2 100644 (file)
@@ -667,7 +667,7 @@ class CRM_Event_Form_Registration_Confirm extends CRM_Event_Form_Registration {
       }
       $this->assign('register_date', $registerDate);
 
-      $this->confirmPostProcess($contactID, $contribution, $payment);
+      $this->confirmPostProcess($contactID, $contribution);
     }
 
     //handle if no additional participant.