From 004d2bcae5c9f23751087cd8e7d624c781a4bbf9 Mon Sep 17 00:00:00 2001 From: Janusz Dobrowolski Date: Mon, 15 Mar 2010 18:04:38 +0000 Subject: [PATCH] Missing checks for customer/branch selection. --- sales/customer_payments.php | 14 ++++++++++++++ sales/sales_order_entry.php | 14 ++++++++++++++ 2 files changed, 28 insertions(+) diff --git a/sales/customer_payments.php b/sales/customer_payments.php index 7300450..939799f 100644 --- a/sales/customer_payments.php +++ b/sales/customer_payments.php @@ -78,6 +78,20 @@ function can_process() { global $Refs; + if (!get_post('customer_id')) + { + display_error(_("There is no customer selected.")); + set_focus('customer_id'); + return false; + } + + if (!get_post('BranchID')) + { + display_error(_("This customer has no branch defined.")); + set_focus('BranchID'); + return false; + } + if (!isset($_POST['DateBanked']) || !is_date($_POST['DateBanked'])) { display_error(_("The entered date is invalid. Please enter a valid date for the payment.")); set_focus('DateBanked'); diff --git a/sales/sales_order_entry.php b/sales/sales_order_entry.php index 690b16e..6ca955d 100644 --- a/sales/sales_order_entry.php +++ b/sales/sales_order_entry.php @@ -313,6 +313,20 @@ function line_start_focus() { function can_process() { global $Refs; + if (!get_post('customer_id')) + { + display_error(_("There is no customer selected.")); + set_focus('customer_id'); + return false; + } + + if (!get_post('branch_id')) + { + display_error(_("This customer has no branch defined.")); + set_focus('branch_id'); + return false; + } + if (!is_date($_POST['OrderDate'])) { display_error(_("The entered date is invalid.")); set_focus('OrderDate'); -- 2.30.2