Changeset 1510 for branches/3.0

Timestamp:

11/25/2006 03:13:03 PM (2 years ago)

Author:

xue

Message:

Fixed #460.

Location:

branches/3.0

Files:

• HISTORY (modified) (1 diff)
• framework/Web/UI/WebControls/TBaseValidator.php (modified) (1 diff)

branches/3.0/HISTORY

@@ -5 +5 @@
 ENH: Ticket#446 - Added TMetaTagCollection.getMetaTagByID method (Qiang)
 CHG: Ticket#437 - __autoload is replaced by spl_autoload_register (Qiang)
-CHG: Ticket#454 - Redundant PHP Version Check
+CHG: Ticket#454 - Redundant PHP Version Check (Qiang)
+CHG: Ticket#460 - Validators will not perform validation if parents are disabled (Qiang)
 CHG: TRepeater does not render <span> anymore for empty item template (Qiang)
 CHG: THttpRequest.constructUrl() now encodes ampersand by default (Qiang)

branches/3.0/framework/Web/UI/WebControls/TBaseValidator.php

@@ -483 +483 @@
         public function validate()
         {
-                if($visible=$this->getVisible(true))
+                $this->setIsValid(true);
+                if($this->getVisible(true) && $this->getEnabled(true))
+                {
                         $this->onValidate();
-                $this->setIsValid(true);
-                if($this->getValidationTarget() && $visible && $this->getEnabled())
-                {
-                        if($this->evaluateIsValid())
+                        if($this->getValidationTarget())
                         {
-                                $this->setIsValid(true);
-                                $this->onSuccess();
-                        }
-                        else
-                        {
-                                $this->setIsValid(false);
-                                $this->onError();
+                                if($this->evaluateIsValid())
+                                {
+                                        $this->setIsValid(true);
+                                        $this->onSuccess();
+                                }
+                                else
+                                {
+                                        $this->setIsValid(false);
+                                        $this->onError();
+                                }
                         }
                 }