Changeset 10894

Timestamp:

08/14/08 14:31:48 (5 months ago)

Author:

hartym

Message:

[1.2] fixes #3929

Files:

• branches/1.2/lib/validator/sfValidatorBase.class.php (modified) (1 diff)
• branches/1.2/test/unit/validator/sfValidatorBaseTest.php (modified) (3 diffs)

branches/1.2/lib/validator/sfValidatorBase.class.php

@@ -327 +327 @@
-  protected function isEmpty($value)
-  {
-)
+, array()), true
-  }
-

branches/1.2/test/unit/validator/sfValidatorBaseTest.php

@@ -11 +11 @@
-require_once(dirname(__FILE__).'/../../bootstrap/unit.php');
-
-38
+42
-
-class ValidatorIdentity extends sfValidatorBase
@@ -19 +19 @@
-    $this->addOption('foo', 'bar');
-    $this->addMessage('foo', 'bar');
+  }
+
+  public function testIsEmpty($value)
+  {
+    return $this->isEmpty($value);
-  }
-
@@ -107 +112 @@
-}
-$t->is($v->clean('  foo  '), '  foo  ', '->clean() does not trim whitespaces by default');
+
+// ->isEmpty()
+$t->diag('->isEmpty()');
+$t->is($v->testIsEmpty(null), true, 'null value isEmpty()');
+$t->is($v->testIsEmpty(''), true, 'empty string value is isEmpty()');
+$t->is($v->testIsEmpty(array()), true, 'null value is considered isEmpty()');
+$t->is($v->testIsEmpty(false), false, 'false value !isEmpty()');
-
-// ->getEmptyValue()