typo

[mono.git] / mcs / class / System.Web / System.Web.UI.WebControls / BaseDataBoundControl.cs

diff --git a/mcs/class/System.Web/System.Web.UI.WebControls/BaseDataBoundControl.cs b/mcs/class/System.Web/System.Web.UI.WebControls/BaseDataBoundControl.cs
index 1412b84c0128cd84109abc43c336f5534f7f3801..71f3f6b5d94e830e33abfed5d1d33f4316167942 100644 (file)
--- a/mcs/class/System.Web/System.Web.UI.WebControls/BaseDataBoundControl.cs
+++ b/mcs/class/System.Web/System.Web.UI.WebControls/BaseDataBoundControl.cs
@@ -68,7 +68,8 @@ namespace System.Web.UI.WebControls {
                                return dataSource;
                        }
                        set {
-                               ValidateDataSource (value);
+                               if(value!=null)
+                                       ValidateDataSource (value);
                                dataSource = value;
                                OnDataPropertyChanged ();
                        }
@@ -96,10 +97,32 @@ namespace System.Web.UI.WebControls {
                
                protected bool RequiresDataBinding {
                        get { return requiresDataBinding; }
-                       set { 
-                               requiresDataBinding = value;
-                               if (value && preRendered && IsBoundUsingDataSourceID && Page != null && !Page.IsCallback)
+                       set {
+                               // MSDN: If you set the RequiresDataBinding
+                               // property to true when the data-bound control
+                               // has already begun to render its output to the
+                               // page, the current HTTP request is not a
+                               // callback, and you are using the DataSourceID
+                               // property to identify the data source control
+                               // to bind to, the DataBind method is called
+                               // immediately. In this case, the
+                               // RequiresDataBinding property is _not_ actually
+                               // set to true.
+                               //
+                               // LAMESPEC, the docs quoted above mention that
+                               // DataBind is called in the described
+                               // case. This is wrong since that way we don't
+                               // break recursion when the property is set from
+                               // within the OnSelect handler in user's
+                               // code. EnsureDataBound makes sure that no
+                               // recursive binding is performed. Also the
+                               // property DOES get set in this case (according
+                               // to tests)
+                               if (value && preRendered && IsBoundUsingDataSourceID && Page != null && !Page.IsCallback) {
+                                       requiresDataBinding = true;
                                        EnsureDataBound ();
+                               } else
+                                       requiresDataBinding = value;
                        }
                }
                
@@ -110,7 +133,6 @@ namespace System.Web.UI.WebControls {
                
                public override void DataBind ()
                {
-                       RequiresDataBinding = false;
                        PerformSelect ();
                }
 
@@ -128,7 +150,8 @@ namespace System.Web.UI.WebControls {
 
                protected virtual void OnDataPropertyChanged ()
                {
-                       RequiresDataBinding = true;
+                       if (Initialized)
+                               RequiresDataBinding = true;
                }
                
                protected internal override void OnInit (EventArgs e)
@@ -151,6 +174,21 @@ namespace System.Web.UI.WebControls {
                        EnsureDataBound ();
                        base.OnPreRender (e);
                }
+
+               internal Control FindDataSource ()
+               {
+                       Control ctrl = null;
+                       Control namingContainer = NamingContainer;
+
+                       while (namingContainer != null) {
+                               ctrl = namingContainer.FindControl (DataSourceID);
+                               if (ctrl != null)
+                                       break;
+                               namingContainer = namingContainer.NamingContainer;
+                       }
+
+                       return ctrl;
+               }
                
                protected abstract void PerformSelect ();
                
@@ -161,3 +199,4 @@ namespace System.Web.UI.WebControls {
 
 #endif
 
+