myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mike Kienenberger <mkien...@gmail.com>
Subject Re: "Starting up Tomahawk on the RI-JSF-Implementation." issued when not on RI
Date Mon, 27 Aug 2012 15:43:03 GMT
Looks like you found something that we need to fix.

Can you open an issue please, preferrably with a unified diff patch?


On Mon, Aug 27, 2012 at 11:38 AM, Jon Bionda <jbionda@csdcsystems.com> wrote:
> myfaces-core-2.1.8 source
>
>     org.apache.myfaces.webapp.WebConfigParamsLogger.logWebContextParams()
>
>
>
> Not a big deal but it caused me a few hours of scratching my head so thought
> I would subscribe and post this.
>
>
>
> I am getting the "Starting up Tomahawk on the RI-JSF-Implementation."
> message on startup with JBoss 7.1.1 when using the
> org.jboss.jbossfaces.WAR_BUNDLES_JSF_IMPL context parm to stop it from
> loading the JSF RI.  I see lots of myfaces initialization msgs in the log so
> I was pretty sure(hopeful) the message was wrong so I downloaded and looked
> at the code and think I see a reverse logic error in the source file
> identified at the top of this email.
>
>
>
> Here’s the code and I think the wrong message is being issued for when it
> checks for availability of both the RI and myfaces classes.
>
>
>
>         if (myfacesConfig.isTomahawkAvailable())
>
>         {
>
>             if(myfacesConfig.isMyfacesImplAvailable())
>
>             {
>
>                 if(log.isLoggable(Level.INFO))
>
>                 {
>
>                     log.info("Starting up Tomahawk on the
> RI-JSF-Implementation.");   ßmsg should be MyFaces
>
>                 }
>
>             }
>
>
>
>             if(myfacesConfig.isRiImplAvailable())
>
>             {
>
>                 if(log.isLoggable(Level.INFO))
>
>                 {
>
>                     log.info("Starting up Tomahawk on the
> MyFaces-JSF-Implementation");  ß msg should be RI
>
>                 }
>
>             }
>
>         }
>
>
>
>

Mime
View raw message