merge from 3.0 branch till 1281.

6 files changed, 76 insertions, 18 deletions

diff --git a/framework/Web/UI/TControl.php b/framework/Web/UI/TControl.php
index aa8ea2a3..2bc6ba6a 100644
--- a/framework/Web/UI/TControl.php
+++ b/framework/Web/UI/TControl.php
@@ -1423,27 +1423,36 @@ class TControl extends TApplicationComponent implements IRenderable, IBindable
 	/**

 	 * Broadcasts an event.

 	 * The event will be sent to all controls on the current page hierarchy.

-	 * If this control is not on a page, the event will be sent to all its

-	 * child controls recursively.

-	 * Controls implementing {@link IBroadcastEventReceiver} will get a chance

-	 * to respond to the event.

-	 * @param TControl sender of the event

-	 * @param TBroadcastEventParameter event parameter

+	 * If a control defines the event, the event will be raised for the control.

+	 * If a control implements {@link IBroadcastEventReceiver}, its

+	 * {@link IBroadcastEventReceiver::broadcastEventReceived broadcastEventReceived()} method will

+	 * be invoked which gives the control a chance to respond to the event.

+	 * For example, when broadcasting event 'OnClick', all controls having 'OnClick'

+	 * event will have this event raised, and all controls implementing

+	 * {@link IBroadcastEventReceiver} will also have its

+	 * {@link IBroadcastEventReceiver::broadcastEventReceived broadcastEventReceived()}

+	 * invoked.

+	 * @param string name of the broadcast event

+	 * @param TControl sender of this event

+	 * @param TEventParameter event parameter

 	 */

-	protected function broadcastEvent($sender,TBroadCastEventParameter $param)

+	public function broadcastEvent($name,$sender,$param)

 	{

-		$origin=(($page=$this->getPage())===null)?$this:$page;

-		$origin->broadcastEventInternal($sender,$param);

+		$rootControl=(($page=$this->getPage())===null)?$this:$page;

+		$rootControl->broadcastEventInternal($name,$sender,new TBroadcastEventParameter($name,$param));

 	}

 

 	/**

 	 * Recursively broadcasts an event.

 	 * This method should only be used by framework developers.

+	 * @param string name of the broadcast event

 	 * @param TControl sender of the event

 	 * @param TBroadcastEventParameter event parameter

 	 */

-	final protected function broadcastEventInternal($sender,$param)

+	private function broadcastEventInternal($name,$sender,$param)

 	{

+		if($this->hasEvent($name))

+			$this->raiseEvent($name,$sender,$param->getParameter());

 		if($this instanceof IBroadcastEventReceiver)

 			$this->broadcastEventReceived($sender,$param);

 		if($this->getHasControls())

@@ -1451,7 +1460,7 @@ class TControl extends TApplicationComponent implements IRenderable, IBindable
 			foreach($this->_rf[self::RF_CONTROLS] as $control)

 			{

 				if($control instanceof TControl)

-					$control->broadcastEventInternal($sender,$param);

+					$control->broadcastEventInternal($name,$sender,$param);

 			}

 		}

 	}

diff --git a/framework/Web/UI/TTemplateControl.php b/framework/Web/UI/TTemplateControl.php
index 0595a9e5..c7364d4b 100644
--- a/framework/Web/UI/TTemplateControl.php
+++ b/framework/Web/UI/TTemplateControl.php
@@ -119,7 +119,6 @@ class TTemplateControl extends TCompositeControl
 	{

 		Prado::trace("Loading template ".get_class($this),'System.Web.UI.TTemplateControl');

 		$template=$this->getService()->getTemplateManager()->getTemplateByClassName(get_class($this));

-		self::$_template[get_class($this)]=$template;

 		return $template;

 	}

 

diff --git a/framework/Web/UI/WebControls/TBaseValidator.php b/framework/Web/UI/WebControls/TBaseValidator.php
index 9366de62..6660184e 100644
--- a/framework/Web/UI/WebControls/TBaseValidator.php
+++ b/framework/Web/UI/WebControls/TBaseValidator.php
@@ -11,6 +11,11 @@
  */
 
 /**
+ * Using TLabel class
+ */
+Prado::using('System.Web.UI.WebControls.TLabel');
+
+/**
  * TBaseValidator class
  *
  * TBaseValidator serves as the base class for validator controls.
@@ -474,10 +479,10 @@ abstract class TBaseValidator extends TLabel implements IValidator
 	 */
 	public function validate()
 	{
-		$this->onValidate();
+		if($visible=$this->getVisible(true))
+			$this->onValidate();
 		$this->setIsValid(true);
-		$control=$this->getValidationTarget();
-		if($control && $this->getVisible(true) && $this->getEnabled())
+		if($this->getValidationTarget() && $visible && $this->getEnabled())
 		{
 			if($this->evaluateIsValid())
 			{
@@ -534,6 +539,9 @@ abstract class TBaseValidator extends TLabel implements IValidator
 
 	/**
 	 * This event is raised right before the validator starts to perform validation.
+	 * You may use this event to change the behavior of validation.
+	 * For example, you may disable the validator if certain condition is satisfied.
+	 * Note, the event will NOT be raised if the validator is invisible.
 	 */
 	public function onValidate()
 	{
@@ -666,4 +674,4 @@ class TClientSideValidatorOptions extends TClientSideOptions
 	}
 }
 
-?>
+?>
\ No newline at end of file
diff --git a/framework/Web/UI/WebControls/TDataGrid.php b/framework/Web/UI/WebControls/TDataGrid.php
index f7697fec..2687ac35 100644
--- a/framework/Web/UI/WebControls/TDataGrid.php
+++ b/framework/Web/UI/WebControls/TDataGrid.php
@@ -43,6 +43,12 @@ Prado::using('System.Web.UI.WebControls.TPanel');
  * To change the state of an item, set {@link setEditItemIndex EditItemIndex}

  * or {@link setSelectedItemIndex SelectedItemIndex} property.

  *

+ * Each datagrid item has a {@link TDataGridItem::getItemType type}

+ * which tells the position and state of the item in the datalist. An item in the header

+ * of the repeater is of type 'Header'. A body item may be of either

+ * 'Item', 'AlternatingItem', 'SelectedItem' or 'EditItem', depending whether the item

+ * index is odd or even, whether it is being selected or edited.

+ *

  * A datagrid is specified with a list of columns. Each column specifies how the corresponding

  * table column will be displayed. For example, the header/footer text of that column,

  * the cells in that column, and so on. The following column types are currently

@@ -128,6 +134,14 @@ Prado::using('System.Web.UI.WebControls.TPanel');
  * every newly created datagrid item. You can respond to this event to customize

  * the content or style of the newly created item.

  *

+ * Note, the data bound to the datagrid are reset to null after databinding.

+ * There are several ways to access the data associated with a datagrid row:

+ * - Access the data in {@link onItemDataBound OnItemDataBound} event

+ * - Use {@link getDataKeys DataKeys} to obtain the data key associated with

+ * the specified datagrid row and use the key to fetch the corresponding data

+ * from some persistent storage such as DB.

+ * - Save the data in viewstate and get it back during postbacks.

+ *

  * @author Qiang Xue <qiang.xue@gmail.com>

  * @version $Revision: $  $Date: $

  * @package System.Web.UI.WebControls

diff --git a/framework/Web/UI/WebControls/TDataList.php b/framework/Web/UI/WebControls/TDataList.php
index 2c562d6d..42c47f86 100644
--- a/framework/Web/UI/WebControls/TDataList.php
+++ b/framework/Web/UI/WebControls/TDataList.php
@@ -85,12 +85,26 @@ Prado::using('System.Web.UI.WebControls.TRepeatInfo');
  * event will be raised. Note, the selected index may not be actually changed.

  * The event mainly informs the server side that the end-user has made a selection.

  *

+ * Each datalist item has a {@link TDataListItem::getItemType type}

+ * which tells the position and state of the item in the datalist. An item in the header

+ * of the repeater is of type 'Header'. A body item may be of either

+ * 'Item', 'AlternatingItem', 'SelectedItem' or 'EditItem', depending whether the item

+ * index is odd or even, whether it is being selected or edited.

+ *

  * TDataList raises an {@link onItemCommand OnItemCommand} whenever a button control

  * within some TDataList item raises a <b>OnCommand</b> event. If the command name

  * is one of the followings: 'edit', 'update', 'select', 'delete', 'cancel' (case-insensitive),

  * another event will also be raised. For example, if the command name is 'edit',

  * then the new event is {@link onEditCommand OnEditCommand}.

  *

+ * Note, the data bound to the datalist are reset to null after databinding.

+ * There are several ways to access the data associated with a datalist item:

+ * - Access the data in {@link onItemDataBound OnItemDataBound} event

+ * - Use {@link getDataKeys DataKeys} to obtain the data key associated with

+ * the specified datalist item and use the key to fetch the corresponding data

+ * from some persistent storage such as DB.

+ * - Save the data in viewstate and get it back during postbacks.

+ *

  * @author Qiang Xue <qiang.xue@gmail.com>

  * @version $Revision: $  $Date: $

  * @package System.Web.UI.WebControls

diff --git a/framework/Web/UI/WebControls/TRepeater.php b/framework/Web/UI/WebControls/TRepeater.php
index eb599341..01c88a30 100644
--- a/framework/Web/UI/WebControls/TRepeater.php
+++ b/framework/Web/UI/WebControls/TRepeater.php
@@ -36,6 +36,12 @@ Prado::using('System.Util.TDataFieldAccessor');
  * {@link setSeparatorTemplate SeparatorTemplate}, if not empty, will be

  * displayed between items.

  *

+ * Each repeater item has a {@link TRepeaterItem::getItemType type}

+ * which tells the position of the item in the repeater. An item in the header

+ * of the repeater is of type TRepeater::IT_HEADER. A body item may be of either

+ * TRepeater::IT_ITEM or TRepeater::IT_ALTERNATINGITEM, depending whether the item

+ * index is odd or even.

+ *

  * You can retrive the repeated contents by the {@link getItems Items} property.

  * The header and footer items can be accessed by {@link getHeader Header}

  * and {@link getFooter Footer} properties, respectively.

@@ -46,10 +52,18 @@ Prado::using('System.Util.TDataFieldAccessor');
  * databinding, an {@link onItemDataBound OnItemDataBound} event will be raised.

  *

  * TRepeater raises an {@link onItemCommand OnItemCommand} whenever a button control

- * within some repeater item raises a <b>Command</b> event. Therefore,

- * you can handle all sorts of <b>Command</b> event in a central place by

+ * within some repeater item raises a <b>OnCommand</b> event. Therefore,

+ * you can handle all sorts of <b>OnCommand</b> event in a central place by

  * writing an event handler for {@link onItemCommand OnItemCommand}.

  *

+ * Note, the data bound to the repeater are reset to null after databinding.

+ * There are several ways to access the data associated with a repeater item:

+ * - Access the data in {@link onItemDataBound OnItemDataBound} event

+ * - Use {@link getDataKeys DataKeys} to obtain the data key associated with

+ * the specified repeater item and use the key to fetch the corresponding data

+ * from some persistent storage such as DB.

+ * - Save the data in viewstate and get it back during postbacks.

+ *

  * @author Qiang Xue <qiang.xue@gmail.com>

  * @version $Revision: $  $Date: $

  * @package System.Web.UI.WebControls