Details
Bug
Major
Description
When making payment with a gateway the uses the authorize/capture flow, there are a couple issues. First, the GatewayPayments authorizePayment() method does not link the payment account to the transaction it creates. Second, the GatewayPayment capturePayment() method does not include the company as an email tag like the processPayment() method does.
Possible solution given to a client.
On line of components/gateway_payments/gateway_payments.php 939 change:
$transaction_id = $this->Transactions->add([ 'client_id' => $client_id, 'amount' => $amount, 'currency' => $currency, 'type' => $type, 'gateway_id' => $gateway->id, 'transaction_id' => $response['transaction_id'], 'reference_id' => isset($response['reference_id']) ? $response['reference_id'] : null, 'status' => $response['status'] ]);to
$transaction_id = $this->Transactions->add([ 'client_id' => $client_id, 'amount' => $amount, 'currency' => $currency, 'type' => $type, 'gateway_id' => $gateway->id, 'account_id' => $account_id, 'transaction_id' => $response['transaction_id'], 'reference_id' => isset($response['reference_id']) ? $response['reference_id'] : null, 'status' => $response['status'] ]);On line 1127 change :
$tags = [ 'contact' => $contact, 'amount' => $this->CurrencyFormat->format($amount, $transaction->currency), 'transaction_id' => $transaction->id, 'response' => $response, 'last_four' => (isset($account->last4) ? $account->last4 : null) ];to
$tags = [ 'contact' => $contact, 'amount' => $this->CurrencyFormat->format($amount, $transaction->currency), 'transaction_id' => $transaction->id, 'response' => $response, 'last_four' => (isset($account->last4) ? $account->last4 : null), 'company' => $this->Companies->get($this->company_id) ];On lines 1134-1137 change from:
// Set credit card specific tags if ($transaction->type == 'cc' && isset($account->type)) { $tags['card_type'] = Language::_('Gateways.card_types.' . $account->type, true); }to
// Set credit card specific tags if ($transaction->type == 'cc' && isset($account->type)) { $card_types = $this->Accounts->getCcTypes(); $tags['card_type'] = isset($card_types[$account->type]) ? $card_types[$account->type] : null; }