diff --git a/cronjobs/payouts.php b/cronjobs/payouts.php index d44a0461..e56ac63d 100755 --- a/cronjobs/payouts.php +++ b/cronjobs/payouts.php @@ -63,19 +63,26 @@ if ($setting->getValue('disable_manual_payouts') != 1) { $monitoring->endCronjob($cron_name, 'E0010', 1, true); } $log->logInfo("\t" . $aData['account_id'] . "\t\t" . $aData['username'] . "\t" . $dBalance . "\t\t" . $aData['coin_address']); - if ($transaction->addTransaction($aData['account_id'], $dBalance - $config['txfee_manual'], 'Debit_MP', NULL, $aData['coin_address'], $txid) && $transaction->addTransaction($aData['account_id'], $config['txfee_manual'], 'TXFee', NULL, $aData['coin_address'])) { + if ($transaction->addTransaction($aData['account_id'], $dBalance - $config['txfee_manual'], 'Debit_MP', NULL, $aData['coin_address'], NULL)) { + // Store debit transaction ID for later update + $transaction_id = $transaction->insert_id; + if (!$transaction->addTransaction($aData['account_id'], $config['txfee_manual'], 'TXFee', NULL, $aData['coin_address'])) + $log->logError('Failed to add TXFee record: ' . $transaction->getCronError()); // Mark all older transactions as archived if (!$transaction->setArchived($aData['account_id'], $transaction->insert_id)) $log->logError('Failed to mark transactions for #' . $aData['account_id'] . ' prior to #' . $transaction->insert_id . ' as archived. ERROR: ' . $transaction->getCronError()); // Run the payouts from RPC now that the user is fully debited try { - $txid = $bitcoin->sendtoaddress($aData['coin_address'], $dBalance - $config['txfee_manual']); + $rpc_txid = $bitcoin->sendtoaddress($aData['coin_address'], $dBalance - $config['txfee_manual']); } catch (Exception $e) { $log->logError('E0078: RPC method did not return 200 OK: Address: ' . $aData['coin_address'] . ' ERROR: ' . $e->getMessage()); // Remove this line below if RPC calls are failing but transactions are still added to it // Don't blame MPOS if you run into issues after commenting this out! $monitoring->endCronjob($cron_name, 'E0078', 1, true); } + // Update our transaction and add the RPC Transaction ID + if (empty($rpc_txid) || !$transaction->setRPCTxId($transaction_id, $rpc_txid)) + $log->logError('Unable to add RPC transaction ID ' . $rpc_txid . ' to transaction record ' . $tx_id . ': ' . $transaction->getCronError()); // Notify user via mail $aMailData['email'] = $user->getUserEmail($user->getUserName($aData['account_id'])); $aMailData['subject'] = 'Manual Payout Completed'; @@ -135,19 +142,26 @@ if ($setting->getValue('disable_auto_payouts') != 1) { // Only run if balance meets threshold and can pay the potential transaction fee if ($dBalance > $aUserData['ap_threshold'] && $dBalance > $config['txfee_auto']) { // Create transaction record - if ($transaction->addTransaction($aUserData['id'], $dBalance - $config['txfee_auto'], 'Debit_AP', NULL, $aUserData['coin_address'], $txid) && $transaction->addTransaction($aUserData['id'], $config['txfee_auto'], 'TXFee', NULL, $aUserData['coin_address'])) { + if ($transaction->addTransaction($aUserData['id'], $dBalance - $config['txfee_auto'], 'Debit_AP', NULL, $aUserData['coin_address'], NULL)) { + // Store debit ID for later update + $transaction_id = $transaction->insert_id; + if ($transaction->addTransaction($aUserData['id'], $config['txfee_auto'], 'TXFee', NULL, $aUserData['coin_address'])) + $log->logError('Failed to add TXFee record: ' . $transaction->getCronError()); // Mark all older transactions as archived if (!$transaction->setArchived($aUserData['id'], $transaction->insert_id)) $log->logError('Failed to mark transactions for user #' . $aUserData['id'] . ' prior to #' . $transaction->insert_id . ' as archived. ERROR: ' . $transaction->getCronError()); // Run the payouts from RPC now that the user is fully debited try { - $txid = $bitcoin->sendtoaddress($aUserData['coin_address'], $dBalance - $config['txfee_auto']); + $rpc_txid = $bitcoin->sendtoaddress($aUserData['coin_address'], $dBalance - $config['txfee_auto']); } catch (Exception $e) { $log->logError('E0078: RPC method did not return 200 OK: Address: ' . $aUserData['coin_address'] . ' ERROR: ' . $e->getMessage()); // Remove this line below if RPC calls are failing but transactions are still added to it // Don't blame MPOS if you run into issues after commenting this out! $monitoring->endCronjob($cron_name, 'E0078', 1, true); } + // Update our transaction and add the RPC Transaction ID + if (empty($rpc_txid) || !$transaction->setRPCTxId($transaction_id, $rpc_txid)) + $log->logError('Unable to add RPC transaction ID ' . $rpc_txid . ' to transaction record ' . $tx_id . ': ' . $transaction->getCronError()); // Notify user via mail $aMailData['email'] = $user->getUserEmail($user->getUserName($aUserData['id'])); $aMailData['subject'] = 'Auto Payout Completed'; diff --git a/public/include/classes/transaction.class.php b/public/include/classes/transaction.class.php index 33098ba7..12bb93ce 100644 --- a/public/include/classes/transaction.class.php +++ b/public/include/classes/transaction.class.php @@ -27,6 +27,19 @@ class Transaction extends Base { return $this->sqlError(); } + /** + * Update a transaction with a RPC transaction ID + * @param id integer Transaction ID + * @param txid string RPC Transaction Identifier + * @return bool true or false + **/ + public function setRPCTxId($transaction_id, $rpc_txid=NULL) { + $stmt = $this->mysqli->prepare("UPDATE $this->table SET txid = ? WHERE id = ?"); + if ($this->checkStmt($stmt) && $stmt->bind_param('si', $rpc_txid, $transaction_id) && $stmt->execute()) + return true; + return $this->sqlError(); + } + /* * Mark transactions of a user as archived * @param account_id int Account ID