X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=admin%2Fvoid_transaction.php;h=68413f9aa3ea54fad6ca76a23c2bc3e2071224dd;hb=df660a601cabb0afe7f3246884f16207b9a0c314;hp=474217a0c32a942cccea23ed0225b8d865884327;hpb=53d942f2a0d20cce5e9c409c6485867ce0869e4d;p=fa-stable.git

diff --git a/admin/void_transaction.php b/admin/void_transaction.php
index 474217a0..68413f9a 100644
--- a/admin/void_transaction.php
+++ b/admin/void_transaction.php
@@ -21,9 +21,9 @@ include_once($path_to_root . "/admin/db/transactions_db.inc");
 
 include_once($path_to_root . "/admin/db/voiding_db.inc");
 $js = "";
-if ($use_date_picker)
+if (user_use_date_picker())
 	$js .= get_js_date_picker();
-if ($use_popup_windows)
+if ($SysPrefs->use_popup_windows)
 	$js .= get_js_open_window(800, 500);
 	
 page(_($help_context = "Void a Transaction"), false, false, "", $js);
@@ -73,7 +73,7 @@ function exist_transaction($type, $type_no)
 			return false;
 
 		case ST_SUPPRECEIVE : // it's a GRN
-			if (exists_grn_on_invoices($type_no))
+			if (!exists_grn($type_no))
 				return false;
 			break;
 
@@ -104,7 +104,6 @@ function exist_transaction($type, $type_no)
 			return false;
 		case ST_COSTUPDATE : // it's a stock cost update
 			return false;
-			break;
 	}
 
 	return true;
@@ -143,6 +142,12 @@ function ref_view($row)
 	return $row['ref'];
 }
 
+function is_selected($row) // Function added by faisal
+{
+	global $selected_id;
+	return $row['trans_no'] == $selected_id ? true : false;
+}
+
 function voiding_controls()
 {
 	global $selected_id;
@@ -173,19 +178,21 @@ function voiding_controls()
     end_table(1);
     
 	$trans_ref = false;
-	$sql = get_sql_for_view_transactions($_POST['filterType'], $_POST['FromTransNo'], $_POST['ToTransNo'], $trans_ref);
+	$sql = get_sql_for_view_transactions(get_post('filterType'), get_post('FromTransNo'), get_post('ToTransNo'), $trans_ref);
 	if ($sql == "")
 		return;
 
 	$cols = array(
-		_("#") => array('insert'=>true, 'fun'=>'view_link'), 
-		_("Reference") => array('fun'=>'ref_view'), 
+		_("#") => array('insert'=>true, 'fun'=>'view_link'),
+		_("Reference") => array('fun'=>'ref_view'),
 		_("Date") => array('type'=>'date', 'fun'=>'date_view'),
 		_("GL") => array('insert'=>true, 'fun'=>'gl_view'),
-		_("Select") => array('insert'=>true, 'fun'=>'select_link') 
+		_("Select") => array('insert'=>true, 'fun'=>'select_link')
 	);
 
 	$table =& new_db_pager('transactions', $sql, $cols);
+	$table->set_marker('is_selected', _("Marked transactions will be voided.")); //Added by Faisal
+
 	$table->width = "40%";
 	display_db_pager($table);
 
@@ -223,7 +230,24 @@ function voiding_controls()
 		}	
  		else
  		{
-    		display_warning(_("Are you sure you want to void this transaction ? This action cannot be undone."), 0, 1);
+           	if ($_POST['filterType'] == ST_SUPPRECEIVE) { 
+                $result = get_grn_items($_POST['trans_no']);
+                if (db_num_rows($result) > 0) {
+                    while ($myrow = db_fetch($result)) {
+                        if (is_inventory_item($myrow["item_code"])) {
+                            if (check_negative_stock($myrow["item_code"], -$myrow["qty_recd"], null, $_POST['date_'])) {
+                                $stock = get_item($myrow["item_code"]);
+                                display_error(_("The void cannot be processed because there is an insufficient quantity for item:") .
+                                    " " . $stock['stock_id'] . " - " . $stock['description'] . " - " .
+                                    _("Quantity On Hand") . " = " . number_format2(get_qoh_on_date($stock['stock_id'], null, 
+                                    $_POST['date_']), get_qty_dec($stock['stock_id'])));
+                                return false;
+                            }
+                        }
+                    }
+                }
+            }
+       		display_warning(_("Are you sure you want to void this transaction ? This action cannot be undone."), 0, 1);
    			br();
     		submit_center_first('ConfirmVoiding', _("Proceed"), '', true);
     		submit_center_last('CancelVoiding', _("Cancel"), '', 'cancel');
@@ -241,7 +265,7 @@ function check_valid_entries()
 	{
 		display_error(_("The selected transaction was closed for edition and cannot be voided."));
 		set_focus('trans_no');
-		return;
+		return false;
 	}
 	if (!is_date($_POST['date_']))
 	{
@@ -251,7 +275,7 @@ function check_valid_entries()
 	}
 	if (!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;
 	}
@@ -283,18 +307,17 @@ function handle_void_transaction()
 			return;
 		}
 
-		$ret = void_transaction($_POST['filterType'], $_POST['trans_no'],
+		$msg = void_transaction($_POST['filterType'], $_POST['trans_no'],
 			$_POST['date_'], $_POST['memo_']);
 
-		if ($ret) 
+		if (!$msg) 
 		{
 			display_notification_centered(_("Selected transaction has been voided."));
 			unset($_POST['trans_no']);
 			unset($_POST['memo_']);
-			unset($_POST['date_']);
 		}
 		else {
-			display_error(_("The entered transaction does not exist or cannot be voided."));
+			display_error($msg);
 			set_focus('trans_no');
 
 		}
@@ -320,6 +343,7 @@ if (isset($_POST['ProcessVoiding']))
 if (isset($_POST['ConfirmVoiding']))
 {
 	handle_void_transaction();
+	$selected_id = '';
 	$Ajax->activate('_page_body');
 }
 
@@ -335,4 +359,3 @@ voiding_controls();
 
 end_page();
 
-?>
\ No newline at end of file