X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=inventory%2Ftransfers.php;h=f7ae7d959b2c9c70179db098e1373a219f4c933a;hb=c85c10b86be48b2b1df728f4751adbb38f2ac8d4;hp=e9e931faeab6829423c84511d8d87636554a192c;hpb=da8311619dd73feae101d246a1957b972e00cbd2;p=fa-stable.git

diff --git a/inventory/transfers.php b/inventory/transfers.php
index e9e931fa..f7ae7d95 100644
--- a/inventory/transfers.php
+++ b/inventory/transfers.php
@@ -1,7 +1,16 @@
 <?php
-
-$page_security = 3;
-$path_to_root="..";
+/**********************************************************************
+    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>.
+***********************************************************************/
+$page_security = 'SA_LOCATIONTRANSFER';
+$path_to_root = "..";
 include_once($path_to_root . "/includes/ui/items_cart.inc");
 
 include_once($path_to_root . "/includes/session.inc");
@@ -14,8 +23,9 @@ include_once($path_to_root . "/inventory/includes/inventory_db.inc");
 $js = "";
 if ($use_popup_windows)
 	$js .= get_js_open_window(800, 500);
-page(_("Inventory Location Transfers"), false, false, "", $js);
-
+if ($use_date_picker)
+	$js .= get_js_date_picker();
+page(_($help_context = "Inventory Location Transfers"), false, false, "", $js);
 
 //-----------------------------------------------------------------------------------------------
 
@@ -28,38 +38,23 @@ check_db_has_movement_types(_("There are no inventory movement types defined in
 if (isset($_GET['AddedID'])) 
 {
 	$trans_no = $_GET['AddedID'];
-	$trans_type = systypes::location_transfer();
+	$trans_type = ST_LOCTRANSFER;
 
 	display_notification_centered(_("Inventory transfer has been processed"));
-	display_note(get_trans_view_str($trans_type, $trans_no, _("View this transfer")));
+	display_note(get_trans_view_str($trans_type, $trans_no, _("&View this transfer")));
 
-	hyperlink_no_params($_SERVER['PHP_SELF'], _("Enter Another Inventory Transfer"));
+	hyperlink_no_params($_SERVER['PHP_SELF'], _("Enter &Another Inventory Transfer"));
 
 	display_footer_exit();
 }
-
 //--------------------------------------------------------------------------------------------------
 
-function copy_to_st()
-{
-	$_SESSION['transfer_items']->from_loc = $_POST['FromStockLocation'];
-	$_SESSION['transfer_items']->to_loc = $_POST['ToStockLocation'];
-	$_SESSION['transfer_items']->tran_date = $_POST['AdjDate'];
-	$_SESSION['transfer_items']->transfer_type = $_POST['type'];
-	$_SESSION['transfer_items']->memo_ = $_POST['memo_'];
-}
-
-//--------------------------------------------------------------------------------------------------
+function line_start_focus() {
+  global 	$Ajax;
 
-function copy_from_st()
-{
-	$_POST['FromStockLocation'] = $_SESSION['transfer_items']->from_loc;
-	$_POST['ToStockLocation'] = $_SESSION['transfer_items']->to_loc;
-	$_POST['AdjDate'] = $_SESSION['transfer_items']->tran_date;
-	$_POST['type'] = $_SESSION['transfer_items']->transfer_type;
-	$_POST['memo_'] = $_SESSION['transfer_items']->memo_;
+  $Ajax->activate('items_table');
+  set_focus('_stock_id_edit');
 }
-
 //-----------------------------------------------------------------------------------------------
 
 function handle_new_order()
@@ -70,10 +65,8 @@ function handle_new_order()
 		unset ($_SESSION['transfer_items']);
 	}
 
-    session_register("transfer_items");
-
-	$_SESSION['transfer_items'] = new items_cart;
-	$_POST['AdjDate'] = Today();
+	$_SESSION['transfer_items'] = new items_cart(ST_LOCTRANSFER);
+	$_POST['AdjDate'] = new_doc_date();
 	if (!is_date_in_fiscalyear($_POST['AdjDate']))
 		$_POST['AdjDate'] = end_fiscalyear();
 	$_SESSION['transfer_items']->tran_date = $_POST['AdjDate'];	
@@ -84,41 +77,51 @@ function handle_new_order()
 if (isset($_POST['Process']))
 {
 
+	$tr = &$_SESSION['transfer_items'];
 	$input_error = 0;
 
-	if (!references::is_valid($_POST['ref'])) 
+	if (count($tr->line_items) == 0)	{
+		display_error(_("You must enter at least one non empty item line."));
+		set_focus('stock_id');
+		$input_error = 1;
+	}
+	if (!$Refs->is_valid($_POST['ref'])) 
 	{
 		display_error(_("You must enter a reference."));
+		set_focus('ref');
 		$input_error = 1;
 	} 
-	elseif (!is_new_reference($_POST['ref'], systypes::location_transfer())) 
+	elseif (!is_new_reference($_POST['ref'], ST_LOCTRANSFER)) 
 	{
 		display_error(_("The entered reference is already in use."));
+		set_focus('ref');
 		$input_error = 1;
 	} 
 	elseif (!is_date($_POST['AdjDate'])) 
 	{
-		display_error(_("The entered date for the adjustment is invalid."));
+		display_error(_("The entered transfer date is invalid."));
+		set_focus('AdjDate');
 		$input_error = 1;
 	} 
 	elseif (!is_date_in_fiscalyear($_POST['AdjDate'])) 
 	{
 		display_error(_("The entered date is not in fiscal year."));
+		set_focus('AdjDate');
 		$input_error = 1;
 	} 
 	elseif ($_POST['FromStockLocation'] == $_POST['ToStockLocation'])
 	{
 		display_error(_("The locations to transfer from and to must be different."));
+		set_focus('FromStockLocation');
 		$input_error = 1;
-	} 
-	else 
+	}
+	elseif (!$SysPrefs->allow_negative_stock())
 	{
-		$failed_item = $_SESSION['transfer_items']->check_qoh($_POST['FromStockLocation'], $_POST['AdjDate'], true);
-		if ($failed_item != null) 
+		$low_stock = $tr->check_qoh($_POST['FromStockLocation'], $_POST['AdjDate'], true);
+
+		if ($low_stock)
 		{
-        	display_error(_("The quantity entered is greater than the available quantity for this item at the source location :") .
-        		" " . $failed_item->stock_id . " - " .  $failed_item->item_description);
-        	echo "<br>";
+    		display_error(_("The transfer cannot be processed because it would cause negative inventory balance in source location for marked items as of document date or later."));
 			$input_error = 1;
 		}
 	}
@@ -135,7 +138,7 @@ if (isset($_POST['Process']))
 	$trans_no = add_stock_transfer($_SESSION['transfer_items']->line_items,
 		$_POST['FromStockLocation'], $_POST['ToStockLocation'],
 		$_POST['AdjDate'], $_POST['type'], $_POST['ref'], $_POST['memo_']);
-
+	new_doc_date($_POST['AdjDate']);
 	$_SESSION['transfer_items']->clear_items();
 	unset($_SESSION['transfer_items']);
 
@@ -146,18 +149,12 @@ if (isset($_POST['Process']))
 
 function check_item_data()
 {
-	if (!is_numeric($_POST['qty']) || ($_POST['qty'] == 0))
-	{
-		display_error( _("The quantity entered is not a valid number."));
-		return false;
-	}
-
-	if ($_POST['qty'] <= 0)
+	if (!check_num('qty', 0))
 	{
 		display_error(_("The quantity entered must be a positive number."));
+		set_focus('qty');
 		return false;
 	}
-
    	return true;
 }
 
@@ -167,17 +164,20 @@ function handle_update_item()
 {
     if($_POST['UpdateItem'] != "" && check_item_data())
     {
+		$id = $_POST['LineNo'];
     	if (!isset($_POST['std_cost']))
-    		$_POST['std_cost'] = $_SESSION['transfer_items']->line_items[$_POST['stock_id']]->standard_cost;
-    	$_SESSION['transfer_items']->update_cart_item($_POST['stock_id'], $_POST['qty'], $_POST['std_cost']);
+    		$_POST['std_cost'] = $_SESSION['transfer_items']->line_items[$id]->standard_cost;
+    	$_SESSION['transfer_items']->update_cart_item($id, input_num('qty'), $_POST['std_cost']);
     }
+	line_start_focus();
 }
 
 //-----------------------------------------------------------------------------------------------
 
-function handle_delete_item()
+function handle_delete_item($id)
 {
-	$_SESSION['transfer_items']->remove_from_cart($_GET['Delete']);
+	$_SESSION['transfer_items']->remove_from_cart($id);
+	line_start_focus();
 }
 
 //-----------------------------------------------------------------------------------------------
@@ -188,19 +188,14 @@ function handle_new_item()
 		return;
 	if (!isset($_POST['std_cost']))
    		$_POST['std_cost'] = 0;
-	add_to_order($_SESSION['transfer_items'], $_POST['stock_id'], $_POST['qty'], $_POST['std_cost']);
+	add_to_order($_SESSION['transfer_items'], $_POST['stock_id'], input_num('qty'), $_POST['std_cost']);
+	line_start_focus();
 }
 
 //-----------------------------------------------------------------------------------------------
-
-if (isset($_GET['Delete']) || isset($_GET['Edit']))
-	copy_from_st();
-
-if (isset($_GET['Delete']))
-	handle_delete_item();
-
-if (isset($_POST['AddItem']) || isset($_POST['UpdateItem']))
-	copy_to_st();
+$id = find_submit('Delete');
+if ($id != -1)
+	handle_delete_item($id);
 	
 if (isset($_POST['AddItem']))
 	handle_new_item();
@@ -208,6 +203,9 @@ if (isset($_POST['AddItem']))
 if (isset($_POST['UpdateItem']))
 	handle_update_item();
 
+if (isset($_POST['CancelItemChanges'])) {
+	line_start_focus();
+}
 //-----------------------------------------------------------------------------------------------
 
 if (isset($_GET['NewTransfer']) || !isset($_SESSION['transfer_items']))
@@ -216,12 +214,11 @@ if (isset($_GET['NewTransfer']) || !isset($_SESSION['transfer_items']))
 }
 
 //-----------------------------------------------------------------------------------------------
-
-start_form(false, true);
+start_form();
 
 display_order_header($_SESSION['transfer_items']);
 
-start_table("$table_style width=70%", 10);
+start_table(TABLESTYLE, "width='70%'", 10);
 start_row();
 echo "<td>";
 display_transfer_items(_("Items"), $_SESSION['transfer_items']);
@@ -230,16 +227,9 @@ echo "</td>";
 end_row();
 end_table(1);
 
-if (!isset($_POST['Process']))
-{
-	if ($_SESSION['transfer_items']->count_items() >= 1)
-	{
-    	submit_center_first('Update', _("Update"));
-	    submit_center_last('Process', _("Process Transfer"));
-	}
-	else
-    	submit_center('Update', _("Update"));
-}
+submit_center_first('Update', _("Update"), '', null);
+submit_center_last('Process', _("Process Transfer"), '',  'default');
+
 end_form();
 end_page();