ifl/iflrandevu
1
1
Fork 0
forked from mirrors/easyappointments
Code Issues 6 Pull requests Releases 3 Activity

Fix appointment required fields validation

Browse source 
We should be using empty to check the fields instead of isset. Otherwise,
appointments may be booked with empty fields if these are not properly validated
client-side for some reason, since they will be empty strings. Also, we should
check first_name.
This commit is contained in:
DiGitHubCap 2018-10-27 21:33:22 -04:00
parent da98768cd0
commit c50c545d98
1 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/application/models/Customers_model.php
View file

@ -68,7 +68,7 @@ class Customers_Model extends CI_Model {
*/
public function exists($customer)
{
if ( ! isset($customer['email']))
if (empty($customer['email']))
{
throw new Exception('Customer\'s email is not provided.');
}
@ -163,7 +163,7 @@ class Customers_Model extends CI_Model {
*/
public function find_record_id($customer)
{
if ( ! isset($customer['email']))
if (empty($customer['email']))
{
throw new Exception('Customer\'s email was not provided: '
. print_r($customer, TRUE));
@ -212,9 +212,10 @@ class Customers_Model extends CI_Model {
}
}
// Validate required fields
if ( ! isset($customer['last_name'])
|| ! isset($customer['email'])
|| ! isset($customer['phone_number']))
if (empty($customer['first_name'])
|| empty($customer['last_name'])
|| empty($customer['email'])
|| empty($customer['phone_number']))
{
throw new Exception('Not all required fields are provided: '
. print_r($customer, TRUE));