[0004904] Customer Credit Note: fixed invalid inventory GL postings for service items.

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

diff --git a/sales/includes/db/sales_credit_db.inc b/sales/includes/db/sales_credit_db.inc
index b35987824408c2102fab500cb8a189ef22c39582..171c0a7e86a78711039b5c73443d8237966a8998 100644 (file)
--- a/sales/includes/db/sales_credit_db.inc
+++ b/sales/includes/db/sales_credit_db.inc
@@ -114,12 +114,13 @@ function write_credit_note(&$credit_note, $write_off_acc)
                $line_tax = get_full_price_for_item($credit_line->stock_id, $credit_line->price,
                  0, $credit_note->tax_included, $credit_note->tax_group_array) - $line_taxfree_price;
 
+               $credit_line->standard_cost = get_unit_cost($credit_line->stock_id); 
+               
                write_customer_trans_detail_item(ST_CUSTCREDIT, $credit_no, $credit_line->stock_id,
                        $credit_line->item_description, $credit_line->qty_dispatched,
                        $credit_line->line_price(), $line_tax, $credit_line->discount_percent,
                        $credit_line->standard_cost, $credit_line->src_id, $trans_no==0 ? 0:  $credit_line->id);
 
-
                if ($credit_type == 'Return')
                        add_credit_movements_item($credit_note, $credit_line,
                                $credit_type, ($line_taxfree_price+$line_tax)*(1-$credit_line->discount_percent), $credit_invoice);
@@ -181,17 +182,16 @@ function add_credit_movements_item(&$credit_note, &$credit_line,
        $credit_type, $price, $credited_invoice=0)
 {
     //Chaitanya : Stamp current cost in stock moves $credit_line does not fetch cost
-    $curr_std_cost = get_standard_cost($credit_line->stock_id);
+    $curr_std_cost = get_unit_cost($credit_line->stock_id);
 
        $reference = _("Return");
-       if ($credited_invoice) {
+       if ($credited_invoice) 
                $reference .= ' '._("Ex Inv:").' ' . $credited_invoice;
 
-           add_stock_move(ST_CUSTCREDIT,  $credit_line->stock_id,
-                   key($credit_note->trans_no), $credit_note->Location,
-                       $credit_note->document_date, $reference, $credit_line->qty_dispatched,
-               $curr_std_cost, $price);
-       }
+    add_stock_move(ST_CUSTCREDIT,  $credit_line->stock_id,
+           key($credit_note->trans_no), $credit_note->Location,
+               $credit_note->document_date, $reference, $credit_line->qty_dispatched,
+        $curr_std_cost, $price);
 }
 
 //----------------------------------------------------------------------------------------
@@ -210,12 +210,13 @@ function add_gl_trans_credit_costs($order, $order_line, $credit_no, $date_,
 
        $total = 0;
        /* insert gl_trans to credit stock and debit cost of sales at standard cost*/
-       $standard_cost = get_standard_cost($order_line->stock_id);
-       if ($standard_cost != 0) {
+       $unit_cost = get_unit_cost($order_line->stock_id);
+
+       if ($stock_gl_codes['mb_flag'] != 'D' && $unit_cost != 0) {
                /*first the cost of sales entry*/
 
                $total += add_gl_trans_std_cost(ST_CUSTCREDIT, $credit_no, $date_, $stock_gl_codes["cogs_account"],
-                       $dim, $dim2, "", -($standard_cost * $order_line->qty_dispatched),
+                       $dim, $dim2, "", -($unit_cost * $order_line->qty_dispatched),
                        PT_CUSTOMER, $order->customer_id,
                        "The cost of sales GL posting could not be inserted");
 
@@ -228,7 +229,7 @@ function add_gl_trans_credit_costs($order, $order_line, $credit_no, $date_,
                }
 
                $total += add_gl_trans_std_cost(ST_CUSTCREDIT, $credit_no, $date_, $stock_entry_account, 0, 0,
-                       "", ($standard_cost * $order_line->qty_dispatched),
+                       "", ($unit_cost * $order_line->qty_dispatched),
                        PT_CUSTOMER, $order->customer_id,
                        "The stock side (or write off) of the cost of sales GL posting could not be inserted");