From a128bcbb38a664bb2a07d0beb8bfb777e4b5da44 Mon Sep 17 00:00:00 2001 From: msquare Date: Thu, 29 Sep 2016 16:04:05 +0200 Subject: use more ValidationResult --- includes/model/AngelType_model.php | 32 ++++++++++++++------------------ 1 file changed, 14 insertions(+), 18 deletions(-) (limited to 'includes/model/AngelType_model.php') diff --git a/includes/model/AngelType_model.php b/includes/model/AngelType_model.php index 21ab3bf2..73b746f0 100644 --- a/includes/model/AngelType_model.php +++ b/includes/model/AngelType_model.php @@ -55,38 +55,34 @@ function AngelType_create($name, $restricted, $description, $requires_driver_lic /** * Validates a name for angeltypes. - * Returns array containing validation success and validated name. + * Returns ValidationResult containing validation success and validated name. * - * @param string $name - * @param AngelType $angeltype + * @param string $name + * Wanted name for the angeltype + * @param AngelType $angeltype + * The angeltype the name is for + * @return ValidationResult result and validated name */ function AngelType_validate_name($name, $angeltype) { $name = strip_item($name); if ($name == "") { - return [ - false, - $name - ]; + return new ValidationResult(false, ""); } - if (isset($angeltype) && isset($angeltype['id'])) { - return [ - sql_num_query(" + if ($angeltype != null && isset($angeltype['id'])) { + $valid = sql_num_query(" SELECT * FROM `AngelTypes` WHERE `name`='" . sql_escape($name) . "' AND NOT `id`='" . sql_escape($angeltype['id']) . "' - LIMIT 1") == 0, - $name - ]; + LIMIT 1") == 0; + return new ValidationResult($valid, $name); } - return [ - sql_num_query(" + $valid = sql_num_query(" SELECT `id` FROM `AngelTypes` WHERE `name`='" . sql_escape($name) . "' - LIMIT 1") == 0, - $name - ]; + LIMIT 1") == 0; + return new ValidationResult($valid, $name); } /** -- cgit v1.2.3-54-g00ecf