[0000094] Report does not show items that have 0 qty but have demand qty

diff --git a/CHANGELOG.txt b/CHANGELOG.txt
index 7ab6280445bde405deaa4c116fe55181c6f06d79..880de99b60e08310f80eb7ba4df4550048fe1975 100644 (file)
--- a/CHANGELOG.txt
+++ b/CHANGELOG.txt
@@ -19,6 +19,11 @@ Legend:
 ! -> Note
 $ -> Affected files
 
+29-Nov-2008 Joe Hunt
+# [0000094] Report does not show items that have 0 qty but have demand qty
+$ /reporting/rep302.php
+  /reporting/rep303.php
+  
 28-Nov-2008 Joe Hunt
 # [0000093] Voiding Supplier Invoices Causes some lines cost update to go to 0 and exchange rate change
 $ /purchasing/includes/db/invoice_db.inc

diff --git a/reporting/rep302.php b/reporting/rep302.php
index 47bf8448c8c2212dd4cb1d255ed30cb14fef7539..68758cd4dbeb5c492eb6dfed82e47f2bba3e46d0 100644 (file)
--- a/reporting/rep302.php
+++ b/reporting/rep302.php
@@ -26,23 +26,22 @@ function getTransactions($category, $location)
                        ".TB_PREF."stock_category.description AS cat_description,
                        ".TB_PREF."stock_master.stock_id,
                        ".TB_PREF."stock_master.description,
-                       ".TB_PREF."stock_moves.loc_code,
-                       SUM(".TB_PREF."stock_moves.qty) AS qty_on_hand
-               FROM ".TB_PREF."stock_master,
-                       ".TB_PREF."stock_category,
-                       ".TB_PREF."stock_moves
-               WHERE ".TB_PREF."stock_master.stock_id=".TB_PREF."stock_moves.stock_id
-               AND ".TB_PREF."stock_master.category_id=".TB_PREF."stock_category.category_id
+                       IF(".TB_PREF."stock_moves.stock_id IS NULL, '', ".TB_PREF."stock_moves.loc_code) AS loc_code,
+                       SUM(IF(".TB_PREF."stock_moves.stock_id IS NULL,0,".TB_PREF."stock_moves.qty)) AS qty_on_hand
+               FROM (".TB_PREF."stock_master,
+                       ".TB_PREF."stock_category)
+               LEFT JOIN ".TB_PREF."stock_moves ON
+                       (".TB_PREF."stock_master.stock_id=".TB_PREF."stock_moves.stock_id OR ".TB_PREF."stock_master.stock_id IS NULL)
+               WHERE ".TB_PREF."stock_master.category_id=".TB_PREF."stock_category.category_id
                AND (".TB_PREF."stock_master.mb_flag='B' OR ".TB_PREF."stock_master.mb_flag='M')";
        if ($category != 0)
                $sql .= " AND ".TB_PREF."stock_master.category_id = '$category'";
        if ($location != 'all')
                $sql .= " AND ".TB_PREF."stock_moves.loc_code = '$location'";
        $sql .= " GROUP BY ".TB_PREF."stock_master.category_id,
-               ".TB_PREF."stock_master.description,
                ".TB_PREF."stock_category.description,
-               ".TB_PREF."stock_moves.stock_id,
-               ".TB_PREF."stock_master.stock_id
+               ".TB_PREF."stock_master.stock_id,
+               ".TB_PREF."stock_master.description
                ORDER BY ".TB_PREF."stock_master.category_id,
                ".TB_PREF."stock_master.stock_id";
 
@@ -55,9 +54,10 @@ function getCustQty($stockid, $location)
        $sql = "SELECT SUM(".TB_PREF."sales_order_details.quantity - ".TB_PREF."sales_order_details.qty_sent) AS qty_demand
                                FROM ".TB_PREF."sales_order_details,
                                        ".TB_PREF."sales_orders
-                               WHERE ".TB_PREF."sales_order_details.order_no=".TB_PREF."sales_orders.order_no AND
-                                       ".TB_PREF."sales_orders.from_stk_loc ='$location' AND
-                                       ".TB_PREF."sales_order_details.stk_code = '$stockid'";
+                               WHERE ".TB_PREF."sales_order_details.order_no=".TB_PREF."sales_orders.order_no AND ";
+       if ($location != "")
+               $sql .= TB_PREF."sales_orders.from_stk_loc ='$location' AND ";
+       $sql .= TB_PREF."sales_order_details.stk_code = '$stockid'";
 
     $TransResult = db_query($sql,"No transactions were returned");
        $DemandRow = db_fetch($TransResult);
@@ -73,9 +73,10 @@ function getCustAsmQty($stockid, $location)
                                                ".TB_PREF."bom,
                                                ".TB_PREF."stock_master
                                   WHERE ".TB_PREF."sales_order_details.stk_code=".TB_PREF."bom.parent AND
-                                  ".TB_PREF."sales_orders.order_no = ".TB_PREF."sales_order_details.order_no AND
-                                  ".TB_PREF."sales_orders.from_stk_loc='$location' AND
-                                  ".TB_PREF."sales_order_details.quantity-".TB_PREF."sales_order_details.qty_sent > 0 AND
+                                  ".TB_PREF."sales_orders.order_no = ".TB_PREF."sales_order_details.order_no AND ";
+       if ($location != "")
+               $sql .= TB_PREF."sales_orders.from_stk_loc ='$location' AND ";
+       $sql .= TB_PREF."sales_order_details.quantity-".TB_PREF."sales_order_details.qty_sent > 0 AND
                                   ".TB_PREF."bom.component='$stockid' AND
                                   ".TB_PREF."stock_master.stock_id=".TB_PREF."bom.parent AND
                                   ".TB_PREF."stock_master.mb_flag='A'";
@@ -98,8 +99,9 @@ function getSuppQty($stockid, $location)
                                FROM ".TB_PREF."purch_order_details,
                                        ".TB_PREF."purch_orders
                                WHERE ".TB_PREF."purch_order_details.order_no = ".TB_PREF."purch_orders.order_no
-                               AND ".TB_PREF."purch_order_details.item_code = '$stockid'
-                               AND ".TB_PREF."purch_orders.into_stock_location= '$location'";
+                               AND ".TB_PREF."purch_order_details.item_code = '$stockid'";
+       if ($location != "")                    
+               $sql .= " AND ".TB_PREF."purch_orders.into_stock_location= '$location'";
 
     $TransResult = db_query($sql,"No transactions were returned");
        $DemandRow = db_fetch($TransResult);
@@ -196,10 +198,13 @@ function print_inventory_planning()
                        $catt = $trans['cat_description'];
                        $rep->NewLine();
                }
-
-               $custqty = getCustQty($trans['stock_id'], $trans['loc_code']);
-               $custqty += getCustAsmQty($trans['stock_id'], $trans['loc_code']);
-               $suppqty = getSuppQty($trans['stock_id'], $trans['loc_code']);
+               if ($location == 'all')
+                       $loc_code = "";
+               else
+                       $loc_code = $trans['loc_code'];
+               $custqty = getCustQty($trans['stock_id'], $loc_code);
+               $custqty += getCustAsmQty($trans['stock_id'], $loc_code);
+               $suppqty = getSuppQty($trans['stock_id'], $loc_code);
                $period = getPeriods($trans['stock_id'], $trans['loc_code']);
                $rep->NewLine();
                $dec = get_qty_dec($trans['stock_id']);

diff --git a/reporting/rep303.php b/reporting/rep303.php
index c984b3cb2fe679b4da45b345a98e1204f6e7f325..bc04f7694e6fba1afdf97e2f90fe2af2b93897a1 100644 (file)
--- a/reporting/rep303.php
+++ b/reporting/rep303.php
@@ -26,13 +26,13 @@ function getTransactions($category, $location)
                        ".TB_PREF."stock_category.description AS cat_description,
                        ".TB_PREF."stock_master.stock_id,
                        ".TB_PREF."stock_master.description,
-                       ".TB_PREF."stock_moves.loc_code,
-                       SUM(".TB_PREF."stock_moves.qty) AS QtyOnHand
-               FROM ".TB_PREF."stock_master,
-                       ".TB_PREF."stock_category,
-                       ".TB_PREF."stock_moves
-               WHERE ".TB_PREF."stock_master.stock_id=".TB_PREF."stock_moves.stock_id
-               AND ".TB_PREF."stock_master.category_id=".TB_PREF."stock_category.category_id
+                       IF(".TB_PREF."stock_moves.stock_id IS NULL, '', ".TB_PREF."stock_moves.loc_code) AS loc_code,
+                       SUM(IF(".TB_PREF."stock_moves.stock_id IS NULL,0,".TB_PREF."stock_moves.qty)) AS QtyOnHand
+               FROM (".TB_PREF."stock_master,
+                       ".TB_PREF."stock_category)
+               LEFT JOIN ".TB_PREF."stock_moves ON
+                       (".TB_PREF."stock_master.stock_id=".TB_PREF."stock_moves.stock_id OR ".TB_PREF."stock_master.stock_id IS NULL)
+               WHERE ".TB_PREF."stock_master.category_id=".TB_PREF."stock_category.category_id
                AND (".TB_PREF."stock_master.mb_flag='B' OR ".TB_PREF."stock_master.mb_flag='M')";
        if ($category != 0)
                $sql .= " AND ".TB_PREF."stock_master.category_id = '$category'";
@@ -53,9 +53,10 @@ function getDemandQty($stockid, $location)
        $sql = "SELECT SUM(".TB_PREF."sales_order_details.quantity - ".TB_PREF."sales_order_details.qty_sent) AS QtyDemand
                                FROM ".TB_PREF."sales_order_details,
                                        ".TB_PREF."sales_orders
-                               WHERE ".TB_PREF."sales_order_details.order_no=".TB_PREF."sales_orders.order_no AND
-                                       ".TB_PREF."sales_orders.from_stk_loc ='$location' AND
-                                       ".TB_PREF."sales_order_details.stk_code = '$stockid'";
+                               WHERE ".TB_PREF."sales_order_details.order_no=".TB_PREF."sales_orders.order_no AND ";
+       if ($location != "")
+               $sql .= TB_PREF."sales_orders.from_stk_loc ='$location' AND ";
+       $sql .= TB_PREF."sales_order_details.stk_code = '$stockid'";
 
     $TransResult = db_query($sql,"No transactions were returned");
        $DemandRow = db_fetch($TransResult);
@@ -71,9 +72,10 @@ function getDemandAsmQty($stockid, $location)
                                                ".TB_PREF."bom,
                                                ".TB_PREF."stock_master
                                   WHERE ".TB_PREF."sales_order_details.stk_code=".TB_PREF."bom.parent AND
-                                  ".TB_PREF."sales_orders.order_no = ".TB_PREF."sales_order_details.order_no AND
-                                  ".TB_PREF."sales_orders.from_stk_loc='$location' AND
-                                  ".TB_PREF."sales_order_details.quantity-".TB_PREF."sales_order_details.qty_sent > 0 AND
+                                  ".TB_PREF."sales_orders.order_no = ".TB_PREF."sales_order_details.order_no AND ";
+       if ($location != "")
+               $sql .= TB_PREF."sales_orders.from_stk_loc ='$location' AND ";
+       $sql .= TB_PREF."sales_order_details.quantity-".TB_PREF."sales_order_details.qty_sent > 0 AND
                                   ".TB_PREF."bom.component='$stockid' AND
                                   ".TB_PREF."stock_master.stock_id=".TB_PREF."bom.parent AND
                                   ".TB_PREF."stock_master.mb_flag='A'";
@@ -163,8 +165,12 @@ function print_stock_check()
                        $catt = $trans['cat_description'];
                        $rep->NewLine();
                }
-               $demandqty = getDemandQty($trans['stock_id'], $trans['loc_code']);
-               $demandqty += getDemandAsmQty($trans['stock_id'], $trans['loc_code']);
+               if ($location == 'all')
+                       $loc_code = "";
+               else
+                       $loc_code = $trans['loc_code'];
+               $demandqty = getDemandQty($trans['stock_id'], $loc_code);
+               $demandqty += getDemandAsmQty($trans['stock_id'], $loc_code);
                $rep->NewLine();
                $dec = get_qty_dec($trans['stock_id']);
                $rep->TextCol(0, 1, $trans['stock_id']);