From: Matthew Wire <devel@mrwire.co.uk>
Date: Sun, 14 Apr 2019 19:37:43 +0000 (+0100)
Subject: Merge pull request #13837 from eileenmcnaughton/event_conf
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=e48ed441e7d36e73a3ea5554a8e59411e8eb9920;p=civicrm-core.git

Merge pull request #13837 from eileenmcnaughton/event_conf

Create contribution before taking payment, per contribution page workflow
---

e48ed441e7d36e73a3ea5554a8e59411e8eb9920
diff --cc CRM/Event/Form/Registration/Confirm.php
index 7ba4fdb95a,71a3fd2bb8..bc800a62a5
--- a/CRM/Event/Form/Registration/Confirm.php
+++ b/CRM/Event/Form/Registration/Confirm.php
@@@ -973,9 -998,12 +998,12 @@@ class CRM_Event_Form_Registration_Confi
      }
  
      // CRM-20264: fetch CC type ID and number (last 4 digit) and assign it back to $params
+     // @todo remove this once backoffice form is doing it - front end is doing it twice.
+     // (in general they are sharing much more code than they should - anything that truly should be shared
+     // should not be on this class).
      CRM_Contribute_Form_AbstractEditPayment::formatCreditCardDetails($params);
  
 -    $contribParams = array(
 +    $contribParams = [
        'contact_id' => $contactID,
        'financial_type_id' => !empty($form->_values['event']['financial_type_id']) ? $form->_values['event']['financial_type_id'] : $params['financial_type_id'],
        'receive_date' => $now,