X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=purchasing%2Fsupplier_invoice_grns.php;h=13f4d3040b78c7182b9b6c1d3668243e1ed07fdc;hb=3a1f2b3032483a04ffca4bb037ddb5c73d4730a7;hp=95c345192056b535f22fea52fa7d48f5d563cd8b;hpb=d34add5e762a470a52fade37173b4491462ee22a;p=fa-stable.git diff --git a/purchasing/supplier_invoice_grns.php b/purchasing/supplier_invoice_grns.php index 95c34519..13f4d304 100644 --- a/purchasing/supplier_invoice_grns.php +++ b/purchasing/supplier_invoice_grns.php @@ -36,23 +36,29 @@ function check_data() if (!check_num('this_quantity_inv', 0) || input_num('this_quantity_inv')==0) { display_error( _("The quantity to invoice must be numeric and greater than zero.")); + set_focus('this_quantity_inv'); return false; } if (!check_num('ChgPrice')) { display_error( _("The price is not numeric.")); + set_focus('ChgPrice'); return false; } if ($check_price_charged_vs_order_price == True) { - if (input_num('ChgPrice')/$_POST['order_price'] > - (1 + (sys_prefs::over_charge_allowance() / 100))) - { + if ($_POST['order_price']!=input_num('ChgPrice')) { + if ($_POST['order_price']==0 || + input_num('ChgPrice')/$_POST['order_price'] > + (1 + (sys_prefs::over_charge_allowance() / 100))) + { display_error(_("The price being invoiced is more than the purchase order price by more than the allowed over-charge percentage. The system is set up to prohibit this. See the system administrator to modify the set up parameters if necessary.") . _("The over-charge percentage allowance is :") . sys_prefs::over_charge_allowance() . "%"); + set_focus('ChgPrice'); return false; + } } } @@ -63,6 +69,7 @@ function check_data() { display_error( _("The quantity being invoiced is more than the outstanding quantity by more than the allowed over-charge percentage. The system is set up to prohibit this. See the system administrator to modify the set up parameters if necessary.") . _("The over-charge percentage allowance is :") . sys_prefs::over_charge_allowance() . "%"); + set_focus('this_quantity_inv'); return false; } }