From ad66c061361eabb80aae520d68393951a0089a34 Mon Sep 17 00:00:00 2001
From: Joe Hunt <joe.hunt.consulting@gmail.com>
Date: Sat, 28 Apr 2007 13:26:19 +0000
Subject: [PATCH] When saving work order entries a lot of debug boxes appeared.
 Fixed When clicking on the link, Issue, on work order inquires an empty
 screen appeared. Fixed

---
 CHANGELOG.txt                                      | 7 +++++++
 manufacturing/includes/db/work_orders_quick_db.inc | 3 ---
 manufacturing/includes/work_order_issue_ui.inc     | 2 +-
 manufacturing/work_order_entry.php                 | 2 ++
 4 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/CHANGELOG.txt b/CHANGELOG.txt
index f0dea4d..976ae83 100644
--- a/CHANGELOG.txt
+++ b/CHANGELOG.txt
@@ -19,6 +19,13 @@ Legend:
 ! -> Note
 $ -> Affected files
 
+28-Apr-2007 Joe Hunt
+ # When saving work order entries a lot of debug boxes appeared. Fixed
+ # When clicking on the link Issue on work order inquires an empty screen appeared. Fixed
+ $ /manufacturing/work_order_entry.php
+   /manufacturing/includes/work_order_issue_ui.inc
+   /manufacturing/includes/db/work_orders_quick_db.inc
+   
 25-Apr-2007 Joe Hunt
  # Missing Date Picker
  $ /sales/customer_invoice.php
diff --git a/manufacturing/includes/db/work_orders_quick_db.inc b/manufacturing/includes/db/work_orders_quick_db.inc
index b3eca14..aec2ee8 100644
--- a/manufacturing/includes/db/work_orders_quick_db.inc
+++ b/manufacturing/includes/db/work_orders_quick_db.inc
@@ -86,7 +86,6 @@ function work_order_quick_costs($woid, $stock_id, $units_reqd, $date_, $addition
 		
 		add_gl_trans_std_cost(systypes::work_order(), $woid, $date_, $bom_accounts["inventory_account"], 0, 0,
 			null, -$bom_cost);
-		alert("bokade bom- cost ".-$bom_cost);
 		
 		$total_cost += $bom_cost;
 	}	
@@ -94,13 +93,11 @@ function work_order_quick_costs($woid, $stock_id, $units_reqd, $date_, $addition
 	$item_accounts = get_stock_gl_code($stock_id);
 	add_gl_trans_std_cost(systypes::work_order(), $woid, $date_, $item_accounts["assembly_account"], 
 		$item_accounts["dimension_id"], $item_accounts["dimension2_id"], null, -$additional_costs);
-	alert("bokade additional cost ".-$additional_costs.", acc = ".$item_accounts["assembly_account"]);
 	
 	// debit total components + additional
 	$total_cost += $additional_costs;	
 	add_gl_trans_std_cost(systypes::work_order(), $woid, $date_, $item_accounts["inventory_account"], 
 		0, 0, null, $total_cost);
-	alert("bokade total- cost ".$total_cost);
 } 
 
 //--------------------------------------------------------------------------------------
diff --git a/manufacturing/includes/work_order_issue_ui.inc b/manufacturing/includes/work_order_issue_ui.inc
index c81b779..531e998 100644
--- a/manufacturing/includes/work_order_issue_ui.inc
+++ b/manufacturing/includes/work_order_issue_ui.inc
@@ -57,7 +57,7 @@ function display_adjustment_items($Title, &$order)
     	 	label_cell("<a target='_blank' href='$path_to_root/inventory/inquiry/stock_status.php?" . SID . "stock_id=" . $stock_item->stock_id . "'>$stock_item->stock_id</a>");
     		label_cell($stock_item->item_description);
     		qty_cell($stock_item_quantity);
-    		if (strlen($stock_item->units)
+    		if (strlen($stock_item->units))
     			label_cell($stock_item->units);
     		else
     			label_cell("");
diff --git a/manufacturing/work_order_entry.php b/manufacturing/work_order_entry.php
index b26ed9c..a8253ed 100644
--- a/manufacturing/work_order_entry.php
+++ b/manufacturing/work_order_entry.php
@@ -146,6 +146,8 @@ function can_process()
         	return false;
         }
 
+		if ($_POST['Costs'] == "")
+			$_POST['Costs'] = 0;
     	if (!is_numeric($_POST['Costs']))
     	{
     		display_error( _("The cost entered must be numeric."));
-- 
2.30.2