X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=purchasing%2Fpo_entry_items.php;h=388eb772158c7449dee682d35dfc5b0d8e91705c;hb=90b3d069d96b99671af51726e2953352738abb75;hp=84fe0e38642443572beeab234325a99f3f0c905e;hpb=4e4ce8577f8582967995919860c66aa94d633afd;p=fa-stable.git

diff --git a/purchasing/po_entry_items.php b/purchasing/po_entry_items.php
index 84fe0e38..388eb772 100644
--- a/purchasing/po_entry_items.php
+++ b/purchasing/po_entry_items.php
@@ -57,6 +57,9 @@ if (isset($_GET['ModifyOrderNumber']) && is_numeric($_GET['ModifyOrderNumber']))
 
 page($_SESSION['page_title'], false, false, "", $js);
 
+if (isset($_GET['ModifyOrderNumber']))
+	check_is_closed(ST_PURCHORDER, $_GET['ModifyOrderNumber']);
+
 //---------------------------------------------------------------------------------------------------
 
 check_db_has_suppliers(_("There are no suppliers defined in the system."));
@@ -327,21 +330,27 @@ function can_commit()
 		set_focus('supplier_id');
 		return false;
 	} 
-	
+
 	if (!is_date($_POST['OrderDate'])) 
 	{
 		display_error(_("The entered order date is invalid."));
 		set_focus('OrderDate');
 		return false;
 	} 
-	
+	if (($_SESSION['PO']->trans_type == ST_SUPPRECEIVE || $_SESSION['PO']->trans_type == ST_SUPPINVOICE) 
+		&& !is_date_in_fiscalyear($_POST['OrderDate'])) {
+		display_error(_("The entered date is out of fiscal year or is closed for further data entry."));
+		set_focus('OrderDate');
+		return false;
+	}
+
 	if (($_SESSION['PO']->trans_type==ST_SUPPINVOICE) && !is_date($_POST['due_date'])) 
 	{
 		display_error(_("The entered due date is invalid."));
 		set_focus('due_date');
 		return false;
 	} 
-	
+
 	if (!$_SESSION['PO']->order_no) 
 	{
     	if (!$Refs->is_valid(get_post('ref'))) 
@@ -350,7 +359,7 @@ function can_commit()
 			set_focus('ref');
     		return false;
     	} 
-    	
+
     	if (!is_new_reference(get_post('ref'), $_SESSION['PO']->trans_type)) 
     	{
     		display_error(_("The entered reference is already in use."));
@@ -358,13 +367,20 @@ function can_commit()
     		return false;
     	}
 	}
-	
+
 	if ($_SESSION['PO']->trans_type == ST_SUPPINVOICE && !$Refs->is_valid(get_post('supp_ref'))) 
 	{
 		display_error(_("You must enter a supplier's invoice reference."));
 		set_focus('supp_ref');
 		return false;
 	}
+	if ($_SESSION['PO']->trans_type==ST_SUPPINVOICE 
+		&& is_reference_already_there($_SESSION['PO']->supplier_id, get_post('supp_ref'), $_SESSION['PO']->order_no))
+	{
+		display_error(_("This invoice number has already been entered. It cannot be entered again.") . " (" . get_post('supp_ref') . ")");
+		set_focus('supp_ref');
+		return false;
+	}
 	if ($_SESSION['PO']->trans_type == ST_PURCHORDER && get_post('delivery_address') == '')
 	{
 		display_error(_("There is no delivery address specified."));
@@ -391,6 +407,8 @@ function can_commit()
 
 function handle_commit_order()
 {
+	global $Refs, $type_shortcuts;
+
 	$cart = &$_SESSION['PO'];
 
 	if (can_commit()) {
@@ -437,24 +455,43 @@ function handle_commit_order()
 			$inv->reference = $ref;
 			$inv->supp_reference = $cart->supp_ref;
 			$inv->tax_included = $cart->tax_included;
+			$inv->tax_algorithm = $cart->tax_algorithm;
+			$inv->stored_algorithm = $cart->stored_algorithm;
 			$supp = get_supplier($cart->supplier_id);
 			$inv->tax_group_id = $supp['tax_group_id'];
-//			$inv->ov_discount 'this isn't used at all'
-			$inv->ov_amount = $inv->ov_gst = 0;
-			
+			$total = 0;
+
 			foreach($cart->line_items as $key => $line) {
 				$inv->add_grn_to_trans($line->grn_item_id, $line->po_detail_rec, $line->stock_id,
 					$line->item_description, $line->receive_qty, 0, $line->receive_qty,
 					$line->price, $line->price, true, get_standard_cost($line->stock_id), '');
-				$inv->ov_amount += round2(($line->receive_qty * $line->price), user_price_dec());
+				$total += round2(($line->receive_qty * $line->price), user_price_dec());
 			}
-			$taxes = $inv->get_taxes($inv->tax_group_id, 0, false);
-			foreach( $taxes as $taxitem) {
-				$inv->ov_gst += round2($taxitem['Value'], user_price_dec());
+			if (!$inv->tax_included) {
+				$taxes = $inv->get_taxes($inv->tax_group_id, 0, false, $inv->tax_algorithm);
+				foreach( $taxes as $taxitem) {
+					$total += $taxitem['Value'];
+				}
 			}
+
 			$inv_no = add_supp_invoice($inv);
-			commit_transaction(); // save PO+GRN+PI
-			// FIXME payment for cash terms. (Needs cash account selection)
+			// presume supplier data need correction
+			if ($inv->stored_algorithm != $inv->tax_algorithm)
+				update_supp_tax_algorithm($inv->supplier_id, $inv->tax_algorithm);
+
+			if (get_post('cash_account')) {
+
+				$pmt_no = add_supp_payment($inv->supplier_id, $inv->tran_date, get_post('cash_account'),
+					$total, 0,	$Refs->get_next(ST_SUPPAYMENT), 
+					_('Payment for:').$inv->supp_reference .' ('.$type_shortcuts[ST_SUPPINVOICE].$inv_no.')');
+				add_supp_allocation($total, ST_SUPPAYMENT, $pmt_no, ST_SUPPINVOICE, $inv_no, $inv->tran_date);
+				update_supp_trans_allocation(ST_SUPPINVOICE, $inv_no, $total);
+				update_supp_trans_allocation(ST_SUPPAYMENT, $pmt_no, $total);
+
+			}
+
+			commit_transaction(); // save PO+GRN+PI(+SP)
+
 			unset($_SESSION['PO']);
        		meta_forward($_SERVER['PHP_SELF'], "AddedPI=$inv_no");
 		}
@@ -501,6 +538,17 @@ echo "<br>";
 display_po_items($_SESSION['PO']);
 
 start_table(TABLESTYLE2);
+
+if (list_updated('tax_algorithm')) {
+	$_SESSION['PO']->tax_algorithm = $_POST['tax_algorithm'];
+    $Ajax->activate('items_table');
+}
+
+if ($_SESSION['PO']->trans_type == ST_SUPPINVOICE) {
+	tax_algorithm_list_row(_("Tax algorithm:"), 'tax_algorithm', null, true);
+	cash_accounts_list_row(_("Payment:"), 'cash_account', null, false, _('Delayed'));
+}
+
 textarea_row(_("Memo:"), 'Comments', null, 70, 4);
 
 end_table(1);