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