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 d9bd000657ba8f51dd5b4f2bffa7481319fdd3d0..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
@@ -4,9 +4,7 @@
 // Authors:
 //     Lluis Sanchez Gual (lluis@novell.com)
 //
-// (C) 2004 Novell, Inc (http://www.novell.com)
-//
-
+// Copyright (C) 2004-2005 Novell, Inc (http://www.novell.com)
 //
 // Permission is hereby granted, free of charge, to any person obtaining
 // a copy of this software and associated documentation files (the
@@ -31,9 +29,14 @@
 #if NET_2_0
 using System.Collections;
 using System.ComponentModel;
+using System.Security.Permissions;
 
-namespace System.Web.UI.WebControls
-{
+namespace System.Web.UI.WebControls {
+
+       // CAS
+       [AspNetHostingPermission (SecurityAction.LinkDemand, Level = AspNetHostingPermissionLevel.Minimal)]
+       [AspNetHostingPermission (SecurityAction.InheritanceDemand, Level = AspNetHostingPermissionLevel.Minimal)]
+       // attributes
        [DefaultProperty ("DataSourceID")]
        [DesignerAttribute ("System.Web.UI.Design.WebControls.BaseDataBoundControlDesigner, " + Consts.AssemblySystem_Design, "System.ComponentModel.Design.IDesigner")]
        public abstract class BaseDataBoundControl: WebControl
@@ -41,8 +44,8 @@ namespace System.Web.UI.WebControls
                public event EventHandler DataBound;
                
                object dataSource;
-               string dataSourceId;
                bool initialized;
+               bool preRendered;
                bool requiresDataBinding;
                
                protected BaseDataBoundControl ()
@@ -65,10 +68,10 @@ namespace System.Web.UI.WebControls
                                return dataSource;
                        }
                        set {
-                               ValidateDataSource (value);
+                               if(value!=null)
+                                       ValidateDataSource (value);
                                dataSource = value;
-                               if (initialized)
-                                       OnDataPropertyChanged ();
+                               OnDataPropertyChanged ();
                        }
                }
                
@@ -76,12 +79,11 @@ namespace System.Web.UI.WebControls
                [ThemeableAttribute (false)]
                public virtual string DataSourceID {
                        get {
-                               return dataSourceId != null ? dataSourceId : string.Empty;
+                               return ViewState.GetString ("DataSourceID", "");
                        }
                        set {
-                               dataSourceId = value;
-                               if (initialized)
-                                       OnDataPropertyChanged ();
+                               ViewState["DataSourceID"] = value;
+                               OnDataPropertyChanged ();
                        }
                }
                
@@ -95,7 +97,33 @@ namespace System.Web.UI.WebControls
                
                protected bool RequiresDataBinding {
                        get { return requiresDataBinding; }
-                       set { requiresDataBinding = value; }
+                       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;
+                       }
                }
                
                protected void ConfirmInitState ()
@@ -105,7 +133,6 @@ namespace System.Web.UI.WebControls
                
                public override void DataBind ()
                {
-                       RequiresDataBinding = false;
                        PerformSelect ();
                }
 
@@ -123,13 +150,17 @@ namespace System.Web.UI.WebControls
 
                protected virtual void OnDataPropertyChanged ()
                {
-                       RequiresDataBinding = true;
+                       if (Initialized)
+                               RequiresDataBinding = true;
                }
                
                protected internal override void OnInit (EventArgs e)
                {
                        base.OnInit (e);
                        Page.PreLoad += new EventHandler (OnPagePreLoad);
+
+                       if (!IsViewStateEnabled && Page != null && Page.IsPostBack)
+                               RequiresDataBinding = true;
                }
                
                protected virtual void OnPagePreLoad (object sender, EventArgs e)
@@ -139,9 +170,25 @@ namespace System.Web.UI.WebControls
                
                protected internal override void OnPreRender (EventArgs e)
                {
+                       preRendered = true;
                        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 ();
                
@@ -152,3 +199,4 @@ namespace System.Web.UI.WebControls
 
 #endif
 
+