From: eileenmcnaugton <eileen@fuzion.co.nz>
Date: Tue, 13 Oct 2015 09:38:04 +0000 (+1300)
Subject: CRM-17387 getBillingDefaults function not available to Membership Form
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=44032b26f370d90cb8ac682bf027c94e1c6b21ab;p=civicrm-core.git

CRM-17387 getBillingDefaults function not available to Membership Form

I think this might be some mix up in the history of merged & perhaps this function
might have been meant to go in favour of anohter.

But, the other places in the code that duplicate this are still present - so perhaps not tackled
---

diff --git a/CRM/Contribute/Form/AbstractEditPayment.php b/CRM/Contribute/Form/AbstractEditPayment.php
index 7fb7a90e40..6c373e3271 100644
--- a/CRM/Contribute/Form/AbstractEditPayment.php
+++ b/CRM/Contribute/Form/AbstractEditPayment.php
@@ -706,4 +706,30 @@ LEFT JOIN  civicrm_contribution on (civicrm_contribution.contact_id = civicrm_co
     }
   }
 
+  /**
+   * Get default values for billing fields.
+   *
+   * @todo this function still replicates code in several other places in the code.
+   *
+   * Also - the call to getProfileDefaults possibly covers the state_province & country already.
+   *
+   * @param $defaults
+   *
+   * @return array
+   */
+  protected function getBillingDefaults($defaults) {
+    // set default country from config if no country set
+    $config = CRM_Core_Config::singleton();
+    if (empty($defaults["billing_country_id-{$this->_bltID}"])) {
+      $defaults["billing_country_id-{$this->_bltID}"] = $config->defaultContactCountry;
+    }
+
+    if (empty($defaults["billing_state_province_id-{$this->_bltID}"])) {
+      $defaults["billing_state_province_id-{$this->_bltID}"] = $config->defaultContactStateProvince;
+    }
+
+    $billingDefaults = $this->getProfileDefaults('Billing', $this->_contactID);
+    return array_merge($defaults, $billingDefaults);
+  }
+
 }
diff --git a/CRM/Member/Form/MembershipRenewal.php b/CRM/Member/Form/MembershipRenewal.php
index 697ed171c8..b419773ea5 100644
--- a/CRM/Member/Form/MembershipRenewal.php
+++ b/CRM/Member/Form/MembershipRenewal.php
@@ -726,24 +726,4 @@ class CRM_Member_Form_MembershipRenewal extends CRM_Member_Form {
     }
   }
 
-  /**
-   * @param $defaults
-   *
-   * @return array
-   */
-  protected function getBillingDefaults($defaults) {
-    // set default country from config if no country set
-    $config = CRM_Core_Config::singleton();
-    if (empty($defaults["billing_country_id-{$this->_bltID}"])) {
-      $defaults["billing_country_id-{$this->_bltID}"] = $config->defaultContactCountry;
-    }
-
-    if (empty($defaults["billing_state_province_id-{$this->_bltID}"])) {
-      $defaults["billing_state_province_id-{$this->_bltID}"] = $config->defaultContactStateProvince;
-    }
-
-    $billingDefaults = $this->getProfileDefaults('Billing', $this->_contactID);
-    return array_merge($defaults, $billingDefaults);
-  }
-
 }