X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;ds=sidebyside;f=gl%2Fincludes%2Fdb%2Fgl_db_banking.inc;h=5774de8577aa229c0f77b859c088e6784459023d;hb=f78376207e893ceea25c6a5a3c8ab633abb6e2ed;hp=6c4395d582abc8e813d813f056e19a0369d86d30;hpb=ea471f0c21d91b7b29ba03038015f67083df496f;p=fa-stable.git diff --git a/gl/includes/db/gl_db_banking.inc b/gl/includes/db/gl_db_banking.inc index 6c4395d5..5774de85 100644 --- a/gl/includes/db/gl_db_banking.inc +++ b/gl/includes/db/gl_db_banking.inc @@ -86,18 +86,6 @@ function get_cust_account_curr_balances($date) return $result; } -function get_cust_account_home_balance($debtor_no, $rec_account, $to_date) -{ - $to = date2sql($to_date); - - $sql = "SELECT SUM(amount) FROM ".TB_PREF."gl_trans t - WHERE t.person_id=".db_escape($debtor_no)." AND t.account='$rec_account' - AND person_type_id=".PT_CUSTOMER." AND t.tran_date <= '$to'"; - $result = db_query($sql, "The AR balance for customer $debtor_no could not be calculated"); - $row = db_fetch_row($result); - return $row[0]; -} - function get_supp_account_curr_balances($date) { $to = date2sql($date); @@ -115,14 +103,14 @@ function get_supp_account_curr_balances($date) return $result; } -function get_supp_account_home_balance($supplier_id, $rec_account, $to_date) +function get_account_home_balance($person_id, $person_type, $rec_account, $to_date) { $to = date2sql($to_date); - $sql = "SELECT SUM(amount) FROM ".TB_PREF."gl_trans t - WHERE t.person_id=".db_escape($supplier_id)." AND t.account='$rec_account' - AND person_type_id=".PT_SUPPLIER." AND t.tran_date <= '$to'"; - $result = db_query($sql, "The AP balance for customer $supplier_id could not be calculated"); + $sql = "SELECT SUM(amount) FROM ".TB_PREF."gl_trans + WHERE person_id=".db_escape($person_id)." AND account='$rec_account' + AND person_type_id=$person_type AND tran_date <= '$to'"; + $result = db_query($sql, "The AR/AP balance for customer $person_id could not be calculated"); $row = db_fetch_row($result); return $row[0]; } @@ -164,7 +152,7 @@ function add_exchange_variation_all($date=null, $ref="", $memo) { $exrate = get_exchange_rate_from_home_currency($row['curr_code'], $date); $foreign = round2($row['amount']*$exrate, user_price_dec()); - $home = round2(get_cust_account_home_balance($row['debtor_no'], $row['receivables_account'], $date), user_price_dec()); + $home = round2(get_account_home_balance($row['debtor_no'], PT_CUSTOMER, $row['receivables_account'], $date), user_price_dec()); if ($foreign != $home) { $amount = $foreign - $home; @@ -180,7 +168,7 @@ function add_exchange_variation_all($date=null, $ref="", $memo) { $exrate = get_exchange_rate_from_home_currency($row['curr_code'], $date); $foreign = round2($row['amount']*$exrate, user_price_dec()); - $home = round2(get_supp_account_home_balance($row['supplier_id'], $row['payable_account'], $date), user_price_dec()); + $home = round2(get_account_home_balance($row['supplier_id'], PT_SUPPLIER, $row['payable_account'], $date), user_price_dec()); if ($foreign != $home) { $amount = $foreign - $home; @@ -190,7 +178,7 @@ function add_exchange_variation_all($date=null, $ref="", $memo) } } //------------- END - + commit_transaction(); return array(($exchanged ? $trans_no : 0), $je); } @@ -202,14 +190,14 @@ function add_exchange_variation_all($date=null, $ref="", $memo) // 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); @@ -250,16 +238,21 @@ function add_bank_transfer($from_account, $to_account, $date_, $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()) { @@ -300,7 +293,7 @@ function add_bank_transfer($from_account, $to_account, $date_, // 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; @@ -315,12 +308,15 @@ function write_bank_transaction($trans_type, $trans_no, $from_account, $items, $ $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); - - if ($trans_no) - clear_bank_transaction($_SESSION['pay_items']->trans_type, $_SESSION['pay_items']->order_id); + $aid = 0; + if ($trans_no) { + $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); $bank_gl_account = get_bank_gl_account($from_account); @@ -333,11 +329,14 @@ function write_bank_transaction($trans_type, $trans_no, $from_account, $items, $ // 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_); + if (!isset($settled_amount)) // leaved for backward/ext compatibility + $cust_amount = exchange_from_to($total_amount, $currency, get_customer_currency($person_id), $date_); + else + $cust_amount = $settled_amount; // we need to negate it too $cust_amount = -$cust_amount; - $trans_no = write_customer_trans($trans_type, $trans_no, $person_id, $person_detail_id, $date_, + $trans_no = write_customer_trans($trans_type, 0, $person_id, $person_detail_id, $date_, $ref, $cust_amount); } @@ -345,24 +344,31 @@ function write_bank_transaction($trans_type, $trans_no, $from_account, $items, $ { // 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($total_amount, $currency, get_supplier_currency($person_id), $date_); + else + $supp_amount = $settled_amount; // we need to negate it too $supp_amount = -$supp_amount; - $trans_no = write_supp_trans($trans_type, $trans_no, $person_id, $date_, '', + $trans_no = write_supp_trans($trans_type, 0, $person_id, $date_, '', $ref, "", $supp_amount, 0, 0); } else { - if (!$trans_no) - $trans_no = get_next_trans_no($trans_type); + $trans_no = get_next_trans_no($trans_type); $do_exchange_variance = $SysPrefs->auto_currency_revaluation(); 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, @@ -396,13 +402,13 @@ function write_bank_transaction($trans_type, $trans_no, $from_account, $items, $ { add_exchange_variation($trans_no1, $date_, $is_bank_to, $gl_item->code_id, $currency, $person_type_id, $person_id); - } + } } // store tax details if the gl account is a tax account $amount = $gl_item->amount; $ex_rate = get_exchange_rate_from_home_currency($currency, $date_); - + add_gl_tax_details($gl_item->code_id, $trans_type, $trans_no, -$amount, $ex_rate, $date_, $memo_); } @@ -415,11 +421,11 @@ function write_bank_transaction($trans_type, $trans_no, $from_account, $items, $ { if ($exchanged || add_exchange_variation($trans_no1, $date_, $from_account, $bank_gl_account, $currency, $person_type_id, $person_id)) - { + { $ref1 = $Refs->get_next(ST_JOURNAL); $Refs->save(ST_JOURNAL, $trans_no1, $ref1); add_audit_trail(ST_JOURNAL, $trans_no1, $date_); - } + } } add_comments($trans_type, $trans_no, $date_, $memo_); @@ -427,6 +433,11 @@ function write_bank_transaction($trans_type, $trans_no, $from_account, $items, $ $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) @@ -434,30 +445,5 @@ function write_bank_transaction($trans_type, $trans_no, $from_account, $items, $ return array($trans_type, $trans_no); } -//---------------------------------------------------------------------------------------- - -function clear_bank_transaction($type, $type_no) -{ - - hook_db_prevoid($type, $type_no); - - $sql = "DELETE FROM ".TB_PREF."bank_trans - WHERE type=".db_escape($type)." AND trans_no=".db_escape($type_no); - - $result = db_query($sql, "could not clear bank transactions for type=$type and trans_no=$type_no"); - - clear_gl_trans($type, $type_no, true); - - // in case it's a customer trans - probably better to check first - void_cust_allocations($type, $type_no); - - // in case it's a supplier trans - probably better to check first - void_supp_allocations($type, $type_no); - clear_supp_trans($type, $type_no); - - clear_trans_tax_details($type, $type_no); - - delete_comments($type, $type_no); -} ?> \ No newline at end of file