Author: gert
Date: 2007-02-17 09:59:47 -0500 (Sat, 17 Feb 2007)
New Revision: 73056

Modified:
   trunk/mcs/class/System.Web/System.Web.Configuration/ChangeLog
   trunk/mcs/class/System.Web/System.Web.Configuration/PagesConfiguration.cs
   
trunk/mcs/class/System.Web/System.Web.Configuration/PagesConfigurationHandler.cs
Log:
* PagesConfiguration.cs: Stop swallowing configuration exceptions.
* PagesConfigurationHandler.cs: Provide better exception message for
invalid 'enableSessionState' value.


Modified: trunk/mcs/class/System.Web/System.Web.Configuration/ChangeLog
===================================================================
--- trunk/mcs/class/System.Web/System.Web.Configuration/ChangeLog       
2007-02-17 14:15:22 UTC (rev 73055)
+++ trunk/mcs/class/System.Web/System.Web.Configuration/ChangeLog       
2007-02-17 14:59:47 UTC (rev 73056)
@@ -1,3 +1,9 @@
+2007-02-17  Gert Driesen  <[EMAIL PROTECTED]>
+
+       * PagesConfiguration.cs: Stop swallowing configuration exceptions.
+       * PagesConfigurationHandler.cs: Provide better exception message for
+       invalid 'enableSessionState' value.
+
 2007-01-20  Miguel de Icaza  <[EMAIL PROTECTED]>
 
        * HandlerFactoryConfiguration.cs (LoadType): remove unused

Modified: 
trunk/mcs/class/System.Web/System.Web.Configuration/PagesConfiguration.cs
===================================================================
--- trunk/mcs/class/System.Web/System.Web.Configuration/PagesConfiguration.cs   
2007-02-17 14:15:22 UTC (rev 73055)
+++ trunk/mcs/class/System.Web/System.Web.Configuration/PagesConfiguration.cs   
2007-02-17 14:59:47 UTC (rev 73056)
@@ -67,15 +67,7 @@
                        PagesConfiguration config;
                        if (context == null)
                                context = HttpContext.Current;
-
-                       try {
-                               config = context.GetConfig ("system.web/pages") 
as PagesConfiguration;
-                       } catch {
-                               return new PagesConfiguration (null);
-                       }
-                       return config;
+                       return context.GetConfig ("system.web/pages") as 
PagesConfiguration;
                }
-
        }
 }
-

Modified: 
trunk/mcs/class/System.Web/System.Web.Configuration/PagesConfigurationHandler.cs
===================================================================
--- 
trunk/mcs/class/System.Web/System.Web.Configuration/PagesConfigurationHandler.cs
    2007-02-17 14:15:22 UTC (rev 73055)
+++ 
trunk/mcs/class/System.Web/System.Web.Configuration/PagesConfigurationHandler.cs
    2007-02-17 14:59:47 UTC (rev 73056)
@@ -49,8 +49,9 @@
                        attvalue = AttValue ("enableSessionState", section);
                        if (attvalue != null) {
                                if (attvalue != "true" && attvalue != "false" 
&& attvalue != "ReadOnly")
-                                       HandlersUtil.ThrowException ("Invalid 
value for 'enableSessionState'", section);
-
+                                       HandlersUtil.ThrowException ("The 
'enableSessionState' attribute"
+                                               + " is case sensitive and must 
be one of the following values:"
+                                               + " false, true, ReadOnly.", 
section);
                                config.EnableSessionState = attvalue;
                        }
 

_______________________________________________
Mono-patches maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-patches

Reply via email to