Skip to content
Snippets Groups Projects
Unverified Commit c0acba3d authored by Eileen McNaughton's avatar Eileen McNaughton Committed by GitHub
Browse files

Merge pull request #20212 from jitendrapurohit/dev-2583-537

#2583 - Fix fatal error on membership renewal
parents 5e738d5f 929155cf
Branches
Tags
No related merge requests found
......@@ -202,7 +202,7 @@ class CRM_Member_Form_MembershipRenewal extends CRM_Member_Form {
$defaults['total_amount'] = CRM_Utils_Money::formatLocaleNumericRoundedForDefaultCurrency(CRM_Core_DAO::getFieldValue('CRM_Member_DAO_MembershipType',
$this->_memType,
'minimum_fee'
));
) ?? 0);
$defaults['record_contribution'] = 0;
$defaults['num_terms'] = 1;
......@@ -276,7 +276,7 @@ class CRM_Member_Form_MembershipRenewal extends CRM_Member_Form {
//CRM-16950
$taxAmount = NULL;
$totalAmount = $values['minimum_fee'] ?? NULL;
$totalAmount = $values['minimum_fee'] ?? 0;
// @todo - feels a bug - we use taxRate from the form default rather than from the specified type?!?
if ($this->getTaxRateForFinancialType($values['financial_type_id'])) {
$taxAmount = ($taxRate / 100) * CRM_Utils_Array::value('minimum_fee', $values);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment