}
$amount = get_gl_trans_from_to("", $date_, $account);
$diff = $amount - $for_amount;
- if ($diff != 0)
+ if (floatcmp($diff,0))
{
if ($trans_no == null)
$trans_no = get_next_trans_no(ST_JOURNAL);
-$diff, null, $person_type_id, $person_id);
add_gl_trans(ST_JOURNAL, $trans_no, $date_, get_company_pref('exchange_diff_act'), 0, 0,
_("Exchange Variance"), $diff, null, $person_type_id, $person_id);
+ return true;
}
- return ($diff != 0);
+ return false;
}
//------------- New helper functions for revaluation of customer/supplier currencies 2011-05-08 Joe Hunt.
//
function add_bank_transfer($from_account, $to_account, $date_,
- $amount, $ref, $memo_, $charge=0)
+ $amount, $ref, $memo_, $charge=0, $target_amount=0)
{
global $Refs, $SysPrefs;
begin_transaction();
$args = func_get_args(); if (count($args) < 7) $args[] = 0;
$args = (object)array_combine(array('from_account', 'to_account', 'date_', 'amount',
- 'ref', 'memo_', 'charge'), $args);
+ 'ref', 'memo_', 'charge', 'target_amount'), $args);
$args->trans_no = 0;
hook_db_prewrite($args, ST_BANKTRANSFER);
$total += add_gl_trans($trans_type, $trans_no, $date_,
$charge_act, 0, 0, $person_id, $charge, $currency);
}
+
+ // provide backward compatibility for extension modules (target amount can be not passed)
+ $to_currency = $target_amount ? $toact['bank_curr_code'] : $currency;
+ $to_amount = $target_amount ? $target_amount : $amount;
+
// do the destination account postings
$total += add_gl_trans($trans_type, $trans_no, $date_, $to_gl_account, 0, 0, $person_id,
- $amount, $currency);
+ $to_amount, $to_currency);
/*Post a balance post if $total != 0 */
add_gl_balance($trans_type, $trans_no, $date_, -$total);
add_bank_trans($trans_type, $trans_no, $to_account, $ref,
- $date_, $amount, PT_MISC, $person_id,
- $currency, "Cannot insert a destination bank transaction");
+ $date_, $to_amount, PT_MISC, $person_id,
+ $to_currency, "Cannot insert a destination bank transaction");
if ($SysPrefs->auto_currency_revaluation())
{
// Add bank payment or deposit to database.
//
// $from_account - bank account id
-// $item - transaction cart (line item's amounts in bank account's currency)
+// $items - transaction cart (line amounts in bank account's currency); negative for deposit
// $person_type_id - defines type of $person_id identifiers
// $person_id - supplier/customer/other id
-// $person_detail_id - customer branch id or not used
+// $person_detail_id - customer branch id or not used
+// $settled_amount - settled amount in AR/AP (if applicable) in customer/supplier currency (always non-negative number)
//
// returns an array of (inserted trans type, trans no)
//
// FIXME -revise code for update case
//
function write_bank_transaction($trans_type, $trans_no, $from_account, $items, $date_,
- $person_type_id, $person_id, $person_detail_id, $ref, $memo_, $use_transaction=true)
+ $person_type_id, $person_id, $person_detail_id, $ref, $memo_, $use_transaction=true, $settled_amount=null)
{
global $Refs, $SysPrefs;
$args = func_get_args(); if (count($args) < 11) $args[] = true;
$args = (object)array_combine(array('trans_type', 'trans_no', 'from_account', 'items', 'date_',
- 'person_type_id', 'person_id', 'person_detail_id', 'ref', 'memo_', 'use_transaction'),
+ 'person_type_id', 'person_id', 'person_detail_id', 'ref', 'memo_', 'use_transaction', 'settled_amount'),
$args);
hook_db_prewrite($args, $trans_type);
-
+ $aid = 0;
if ($trans_no) {
- void_transaction($trans_type, $trans_no, Today(), _("Document reentered."));
+ $old_trans = $trans_no;
$Refs->restore_last($trans_type, $trans_no);
+ $aid = has_attachment($trans_type, $trans_no);
}
$currency = get_bank_account_currency($from_account);
if ($person_type_id == PT_CUSTOMER)
{
// we need to add a customer transaction record
-
// convert to customer currency
- $cust_amount = exchange_from_to($total_amount, $currency, get_customer_currency($person_id), $date_);
- // we need to negate it too
- $cust_amount = -$cust_amount;
+ if (!isset($settled_amount)) // leaved for backward/ext compatibility
+ $cust_amount = exchange_from_to(abs($total_amount), $currency, get_customer_currency($person_id), $date_);
+ else
+ $cust_amount = $settled_amount;
+
+ if ($trans_type == ST_BANKPAYMENT)
+ $cust_amount = -$cust_amount;
$trans_no = write_customer_trans($trans_type, 0, $person_id, $person_detail_id, $date_,
$ref, $cust_amount);
{
// we need to add a supplier transaction record
// convert to supp currency
- $supp_amount = exchange_from_to($total_amount, $currency, get_supplier_currency($person_id), $date_);
+ if (!isset($settled_amount)) // leaved for for backward/ext compatibility
+ $supp_amount = exchange_from_to(abs($total_amount), $currency, get_supplier_currency($person_id), $date_);
+ else
+ $supp_amount = $settled_amount;
- // we need to negate it too
- $supp_amount = -$supp_amount;
+ if ($trans_type == ST_BANKPAYMENT)
+ $supp_amount = -$supp_amount;
$trans_no = write_supp_trans($trans_type, 0, $person_id, $date_, '',
$ref, "", $supp_amount, 0, 0);
if ($do_exchange_variance)
$trans_no1 = get_next_trans_no(ST_JOURNAL);
}
-
+ if ($aid != 0)
+ {
+ $row = get_attachment($aid);
+ update_attachment($aid, $row['type_no'], $trans_no, $row['description'],
+ $row['filename'], $row['unique_name'], $row['filesize'], $row['filetype']);
+ }
// do the source account postings
add_bank_trans($trans_type, $trans_no, $from_account, $ref,
$Refs->save($trans_type, $trans_no, $ref);
add_audit_trail($trans_type, $trans_no, $date_);
+ // old transaction can be voided only after new transaction is entered,
+ // otherwise the operation could fail for cash accounts due to temporary negative balance
+ if (@$old_trans)
+ void_transaction($trans_type, $old_trans, Today(), _("Document reentered."));
+
$args->trans_no = $trans_no;
hook_db_postwrite($args, $trans_type);
if ($use_transaction)
projects / fa-stable.git / blobdiff