Fixed an accounting sync in GL when adding bank account transactions (no customer...

[fa-stable.git] / gl / includes / db / gl_db_banking.inc

diff --git a/gl/includes/db/gl_db_banking.inc b/gl/includes/db/gl_db_banking.inc
index a5fe71ad2250ee84ee1d05764b9d153aa925025d..237626a4f76973153c33d2b98a8e6d427aa199b0 100644 (file)
--- a/gl/includes/db/gl_db_banking.inc
+++ b/gl/includes/db/gl_db_banking.inc
@@ -1,5 +1,45 @@
 <?php
 
+//----------------------------------------------------------------------------------
+function add_exchange_variation($trans_type, $trans_no, $date_, $account,
+    $currency, $person_type_id=null, $person_id = "")
+{
+       if (is_company_currency($currency))
+               return;
+       if ($date_ == null)
+               $date_ = Today();
+       $rate = get_exchange_rate_from_home_currency($currency, $date_);        
+       $result = db_query("SELECT SUM(amount) FROM ".TB_PREF."bank_trans WHERE 
+               bank_act='$account' AND trans_date<='".date2sql($date_)."'", 
+               "Transactions for account $account could not be calculated");
+       $row = db_fetch_row($result);
+       $foreign_amount = $row[0];
+       $amount = get_gl_trans_from_to("", $date_, $account);
+       $diff = $amount - (round2($foreign_amount * $rate, user_price_dec()));
+       if ($diff != 0)
+       {
+               if ($trans_type == null)
+                       $trans_type = systypes::journal_entry();
+               if ($trans_no == null)
+                       $trans_no = get_next_trans_no($trans_type);
+               if ($person_type_id == null)
+                       $person_type_id = payment_person_types::misc();
+               add_gl_trans($trans_type, $trans_no, $date_, $account, 0, 0, "",
+               -$diff, null, $person_type_id, $person_id);
+       add_gl_trans($trans_type, $trans_no, $date_, get_company_pref('exchange_diff_act'), 0, 0, "",
+               $diff, null, $person_type_id, $person_id);
+       }       
+}
+
+function add_exchange_variation_all()
+{
+       $trans_no = get_next_trans_no(0);
+       $sql = "SELECT * FROM ".TB_PREF."bank_accounts";
+       $result = db_query($sql, "could not retreive bank accounts");
+       while ($myrow = db_fetch($result))
+               add_exchange_variation(0, $trans_no, null, $myrow['account_code'],
+                       $myrow['currency_code']);
+}
 //----------------------------------------------------------------------------------
 
 function add_bank_transfer($from_account, $to_account, $date_,
@@ -13,8 +53,9 @@ function add_bank_transfer($from_account, $to_account, $date_,
 
        $trans_no = get_next_trans_no($trans_type);
 
+       $total = 0;
        // do the source account postings
-    add_gl_trans($trans_type, $trans_no, $date_, $from_account, 0, 0, "",
+    $total += add_gl_trans($trans_type, $trans_no, $date_, $from_account, 0, 0, "",
                -$amount, $currency);
 
     add_bank_trans($trans_type, $trans_no, $from_account, $ref,
@@ -22,16 +63,25 @@ function add_bank_transfer($from_account, $to_account, $date_,
                payment_person_types::misc(), "", $currency,
                "Cannot insert a source bank transaction");
 
+       add_exchange_variation($trans_type, $trans_no, $date_, $from_account, 
+               $currency, payment_person_types::misc(), "");
+
        // do the destination account postings
-       add_gl_trans($trans_type, $trans_no, $date_, $to_account, 0, 0, "",
+       $total += add_gl_trans($trans_type, $trans_no, $date_, $to_account, 0, 0, "",
                $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_, $payment_type, $amount,
                payment_person_types::misc(), "",
                $currency,
                "Cannot insert a destination bank transaction");
 
+       add_exchange_variation($trans_type, $trans_no, $date_, $from_account, 
+               $currency, payment_person_types::misc(), "");
+       
        add_comments($trans_type, $trans_no, $date_, $memo_);
 
        references::save_last($ref, $trans_type);
@@ -54,6 +104,8 @@ function add_bank_transaction($trans_type, $from_account, $items, $date_,
        if ($trans_type != systypes::bank_payment() && $trans_type != systypes::bank_deposit())
                display_db_error("Invalid type ($trans_type) sent to add_bank_transaction");
 
+       $do_exchange_variance = false;
+       
        begin_transaction();
 
        $currency = get_bank_account_currency($from_account);
@@ -90,18 +142,15 @@ function add_bank_transaction($trans_type, $from_account, $items, $date_,
     else
     {
        $trans_no = get_next_trans_no($trans_type);
+       $do_exchange_variance = true;
     }
 
-       // do the source account postings
-    add_gl_trans($trans_type, $trans_no, $date_, $from_account, 0, 0, "",
-       -$total_amount, $currency, $person_type_id, $person_id);
-
     add_bank_trans($trans_type, $trans_no, $from_account, $ref,
        $date_, $type, -$total_amount,
        $person_type_id, $person_id,
        $currency,
        "Cannot insert a source bank transaction");
-
+       $total = 0;
        foreach ($items->gl_items as $gl_item)
        {
                $is_bank_to = is_bank_account($gl_item->code_id);
@@ -113,7 +162,7 @@ function add_bank_transaction($trans_type, $from_account, $items, $date_,
                }
 
        // do the destination account postings
-       add_gl_trans($trans_type, $trans_no, $date_, $gl_item->code_id,
+       $total += add_gl_trans($trans_type, $trans_no, $date_, $gl_item->code_id,
                $gl_item->dimension_id, $gl_item->dimension2_id, $gl_item->reference,
                $gl_item->amount, $currency, $person_type_id, $person_id);
 
@@ -123,9 +172,20 @@ function add_bank_transaction($trans_type, $from_account, $items, $date_,
                        $date_, $type, $gl_item->amount,
                        $person_type_id, $person_id, $currency,
                        "Cannot insert a destination bank transaction");
+               if ($do_exchange_variance)
+                       add_exchange_variation($trans_type, $trans_no, $date_, $gl_item->code_id, 
+                               $currency, $person_type_id, $person_id);
        }
        }
 
+       // do the source account postings
+    add_gl_trans($trans_type, $trans_no, $date_, $from_account, 0, 0, "",
+       -$total, null, $person_type_id, $person_id);
+
+    if ($do_exchange_variance)
+       add_exchange_variation($trans_type, $trans_no, $date_, $from_account, 
+               $currency, $person_type_id, $person_id);
+
        add_comments($trans_type, $trans_no, $date_, $memo_);
 
        references::save_last($ref, $trans_type);