Reverted invalid changes in manufacturing accounts back to cogs_account for BOM servi...
[fa-stable.git] / manufacturing / includes / db / work_order_produce_items_db.inc
index 1c72f94f3cc8d8e25409aa9d93380a549d35753a..d67d2e9048dce7891e853f813b9ed290434c0077 100644 (file)
@@ -44,26 +44,39 @@ function work_order_produce($woid, $ref, $quantity, $date_, $memo_, $close_wo)
        db_query($sql,"A work order manufacture could not be added");
 
        $id = db_insert_id();
-       \r
-       // -------------------------------------------------------------------------\r
+       
+       // -------------------------------------------------------------------------
 
-       work_order_quick_costs($woid, $details["stock_id"], $quantity, $date_, $id);\r
-       \r
+       work_order_quick_costs($woid, $details["stock_id"], $quantity, $date_, $id);
+       
        // -------------------------------------------------------------------------
 
-       // Chaitanya: stamp BOM cost to finished item\r
-       $m_cost = 0;\r
-    $result = get_bom($details["stock_id"]);\r
-       while ($bom_item = db_fetch($result))\r
-       {\r
-               $standard_cost = get_standard_cost($bom_item['component']);\r
-               $m_cost += ($bom_item['quantity'] * $standard_cost);\r
-       }       \r
+       // Chaitanya: stamp BOM cost to finished item
+       $m_cost = 0;
+       $result = get_bom($details["stock_id"]);
+       while ($bom_item = db_fetch($result))
+       {
+               $standard_cost = get_standard_cost($bom_item['component']);
+               $m_cost += ($bom_item['quantity'] * $standard_cost);
+       }
+       // new Joe Hunt 2015.10.15      
+       // additilnal costs.
+       if (work_order_has_issues($woid))
+       {
+               $res = get_additional_issues($woid);
+               while ($issue = db_fetch($res))
+               {
+                       $standard_cost = get_standard_cost($issue['stock_id']);
+                       $m_cost += ($issue['qty_issued'] * $standard_cost) / $quantity;
+               }
+       }
+       $m_cost += (get_gl_wo_cost($woid, WO_LABOUR) / $quantity);
+       $m_cost += (get_gl_wo_cost($woid, WO_OVERHEAD) / $quantity);
 
        // insert a +ve stock move for the item being manufactured
        // negative means "unproduce" or unassemble
        add_stock_move(ST_MANURECEIVE, $details["stock_id"], $id,
-               $details["loc_code"], $date_, $ref, $quantity, $m_cost);\r
+               $details["loc_code"], $date_, $ref, $quantity, $m_cost);
        // update wo quantity and close wo if requested
        work_order_update_finished_quantity($woid, $quantity, $close_wo);
 
@@ -118,53 +131,53 @@ function void_work_order_produce($type_no)
 {
        begin_transaction();
        hook_db_prevoid(ST_MANURECEIVE, $type_no);
-       \r
-       //Chaitanya : Skip processing already voided entry i.e. explicitly voided\r
-       $void_entry = get_voided_entry(ST_MANURECEIVE, $type_no);\r
-       if ($void_entry)\r
-               return; \r
+       
+       //Chaitanya : Skip processing already voided entry i.e. explicitly voided
+       $void_entry = get_voided_entry(ST_MANURECEIVE, $type_no);
+       if ($void_entry)
+               return; 
 
        $row = get_work_order_produce($type_no);
 
        // deduct the quantity of this production from the parent work order
        work_order_update_finished_quantity($row["workorder_id"], -$row["quantity"]);
 
-       //Chaitanya : skipped this step as BOM may have got changed\r
-       //work_order_quick_costs($row['workorder_id'], $row['stock_id'], -$row['quantity'], sql2date($row['date_']), $type_no);\r
-
-       // void any related gl trans\r
-       //Chaitanya : Nothing happens due to next statement as all gl postings are done against WO\r
-       //void_gl_trans(ST_MANURECEIVE, $type_no, true);\r
-       \r
-       $woid = $row["workorder_id"];\r
-       $date_ = sql2date($row["date_"]);\r
-               \r
-       $result = get_stock_moves(ST_MANURECEIVE, $type_no);\r
-       while ($myrow = db_fetch($result))\r
-       {\r
-               $issue_cost = $myrow["qty"]*$myrow["standard_cost"];\r
-               $issue = get_stock_gl_code($myrow["stock_id"]);\r
-        $stockitem = get_item($myrow["stock_id"]);\r
-               \r
-               //Chaitanya : Compatibility for Service Items\r
-               if (!is_service($issue["mb_flag"]))\r
-                       $ivaccount = $issue["inventory_account"];\r
-               else\r
-                       $ivaccount = $issue["cogs_account"];            \r
-               \r
-               if ($issue_cost != 0)\r
-               {\r
-                       add_gl_trans_std_cost(ST_WORKORDER, $woid, $date_, $ivaccount, 0, 0,\r
-                               $date_.": "._("Reversed the production ")." ".$stockitem["description"],\r
-                               -$issue_cost);\r
-               }\r
-       }       \r
-                       \r
+       //Chaitanya : skipped this step as BOM may have got changed
+       //work_order_quick_costs($row['workorder_id'], $row['stock_id'], -$row['quantity'], sql2date($row['date_']), $type_no);
+
+       // void any related gl trans
+       //Chaitanya : Nothing happens due to next statement as all gl postings are done against WO
+       //void_gl_trans(ST_MANURECEIVE, $type_no, true);
+       
+       $woid = $row["workorder_id"];
+       $date_ = sql2date($row["date_"]);
+               
+       $result = get_stock_moves(ST_MANURECEIVE, $type_no);
+       while ($myrow = db_fetch($result))
+       {
+               $issue_cost = $myrow["qty"]*$myrow["standard_cost"];
+               $issue = get_stock_gl_code($myrow["stock_id"]);
+        $stockitem = get_item($myrow["stock_id"]);
+               
+               //Chaitanya : Compatibility for Service Items
+               if (!is_service($issue["mb_flag"]))
+                       $ivaccount = $issue["inventory_account"];
+               else
+                       $ivaccount = $issue["cogs_account"];            
+               
+               if ($issue_cost != 0)
+               {
+                       add_gl_trans_std_cost(ST_WORKORDER, $woid, $date_, $ivaccount, 0, 0,
+                               $date_.": "._("Reversed the production ")." ".$stockitem["description"],
+                               -$issue_cost);
+               }
+       }       
+                       
        // clear the production record
        $sql = "UPDATE ".TB_PREF."wo_manufacture SET quantity=0 WHERE id=".db_escape($type_no);
-       db_query($sql, "Cannot void a wo production");                  \r
-       \r
-       //Chaitanya : Shifted below\r
+       db_query($sql, "Cannot void a wo production");                  
+       
+       //Chaitanya : Shifted below
        // void all related stock moves
        void_stock_move(ST_MANURECEIVE, $type_no);