Changeset 459

Timestamp:

Jan 27, 2014 5:52:20 PM (10 years ago)

Author:

anonymous

Message:

Revent null values going to validateStrDate

Location:

trunk/lib

Files:

• FormValidator.inc.php (modified) (13 diffs)
• Validator.inc.php (modified) (1 diff)

trunk/lib/FormValidator.inc.php

@@ -4 +4 @@
  * For details visit the project site: <http://trac.strangecode.com/codebase/>
  * Copyright 2001-2012 Strangecode, LLC
- * 
+ *
  * This file is part of The Strangecode Codebase.
  *
@@ -11 +11 @@
  * Free Software Foundation, either version 3 of the License, or (at your option)
  * any later version.
- * 
+ *
  * The Strangecode Codebase is distributed in the hope that it will be useful, but
  * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
  * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for more
  * details.
- * 
+ *
  * You should have received a copy of the GNU General Public License along with
  * The Strangecode Codebase. If not, see <http://www.gnu.org/licenses/>.
@@ -70 +70 @@
     // Array filling with error messages.
     var $errors = array();
-    
+
     /**
      * Set (or overwrite existing) parameters by passing an array of new parameters.
@@ -80 +80 @@
     {
         $app =& App::getInstance();
-    
+
         if (isset($params) && is_array($params)) {
             // Merge new parameters with old overriding only those passed.
@@ -99 +99 @@
     {
         $app =& App::getInstance();
-    
+
         if (isset($this->_params[$param])) {
             return $this->_params[$param];
@@ -107 +107 @@
         }
     }
-    
+
     /**
      * Return the current list of errors.
@@ -159 +159 @@
             return false;
         } else {
-            return (sizeof($this->errors) > 0);            
+            return (sizeof($this->errors) > 0);
         }
     }
@@ -283 +283 @@
         }
     }
-    
+
     /*
     * We were using the isEmpty method *wrong* all these years and should have been using notEmpty.
-    * But the fact is the only use is to ensure a value is not empty, so this function simply becomes 
+    * But the fact is the only use is to ensure a value is not empty, so this function simply becomes
     * an alias of the one-true notEmpty() function.
     * @since    03 Jun 2006 22:56:46
@@ -482 +482 @@
             return false;
             break;
-            
+
         case VALIDATE_EMAIL_LENGTH_FAIL :
             // Failed length requirements.
@@ -489 +489 @@
             return false;
             break;
-            
+
         case VALIDATE_EMAIL_MX_FAIL :
             // Failed MX record test.
@@ -496 +496 @@
             return false;
             break;
-            
+
         case VALIDATE_EMAIL_SUCCESS :
         default :
@@ -535 +535 @@
         $app =& App::getInstance();
 
-        if (parent::validateStrDate(getFormData($form_name))) {
+        if (parent::validateStrDate(getFormData($form_name, ''))) {
             return true;
         } else {
@@ -557 +557 @@
     {
         $cc_num = getFormData($form_name);
-        
+
         if (parent::validateCCNumber($cc_num, $cc_type)) {
             return true;

trunk/lib/Validator.inc.php

@@ -218 +218 @@
     /**
      * Verifies that date can be processed by the strtotime function.
+     * Empty strings are considered valid. Other values are tested on their return value from strtotime(). Null values will fail.
      *
      * @param  string  $val The input data to validate.