[0004212] Work Order Entry: fixed error when voided WO refence is reused.

[fa-stable.git] / manufacturing / work_order_issue.php

diff --git a/manufacturing/work_order_issue.php b/manufacturing/work_order_issue.php
index 8809b61f1453e97dba64481805fd3eff8f4d5bdb..025a91258c2dcfb7f76c398c90927b23c7ea9951 100644 (file)
--- a/manufacturing/work_order_issue.php
+++ b/manufacturing/work_order_issue.php
@@ -1,16 +1,17 @@
 <?php
 /**********************************************************************
     Copyright (C) FrontAccounting, LLC.
-       Released under the terms of the GNU Affero General Public License,
-       AGPL, as published by the Free Software Foundation, either version 
-       3 of the License, or (at your option) any later version.
+       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/agpl-3.0.html>.
+    See the License here <http://www.gnu.org/licenses/gpl-3.0.html>.
 ***********************************************************************/
-$page_security = 3;
-$path_to_root="..";
+$page_security = 'SA_MANUFISSUE';
+$path_to_root = "..";
+
 include_once($path_to_root . "/includes/ui/items_cart.inc");
 
 include_once($path_to_root . "/includes/session.inc");
@@ -22,20 +23,26 @@ include_once($path_to_root . "/manufacturing/includes/manufacturing_db.inc");
 include_once($path_to_root . "/manufacturing/includes/manufacturing_ui.inc");
 include_once($path_to_root . "/manufacturing/includes/work_order_issue_ui.inc");
 $js = "";
-if ($use_popup_windows)
+if ($SysPrefs->use_popup_windows)
        $js .= get_js_open_window(800, 500);
-if ($use_date_picker)
+if (user_use_date_picker())
        $js .= get_js_date_picker();
-page(_("Issue Items to Work Order"), false, false, "", $js);
+
+page(_($help_context = "Issue Items to Work Order"), false, false, "", $js);
 
 //-----------------------------------------------------------------------------------------------
 
 if (isset($_GET['AddedID'])) 
 {
-       echo "<center>" . _("The work order issue has been entered.");
-       echo "<br>";
+       $id = $_GET['AddedID'];
+       display_notification(_("The work order issue has been entered."));
+
+    display_note(get_trans_view_str(ST_WORKORDER, $id, _("View this Work Order")));
+
+       display_note(get_gl_view_str(ST_WORKORDER, $id, _("View the GL Journal Entries for this Work Order")), 1);
+
        hyperlink_no_params("search_work_orders.php", _("Select another &Work Order to Process"));
-       echo "<br><br>";
+
        display_footer_exit();
 }
 //--------------------------------------------------------------------------------------------------
@@ -57,47 +64,35 @@ function handle_new_order()
                unset ($_SESSION['issue_items']);
        }
 
-     Session_register("issue_items");
-
-     $_SESSION['issue_items'] = new items_cart(28);
+     $_SESSION['issue_items'] = new items_cart(ST_MANUISSUE);
      $_SESSION['issue_items']->order_id = $_GET['trans_no'];
 }
 
 //-----------------------------------------------------------------------------------------------
-
 function can_process()
 {
-       if (!is_date($_POST['date_'])) 
+       if (!is_date($_POST['date_']))
        {
                display_error(_("The entered date for the issue is invalid."));
                set_focus('date_');
                return false;
        } 
-       elseif (!is_date_in_fiscalyear($_POST['date_'])) 
+       elseif (!is_date_in_fiscalyear($_POST['date_']))
        {
-               display_error(_("The entered date is not in fiscal year."));
+               display_error(_("The entered date is out of fiscal year or is closed for further data entry."));
                set_focus('date_');
                return false;
        }
-       if (!references::is_valid($_POST['ref'])) 
-       {
-               display_error(_("You must enter a reference."));
-               set_focus('ref');
-               return false;
-       }
-
-       if (!is_new_reference($_POST['ref'], 28)) 
+       if (!check_reference($_POST['ref'], ST_MANUISSUE))
        {
-               display_error(_("The entered reference is already in use."));
                set_focus('ref');
                return false;
        }
 
        $failed_item = $_SESSION['issue_items']->check_qoh($_POST['Location'], $_POST['date_'], !$_POST['IssueType']);
-       if ($failed_item >= 0) 
+       if ($failed_item)
        {
-       display_error( _("The issue cannot be processed because an entered item would cause a negative inventory balance :") .
-               " " . $failed_item->stock_id . " - " .  $failed_item->item_description);
+               display_error(_("The issue cannot be processed because it would cause negative inventory balance for marked items as of document date or later."));
                return false;
        }
 
@@ -120,7 +115,7 @@ if (isset($_POST['Process']) && can_process())
        } 
        else 
        {
-               meta_forward($_SERVER['PHP_SELF'], "AddedID=1");
+               meta_forward($_SERVER['PHP_SELF'], "AddedID=".$_SESSION['issue_items']->order_id);
        }
 
 } /*end of process credit note */
@@ -129,7 +124,7 @@ if (isset($_POST['Process']) && can_process())
 
 function check_item_data()
 {
-       if (!check_num('qty', 0))
+       if (input_num('qty') == 0 || !check_num('qty', 0))
        {
                display_error(_("The quantity entered is negative or invalid."));
                set_focus('qty');
@@ -205,9 +200,9 @@ if (isset($_GET['trans_no']))
 display_wo_details($_SESSION['issue_items']->order_id);
 echo "<br>";
 
-start_form(false, true);
+start_form();
 
-start_table("$table_style width=90%", 10);
+start_table(TABLESTYLE, "width='90%'", 10);
 echo "<tr><td>";
 display_issue_items(_("Items to Issue"), $_SESSION['issue_items']);
 issue_options_controls();
@@ -215,7 +210,7 @@ echo "</td></tr>";
 
 end_table();
 
-submit_center('Process', _("Process Issue"), true, '', true);
+submit_center('Process', _("Process Issue"), true, '', 'default');
 
 end_form();
 
@@ -223,4 +218,3 @@ end_form();
 
 end_page();
 
-?>