From 412eec8a5fb6208d361c033653015ff3c74ad3ab Mon Sep 17 00:00:00 2001 From: ftimme <ft@falkotimme.com> Date: Thu, 11 Jul 2013 10:59:00 -0400 Subject: [PATCH] - Improved recognition of when a service (Apache, nginx, etc.) was really restarted/reloaded successfully. Checking port 80 is not enough because nginx reloads silently, ignoring faulty configuration files, but returns a value > 0. If we don't check the return value, nginx will refuse to start when someone issues a restart the next time. --- interface/lib/classes/validate_datetime.inc.php | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/interface/lib/classes/validate_datetime.inc.php b/interface/lib/classes/validate_datetime.inc.php index b13e0ac..42afeaf 100755 --- a/interface/lib/classes/validate_datetime.inc.php +++ b/interface/lib/classes/validate_datetime.inc.php @@ -74,7 +74,8 @@ */ function _get_timestamp_value($field_value) { - $second = 0; + if(!is_array($field_value)) return 0; + $second = 0; $filtered_values = array_map(create_function('$item','return (int)$item;'), $field_value); extract($filtered_values, EXTR_OVERWRITE); @@ -93,6 +94,7 @@ */ function not_empty($field_name, $field_value, $validator) { + if(!is_array($field_value)) return $this->_get_error($validator['errmsg']); extract($field_value); if ( !($day > 0 && $month > 0 && $year > 0) ) { return $this->_get_error($validator['errmsg']); -- Gitblit v1.9.1