2.4.4 no Longer Displays 0.00 Sales Invoices in customer_inquiry.php. Fixed. $Braath...
[fa-stable.git] / sales / includes / db / cust_trans_db.inc
index c9fccf11522d61421f37cfa8943edcf13f52119e..888181ebae9b811a4b04c863364e5a488d8879ad 100644 (file)
@@ -9,38 +9,6 @@
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  
     See the License here <http://www.gnu.org/licenses/gpl-3.0.html>.
 ***********************************************************************/
-//------------------------------------------------------------------------------
-//     Retreive parent document number(s) for given transaction
-//
-function get_parent_trans($trans_type, $trans_no) {
-
-       $sql = 'SELECT trans_link FROM
-                       '.TB_PREF.'debtor_trans WHERE
-                       (trans_no='.db_escape($trans_no).' AND type='.db_escape($trans_type).' AND trans_link!=0)';
-
-       $result = db_query($sql, 'Parent document numbers cannot be retrieved');
-
-       if (db_num_rows($result)) {
-               $link = db_fetch($result);
-               return array($link['trans_link']);
-       }
-       if ($trans_type!=ST_SALESINVOICE) return 0;     // this is credit note with no parent invoice
-       // invoice: find batch invoice parent trans.
-       $sql = 'SELECT trans_no FROM
-                       '.TB_PREF.'debtor_trans WHERE
-                       (trans_link='.db_escape($trans_no).' AND type='. get_parent_type($trans_type) .')';
-
-       $result = db_query($sql, 'Delivery links cannot be retrieved');
-
-       $delivery = array();
-       if(db_num_rows($result)>0) {
-               while($link = db_fetch($result)) {
-                       $delivery[] = $link['trans_no'];
-               }
-       }
-       return count($delivery) ? $delivery : 0;
-}
-
 //----------------------------------------------------------------------------------------
 // Mark changes in debtor_trans_details
 //
@@ -70,7 +38,7 @@ function get_customer_trans_version($type, $trans_no) {
                        WHERE type='.db_escape($type).' AND (';
 
        foreach ($trans_no as $key=>$trans)
-               $trans_no[$key] =       'trans_no='.$trans_no[$key];
+               $trans_no[$key] =       'trans_no='.db_escape($trans_no[$key]);
 
        $sql .= implode(' OR ', $trans_no) . ')';
 
@@ -87,8 +55,8 @@ function get_customer_trans_version($type, $trans_no) {
 // date_ is display date (non-sql)
 function write_customer_trans($trans_type, $trans_no, $debtor_no, $BranchNo,
        $date_, $reference, $Total, $discount=0, $Tax=0, $Freight=0, $FreightTax=0,
-       $sales_type=0, $order_no=0, $trans_link=0, $ship_via=0, $due_date="",
-       $AllocAmt=0, $rate=0, $dimension_id=0, $dimension2_id=0, $payment_terms=null)
+       $sales_type=0, $order_no=0, $ship_via=0, $due_date="",
+       $AllocAmt=0, $rate=0, $dimension_id=0, $dimension2_id=0, $payment_terms=null, $tax_included=0, $prep_amount=0)
 {
        $new = $trans_no==0;
        $curr = get_customer_currency($debtor_no);
@@ -104,8 +72,10 @@ function write_customer_trans($trans_type, $trans_no, $debtor_no, $BranchNo,
        if ($trans_type == ST_BANKPAYMENT)
                $Total = -$Total;
 
-       if ($new) {
-       $trans_no = get_next_trans_no($trans_type);
+    if ($new || !exists_customer_trans($trans_type, $trans_no))
+    {
+        if ($new)
+        $trans_no = get_next_trans_no($trans_type);
 
        $sql = "INSERT INTO ".TB_PREF."debtor_trans (
                trans_no, type,
@@ -114,16 +84,17 @@ function write_customer_trans($trans_type, $trans_no, $debtor_no, $BranchNo,
                reference, tpe,
                order_, ov_amount, ov_discount,
                ov_gst, ov_freight, ov_freight_tax,
-               rate, ship_via, alloc, trans_link,
-               dimension_id, dimension2_id, payment_terms
-               ) VALUES ($trans_no, ".db_escape($trans_type).",
+               rate, ship_via, alloc,
+               dimension_id, dimension2_id, payment_terms, tax_included, prep_amount
+               ) VALUES (".db_escape($trans_no).", ".db_escape($trans_type).",
                ".db_escape($debtor_no).", ".db_escape($BranchNo).",
                '$SQLDate', '$SQLDueDate', ".db_escape($reference).",
                ".db_escape($sales_type).", ".db_escape($order_no).", $Total, ".db_escape($discount).", $Tax,
                ".db_escape($Freight).",
-               $FreightTax, $rate, ".db_escape($ship_via).", $AllocAmt, ".db_escape($trans_link).",
+               $FreightTax, $rate, ".db_escape($ship_via).", $AllocAmt,
                ".db_escape($dimension_id).", ".db_escape($dimension2_id).", "
-               .db_escape($payment_terms).")";
+               .db_escape($payment_terms, true).", "
+               .db_escape($tax_included).", ".db_escape($prep_amount).")";
        } else {        // may be optional argument should stay unchanged ?
        $sql = "UPDATE ".TB_PREF."debtor_trans SET
                debtor_no=".db_escape($debtor_no)." , branch_code=".db_escape($BranchNo).",
@@ -131,32 +102,40 @@ function write_customer_trans($trans_type, $trans_no, $debtor_no, $BranchNo,
                reference=".db_escape($reference).", tpe=".db_escape($sales_type).", order_=".db_escape($order_no).",
                ov_amount=$Total, ov_discount=".db_escape($discount).", ov_gst=$Tax,
                ov_freight=".db_escape($Freight).", ov_freight_tax=$FreightTax, rate=$rate,
-               ship_via=".db_escape($ship_via).", alloc=$AllocAmt, trans_link=$trans_link,
+               ship_via=".db_escape($ship_via).", alloc=$AllocAmt,
                dimension_id=".db_escape($dimension_id).", dimension2_id=".db_escape($dimension2_id).",
-               payment_terms=".db_escape($payment_terms)."
-               WHERE trans_no=$trans_no AND type=".db_escape($trans_type);
+               payment_terms=".db_escape($payment_terms, true).",
+               tax_included=".db_escape($tax_included).",
+               prep_amount =".db_escape($prep_amount)."
+               WHERE trans_no=".db_escape($trans_no)." AND type=".db_escape($trans_type);
        }
        db_query($sql, "The debtor transaction record could not be inserted");
 
-       add_audit_trail($trans_type, $trans_no, $date_, $new ? '': _("Updated."));
+       if ($trans_type != ST_JOURNAL) // skip for journal entries
+               add_audit_trail($trans_type, $trans_no, $date_, $new ? '': _("Updated."));
 
        return $trans_no;
 }
-
 //----------------------------------------------------------------------------------------
 
-function get_customer_trans($trans_id, $trans_type)
+function get_customer_trans($trans_id, $trans_type, $customer_id=null)
 {
+       global $SysPrefs;
+
        $sql = "SELECT trans.*,"
                ."ov_amount+ov_gst+ov_freight+ov_freight_tax+ov_discount AS Total,"
                ."cust.name AS DebtorName, cust.address, "
-               ."cust.email AS email2, cust.curr_code, "
-               ."cust.tax_id ";
+               ."cust.curr_code, "
+               ."cust.tax_id,
+               trans.prep_amount>0 as prepaid,"
+               ."com.memo_";
 
-       if ($trans_type == ST_CUSTPAYMENT) {
+       if ($trans_type == ST_CUSTPAYMENT || $trans_type == ST_BANKDEPOSIT) {
                // it's a payment so also get the bank account
-               $sql .= ", ".TB_PREF."bank_accounts.bank_name, ".TB_PREF."bank_accounts.bank_account_name,
-                       ".TB_PREF."bank_accounts.account_type AS BankTransType ";
+               $sql .= ",bank_act,".TB_PREF."bank_accounts.bank_name, ".TB_PREF."bank_accounts.bank_account_name,
+                       ".TB_PREF."bank_accounts.account_type AS BankTransType,
+                       ".TB_PREF."bank_accounts.bank_curr_code,
+                       ".TB_PREF."bank_trans.amount as bank_amount";
        }
 
        if ($trans_type == ST_SALESINVOICE || $trans_type == ST_CUSTCREDIT || $trans_type == ST_CUSTDELIVERY) {
@@ -170,57 +149,73 @@ function get_customer_trans($trans_id, $trans_type)
                .TB_PREF."tax_groups.id AS tax_group_id ";
        }
 
-       $sql .= ", IF(ISNULL(branch.rep_lang),cust.rep_lang,branch.rep_lang) AS rep_lang";
+       if ($trans_type == ST_JOURNAL) {
+               $sql .= ", branch.*";
+       }
 
-       $sql .= " FROM ".TB_PREF."debtor_trans trans, ".TB_PREF."debtors_master cust";
+       $sql .= " FROM ".TB_PREF."debtor_trans trans
+                                       LEFT JOIN ".TB_PREF."comments com ON trans.type=com.type AND trans.trans_no=com.id
+                                       LEFT JOIN ".TB_PREF."shippers ON ".TB_PREF."shippers.shipper_id=trans.ship_via, 
+                                       ".TB_PREF."debtors_master cust";
 
-       if ($trans_type == ST_CUSTPAYMENT) {
+       if ($trans_type == ST_CUSTPAYMENT || $trans_type == ST_BANKDEPOSIT) {
                // it's a payment so also get the bank account
                $sql .= ", ".TB_PREF."bank_trans, ".TB_PREF."bank_accounts";
        }
 
        if ($trans_type == ST_SALESINVOICE || $trans_type == ST_CUSTCREDIT || $trans_type == ST_CUSTDELIVERY) {
                // it's an invoice so also get the shipper, salestypes
-               $sql .= ", ".TB_PREF."shippers, "
-               .TB_PREF."sales_types, "
+               $sql .= ", ".TB_PREF."sales_types, "
                .TB_PREF."cust_branch branch, "
                .TB_PREF."tax_groups ";
        }
 
+       if ($trans_type == ST_JOURNAL) {
+               $sql .= ", ".TB_PREF."cust_branch branch ";
+       }
+
        $sql .= " WHERE trans.trans_no=".db_escape($trans_id)."
                AND trans.type=".db_escape($trans_type)."
                AND trans.debtor_no=cust.debtor_no";
 
-       if ($trans_type == ST_CUSTPAYMENT) {
+       if (isset($customer_id))
+               $sql .= " AND trans.debtor_no=".db_escape($customer_id);
+
+       if ($trans_type == ST_CUSTPAYMENT || $trans_type == ST_BANKDEPOSIT) {
                // it's a payment so also get the bank account
-               $sql .= " AND ".TB_PREF."bank_trans.trans_no =$trans_id
+               $sql .= " AND ".TB_PREF."bank_trans.trans_no =".db_escape($trans_id)."
                        AND ".TB_PREF."bank_trans.type=$trans_type
+                       AND ".TB_PREF."bank_trans.amount != 0
                        AND ".TB_PREF."bank_accounts.id=".TB_PREF."bank_trans.bank_act ";
        }
        if ($trans_type == ST_SALESINVOICE || $trans_type == ST_CUSTCREDIT || $trans_type == ST_CUSTDELIVERY) {
                // it's an invoice so also get the shipper
-               $sql .= " AND ".TB_PREF."shippers.shipper_id=trans.ship_via
-                       AND ".TB_PREF."sales_types.id = trans.tpe
+               $sql .= " AND ".TB_PREF."sales_types.id = trans.tpe
                        AND branch.branch_code = trans.branch_code
                        AND branch.tax_group_id = ".TB_PREF."tax_groups.id ";
        }
+       if ($trans_type == ST_JOURNAL) {
+               $sql .= " AND branch.branch_code = trans.branch_code ";
+       }
 
        $result = db_query($sql, "Cannot retreive a debtor transaction");
 
        if (db_num_rows($result) == 0) {
                // can't return nothing
+               if ($SysPrefs->go_debug)
+                       display_backtrace();
                display_db_error("no debtor trans found for given params", $sql, true);
                exit;
        }
-
        if (db_num_rows($result) > 1) {
                // can't return multiple
+               if($SysPrefs->go_debug)
+                       display_backtrace();
                display_db_error("duplicate debtor transactions found for given params", $sql, true);
                exit;
        }
 
        $row = db_fetch($result);
-       $row['email'] = $row['email2'];
        return $row;
 }
 
@@ -253,20 +248,16 @@ function get_customer_trans_order($type, $type_no)
 
 //----------------------------------------------------------------------------------------
 
-function get_related_documents($type, $trans_no)
-{
-       $sql = "SELECT * FROM ".TB_PREF."debtor_trans WHERE type=$type AND order_=".db_escape($trans_no);
-       return db_query($sql,"The related documents could not be retreived");
-}
-//----------------------------------------------------------------------------------------
-
 function get_customer_details_from_trans($type, $type_no)
 {
-       $sql = "SELECT ".TB_PREF."debtors_master.name, ".TB_PREF."debtors_master.curr_code, ".TB_PREF."cust_branch.br_name
-               FROM ".TB_PREF."debtors_master,".TB_PREF."cust_branch,".TB_PREF."debtor_trans
-               WHERE ".TB_PREF."debtor_trans.type=".db_escape($type)." AND ".TB_PREF."debtor_trans.trans_no=".db_escape($type_no)."
-               AND ".TB_PREF."debtors_master.debtor_no = ".TB_PREF."debtor_trans.debtor_no
-               AND     ".TB_PREF."cust_branch.branch_code = ".TB_PREF."debtor_trans.branch_code";
+       $sql = "SELECT debtor.name, debtor.curr_code, branch.br_name
+               FROM "
+                       .TB_PREF."debtors_master debtor,"
+                       .TB_PREF."cust_branch branch,"
+                       .TB_PREF."debtor_trans trans
+               WHERE trans.type=".db_escape($type)." AND trans.trans_no=".db_escape($type_no)."
+               AND debtor.debtor_no = trans.debtor_no
+               AND     branch.branch_code = trans.branch_code";
 
        $result = db_query($sql, "could not get customer details from trans");
        return db_fetch($result);
@@ -278,13 +269,22 @@ function void_customer_trans($type, $type_no)
 {
        // clear all values and mark as void
        $sql = "UPDATE ".TB_PREF."debtor_trans SET ov_amount=0, ov_discount=0, ov_gst=0, ov_freight=0,
-               ov_freight_tax=0, alloc=0, version=version+1 WHERE type=".db_escape($type)." AND trans_no=".db_escape($type_no);
+               ov_freight_tax=0, alloc=0, prep_amount=0, version=version+1 WHERE type=".db_escape($type)." AND trans_no=".db_escape($type_no);
 
        db_query($sql, "could not void debtor transactions for type=$type and trans_no=$type_no");
 }
 
 //----------------------------------------------------------------------------------------
 
+function clear_customer_trans($type, $type_no)
+{
+       // Delete
+       $sql = "DELETE FROM ".TB_PREF."debtor_trans WHERE type=".db_escape($type)." AND trans_no=".db_escape($type_no);
+
+       db_query($sql, "could not clear debtor transactions for type=$type and trans_no=$type_no");
+}
+//----------------------------------------------------------------------------------------
+
 function post_void_customer_trans($type, $type_no)
 {
        switch ($type) {
@@ -302,20 +302,10 @@ function post_void_customer_trans($type, $type_no)
 }
 
 //----------------------------------------------------------------------------------------
-
-function get_customer_trans_link($type, $type_no)
-{
-       $row = db_query("SELECT trans_link from ".TB_PREF."debtor_trans
-               WHERE type=".db_escape($type)." AND trans_no=".db_escape($type_no),
-               "could not get transaction link for type=$type and trans_no=$type_no");
-       return $row[0];
-}
-//----------------------------------------------------------------------------------------
-
-function get_sql_for_customer_inquiry()
+function get_sql_for_customer_inquiry($from, $to, $cust_id = ALL_TEXT, $filter = ALL_TEXT, $show_voided = 0)
 {
-    $date_after = date2sql($_POST['TransAfterDate']);
-    $date_to = date2sql($_POST['TransToDate']);
+    $date_after = date2sql($from);
+    $date_to = date2sql($to);
 
        $sql = "SELECT 
                trans.type, 
@@ -327,55 +317,76 @@ function get_sql_for_customer_inquiry()
                debtor.name, 
                branch.br_name,
                debtor.curr_code,
-               (trans.ov_amount + trans.ov_gst + trans.ov_freight 
-                       + trans.ov_freight_tax + trans.ov_discount)     AS TotalAmount, "; 
-       if ($_POST['filterType'] != ALL_TEXT)
-               $sql .= "@bal := @bal+(trans.ov_amount + trans.ov_gst + trans.ov_freight + trans.ov_freight_tax + trans.ov_discount), ";
+               IF(prep_amount, prep_amount, trans.ov_amount + trans.ov_gst + trans.ov_freight 
+                       + trans.ov_freight_tax + trans.ov_discount)     AS TotalAmount,"
+//     if ($filter != ALL_TEXT)
+//             $sql .= "@bal := @bal+IF(prep_amount, prep_amount, trans.ov_amount + trans.ov_gst + trans.ov_freight + trans.ov_freight_tax + trans.ov_discount), ";
 
 //     else
 //             $sql .= "IF(trans.type=".ST_CUSTDELIVERY.",'', IF(trans.type=".ST_SALESINVOICE." OR trans.type=".ST_BANKPAYMENT.",@bal := @bal+
 //                     (trans.ov_amount + trans.ov_gst + trans.ov_freight + trans.ov_freight_tax + trans.ov_discount), @bal := @bal-
 //                     (trans.ov_amount + trans.ov_gst + trans.ov_freight + trans.ov_freight_tax + trans.ov_discount))) , ";
+               . "IF(trans.type IN(".implode(',',  array(ST_CUSTCREDIT,ST_CUSTPAYMENT,ST_BANKDEPOSIT))."), -1, 1)
+                               *(IF(prep_amount, prep_amount, trans.ov_amount + trans.ov_gst + trans.ov_freight 
+                       + trans.ov_freight_tax + trans.ov_discount)-trans.alloc) Balance, 
+               debtor.debtor_no,";
+
                $sql .= "trans.alloc AS Allocated,
-               ((trans.type = ".ST_SALESINVOICE.")
-                       AND trans.due_date < '" . date2sql(Today()) . "') AS OverDue
+               ((trans.type = ".ST_SALESINVOICE." || trans.type = ".ST_JOURNAL.")
+                       AND trans.due_date < '" . date2sql(Today()) . "') AS OverDue ,
+               Sum(line.quantity-line.qty_done) AS Outstanding,
+               Sum(line.qty_done) AS HasChild,
+               prep_amount
                FROM "
-                       .TB_PREF."debtor_trans as trans, "
-                       .TB_PREF."debtors_master as debtor, "
-                       .TB_PREF."cust_branch as branch
-               WHERE debtor.debtor_no = trans.debtor_no
-                       AND trans.tran_date >= '$date_after'
-                       AND trans.tran_date <= '$date_to'
-                       AND trans.branch_code = branch.branch_code";
-
-       if ($_POST['customer_id'] != ALL_TEXT)
-               $sql .= " AND trans.debtor_no = ".db_escape($_POST['customer_id']);
-
-       if ($_POST['filterType'] != ALL_TEXT)
+                       .TB_PREF."debtor_trans as trans
+                       LEFT JOIN ".TB_PREF."debtor_trans_details as line
+                               ON trans.trans_no=line.debtor_trans_no AND trans.type=line.debtor_trans_type
+                       LEFT JOIN ".TB_PREF."voided as v
+                               ON trans.trans_no=v.id AND trans.type=v.type
+                        LEFT JOIN ".TB_PREF."audit_trail as audit ON (trans.type=audit.type AND trans.trans_no=audit.trans_no)
+                        LEFT JOIN ".TB_PREF."users as user ON (audit.user=user.id)
+                       LEFT JOIN ".TB_PREF."cust_branch as branch ON trans.branch_code=branch.branch_code,"
+                       .TB_PREF."debtors_master as debtor
+               WHERE debtor.debtor_no = trans.debtor_no"; // exclude voided transactions and self-balancing (journal) transactions:
+    if (!$show_voided)         
+               $sql .= " AND ISNULL(v.date_) AND (trans.ov_amount + trans.ov_gst + trans.ov_freight + trans.ov_freight_tax + trans.ov_discount) != 0";
+
+       if ($filter == '2')
+               $sql .= " AND ABS(IF(prep_amount, prep_amount, trans.ov_amount + trans.ov_gst + trans.ov_freight + trans.ov_freight_tax + trans.ov_discount)-trans.alloc)>"
+                       .FLOAT_COMP_DELTA;
+       else {
+               $sql .= " AND trans.tran_date >= '$date_after'
+                       AND trans.tran_date <= '$date_to'";
+       }
+
+       if ($cust_id != ALL_TEXT)
+               $sql .= " AND trans.debtor_no = ".db_escape($cust_id);
+
+       if ($filter != ALL_TEXT)
        {
-               if ($_POST['filterType'] == '1')
+               if ($filter == '1')
                {
-                       $sql .= " AND (trans.type = ".ST_SALESINVOICE." OR trans.type = ".ST_BANKPAYMENT.") ";
+                       $sql .= " AND (trans.type = ".ST_SALESINVOICE.") ";
                }
-               elseif ($_POST['filterType'] == '2')
+               elseif ($filter == '2')
                {
-                       $sql .= " AND (trans.type = ".ST_SALESINVOICE.") ";
+                       $sql .= " AND (trans.type <> ".ST_CUSTDELIVERY.") ";
                }
-               elseif ($_POST['filterType'] == '3')
+               elseif ($filter == '3')
                {
                        $sql .= " AND (trans.type = " . ST_CUSTPAYMENT 
-                                       ." OR trans.type = ".ST_BANKDEPOSIT.") ";
+                                       ." OR trans.type = ".ST_BANKDEPOSIT." OR trans.type = ".ST_BANKPAYMENT.") ";
                }
-               elseif ($_POST['filterType'] == '4')
+               elseif ($filter == '4')
                {
                        $sql .= " AND trans.type = ".ST_CUSTCREDIT." ";
                }
-               elseif ($_POST['filterType'] == '5')
+               elseif ($filter == '5')
                {
                        $sql .= " AND trans.type = ".ST_CUSTDELIVERY." ";
                }
 
-       if ($_POST['filterType'] == '2')
+       if ($filter == '2')
        {
                $today =  date2sql(Today());
                $sql .= " AND trans.due_date < '$today'
@@ -383,10 +394,13 @@ function get_sql_for_customer_inquiry()
                                trans.ov_freight + trans.ov_discount - trans.alloc > 0) ";
        }
        }
+
+       $sql .= " GROUP BY trans.trans_no, trans.type";
+
        return $sql;
 }
 
-function get_sql_for_sales_deliveries_view($selected_customer, $selected_stock_item=null)
+function get_sql_for_sales_deliveries_view($from, $to, $customer_id, $stock_item, $location, $delivery, $outstanding=false)
 {
        $sql = "SELECT trans.trans_no,
                        debtor.name,
@@ -399,7 +413,7 @@ function get_sql_for_sales_deliveries_view($selected_customer, $selected_stock_i
                        trans.due_date,
                        (ov_amount+ov_gst+ov_freight+ov_freight_tax) AS DeliveryValue,
                        debtor.curr_code,
-                       Sum(line.quantity-line.qty_done) AS Outstanding,
+                       Sum(line.quantity-line.qty_done) AND sorder.prep_amount=0 AS Outstanding,
                        Sum(line.qty_done) AS Done
                FROM "
                 .TB_PREF."sales_orders as sorder, "
@@ -416,34 +430,32 @@ function get_sql_for_sales_deliveries_view($selected_customer, $selected_stock_i
                                AND trans.branch_code = branch.branch_code
                                AND trans.debtor_no = branch.debtor_no ";
 
-       if ($_POST['OutstandingOnly'] == true) {
+       if ($outstanding == true) {
                 $sql .= " AND line.qty_done < line.quantity ";
        }
 
        //figure out the sql required from the inputs available
-       if (isset($_POST['DeliveryNumber']) && $_POST['DeliveryNumber'] != "")
+       if ($delivery)
        {
-               $delivery = "%".$_POST['DeliveryNumber'];
-               $sql .= " AND trans.trans_no LIKE ".db_escape($delivery);
+               $sql .= " AND trans.trans_no LIKE ".db_escape('%' . $delivery . '%');
                $sql .= " GROUP BY trans.trans_no";
        }
        else
        {
-               $sql .= " AND trans.tran_date >= '".date2sql($_POST['DeliveryAfterDate'])."'";
-               $sql .= " AND trans.tran_date <= '".date2sql($_POST['DeliveryToDate'])."'";
+               $sql .= " AND trans.tran_date >= '".date2sql($from)."'";
+               $sql .= " AND trans.tran_date <= '".date2sql($to)."'";
 
-               if ($selected_customer != -1)
-                       $sql .= " AND trans.debtor_no=".db_escape($selected_customer)." ";
+               if ($stock_item != ALL_TEXT)
+                       $sql .= " AND line.stock_id=".db_escape($stock_item)." ";
 
-               if (isset($selected_stock_item))
-                       $sql .= " AND line.stock_id=".db_escape($selected_stock_item)." ";
+               if ($location != ALL_TEXT)
+                       $sql .= " AND sorder.from_stk_loc = ".db_escape($location)." ";
 
-               if (isset($_POST['StockLocation']) && $_POST['StockLocation'] != ALL_TEXT)
-                       $sql .= " AND sorder.from_stk_loc = ".db_escape($_POST['StockLocation'])." ";
+               if ($customer_id != ALL_TEXT)
+                       $sql .= " AND trans.debtor_no = ".db_escape($customer_id);
 
                $sql .= " GROUP BY trans.trans_no ";
 
        } //end no delivery number selected
        return $sql;
 }
-?>
\ No newline at end of file