Yes, another good example of an area I need to revisit. I'll wrap these two items up in a new ticket.

Don

Matt Raible wrote:
On 8/30/06, Ted Husted <[EMAIL PROTECTED]> wrote:
On 8/30/06, Don Brown <[EMAIL PROTECTED]> wrote:
> The jasper reports result _should_ be there,

Why?

> however, we do need to
> reduce the warn message to perhaps an info. The idea is we should list > results that may not be available to make it easy to use the new results
> when the struts-extras.jar is present.

It's already easy to add new results to any package. I don't think we
should go overboard by trying to bundle everything into the default
package. People who use JSF or JasperReport results can just add them
to their own package.

I think it's OK that they're there, as long as users aren't *aware*
they're there.  Here's what I get in regards to JSF:

WARN - InterceptorBuilder.constructInterceptorReference(59) | Unable to load con fig class org.apache.struts2.jsf.FacesSetupInterceptor at interceptor - jar:file :/Users/mraible/Work/appfuse/web/struts-war/target/work/webapp/WEB-INF/lib/strut s2-core-2.0.0-SNAPSHOT.jar!/struts-default.xml:54:97 probably due to a missing j
ar, which might be fine if you never plan to use the jsfSetup interceptor
WARN - InterceptorBuilder.constructInterceptorReference(59) | Unable to load con fig class org.apache.struts2.jsf.RestoreViewInterceptor at interceptor - jar:fil e:/Users/mraible/Work/appfuse/web/struts-war/target/work/webapp/WEB-INF/lib/stru ts2-core-2.0.0-SNAPSHOT.jar!/struts-default.xml:55:100 probably due to a missing jar, which might be fine if you never plan to use the jsfRestore interceptor WARN - InterceptorBuilder.constructInterceptorReference(59) | Unable to load con fig class org.apache.struts2.jsf.ApplyRequestValuesInterceptor at interceptor - jar:file:/Users/mraible/Work/appfuse/web/struts-war/target/work/webapp/WEB-INF/l ib/struts2-core-2.0.0-SNAPSHOT.jar!/struts-default.xml:56:105 probably due to a missing jar, which might be fine if you never plan to use the jsfApply intercept
or
WARN - InterceptorBuilder.constructInterceptorReference(59) | Unable to load con fig class org.apache.struts2.jsf.ProcessValidationsInterceptor at interceptor - jar:file:/Users/mraible/Work/appfuse/web/struts-war/target/work/webapp/WEB-INF/l ib/struts2-core-2.0.0-SNAPSHOT.jar!/struts-default.xml:57:108 probably due to a missing jar, which might be fine if you never plan to use the jsfValidate interc
eptor
WARN - InterceptorBuilder.constructInterceptorReference(59) | Unable to load con fig class org.apache.struts2.jsf.UpdateModelValuesInterceptor at interceptor - j ar:file:/Users/mraible/Work/appfuse/web/struts-war/target/work/webapp/WEB-INF/li b/struts2-core-2.0.0-SNAPSHOT.jar!/struts-default.xml:58:105 probably due to a m issing jar, which might be fine if you never plan to use the jsfUpdate intercept
or
WARN - InterceptorBuilder.constructInterceptorReference(59) | Unable to load con fig class org.apache.struts2.jsf.InvokeApplicationInterceptor at interceptor - j ar:file:/Users/mraible/Work/appfuse/web/struts-war/target/work/webapp/WEB-INF/li b/struts2-core-2.0.0-SNAPSHOT.jar!/struts-default.xml:59:105 probably due to a m issing jar, which might be fine if you never plan to use the jsfInvoke intercept
or

I think it'd be best if I didn't have to add results to my struts.xml
- I'd prefer that they're available when I add the appropriate
dependencies (i.e. JasperReports or MyFaces).  I'd be more than happy
to see a warning if I tried to use one of these results and I didn't
have the dependent JARs in my classpath.

Matt

-Ted.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]




---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to