Fixed #223 #225

2 files changed, 74 insertions, 11 deletions

diff --git a/framework/Web/UI/WebControls/TRadioButton.php b/framework/Web/UI/WebControls/TRadioButton.php
index 9198be1b..6ddd7c96 100644
--- a/framework/Web/UI/WebControls/TRadioButton.php
+++ b/framework/Web/UI/WebControls/TRadioButton.php
@@ -37,6 +37,8 @@ Prado::using('System.Web.UI.WebControls.TRadioButtonList');
  * between posts to the server.

  *

  * TRadioButton uses {@link setGroupName GroupName} to group together a set of radio buttons.

+ * Once the {@link setGroupName GroupName} is set, you can use the {@link getRadioButtonsInGroup}

+ * method to get an array of TRadioButtons having the same group name.

  *

  * If {@link setAutoPostBack AutoPostBack} is set true, changing the radio button state

  * will cause postback action. And if {@link setCausesValidation CausesValidation}

@@ -54,6 +56,10 @@ Prado::using('System.Web.UI.WebControls.TRadioButtonList');
 class TRadioButton extends TCheckBox

 {

 	/**

+	 * @param array list of radio buttons registered.

+	 */

+	private static $_radiobuttons=array();	

+	/**

 	 * @var string the name used to fetch radiobutton post data

 	 */

 	private $_uniqueGroupName=null;

@@ -99,6 +105,36 @@ class TRadioButton extends TCheckBox
 	public function setGroupName($value)

 	{

 		$this->setViewState('GroupName',$value,'');

+		$this->registerRadioButton($this);

+	}

+	

+	/**

+	 * Register radio button control grouping.

+	 * @param TRadioButton control to add

+	 */

+	private function registerRadioButton(TRadioButton $control)

+	{

+		$id = $control->getUniqueID();

+		if(!isset(self::$_radiobuttons[$id]))

+			self::$_radiobuttons[$id] = $control;

+	}

+	

+	/**  

+	 * Gets an array of RadioButtons with same group name.

+	 * Radio buttons are grouped when the GroupName property is set.

+	 * This method will always return at least the current radio button in the array.

+	 * @return array list of TRadioButton with the same group

+	 */

+	public function getRadioButtonsInGroup()

+	{

+		$group = $this->getUniqueGroupName();

+		$buttons = array();

+		foreach(self::$_radiobuttons as $control)

+		{

+			if($control->getUniqueGroupName() === $group)

+				$buttons[] = $control;

+		}

+		return count($buttons) > 0 ? $buttons : array($this);

 	}

 

 	/**

diff --git a/framework/Web/UI/WebControls/TRequiredFieldValidator.php b/framework/Web/UI/WebControls/TRequiredFieldValidator.php
index b8a939ad..c99d9c19 100644
--- a/framework/Web/UI/WebControls/TRequiredFieldValidator.php
+++ b/framework/Web/UI/WebControls/TRequiredFieldValidator.php
@@ -76,22 +76,47 @@ class TRequiredFieldValidator extends TBaseValidator
 	protected function evaluateIsValid()

 	{

 		$control = $this->getValidationTarget();

-		$initial = trim($this->getInitialValue());

 		if($control instanceof TListControl)

+			return $this->validateListControl($control);

+		else if($control instanceof TRadioButton && strlen($control->getGroupName()) > 0)

+			return $this->validateRadioButtonGroup($control);

+		else

+			return $this->validateStandardControl($control);

+	}

+	

+	private function validateListControl($control)

+	{

+		$initial = trim($this->getInitialValue());

+		$count = 0;

+		foreach($control->getItems() as $item)

 		{

-			$count = 0;

-			foreach($control->getItems() as $item)

-			{

-				if($item->getSelected() && $item->getValue() != $initial)

-					$count++;

-			}

-			return $count > 0;

+			if($item->getSelected() && $item->getValue() != $initial)

+				$count++;

 		}

-		else

+		return $count > 0;

+	}

+	

+	private function validateRadioButtonGroup($control)

+	{

+		$initial = trim($this->getInitialValue());

+		foreach($control->getRadioButtonsInGroup() as $radio)

 		{

-			$value=$this->getValidationValue($control);

-			return trim($value)!==$initial || (is_bool($value) && $value);

+			if($radio->getChecked())

+			{

+				if(strlen($value = $radio->getValue()) > 0)

+					return $value !== $initial;

+				else

+					return true;

+			}

 		}

+		return false;

+	}

+	

+	private function validateStandardControl($control)

+	{

+		$initial = trim($this->getInitialValue());

+		$value=$this->getValidationValue($control);

+		return trim($value)!==$initial || (is_bool($value) && $value);

 	}

 

 	/**

@@ -105,6 +130,8 @@ class TRequiredFieldValidator extends TBaseValidator
 		$control = $this->getValidationTarget();

 		if($control instanceof TListControl)

 			$options['TotalItems'] = $control->getItemCount();

+		if($control instanceof TRadioButton && strlen($control->getGroupName()) > 0)

+			$options['GroupName'] = $control->getGroupName();

 		return $options;

 	}

 }