0001503: Request - fee account included in bank account table

[fa-stable.git] / sales / includes / db / payment_db.inc

diff --git a/sales/includes/db/payment_db.inc b/sales/includes/db/payment_db.inc
index 8ae5bdf3c1d3e0f4fd7227f1f80b11c0ce2fe70f..baabf0fed5623c83f2fe7ec010981226ba25d745 100644 (file)
--- a/sales/includes/db/payment_db.inc
+++ b/sales/includes/db/payment_db.inc
@@ -15,28 +15,37 @@
 function write_customer_payment($trans_no, $customer_id, $branch_id, $bank_account,
        $date_, $ref, $amount, $discount, $memo_, $rate=0, $charge=0)
 {
+       global $Refs;
+
        begin_transaction();
+       $args = func_get_args(); while (count($args) < 11) $args[] = 0;
+       $args = (object)array_combine(array('trans_no', 'customer_id', 'branch_id', 'bank_account', 
+               'date_', 'ref', 'amount', 'discount', 'memo_','rate','charge'), $args);
+       hook_db_prewrite($args, ST_CUSTPAYMENT);
 
        $company_record = get_company_prefs();
+       
+       //Chaitanya : 13_OCT_2011 : Voiding Tasks first
+       //Reason : After modifying the customer trans, it was getting voided later
+       if ($trans_no != 0) {
+         delete_comments(ST_CUSTPAYMENT, $trans_no);
+         void_bank_trans(ST_CUSTPAYMENT, $trans_no, true);
+         void_gl_trans(ST_CUSTPAYMENT, $trans_no, true);
+         void_cust_allocations(ST_CUSTPAYMENT, $trans_no, $date_);
+       }
 
-       $payment_no = write_customer_trans(12, $trans_no, $customer_id, $branch_id, 
-               $date_, $ref, $amount, $discount, 0, 0, 0, 0, 0, 0, 0, "", 0, $rate);
-
+       $payment_no = write_customer_trans(ST_CUSTPAYMENT, $trans_no, $customer_id, $branch_id, 
+               $date_, $ref, $amount, $discount, 0, 0, 0, 0, 0, 0, "", 0, $rate);
+   
        $bank_gl_account = get_bank_gl_account($bank_account);
 
-       if ($trans_no != 0) {
-         delete_comments(12, $trans_no);
-         void_bank_trans(12, $trans_no, true);
-         void_gl_trans(12, $trans_no, true);
-         void_cust_allocations(12, $trans_no, $date_);
-       }
        $total = 0;
        /* Bank account entry first */
-       $total += add_gl_trans_customer(12, $payment_no, $date_,
+       $total += add_gl_trans_customer(ST_CUSTPAYMENT, $payment_no, $date_,
                $bank_gl_account, 0, 0, $amount - $charge,  $customer_id,
                "Cannot insert a GL transaction for the bank account debit", $rate);
 
-       if ($branch_id != reserved_words::get_any_numeric()) {
+       if ($branch_id != ANY_NUMERIC) {
 
                $branch_data = get_branch_accounts($branch_id);
 
@@ -50,36 +59,38 @@ function write_customer_payment($trans_no, $customer_id, $branch_id, $bank_accou
 
        if (($discount + $amount) != 0) {
        /* Now Credit Debtors account with receipts + discounts */
-       $total += add_gl_trans_customer(12, $payment_no, $date_,
+       $total += add_gl_trans_customer(ST_CUSTPAYMENT, $payment_no, $date_,
                $debtors_account, 0, 0, -($discount + $amount), $customer_id,
                "Cannot insert a GL transaction for the debtors account credit", $rate);
        }
        if ($discount != 0)     {
                /* Now Debit discount account with discounts allowed*/
-               $total += add_gl_trans_customer(12, $payment_no, $date_,
+               $total += add_gl_trans_customer(ST_CUSTPAYMENT, $payment_no, $date_,
                        $discount_account, 0, 0, $discount, $customer_id,
                        "Cannot insert a GL transaction for the payment discount debit", $rate);
        }
 
        if ($charge != 0)       {
                /* Now Debit bank charge account with charges */
-               $charge_act = get_company_pref('bank_charge_act');
-               $total += add_gl_trans_customer(12, $payment_no, $date_,
+               $charge_act = get_bank_charge_account($bank_account);
+               $total += add_gl_trans_customer(ST_CUSTPAYMENT, $payment_no, $date_,
                        $charge_act, 0, 0, $charge, $customer_id,
                        "Cannot insert a GL transaction for the payment bank charge debit", $rate);
        }
        /*Post a balance post if $total != 0 */
-       add_gl_balance(12, $payment_no, $date_, -$total, payment_person_types::customer(), $customer_id);       
+       add_gl_balance(ST_CUSTPAYMENT, $payment_no, $date_, -$total, PT_CUSTOMER, $customer_id);        
 
        /*now enter the bank_trans entry */
-       add_bank_trans(12, $payment_no, $bank_account, $ref,
-               $date_, $amount - $charge, payment_person_types::customer(), $customer_id,
-               get_customer_currency($customer_id));
+       add_bank_trans(ST_CUSTPAYMENT, $payment_no, $bank_account, $ref,
+               $date_, $amount - $charge, PT_CUSTOMER, $customer_id,
+               get_customer_currency($customer_id), "", $rate);
 
-       add_comments(12, $payment_no, $date_, $memo_);
+       add_comments(ST_CUSTPAYMENT, $payment_no, $date_, $memo_);
 
-       references::save(12, $payment_no, $ref);
+       $Refs->save(ST_CUSTPAYMENT, $payment_no, $ref);
 
+       $args->trans_no = $payment_no;
+       hook_db_postwrite($args, ST_CUSTPAYMENT);
        commit_transaction();
 
        return $payment_no;
@@ -91,6 +102,7 @@ function void_customer_payment($type, $type_no)
 {
        begin_transaction();
 
+       hook_db_prevoid($type, $type_no);
        void_bank_trans($type, $type_no, true);
        void_gl_trans($type, $type_no, true);
        void_cust_allocations($type, $type_no);