Re: JRebel & ERXLog4JConfiguration
Re: JRebel & ERXLog4JConfiguration
- Subject: Re: JRebel & ERXLog4JConfiguration
- From: David Avendasora <email@hidden>
- Date: Wed, 13 Oct 2010 07:32:28 -0400
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@hidden 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@hidden)
> Help/Unsubscribe/Update your Subscription:
>
> This email sent to email@hidden
>
>
_______________________________________________
Do not post admin requests to the list. They will be ignored.
Webobjects-dev mailing list (email@hidden)
Help/Unsubscribe/Update your Subscription:
This email sent to email@hidden