Fixed broken excel reportsfor non-latin1 encoding.
[fa-stable.git] / gl / includes / db / gl_db_banking.inc
index 58825018e3e12db6fb01beaac3bb9aa4aec86079..721bff1eb34abed556cccd099448713be56fbb8e 100644 (file)
@@ -93,6 +93,10 @@ function add_bank_transfer($from_account, $to_account, $date_,
        $currency = get_bank_account_currency($from_account);
 
        $trans_no = get_next_trans_no($trans_type);
+       
+    $fromact = get_bank_account($from_account);
+    $toact = get_bank_account($to_account);
+    $person_id = _("From")." ".$fromact['bank_account_name']." "._("To")." ".$toact['bank_account_name'];
 
        $from_gl_account = get_bank_gl_account($from_account);
        $to_gl_account = get_bank_gl_account($to_account);
@@ -100,17 +104,18 @@ function add_bank_transfer($from_account, $to_account, $date_,
        $exchanged = false;
        $total = 0;
        // do the source account postings
-    $total += add_gl_trans($trans_type, $trans_no, $date_, $from_gl_account, 0, 0, "",
+    $total += add_gl_trans($trans_type, $trans_no, $date_, $from_gl_account, 0, 0, $person_id,
                -($amount + $charge), $currency);
 
     add_bank_trans($trans_type, $trans_no, $from_account, $ref,
                $date_, -($amount + $charge),
-               PT_MISC, "", $currency,
+               PT_MISC, $person_id, $currency,
                "Cannot insert a source bank transaction");
        if ($SysPrefs->auto_currency_revaluation())
        {
                $trans_no1 = get_next_trans_no(ST_JOURNAL);
-               if (add_exchange_variation($trans_no1, $date_, $from_account, $from_gl_account, $currency))
+               if (add_exchange_variation($trans_no1, $date_, $from_account, $from_gl_account, 
+                       $currency, PT_MISC, $person_id))
                        $exchanged = true;
        }
        if ($charge != 0)
@@ -118,24 +123,25 @@ function add_bank_transfer($from_account, $to_account, $date_,
                /* Now Debit bank charge account with charges */
                $charge_act = get_company_pref('bank_charge_act');
                $total += add_gl_trans($trans_type, $trans_no, $date_,
-                       $charge_act, 0, 0, "", $charge, $currency);
+                       $charge_act, 0, 0, $person_id, $charge, $currency);
        }
        // do the destination account postings
-       $total += add_gl_trans($trans_type, $trans_no, $date_, $to_gl_account, 0, 0, "",
+       $total += add_gl_trans($trans_type, $trans_no, $date_, $to_gl_account, 0, 0, $person_id,
                $amount, $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, "",
+               $date_, $amount, PT_MISC, $person_id,
                $currency, "Cannot insert a destination bank transaction");
 
        if ($SysPrefs->auto_currency_revaluation())
        {
                $currency = get_bank_account_currency($to_account);
        
-               if ($exchanged || add_exchange_variation($trans_no1, $date_, $to_account, $to_gl_account,       $currency))
+               if ($exchanged || add_exchange_variation($trans_no1, $date_, $to_account, $to_gl_account,       
+                       $currency, PT_MISC, $person_id))
                {
                        $ref1 = $Refs->get_next(ST_JOURNAL);
                        $Refs->save(ST_JOURNAL, $trans_no1, $ref1);
@@ -151,7 +157,6 @@ function add_bank_transfer($from_account, $to_account, $date_,
 
        return $trans_no;
 }
-
 //----------------------------------------------------------------------------------
 //     Add bank payment or deposit to database.
 //
@@ -162,8 +167,10 @@ function add_bank_transfer($from_account, $to_account, $date_,
 // $person_detail_id - customer branch id or not used
 //
 // returns an array of (inserted trans type, trans no)
-
-function add_bank_transaction($trans_type, $from_account, $items, $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)
 {
        global $Refs, $SysPrefs;
@@ -177,6 +184,9 @@ function add_bank_transaction($trans_type, $from_account, $items, $date_,
        if ($use_transaction)
                begin_transaction();
 
+       if ($trans_no)
+               clear_bank_transaction($_SESSION['pay_items']->trans_type, $_SESSION['pay_items']->order_id);
+
        $currency = get_bank_account_currency($from_account);
        $bank_gl_account = get_bank_gl_account($from_account);
 
@@ -192,7 +202,7 @@ function add_bank_transaction($trans_type, $from_account, $items, $date_,
                // we need to negate it too
                $cust_amount = -$cust_amount;
 
-               $trans_no = write_customer_trans($trans_type, 0, $person_id, $person_detail_id, $date_,
+               $trans_no = write_customer_trans($trans_type, $trans_no, $person_id, $person_detail_id, $date_,
                $ref, $cust_amount);
 
     }
@@ -205,13 +215,14 @@ function add_bank_transaction($trans_type, $from_account, $items, $date_,
                // we need to negate it too
                $supp_amount = -$supp_amount;
 
-               $trans_no = add_supp_trans($trans_type, $person_id, $date_, '',
-               $ref, "", $supp_amount, 0, 0);
+               $trans_no = write_supp_trans($trans_type, $trans_no, $person_id, $date_, '',
+                       $ref, "", $supp_amount, 0, 0);
 
     }
     else
     {
-       $trans_no = get_next_trans_no($trans_type);
+       if (!$trans_no)
+               $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);
@@ -248,9 +259,8 @@ function add_bank_transaction($trans_type, $from_account, $items, $date_,
                        "Cannot insert a destination bank transaction");
                if ($do_exchange_variance)
                {
-                       if (add_exchange_variation($trans_no1, $date_, $is_bank_to, $gl_item->code_id, 
-                               $currency, $person_type_id, $person_id))
-                               $exchanged = true;
+                       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
@@ -276,6 +286,7 @@ function add_bank_transaction($trans_type, $from_account, $items, $date_,
                        add_audit_trail(ST_JOURNAL, $trans_no1, $date_);
                }       
        }
+
        add_comments($trans_type, $trans_no, $date_, $memo_);
 
        $Refs->save($trans_type, $trans_no, $ref);
@@ -286,7 +297,28 @@ function add_bank_transaction($trans_type, $from_account, $items, $date_,
 
        return array($trans_type, $trans_no);
 }
-
 //----------------------------------------------------------------------------------------
 
+function clear_bank_transaction($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