Ahg! Here's the right patch (obviously setting them to private isn't going to 
improve the situation):

Index: Sources/er/extensions/logging/ERXLog4JConfiguration.java
===================================================================
--- Sources/er/extensions/logging/ERXLog4JConfiguration.java    (revision 11514)
+++ Sources/er/extensions/logging/ERXLog4JConfiguration.java    (working copy)
@@ -39,7 +39,7 @@
     /**
      * A representation of the various Logger levels.
      */
-    protected enum LoggerLevel {
+    public enum LoggerLevel {
         ALL(Level.ALL, "All"),
         TRACE(Level.TRACE, "Trace"),
         DEBUG(Level.DEBUG, "Debug"),
@@ -76,7 +76,7 @@
     /**
      * A representation of the available page sections/views.
      */
-    protected enum PageSection {
+    public enum PageSection {
         LOGGERS("Loggers", "Loggers"),
         REPOSITORY("Repository", "Repository"),
         APPENDERS("Appenders", "Appenders"),



On Oct 13, 2010, at 7:24 AM, David Avendasora wrote:

> Hi all,
> 
> This is just a heads up for all you D2Wers using JRebel. JRebel chokes 
> (java.lang.IllegalAccessException) on the protected enums in the 
> ERXLog4JConfiguration class, so you can't use the awesome new Log4J 
> configuration page. :-(
> 
> If you change them to public then the Log4J configuration page works just 
> fine.
> 
> After discussions with Q on the subject I've contacted 
> [email protected] to report this bug.
> 
> In the meantime below is a patch you can apply to ERXExtensions.
> 
> Dave
> 
> 
> Index: Sources/er/extensions/logging/ERXLog4JConfiguration.java
> ===================================================================
> --- Sources/er/extensions/logging/ERXLog4JConfiguration.java  (revision 11514)
> +++ Sources/er/extensions/logging/ERXLog4JConfiguration.java  (working copy)
> @@ -39,7 +39,7 @@
>     /**
>      * A representation of the various Logger levels.
>      */
> -    protected enum LoggerLevel {
> +    private enum LoggerLevel {
>         ALL(Level.ALL, "All"),
>         TRACE(Level.TRACE, "Trace"),
>         DEBUG(Level.DEBUG, "Debug"),
> @@ -76,7 +76,7 @@
>     /**
>      * A representation of the available page sections/views.
>      */
> -    protected enum PageSection {
> +    private enum PageSection {
>         LOGGERS("Loggers", "Loggers"),
>         REPOSITORY("Repository", "Repository"),
>         APPENDERS("Appenders", "Appenders"),
> 
> _______________________________________________
> Do not post admin requests to the list. They will be ignored.
> Webobjects-dev mailing list      ([email protected])
> Help/Unsubscribe/Update your Subscription:
> http://lists.apple.com/mailman/options/webobjects-dev/webobjects%40avendasora.com
> 
> This email sent to [email protected]
> 
> 

 _______________________________________________
Do not post admin requests to the list. They will be ignored.
Webobjects-dev mailing list      ([email protected])
Help/Unsubscribe/Update your Subscription:
http://lists.apple.com/mailman/options/webobjects-dev/archive%40mail-archive.com

This email sent to [email protected]

Reply via email to