Merged changes from main CVS up to 2.1.5
[fa-stable.git] / sales / customer_delivery.php
index c7ba4b6183df4eb194e91aaed9d81ede66f365f5..23cbd79a10a2ed31f3bf1478eb9ee29660932012 100644 (file)
 <?php
-
+/**********************************************************************
+    Copyright (C) FrontAccounting, LLC.
+       Released under the terms of the GNU General Public License, GPL, 
+       as published by the Free Software Foundation, either version 3 
+       of the License, or (at your option) any later version.
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  
+    See the License here <http://www.gnu.org/licenses/gpl-3.0.html>.
+***********************************************************************/
+//-----------------------------------------------------------------------------
+//
+//     Entry/Modify Delivery Note against Sales Order
+//
 $page_security = 2;
 $path_to_root="..";
+
 include_once($path_to_root . "/sales/includes/cart_class.inc");
 include_once($path_to_root . "/includes/session.inc");
-
 include_once($path_to_root . "/includes/data_checks.inc");
-
 include_once($path_to_root . "/includes/manufacturing.inc");
 include_once($path_to_root . "/sales/includes/sales_db.inc");
 include_once($path_to_root . "/sales/includes/sales_ui.inc");
-
+include_once($path_to_root . "/reporting/includes/reporting.inc");
 include_once($path_to_root . "/taxes/tax_calc.inc");
 
 $js = "";
-if ($use_popup_windows)
+if ($use_popup_windows) {
        $js .= get_js_open_window(900, 500);
-if ($use_date_picker)
+}
+if ($use_date_picker) {
        $js .= get_js_date_picker();
-
-if(isset($_GET['ModifyDelivery'])) {
-       $_SESSION['page_title'] = _("Modifying Delivery Note") . " #".$_GET['ModifyDelivery']; 
 }
-else { 
+
+if (isset($_GET['ModifyDelivery'])) {
+       $_SESSION['page_title'] = sprintf(_("Modifying Delivery Note # %d."), $_GET['ModifyDelivery']);
+       $help_page_title = _("Modifying Delivery Note");
+       processing_start();
+} elseif (isset($_GET['OrderNumber'])) {
        $_SESSION['page_title'] = _("Deliver Items for a Sales Order");
+       processing_start();
 }
 
-page($_SESSION['page_title'], false, false, "", $js); 
+page($_SESSION['page_title'], false, false, "", $js);
 
-if (isset($_GET['AddedID'])) 
-{
+if (isset($_GET['AddedID'])) {
        $dispatch_no = $_GET['AddedID'];
-       $trans_type = 13;
 
-       display_notification(_("Dispatch processed:") . ' '.$_GET['AddedID'], true);
+       display_notification_centered(sprintf(_("Delivery # %d has been entered."),$dispatch_no));
 
-       display_note(get_customer_trans_view_str($trans_type, $dispatch_no, _("View this dispatch")), 0, 1);
+       display_note(get_customer_trans_view_str(13, $dispatch_no, _("&View This Delivery")), 0, 1);
 
-       display_note(get_gl_view_str($trans_type, $dispatch_no, _("View the GL Journal Entries for this Dispatch")));
+       display_note(print_document_link($dispatch_no, _("&Print Delivery Note"), true, 13));
+       display_note(print_document_link($dispatch_no, _("&Email Delivery Note"), true, 13, false, "", "", 1),0, 1);
+
+       display_note(get_gl_view_str(13, $dispatch_no, _("View the GL Journal Entries for this Dispatch")),1);
 
        hyperlink_params("$path_to_root/sales/customer_invoice.php", _("Invoice This Delivery"), "DeliveryNumber=$dispatch_no");
 
-       if ($_SESSION['Items']->trans_type=='invoice')
-               hyperlink_params("$path_to_root/sales/sales_order_entry.php", _("Make Another Dispatch"), "NewDispatch=Yes");
-       else
-               hyperlink_params("$path_to_root/sales/inquiry/sales_orders_view.php", _("Select Another Order For Dispatch"), "OutstandingOnly=1");
+       hyperlink_params("$path_to_root/sales/inquiry/sales_orders_view.php", _("Select Another Order For Dispatch"), "OutstandingOnly=1");
 
        display_footer_exit();
-}
 
-//---------------------------------------------------------------------------------------------------------------
+} elseif (isset($_GET['UpdatedID'])) {
 
-if (!isset($_GET['OrderNumber']) && !isset($_SESSION['ProcessingOrder']) && 
- !isset($_GET['ModifyDelivery']) && !isset($_GET['process_delivery'])) 
-{
-       /* This page can only be called with an order number for invoicing*/
-       display_error(_("This page can only be opened if an order or delivery note has been selected. Please select it first."));
+       $delivery_no = $_GET['UpdatedID'];
 
-       hyperlink_params("$path_to_root/sales/inquiry/sales_orders_view.php", _("Select a Sales Order to Delivery"), "OutstandingOnly=1");
+       display_notification_centered(sprintf(_('Delivery Note # %d has been updated.'),$delivery_no));
 
-       end_page();
-       exit;
+       display_note(get_trans_view_str(13, $delivery_no, _("View this delivery")));
+       echo '<br>';
+       display_note(print_document_link($delivery_no, _("Print this delivery"), true, 13));
 
-} 
-elseif (isset($_GET['OrderNumber']) && $_GET['OrderNumber'] > 0) 
-{
+       hyperlink_params($path_to_root . "/sales/customer_invoice.php", _("Confirm Delivery and Invoice"), "DeliveryNumber=$delivery_no");
 
-       if (isset($_SESSION['Items']))
-       {
-               unset($_SESSION['Items']->line_items);
-               unset ($_SESSION['Items']);
-       }
+       hyperlink_params($path_to_root . "/sales/inquiry/sales_deliveries_view.php", _("Select A Different Delivery"), "OutstandingOnly=1");
+
+       display_footer_exit();
+}
+//-----------------------------------------------------------------------------
+
+if (isset($_GET['OrderNumber']) && $_GET['OrderNumber'] > 0) {
+
+       $ord = new Cart(30, $_GET['OrderNumber'], true);
 
-       $_SESSION['ProcessingOrder'] = $_GET['OrderNumber'];
-       $_SESSION['Items'] = new cart;
-       
        /*read in all the selected order into the Items cart  */
 
-       if (read_sales_order($_SESSION['ProcessingOrder'], $_SESSION['Items'])) 
-       {
-               if ($_SESSION['Items']->count_items() == 0) 
-               {
-               hyperlink_params($path_to_root . "/sales/inquiry/sales_orders_view.php", _("Select a different sales order to delivery"), "OutstandingOnly=1");
-               die ("<br><b>" . _("This order has no items. There is nothing to delivery.") . "</b>");
-               }
-       } 
-       else 
-       {
-               hyperlink_no_params("/sales_orders_view.php", _("Select a sales order to dispatch"));
-               die ("<br><b>" . _("This order items could not be retrieved. Please select another order.") . "</b>");
+       if ($ord->count_items() == 0) {
+               hyperlink_params($path_to_root . "/sales/inquiry/sales_orders_view.php",
+                       _("Select a different sales order to delivery"), "OutstandingOnly=1");
+               die ("<br><b>" . _("This order has no items. There is nothing to delivery.") . "</b>");
        }
 
-} 
-elseif (isset($_GET['ModifyDelivery']) && $_GET['ModifyDelivery'] > 0) {
-       if (isset($_SESSION['Items']))
-       {
-               unset($_SESSION['Items']->line_items);
-               unset ($_SESSION['Items']);
-       }
-       $_SESSION['Items'] = new cart;
-       
-       if(read_sales_delivery($_GET['ModifyDelivery'],$_SESSION['Items'] ))
-       {
-           if ($_SESSION['Items']->count_items() == 0) 
-           {
-               hyperlink_params($path_to_root . "/sales/inquiry/sales_orders_view.php", _("Select a different delivery"), "OutstandingOnly=1");
-               die ("<br><b>" . _("This delivery has all items invoiced. There is nothing to modify.") . "</b>");
-           }
-       } 
-       else 
-       {
-               hyperlink_no_params("/sales_orders_view.php", _("Select a sales order to dispatch"));
-               die ("<br><b>" . _("This sales delivery item could not be retrieved. Please select another delivery.") . "</b>");
+       $ord->trans_type = 13;
+       $ord->src_docs = $ord->trans_no;
+       $ord->order_no = key($ord->trans_no);
+       $ord->trans_no = 0;
+       $ord->reference = references::get_next(13);
+       $ord->document_date = new_doc_date();
+       $_SESSION['Items'] = $ord;
+       copy_from_cart();
+
+} elseif (isset($_GET['ModifyDelivery']) && $_GET['ModifyDelivery'] > 0) {
+
+       $_SESSION['Items'] = new Cart(13,$_GET['ModifyDelivery']);
+
+       if ($_SESSION['Items']->count_items() == 0) {
+               hyperlink_params($path_to_root . "/sales/inquiry/sales_orders_view.php",
+                       _("Select a different delivery"), "OutstandingOnly=1");
+               echo "<br><center><b>" . _("This delivery has all items invoiced. There is nothing to modify.") .
+                       "</center></b>";
+               display_footer_exit();
        }
 
-       $_SESSION['ProcessingOrder'] = $_SESSION['Items']->order_no;
-} else 
-{
- // Update cart delivery quantities/descriptions
-       foreach ($_SESSION['Items']->line_items as $line=>$itm) 
-       {
-         if(isset($_POST['Line'.$line])) {
-               $line_qty = $_POST['Line'.$line];
-               if (is_numeric($line_qty) && $_POST['Line'.$line] <= ($itm->quantity - $itm->qty_done))
-               {
-                       $_SESSION['Items']->line_items[$line]->qty_dispatched = $line_qty;
-               }
-         }
+       copy_from_cart();
        
-         if(isset($_POST['Line'.$line.'Desc'])) {
-               $line_desc = $_POST['Line'.$line.'Desc'];
-               if (strlen($line_desc) > 0) 
-               {
-                       $_SESSION['Items']->line_items[$line]->item_description = $line_desc;
-               }
-         }
-       }
-}
-
-//---------------------------------------------------------------------------------------------------------------
+} elseif ( !processing_active() ) {
+       /* This page can only be called with an order number for invoicing*/
 
-function order_changed_error()
-{
-       global $path_to_root;
-       display_note(_("This order has been changed or invoiced since this delivery was started to be confirmed. Processing halted."), 1, 0);
-       display_note(_("To enter and confirm this dispatch the order must be re-selected and re-read again to update the changes made by the other user."), 1, 0);
+       display_error(_("This page can only be opened if an order or delivery note has been selected. Please select it first."));
 
-       hyperlink_no_params("$path_to_root/sales/inquiry/sales_orders_view.php", _("Select a sales order for confirming deliveries"));
+       hyperlink_params("$path_to_root/sales/inquiry/sales_orders_view.php", _("Select a Sales Order to Delivery"), "OutstandingOnly=1");
 
-       unset($_SESSION['ProcessingOrder']);
+       end_page();
        exit;
-}
-
-//---------------------------------------------------------------------------------------------------------------
-
-function check_order_changed()
-{
-       global $debug;
 
-       /*Now need to check that the order details are the same as they were when
-                       they were read into the Items array.
-       If they've changed then someone else may have dispatch them */
+} else {
+       check_edit_conflicts();
 
-       $sql = "SELECT id, stk_code, quantity, qty_sent FROM ".TB_PREF."sales_order_details WHERE
-               order_no = " . $_SESSION['ProcessingOrder']. " ORDER BY id";
+       if (!check_quantities()) {
+               display_error(_("Selected quantity cannot be less than quantity invoiced nor more than quantity not dispatched on sales order."));
 
-       $result = db_query($sql,"retreive sales order details");
-
-       if (db_num_rows($result) != count($_SESSION['Items']->line_items))
-       {
-
-               /*there should be the same number of items returned from this query as 
-                  count of lines on the delivery notes - if  not       then someone has 
-                  already invoiced or credited some lines */
-       if ($debug == 1)
-       {
-               display_note($sql, 1, 0);
-               display_note("No rows returned by sql:" . db_num_rows($result), 1, 0);
-               display_note("Count of items in the cart " . count($_SESSION['Items']->line_items), 1, 0);
-       }
-
-               return false;
-       }
-       $line=0;
-       while ($myrow = db_fetch($result)) 
-       {
-               $stk_itm = $myrow["stk_code"];
-               if ($_SESSION['Items']->line_items[$line]->quantity != $myrow["quantity"] ||
-                       $_SESSION['Items']->line_items[$line]->qty_done != $myrow["qty_sent"])
-               {
-                       display_note(_("Original order for") . " '" . $myrow["stk_code"] . "' " .
-                               _("has a quantity of") . " " . $myrow["quantity"] . " " . 
-                               _("and an delivered quantity of") . " " . $myrow["qty_sent"] . " " .
-                               _("the cart shows quantity of") . " " . 
-                               $_SESSION['Items']->line_items[$line]->quantity . " " . 
-                               _("and delivered quantity of") . " " . 
-                               $_SESSION['Items']->line_items[$line]->qty_done, 1, 0);
-
-                       return false;
-               }
-       $line++;
-       } /*loop through all line items of the order to ensure none have been invoiced */
-       return true;
+       } elseif(!check_num('ChargeFreightCost', 0))
+               display_error(_("Freight cost cannot be less than zero"));
+               set_focus('ChargeFreightCost');
 }
 
-//---------------------------------------------------------------------------------------------------------------
+//-----------------------------------------------------------------------------
 
 function check_data()
 {
-       if (!isset($_POST['DispatchDate']) || !is_date($_POST['DispatchDate'])) 
-       {
+       if (!isset($_POST['DispatchDate']) || !is_date($_POST['DispatchDate'])) {
                display_error(_("The entered date of delivery is invalid."));
+               set_focus('DispatchDate');
                return false;
        }
-       if (!is_date_in_fiscalyear($_POST['DispatchDate'])) 
-       {
+
+       if (!is_date_in_fiscalyear($_POST['DispatchDate'])) {
                display_error(_("The entered date of delivery is not in fiscal year."));
+               set_focus('DispatchDate');
                return false;
        }
-       if (!isset($_POST['due_date']) || !is_date($_POST['due_date'])) 
-       {
+
+       if (!isset($_POST['due_date']) || !is_date($_POST['due_date'])) {
                display_error(_("The entered dead-line for invoice is invalid."));
+               set_focus('due_date');
                return false;
        }
 
-       if (!references::is_valid($_POST['ref'])) 
-       {
-               display_error(_("You must enter a reference."));
-               return false;
-       }
+       if ($_SESSION['Items']->trans_no==0) {
+               if (!references::is_valid($_POST['ref'])) {
+                       display_error(_("You must enter a reference."));
+                       set_focus('ref');
+                       return false;
+               }
 
-       if (!is_new_reference($_POST['ref'], 13)) 
-       {
-               display_error(_("The entered reference is already in use."));
-               return false;
+               if ($_SESSION['Items']->trans_no==0 && !is_new_reference($_POST['ref'], 13)) {
+                       display_error(_("The entered reference is already in use."));
+                       set_focus('ref');
+                       return false;
+               }
+       }
+       if ($_POST['ChargeFreightCost'] == "") {
+               $_POST['ChargeFreightCost'] = price_format(0);
        }
-       if ($_POST['ChargeFreightCost'] == "")
-               $_POST['ChargeFreightCost'] = 0;
-       if (!is_numeric($_POST['ChargeFreightCost']) || $_POST['ChargeFreightCost'] < 0)        
-       {
+
+       if (!check_num('ChargeFreightCost',0)) {
                display_error(_("The entered shipping value is not numeric."));
+               set_focus('ChargeFreightCost');
                return false;
        }
 
-       if ($_SESSION['Items']->has_items_dispatch() == 0 && $_POST['ChargeFreightCost'] == 0)  
-       {
+       if ($_SESSION['Items']->has_items_dispatch() == 0 && input_num('ChargeFreightCost') == 0) {
                display_error(_("There are no item quantities on this delivery note."));
                return false;
        }
 
+       if (!check_quantities()) {
+               display_error(_("Selected quantity cannot be less than quantity invoiced nor more than quantity
+               not dispatched on sales order."));
+               return false;
+       }
+
        return true;
 }
+//------------------------------------------------------------------------------
+function copy_to_cart()
+{
+       $cart = &$_SESSION['Items'];
+       $cart->ship_via = $_POST['ship_via'];
+       $cart->freight_cost = input_num('ChargeFreightCost');
+       $cart->document_date = $_POST['DispatchDate'];
+       $cart->due_date =  $_POST['due_date'];
+       $cart->Location = $_POST['Location'];
+       $cart->Comments = $_POST['Comments'];
+       if ($cart->trans_no == 0)
+               $dn->ref = $_POST['ref'];
+
+}
+//------------------------------------------------------------------------------
+
+function copy_from_cart()
+{
+       $cart = &$_SESSION['Items'];
+       $_POST['ship_via'] = $cart->ship_via;
+       $_POST['ChargeFreightCost'] = price_format($cart->freight_cost);
+       $_POST['DispatchDate'] = $cart->document_date;
+       $_POST['due_date'] = $cart->due_date;
+       $_POST['Location'] = $cart->Location;
+       $_POST['Comments'] = $cart->Comments;
+       $_POST['cart_id'] = $cart->cart_id;
+       $_POST['ref'] = $cart->reference;
+}
+//------------------------------------------------------------------------------
+
+function check_quantities()
+{
+       $ok =1;
+       // Update cart delivery quantities/descriptions
+       foreach ($_SESSION['Items']->line_items as $line=>$itm) {
+               if (isset($_POST['Line'.$line])) {
+               if($_SESSION['Items']->trans_no) {
+                       $min = $itm->qty_done;
+                       $max = $itm->quantity;
+               } else {
+                       $min = 0;
+                       $max = $itm->quantity - $itm->qty_done;
+               }
+               
+                       if (check_num('Line'.$line, $min, $max)) {
+                               $_SESSION['Items']->line_items[$line]->qty_dispatched =
+                                 input_num('Line'.$line);
+                       } else {
+                               set_focus('Line'.$line);
+                               $ok = 0;
+                       }
+
+               }
 
-//---------------------------------------------------------------------------------------------------------------
+               if (isset($_POST['Line'.$line.'Desc'])) {
+                       $line_desc = $_POST['Line'.$line.'Desc'];
+                       if (strlen($line_desc) > 0) {
+                               $_SESSION['Items']->line_items[$line]->item_description = $line_desc;
+                       }
+               }
+       }
+// ...
+//     else
+//       $_SESSION['Items']->freight_cost = input_num('ChargeFreightCost');
+       return $ok;
+}
+//------------------------------------------------------------------------------
 
 function check_qoh()
 {
-       if (!sys_prefs::allow_negative_stock())
-       {
-       foreach ($_SESSION['Items']->line_items as $itm) 
-       {
+       if (!sys_prefs::allow_negative_stock()) {
+               foreach ($_SESSION['Items']->line_items as $itm) {
 
-                       if ($itm->qty_dispatched && has_stock_holding($itm->mb_flag))
-                       {
+                       if ($itm->qty_dispatched && has_stock_holding($itm->mb_flag)) {
                                $qoh = get_qoh_on_date($itm->stock_id, $_POST['Location'], $_POST['DispatchDate']);
 
-                       if ($itm->qty_dispatched > $qoh) 
-                       {
-                               display_error(_("The delivery cannot be processed because there is an insufficient quantity for item:") .
-                                       " " . $itm->stock_id . " - " .  $itm->item_description);
-                               return false;
-                       }
-               }
-       }
+                               if ($itm->qty_dispatched > $qoh) {
+                                       display_error(_("The delivery cannot be processed because there is an insufficient quantity for item:") .
+                                               " " . $itm->stock_id . " - " .  $itm->item_description);
+                                       return false;
+                               }
+                       }
+               }
        }
-
        return true;
 }
+//------------------------------------------------------------------------------
 
-//---------------------------------------------------------------------------------------------------------------
+if (isset($_POST['process_delivery']) && check_data() && check_qoh()) {
 
-function process_delivery($get_from_order=false)
-{
-       if ($get_from_order)
-       {   // entry point for direct delivery - cart contains completed order;
-           // we should have qty_dispatched and standard cost set anyway
-               unset($_SESSION['Items']->line_items);
-           read_sales_order($_SESSION['Items']->order_no, $_SESSION['Items']);
-
-               $duedate = get_invoice_duedate($_SESSION['Items']->customer_id, $_SESSION['Items']->delivery_date);
-               $delivery_no = add_sales_delivery($_SESSION['Items'],
-                       $_SESSION['Items']->delivery_date, $duedate, $_SESSION['Items']->order_no,
-                       $_SESSION['Items']->tax_group_id, $_SESSION['Items']->freight_cost,
-                       $_SESSION['Items']->Location, $_SESSION['Items']->ship_via,
-                       $_SESSION['Items']->default_sales_type, references::get_next(13),
-                       $_SESSION['Items']->memo_,0);
-       } 
-       else 
-       {
-       
-               if (!check_data())
-                       return;
-
-               if (!check_order_changed())
-                       order_changed_error();
-
-               if (!check_qoh())
-                       return;
-
-               if ($_POST['bo_policy'])
-                       $bo_policy = 0;
-               else
-                       $bo_policy = 1;
-                       
-               $delivery_no = add_sales_delivery($_SESSION['Items'],
-                       $_POST['DispatchDate'], $_POST['due_date'],     $_SESSION['ProcessingOrder'],
-                       $_SESSION['Items']->tax_group_id,$_POST['ChargeFreightCost'], $_POST['Location'],
-                       $_POST['ship_via'],     $_POST['sales_type_id'], $_POST['ref'],
-                       $_POST['InvoiceText'], $bo_policy);
-               unset($_SESSION['ProcessingOrder']);
+       $dn = &$_SESSION['Items'];
+
+       if ($_POST['bo_policy']) {
+               $bo_policy = 0;
+       } else {
+               $bo_policy = 1;
        }
-       meta_forward($_SERVER['PHP_SELF'], "AddedID=$delivery_no");
-}
+       $newdelivery = ($dn->trans_no == 0);
 
-//---------------------------------------------------------------------------------------------------------------
-if (isset($_GET['process_delivery']))
-       process_delivery(true);
-elseif (isset($_POST['process_delivery']))
-       process_delivery();
+       copy_to_cart();
+       if ($newdelivery) new_doc_date($dn->document_date);
+       $delivery_no = $dn->write($bo_policy);
 
-//-------------------------------------------------------------------------------------------------
+       processing_end();
+       if ($newdelivery) {
+               meta_forward($_SERVER['PHP_SELF'], "AddedID=$delivery_no");
+       } else {
+               meta_forward($_SERVER['PHP_SELF'], "UpdatedID=$delivery_no");
+       }
+}
 
-start_form(false, true);
+if (isset($_POST['Update']) || isset($_POST['_Location_update'])) {
+       $Ajax->activate('Items');
+}
+//------------------------------------------------------------------------------
+start_form();
+hidden('cart_id');
 
 start_table("$table_style2 width=80%", 5);
 echo "<tr><td>"; // outer table
@@ -344,39 +323,41 @@ label_cells(_("Currency"), $_SESSION['Items']->customer_currency, "class='tableh
 end_row();
 start_row();
 
-if (!isset($_POST['ref']))
-       $_POST['ref'] = references::get_next(13);
-
-ref_cells(_("Reference"), 'ref', null, "class='tableheader2'");
+//if (!isset($_POST['ref']))
+//     $_POST['ref'] = references::get_next(13);
 
-label_cells(_("For Sales Order"), get_customer_trans_view_str(systypes::sales_order(), $_SESSION['ProcessingOrder']), "class='tableheader2'");
+if ($_SESSION['Items']->trans_no==0) {
+       ref_cells(_("Reference"), 'ref', '', null, "class='tableheader2'");
+} else {
+       label_cells(_("Reference"), $_SESSION['Items']->reference, "class='tableheader2'");
+}
 
-if (!isset($_POST['sales_type_id']))
-       $_POST['sales_type_id'] = $_SESSION['Items']->default_sales_type;
-label_cell(_("Sales Type"), "class='tableheader2'");   
-sales_types_list_cells(null, 'sales_type_id', $_POST['sales_type_id']);
+label_cells(_("For Sales Order"), get_customer_trans_view_str(systypes::sales_order(), $_SESSION['Items']->order_no), "class='tableheader2'");
 
+label_cells(_("Sales Type"), $_SESSION['Items']->sales_type_name, "class='tableheader2'");
 end_row();
 start_row();
 
-if (!isset($_POST['Location']))
+if (!isset($_POST['Location'])) {
        $_POST['Location'] = $_SESSION['Items']->Location;
-label_cell(_("Delivery From"), "class='tableheader2'");        
-locations_list_cells(null, 'Location', $_POST['Location'], false, true);
+}
+label_cell(_("Delivery From"), "class='tableheader2'");
+locations_list_cells(null, 'Location', null, false, true);
 
-if (!isset($_POST['ship_via']))
+if (!isset($_POST['ship_via'])) {
        $_POST['ship_via'] = $_SESSION['Items']->ship_via;
-label_cell(_("Shipping Company"), "class='tableheader2'");     
+}
+label_cell(_("Shipping Company"), "class='tableheader2'");
 shippers_list_cells(null, 'ship_via', $_POST['ship_via']);
 
 // set this up here cuz it's used to calc qoh
-if (!isset($_POST['DispatchDate']) || !is_date($_POST['DispatchDate']))
-{
-       $_POST['DispatchDate'] = Today();
-       if (!is_date_in_fiscalyear($_POST['DispatchDate']))
+if (!isset($_POST['DispatchDate']) || !is_date($_POST['DispatchDate'])) {
+       $_POST['DispatchDate'] = new_doc_date();
+       if (!is_date_in_fiscalyear($_POST['DispatchDate'])) {
                $_POST['DispatchDate'] = end_fiscalyear();
+       }
 }
-date_cells(_("Date"), 'DispatchDate', $_POST['DispatchDate'], 0, 0, 0, "class='tableheader'");
+date_cells(_("Date"), 'DispatchDate', '', $_SESSION['Items']->trans_no==0, 0, 0, 0, "class='tableheader2'");
 end_row();
 
 end_table();
@@ -385,56 +366,65 @@ echo "</td><td>";// outer table
 
 start_table("$table_style width=90%");
 
-if (!isset($_POST['due_date']) || !is_date($_POST['due_date']))
-
+if (!isset($_POST['due_date']) || !is_date($_POST['due_date'])) {
        $_POST['due_date'] = get_invoice_duedate($_SESSION['Items']->customer_id, $_POST['DispatchDate']);
-
-date_row(_("Invoice Dead-line"), 'due_date', $_POST['due_date'], 0, 0, 0, "class='tableheader'");
+}
+date_row(_("Invoice Dead-line"), 'due_date', '', null, 0, 0, 0, "class='tableheader2'");
 end_table();
 
 echo "</td></tr>";
 end_table(1); // outer table
 
-display_heading(_("Invoice Items"));
-
+$row = get_customer_to_order($_SESSION['Items']->customer_id);
+if ($row['dissallow_invoices'] == 1)
+{
+       display_error(_("The selected customer account is currently on hold. Please contact the credit control personnel to discuss."));
+       end_form();
+       end_page();
+       exit();
+}      
+display_heading(_("Delivery Items"));
+div_start('Items');
 start_table("$table_style width=80%");
 $th = array(_("Item Code"), _("Item Description"), _("Ordered"), _("Units"), _("Delivered"),
        _("This Delivery"), _("Price"), _("Tax Type"), _("Discount"), _("Total"));
 table_header($th);
 $k = 0;
 $has_marked = false;
-$show_qoh = true;
 
-foreach ($_SESSION['Items']->line_items as $line=>$ln_itm) 
-{      
-       if($ln_itm->quantity==$ln_itm->qty_done) continue; //this line is fully delivered
-    // if it's a non-stock item (eg. service) don't show qoh
-    if (sys_prefs::allow_negative_stock() || !has_stock_holding($ln_itm->mb_flag) ||
-               $ln_itm->qty_dispatched == 0)
-       $show_qoh = false;
+foreach ($_SESSION['Items']->line_items as $line=>$ln_itm) {
+       if ($ln_itm->quantity==$ln_itm->qty_done) {
+               continue; //this line is fully delivered
+       }
+       // if it's a non-stock item (eg. service) don't show qoh
+       $show_qoh = true;
+       if (sys_prefs::allow_negative_stock() || !has_stock_holding($ln_itm->mb_flag) ||
+               $ln_itm->qty_dispatched == 0) {
+               $show_qoh = false;
+       }
 
-       if ($show_qoh)
+       if ($show_qoh) {
                $qoh = get_qoh_on_date($ln_itm->stock_id, $_POST['Location'], $_POST['DispatchDate']);
+       }
 
-       if ($show_qoh && ($ln_itm->qty_dispatched > $qoh)) 
-       {
+       if ($show_qoh && ($ln_itm->qty_dispatched > $qoh)) {
                // oops, we don't have enough of one of the component items
                start_row("class='stockmankobg'");
                $has_marked = true;
-       } 
-       else
+       } else {
                alt_table_row_color($k);
-
+       }
        view_stock_status_cell($ln_itm->stock_id);
 
        text_cells(null, 'Line'.$line.'Desc', $ln_itm->item_description, 30, 50);
-       qty_cell($ln_itm->quantity);
+       $dec = get_qty_dec($ln_itm->stock_id);
+       qty_cell($ln_itm->quantity, false, $dec);
        label_cell($ln_itm->units);
-       qty_cell($ln_itm->qty_done);
+       qty_cell($ln_itm->qty_done, false, $dec);
 
-       text_cells(null, 'Line'.$line, $ln_itm->qty_dispatched, 10, 10);
+       small_qty_cells(null, 'Line'.$line, qty_format($ln_itm->qty_dispatched, $ln_itm->stock_id, $dec), null, null, $dec);
 
-       $display_discount_percent = number_format2($ln_itm->discount_percent*100,user_percent_dec()) . "%";
+       $display_discount_percent = percent_format($ln_itm->discount_percent*100) . "%";
 
        $line_total = ($ln_itm->qty_dispatched * $ln_itm->price * (1 - $ln_itm->discount_percent));
 
@@ -445,50 +435,47 @@ foreach ($_SESSION['Items']->line_items as $line=>$ln_itm)
 
        end_row();
 }
-  
-$_POST['ChargeFreightCost'] = $_SESSION['Items']->freight_cost;
 
-if (!is_numeric($_POST['ChargeFreightCost']))
-{
-       $_POST['ChargeFreightCost'] = 0;
-}
+$_POST['ChargeFreightCost'] =  get_post('ChargeFreightCost', 
+       price_format($_SESSION['Items']->freight_cost));
 
 start_row();
 
-small_amount_cells(_("Shipping Cost"), 'ChargeFreightCost', null, "colspan=9 align=right");
+small_amount_cells(_("Shipping Cost"), 'ChargeFreightCost', $_SESSION['Items']->freight_cost, "colspan=9 align=right");
 
 $inv_items_total = $_SESSION['Items']->get_items_total_dispatch();
 
-$display_sub_total = number_format2($inv_items_total + $_POST['ChargeFreightCost'],user_price_dec());
+$display_sub_total = price_format($inv_items_total + input_num('ChargeFreightCost'));
 
 label_row(_("Sub-total"), $display_sub_total, "colspan=9 align=right","align=right");
 
-$taxes = $_SESSION['Items']->get_taxes($_SESSION['Items']->tax_group_id, $_POST['ChargeFreightCost']);
-$tax_total = display_edit_tax_items($taxes, 9);
+$taxes = $_SESSION['Items']->get_taxes(input_num('ChargeFreightCost'));
+$tax_total = display_edit_tax_items($taxes, 9, $_SESSION['Items']->tax_included);
 
-$display_total = number_format2(($inv_items_total + $_POST['ChargeFreightCost'] + $tax_total), user_price_dec());
+$display_total = price_format(($inv_items_total + input_num('ChargeFreightCost') + $tax_total));
 
 label_row(_("Amount Total"), $display_total, "colspan=9 align=right","align=right");
 
 end_table(1);
 
-if ($has_marked)
-       display_note(_("Marked items have insufficient quantities in stock."), 0, 1, "class='red'");
-
+if ($has_marked) {
+       display_note(_("Marked items have insufficient quantities in stock as on day of delivery."), 0, 1, "class='red'");
+}
 start_table($table_style2);
 
 policy_list_row(_("Action For Balance"), "bo_policy", null);
 
-textarea_row(_("Memo"), 'InvoiceText', null, 50, 4);
+textarea_row(_("Memo"), 'Comments', null, 50, 4);
 
 end_table(1);
-
-submit_center_first('Update', _("Update"));
-submit_center_last('process_delivery', _("Process Dispatch"));
+div_end();
+submit_center_first('Update', _("Update"),
+  _('Refresh document page'), true);
+submit_center_last('process_delivery', _("Process Dispatch"),
+  _('Check entered data and save document'), 'default');
 
 end_form();
 
-//---------------------------------------------------------------------------------------------
 
 end_page();