messageformat error blows up (JBSEAM-1859)
atao Aug 28, 2007 8:09 AMWhen an EL expression is not closed by a brace, an error is thrown.
It would be better to show the wrong expression as it is:
1/ it's already the way of doing the thing, see in InterpolatorTest the existing case of
// test that a messageformat error doesn't blow up
Assert.assertEquals("{nosuchmessage}", interpolator.interpolate("{nosuchmessage}"));
2/ atm when an EL expression is not closed by a brace, the trace doesn't give a clear clue of what is wrong (see trace below). I wasted too much time with such an error.
With the wrong expression shown in place of the expected result, it'd be straightforward.
In any case I suggest to check that the end bracket is present and at least to throw a exception with a specific message.
====================
Error in Authenticator:
public boolean authenticate() { try { User user = (User) entityManager .createQuery( "from User where username = :username and password = :password") .setParameter("username", identity.getUsername()) .setParameter("password", identity.getPassword()) .getSingleResult(); if (user.getRoles() != null) { for (UserRole mr : user.getRoles()) Identity.instance().addRole(mr.getName()); } return true; } catch (NoResultException ex) { FacesMessages.instance().add( "#{messages['entity.user.login.invalid']"); return false; } }
and the trace (only the first occurence, as it starts a loop):
13:47:43,500 INFO [STDOUT] Hibernate: select user0_.username as username11_, user0_.name as name11_, user0_.password as password11_, user0_.currentBranch as currentB4_11_ from User user0_ where user0_.username=? and user0_.password=?
13:47:43,531 ERROR [SeamPhaseListener] uncaught exception
java.util.NoSuchElementException
at java.util.StringTokenizer.nextToken(StringTokenizer.java:332)
at org.jboss.seam.core.Interpolator.interpolateExpressions(Interpolator.java:94)
at org.jboss.seam.core.Interpolator.interpolate(Interpolator.java:66)
at org.jboss.seam.faces.FacesMessages.createFacesMessage(FacesMessages.java:371)
at org.jboss.seam.faces.FacesMessages.createFacesMessage(FacesMessages.java:379)
at org.jboss.seam.faces.FacesMessages$1.run(FacesMessages.java:425)
at org.jboss.seam.faces.FacesMessages.runTasks(FacesMessages.java:156)
at org.jboss.seam.faces.FacesMessages.afterPhase(FacesMessages.java:166)
at org.jboss.seam.jsf.SeamPhaseListener.afterServletPhase(SeamPhaseListener.java:224)
at org.jboss.seam.jsf.SeamPhaseListener.afterPhase(SeamPhaseListener.java:184)
at com.sun.faces.lifecycle.LifecycleImpl.phase(LifecycleImpl.java:280)
at com.sun.faces.lifecycle.LifecycleImpl.execute(LifecycleImpl.java:117)
at javax.faces.webapp.FacesServlet.service(FacesServlet.java:244)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:290)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
at org.jboss.seam.servlet.SeamFilter$FilterChainImpl.doFilter(SeamFilter.java:82)
at org.jboss.seam.debug.hot.HotDeployFilter.doFilter(HotDeployFilter.java:68)
at org.jboss.seam.servlet.SeamFilter$FilterChainImpl.doFilter(SeamFilter.java:68)
at org.jboss.seam.web.MultipartFilter.doFilter(MultipartFilter.java:85)
at org.jboss.seam.servlet.SeamFilter$FilterChainImpl.doFilter(SeamFilter.java:68)
at org.jboss.seam.web.ExceptionFilter.doFilter(ExceptionFilter.java:64)
at org.jboss.seam.servlet.SeamFilter$FilterChainImpl.doFilter(SeamFilter.java:68)
at org.jboss.seam.web.RedirectFilter.doFilter(RedirectFilter.java:44)
at org.jboss.seam.servlet.SeamFilter$FilterChainImpl.doFilter(SeamFilter.java:68)
at org.ajax4jsf.framework.ajax.xmlfilter.BaseXMLFilter.doXmlFilter(BaseXMLFilter.java:127)
at org.ajax4jsf.framework.ajax.xmlfilter.BaseFilter.doFilter(BaseFilter.java:277)
at org.jboss.seam.web.Ajax4jsfFilter.doFilter(Ajax4jsfFilter.java:60)
at org.jboss.seam.servlet.SeamFilter$FilterChainImpl.doFilter(SeamFilter.java:68)
at org.jboss.seam.web.LoggingFilter.doFilter(LoggingFilter.java:58)
at org.jboss.seam.servlet.SeamFilter$FilterChainImpl.doFilter(SeamFilter.java:68)
at org.jboss.seam.servlet.SeamFilter.doFilter(SeamFilter.java:149)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:235)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
at org.jboss.web.tomcat.filters.ReplyHeaderFilter.doFilter(ReplyHeaderFilter.java:96)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:235)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:228)
at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:175)
at org.jboss.web.tomcat.security.SecurityAssociationValve.invoke(SecurityAssociationValve.java:179)
at org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:433)
at org.jboss.web.tomcat.security.JaccContextValve.invoke(JaccContextValve.java:84)
at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:128)
at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:104)
at org.jboss.web.tomcat.service.jca.CachedConnectionValve.invoke(CachedConnectionValve.java:156)
at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:109)
at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:216)
at org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:844)
at org.apache.coyote.http11.Http11Protocol$Http11ConnectionHandler.process(Http11Protocol.java:624)
at org.apache.tomcat.util.net.JIoEndpoint$Worker.run(JIoEndpoint.java:445)
at java.lang.Thread.run(Thread.java:595)
13:47:43,562 ERROR [CachedConnectionValve] Application error: Faces Servlet did not complete its transaction
13:47:43,609 ERROR [SeamPhaseListener] uncaught exception
java.util.NoSuchElementException
at java.util.StringTokenizer.nextToken(StringTokenizer.java:332)
at org.jboss.seam.core.Interpolator.interpolateExpressions(Interpolator.java:94)
at org.jboss.seam.core.Interpolator.interpolate(Interpolator.java:66)
at org.jboss.seam.faces.FacesMessages.createFacesMessage(FacesMessages.java:371)
at org.jboss.seam.faces.FacesMessages.createFacesMessage(FacesMessages.java:379)
at org.jboss.seam.faces.FacesMessages$1.run(FacesMessages.java:425)
[...]