Fixed amibiguity in few text messages to make translations easier, updated gettext...

[fa-stable.git] / inventory / adjustments.php

diff --git a/inventory/adjustments.php b/inventory/adjustments.php
index 33c24b62d3d74424f8507674a7b7bf484db35ed8..3d2c39baf1c1fc89a3794b834887a50335e60d21 100644 (file)
--- a/inventory/adjustments.php
+++ b/inventory/adjustments.php
@@ -21,9 +21,9 @@ include_once($path_to_root . "/includes/data_checks.inc");
 include_once($path_to_root . "/inventory/includes/item_adjustments_ui.inc");
 include_once($path_to_root . "/inventory/includes/inventory_db.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(_($help_context = "Item Adjustments Note"), false, false, "", $js);
 
@@ -31,8 +31,6 @@ page(_($help_context = "Item Adjustments Note"), false, false, "", $js);
 
 check_db_has_costable_items(_("There are no inventory items defined in the system which can be adjusted (Purchased or Manufactured)."));
 
-check_db_has_movement_types(_("There are no inventory movement types defined in the system. Please define at least one inventory adjustment type."));
-
 //-----------------------------------------------------------------------------------------------
 
 if (isset($_GET['AddedID'])) 
@@ -89,16 +87,9 @@ function can_process()
                set_focus('stock_id');
                return false;
        }
-       if (!$Refs->is_valid($_POST['ref'])) 
-       {
-               display_error( _("You must enter a reference."));
-               set_focus('ref');
-               return false;
-       }
 
-       if (!is_new_reference($_POST['ref'], ST_INVADJUST)) 
+       if (!check_reference($_POST['ref'], ST_INVADJUST))
        {
-               display_error( _("The entered reference is already in use."));
                set_focus('ref');
                return false;
        }
@@ -111,13 +102,13 @@ function can_process()
        } 
        elseif (!is_date_in_fiscalyear($_POST['AdjDate'])) 
        {
-               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('AdjDate');
                return false;
        }
        elseif (!$SysPrefs->allow_negative_stock())
        {
-               $low_stock = $adj->check_qoh($_POST['StockLocation'], $_POST['AdjDate'], !$_POST['Increase']);
+               $low_stock = $adj->check_qoh($_POST['StockLocation'], $_POST['AdjDate']);
 
                if ($low_stock)
                {
@@ -134,8 +125,7 @@ function can_process()
 if (isset($_POST['Process']) && can_process()){
 
        $trans_no = add_stock_adjustment($_SESSION['adj_items']->line_items,
-               $_POST['StockLocation'], $_POST['AdjDate'],     $_POST['type'], $_POST['Increase'],
-               $_POST['ref'], $_POST['memo_']);
+               $_POST['StockLocation'], $_POST['AdjDate'],     $_POST['ref'], $_POST['memo_']);
        new_doc_date($_POST['AdjDate']);
        $_SESSION['adj_items']->clear_items();
        unset($_SESSION['adj_items']);
@@ -148,9 +138,9 @@ if (isset($_POST['Process']) && can_process()){
 
 function check_item_data()
 {
-       if (!check_num('qty',0))
+       if (input_num('qty') == 0)
        {
-               display_error(_("The quantity entered is negative or invalid."));
+               display_error(_("The quantity entered is invalid."));
                set_focus('qty');
                return false;
        }
@@ -169,12 +159,9 @@ function check_item_data()
 
 function handle_update_item()
 {
-    if($_POST['UpdateItem'] != "" && check_item_data())
-    {
-               $id = $_POST['LineNo'];
-       $_SESSION['adj_items']->update_cart_item($id, input_num('qty'), 
-                       input_num('std_cost'));
-    }
+       $id = $_POST['LineNo'];
+       $_SESSION['adj_items']->update_cart_item($id, input_num('qty'), 
+               input_num('std_cost'));
        line_start_focus();
 }
 
@@ -190,9 +177,6 @@ function handle_delete_item($id)
 
 function handle_new_item()
 {
-       if (!check_item_data())
-               return;
-
        add_to_order($_SESSION['adj_items'], $_POST['stock_id'], 
          input_num('qty'), input_num('std_cost'));
        line_start_focus();
@@ -203,10 +187,10 @@ $id = find_submit('Delete');
 if ($id != -1)
        handle_delete_item($id);
 
-if (isset($_POST['AddItem']))
+if (isset($_POST['AddItem']) && check_item_data())
        handle_new_item();
 
-if (isset($_POST['UpdateItem']))
+if (isset($_POST['UpdateItem']) && check_item_data())
        handle_update_item();
 
 if (isset($_POST['CancelItemChanges'])) {
@@ -237,4 +221,3 @@ submit_center_last('Process', _("Process Adjustment"), '', 'default');
 end_form();
 end_page();
 
-?>