projects
/
fa-stable.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
[0003964] Fixed bug in voiding GRN.
[fa-stable.git]
/
admin
/
void_transaction.php
diff --git
a/admin/void_transaction.php
b/admin/void_transaction.php
index a77cbce983571970a3629fcecc9a048179bc0f6a..3016e88d9bf2643dfffe35eaa487ac10109be939 100644
(file)
--- a/
admin/void_transaction.php
+++ b/
admin/void_transaction.php
@@
-23,7
+23,7
@@
include_once($path_to_root . "/admin/db/voiding_db.inc");
$js = "";
if (user_use_date_picker())
$js .= get_js_date_picker();
$js = "";
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);
$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
return false;
case ST_SUPPRECEIVE : // it's a GRN
- if (
exists_grn_on_invoices
($type_no))
+ if (
!exists_grn
($type_no))
return false;
break;
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;
return false;
case ST_COSTUPDATE : // it's a stock cost update
return false;
- break;
}
return true;
}
return true;
@@
-241,7
+240,7
@@
function check_valid_entries()
{
display_error(_("The selected transaction was closed for edition and cannot be voided."));
set_focus('trans_no');
{
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_']))
{
}
if (!is_date($_POST['date_']))
{
@@
-251,7
+250,7
@@
function check_valid_entries()
}
if (!is_date_in_fiscalyear($_POST['date_']))
{
}
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;
}
set_focus('date_');
return false;
}
@@
-283,10
+282,10
@@
function handle_void_transaction()
return;
}
return;
}
- $
ret
= void_transaction($_POST['filterType'], $_POST['trans_no'],
+ $
msg
= void_transaction($_POST['filterType'], $_POST['trans_no'],
$_POST['date_'], $_POST['memo_']);
$_POST['date_'], $_POST['memo_']);
- if (
$ret
)
+ if (
!$msg
)
{
display_notification_centered(_("Selected transaction has been voided."));
unset($_POST['trans_no']);
{
display_notification_centered(_("Selected transaction has been voided."));
unset($_POST['trans_no']);
@@
-294,7
+293,7
@@
function handle_void_transaction()
unset($_POST['date_']);
}
else {
unset($_POST['date_']);
}
else {
- display_error(
_("The entered transaction does not exist or cannot be voided.")
);
+ display_error(
$msg
);
set_focus('trans_no');
}
set_focus('trans_no');
}