[0003780] Sales Invoice to partial delivery was improperly issued as final prepayment...
[fa-stable.git] / sales / customer_invoice.php
index 588f0f56a652386616ecccc7846b72b3e75f282e..6a8e82bf041a1674f6f4afb2772613e769892e0a 100644 (file)
@@ -19,7 +19,6 @@ $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");
@@ -27,7 +26,7 @@ include_once($path_to_root . "/taxes/tax_calc.inc");
 include_once($path_to_root . "/admin/db/shipping_db.inc");
 
 $js = "";
-if ($use_popup_windows) {
+if ($SysPrefs->use_popup_windows) {
        $js .= get_js_open_window(900, 500);
 }
 if (user_use_date_picker()) {
@@ -109,7 +108,7 @@ if (isset($_GET['AddedID'])) {
 //-----------------------------------------------------------------------------
 
 if ( (isset($_GET['DeliveryNumber']) && ($_GET['DeliveryNumber'] > 0) )
-|| isset($_GET['BatchInvoice'])) {
+       || isset($_GET['BatchInvoice'])) {
 
        processing_start();
 
@@ -135,14 +134,7 @@ if ( (isset($_GET['DeliveryNumber']) && ($_GET['DeliveryNumber'] > 0) )
 } elseif (isset($_GET['ModifyInvoice']) && $_GET['ModifyInvoice'] > 0) {
 
        check_is_editable(ST_SALESINVOICE, $_GET['ModifyInvoice']);
-/*
-       if ( get_sales_parent_numbers(ST_SALESINVOICE, $_GET['ModifyInvoice']) == 0) { // 1.xx compatibility hack
-               echo"<center><br><b>" . _("There are no delivery notes for this invoice.<br>
-               Most likely this invoice was created in Front Accounting version prior to 2.0
-               and therefore can not be modified.") . "</b></center>";
-               display_footer_exit();
-       }
-*/
+
        processing_start();
        $_SESSION['Items'] = new Cart(ST_SALESINVOICE, $_GET['ModifyInvoice']);
 
@@ -153,44 +145,36 @@ if ( (isset($_GET['DeliveryNumber']) && ($_GET['DeliveryNumber'] > 0) )
        copy_from_cart();
 } elseif (isset($_GET['AllocationNumber']) || isset($_GET['InvoicePrepayments'])) {
 
-               check_deferred_income_act(_("You have to set Deferred Income Account in GL Setup to entry prepayment invoices."));
+       check_deferred_income_act(_("You have to set Deferred Income Account in GL Setup to entry prepayment invoices."));
 
-               if (isset($_GET['AllocationNumber']))
-               {
-                       $payments = array(get_cust_allocation($_GET['AllocationNumber']));
+       if (isset($_GET['AllocationNumber']))
+       {
+               $payments = array(get_cust_allocation($_GET['AllocationNumber']));
 
-                       if (!$payments || ($payments[0]['trans_type_to'] != ST_SALESORDER))
-                       {
-                               display_error(_("Please select correct Sales Order Prepayment to be invoiced and try again."));
-                               display_footer_exit();
-                       }
-                       $order_no = $payments[0]['trans_no_to'];
-               }
-               else {
-                       $order_no = $_GET['InvoicePrepayments'];
-                       $payments =  get_payments_for($_GET['InvoicePrepayments'], ST_SALESORDER);
+               if (!$payments || ($payments[0]['trans_type_to'] != ST_SALESORDER))
+               {
+                       display_error(_("Please select correct Sales Order Prepayment to be invoiced and try again."));
+                       display_footer_exit();
                }
-               processing_start();
-
-               $_SESSION['Items'] = new Cart(ST_SALESORDER, $order_no, ST_SALESINVOICE);
-               $_SESSION['Items']->order_no = $order_no;
-               $_SESSION['Items']->src_docs = array($order_no);
-               $_SESSION['Items']->trans_no = 0;
-               $_SESSION['Items']->trans_type = ST_SALESINVOICE;
-               $_SESSION['Items']->prepayments = $payments;
-               // prepayment invoice has all line quantities as stated on order.
-//             foreach($_SESSION['Items']->line_items as &$line)
-//             {
-//                     $line->src_id = $line_id;
-//                     $line->qty_dispatched = $line->quantity;
-//             }
-//             unset($line);
-               $_SESSION['Items']->update_payments();
-
-//             $_SESSION['Items']->alloc = $this_amount;
-               copy_from_cart();
+               $order_no = $payments[0]['trans_no_to'];
+       }
+       else {
+               $order_no = $_GET['InvoicePrepayments'];
+               $payments =  get_payments_for($_GET['InvoicePrepayments'], ST_SALESORDER);
+       }
+       processing_start();
+
+       $_SESSION['Items'] = new Cart(ST_SALESORDER, $order_no, ST_SALESINVOICE);
+       $_SESSION['Items']->order_no = $order_no;
+       $_SESSION['Items']->src_docs = array($order_no);
+       $_SESSION['Items']->trans_no = 0;
+       $_SESSION['Items']->trans_type = ST_SALESINVOICE;
+       $_SESSION['Items']->prepayments = $payments;
+       $_SESSION['Items']->update_payments();
+
+       copy_from_cart();
 }
- elseif (!processing_active()) {
+elseif (!processing_active()) {
        /* This page can only be called with a delivery for invoicing or invoice no for edit */
        display_error(_("This page can only be opened after delivery selection. Please select delivery to invoicing first."));
 
@@ -250,11 +234,8 @@ function set_delivery_shipping_sum($delivery_notes)
     
     foreach($delivery_notes as $delivery_num) 
     {
-        $myrow = get_customer_trans($delivery_num, 13);
-        //$branch = get_branch($myrow["branch_code"]);
-        //$sales_order = get_sales_order_header($myrow["order_"]);
-        
-        //$shipping += $sales_order['freight_cost'];
+        $myrow = get_customer_trans($delivery_num, ST_CUSTDELIVERY);
+
         $shipping += $myrow['ov_freight'];
     }
     $_POST['ChargeFreightCost'] = price_format($shipping);
@@ -332,7 +313,7 @@ function check_data()
        }
 
        if ($_SESSION['Items']->trans_no == 0) {
-               if (!$Refs->is_valid($_POST['ref'])) {
+               if (!$Refs->is_valid($_POST['ref'], ST_SALESINVOICE)) {
                        display_error(_("You must enter a reference."));
                        set_focus('ref');
                        return false;
@@ -361,7 +342,7 @@ function check_data()
                        return false;
                }
        } else {
-               if (($_SESSION['Items']->payment_terms['days_before_due'] < 0) && !count($_SESSION['Items']->prepayments)) {
+               if (($_SESSION['Items']->payment_terms['days_before_due'] == -1) && !count($_SESSION['Items']->prepayments)) {
                        display_error(_("There is no non-invoiced payments for this order. If you want to issue final invoice, select delayed or cash payment terms."));
                        return false;
                }
@@ -374,6 +355,7 @@ function check_data()
 if (isset($_POST['process_invoice']) && check_data()) {
        $newinvoice=  $_SESSION['Items']->trans_no == 0;
        copy_to_cart();
+
        if ($newinvoice) 
                new_doc_date($_SESSION['Items']->document_date);
 
@@ -440,7 +422,7 @@ $is_edition = $_SESSION['Items']->trans_type == ST_SALESINVOICE && $_SESSION['It
 start_form();
 hidden('cart_id');
 
-start_table(TABLESTYLE2, "width=80%", 5);
+start_table(TABLESTYLE2, "width='80%'", 5);
 
 start_row();
 $colspan = 1;
@@ -461,20 +443,17 @@ end_row();
 start_row();
 
 if ($_SESSION['Items']->trans_no == 0) {
-       ref_cells(_("Reference"), 'ref', '', null, "class='tableheader2'");
+       ref_cells(_("Reference"), 'ref', '', null, "class='tableheader2'", false, ST_SALESINVOICE,
+               array('customer' => $_SESSION['Items']->customer_id,
+                       'branch' => $_SESSION['Items']->Branch,
+                       'date' => get_post('InvoiceDate')));
 } else {
        label_cells(_("Reference"), $_SESSION['Items']->reference, "class='tableheader2'");
 }
 
-//label_cells(_("Delivery Notes:"),
-//get_customer_trans_view_str(ST_CUSTDELIVERY, array_keys($_SESSION['Items']->src_docs)), "class='tableheader2'");
-
 label_cells(_("Sales Type"), $_SESSION['Items']->sales_type_name, "class='tableheader2'");
 
 label_cells(_("Currency"), $_SESSION['Items']->customer_currency, "class='tableheader2'");
-// 2010-09-03 Joe Hunt
-//if ($dim > 0) 
-//     label_cells(_("Dimension"), get_dimension_string($_SESSION['Items']->dimension_id), "class='tableheader2'");
 if ($dim > 0) {
        label_cell(_("Dimension").":", "class='tableheader2'");
        $_POST['dimension_id'] = $_SESSION['Items']->dimension_id;
@@ -512,12 +491,6 @@ if (!isset($_POST['due_date']) || !is_date($_POST['due_date'])) {
 }
 
 date_cells(_("Due Date"), 'due_date', '', null, 0, 0, 0, "class='tableheader2'");
-/*
-if ($dim > 1) 
-       label_cells(_("Dimension"). " 2", get_dimension_string($_SESSION['Items']->dimension2_id), "class='tableheader2'");
-else if ($dim > 0)
-       label_cell("&nbsp;", "colspan=2");
-*/
 if ($dim > 1) {
        label_cell(_("Dimension")." 2:", "class='tableheader2'");
        $_POST['dimension2_id'] = $_SESSION['Items']->dimension2_id;
@@ -540,13 +513,15 @@ if ($row['dissallow_invoices'] == 1)
 display_heading($prepaid ? _("Sales Order Items") : _("Invoice Items"));
 
 div_start('Items');
-start_table(TABLESTYLE, "width=80%");
+
+start_table(TABLESTYLE, "width='80%'");
 if ($prepaid)
        $th = array(_("Item Code"), _("Item Description"), _("Units"), _("Quantity"),
                _("Price"), _("Tax Type"), _("Discount"), _("Total"));
 else
        $th = array(_("Item Code"), _("Item Description"), _("Delivered"), _("Units"), _("Invoiced"),
                _("This Invoice"), _("Price"), _("Tax Type"), _("Discount"), _("Total"));
+
 if ($is_batch_invoice) {
     $th[] = _("DN");
     $th[] = "";
@@ -602,9 +577,9 @@ foreach ($_SESSION['Items']->line_items as $line=>$ln_itm) {
                if ($dn_line_cnt == 0) {
                        $dn_line_cnt = $dspans[0];
                        $dspans = array_slice($dspans, 1);
-                       label_cell($ln_itm->src_no, "rowspan=$dn_line_cnt class=oddrow");
+                       label_cell($ln_itm->src_no, "rowspan=$dn_line_cnt class='oddrow'");
                        label_cell("<a href='" . $_SERVER['PHP_SELF'] . "?RemoveDN=".
-                               $ln_itm->src_no."'>" . _("Remove") . "</a>", "rowspan=$dn_line_cnt class=oddrow");
+                               $ln_itm->src_no."'>" . _("Remove") . "</a>", "rowspan=$dn_line_cnt class='oddrow'");
                }
                $dn_line_cnt--;
        }