Hey Matthias, I saw a couple of commits go through over the weekend that looked related to the problem I am seeing with the resolvers in the current12 branch. I wasn't sure if you are still planning to apply the patch or if you preferred a different approach, but I wanted to let you know that I am still seeing the NPE. You should be able to recreate the error by adding a JSP like this to current12/test-webapp.
<%@ taglib prefix="c" uri="http://java.sun.com/jsp/jstl/core"; %> <% java.util.ArrayList mylist = new java.util.ArrayList(); mylist.add("one"); pageContext.setAttribute("mylist", mylist); %> <c:forEach var="x" items="${mylist}"> <p>${x}</p> </c:forEach> Best wishes, Paul On 2/18/07, Matthias Wessendorf <[EMAIL PROTECTED]> wrote:
Hi Paul, ok, I'll apply your patch. and yes we have a tck@ list. Since you signed the NDA for Java EE, I guess you can subscribe easily to the myfaces list. -m On 2/18/07, Paul McMahan <[EMAIL PROTECTED]> wrote: > Hi Matthias, Thanks for taking a look at the patch. When I apply it > to my local copy of current12/core I am able to use JSF in a JSP > without breaking any JSTL apps. Without the patch JSPs that use JSTL > without also using JSF produce the NullPointerException mentioned > earlier. In that case org.apache.myfaces.application.ApplicationImpl > has created several resolvers that participate in JSTL's variable > resolution while assuming that the MyFaces context factory has already > put a FacesContext into the ELContext. > > Does the MyFaces project have a private list where details about TCK > are discussed that I could subscribe to? Alternatively I can provide > you with the address for Geronimo's TCK list that you can subscribe to > if you have signed the NDA. > > Best wishes, > Paul > > On 2/17/07, Matthias Wessendorf <[EMAIL PROTECTED]> wrote: > > Paul, > > > > looks ok for me. What are the TCK tests saying ? > > Better now ? Also are the full JSF TCK tests part of the Java EE TCK ? > > > > -M > > > > On 2/16/07, Paul McMahan <[EMAIL PROTECTED]> wrote: > > > On 2/16/07, Paul McMahan <[EMAIL PROTECTED]> wrote: > > > > Question 2: > > > > After integrating MyFaces into Geronimo's Tomcat container (v6.0.8) I > > > > found that I could deploy JSF applications and pass many tests. > > > > However, I found that several of the JSTL tests no longer worked. It > > > > seems that when MyFaces has been enabled it assumes that all the JSPs > > > > in a web app will produce a FacesContext. But that's not the case for > > > > JSPs like the following: > > > [...] > > > > > > I created MYFACES-1536 for this and attached a patch to it that checks > > > for null after calling elContext.getContext(FacesContext.class). > > > Please commit if it looks ok or advise if it needs more work. > > > > > > Best wishes, > > > Paul > > > > > > > > > -- > > Matthias Wessendorf > > http://tinyurl.com/fmywh > > > > further stuff: > > blog: http://jroller.com/page/mwessendorf > > mail: mwessendorf-at-gmail-dot-com > > > -- Matthias Wessendorf http://tinyurl.com/fmywh further stuff: blog: http://jroller.com/page/mwessendorf mail: mwessendorf-at-gmail-dot-com
