Skip to content

Commit

Permalink
Remove unused variable
Browse files Browse the repository at this point in the history
This is over-written a couple of lines later in the template so
we don't need it & ergo don't need to assign it either

Note that the paramter is assigned in postProcess for the
receipt so this is just this usage
  • Loading branch information
eileenmcnaughton committed Jan 17, 2022
1 parent 9401c3b commit 38129a1
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
5 changes: 0 additions & 5 deletions CRM/Contribute/Form/Contribution.php
Original file line number Diff line number Diff line change
Expand Up @@ -516,11 +516,6 @@ public function buildQuickForm() {
$this->assign('payNow', FALSE);
$buildRecurBlock = FALSE;

// display tax amount on edit contribution page
if ($invoicing && $this->_action & CRM_Core_Action::UPDATE && isset($this->_values['tax_amount'])) {
$this->assign('totalTaxAmount', $this->_values['tax_amount']);
}

if (empty($this->_lineItems) &&
($this->_priceSetId || !empty($_POST['price_set_id']))
) {
Expand Down
1 change: 0 additions & 1 deletion templates/CRM/Contribute/Form/Contribution.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -597,7 +597,6 @@
// replace all thousandMarker and change the separator to a dot
totalAmount = totalAmount.replace(thousandMarker,'').replace(separator,'.');
var totalTaxAmount = '{/literal}{$totalTaxAmount}{literal}';
var taxAmount = (taxRate/100)*totalAmount;
taxAmount = isNaN (taxAmount) ? 0:taxAmount;
var totalTaxAmount = taxAmount + Number(totalAmount);
Expand Down

0 comments on commit 38129a1

Please sign in to comment.