[ https://issues.apache.org/jira/browse/MYFACES-4614?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17775828#comment-17775828 ]
Volodymyr Siedlecki commented on MYFACES-4614: ---------------------------------------------- [~tandraschko] Should this be ported to 2.3.x? > Print error message when action method not found > ------------------------------------------------ > > Key: MYFACES-4614 > URL: https://issues.apache.org/jira/browse/MYFACES-4614 > Project: MyFaces Core > Issue Type: Improvement > Components: General > Affects Versions: 4.0.1 > Reporter: Manuel K > Assignee: Thomas Andraschko > Priority: Major > Fix For: 4.0.2, 2.3-next-M9, 4.1.0 > > Attachments: MYFACES-4614.zip > > > Given the following example (also attached), where the number of method > parameters does not match and the method cannot be called, an exception > should be thrown. > XHTML: > {code:java} > <h:form id="frmTest"> > <p:commandButton value="Click" > action="#{testView.onClick}"/> > </h:form> {code} > Java: > {code:java} > public void onClick(Object parameter) { > System.out.println("Button clicked"); > } {code} > MyFaces does not throw an exception, while Mojarra throws: > {code:java} > Juli 12, 2023 1:44:32 PM com.sun.faces.lifecycle.InvokeApplicationPhase > execute > WARNUNG: #{testView.onClick}: /test.xhtml @17,60 > action="#{testView.onClick}": Method not found: TestView(string=Welcome to > PrimeFaces!!!, integer=null, decimal=null, localDateTime=null, > list=[TestObject(id=8e4d3985-c2f7-47dc-a5de-7e51acc41ec1, name=Thriller, > artist=Michael Jackson, released=1982), > TestObject(id=93adce20-fadb-4e8a-b16b-72eb9d03ae0e, name=Back in Black, > artist=AC/DC, released=1980), > TestObject(id=93e267a0-d6f9-4303-a82b-51ef16f88b02, name=The Bodyguard, > artist=Whitney Houston, released=1992), > TestObject(id=29ebc993-d665-47ec-844f-c14a34dc0554, name=The Dark Side of the > Moon, artist=Pink Floyd, released=1973)]).onClick() > jakarta.faces.FacesException: #{testView.onClick}: /test.xhtml @17,60 > action="#{testView.onClick}": Method not found: TestView(string=Welcome to > PrimeFaces!!!, integer=null, decimal=null, localDateTime=null, > list=[TestObject(id=8e4d3985-c2f7-47dc-a5de-7e51acc41ec1, name=Thriller, > artist=Michael Jackson, released=1982), > TestObject(id=93adce20-fadb-4e8a-b16b-72eb9d03ae0e, name=Back in Black, > artist=AC/DC, released=1980), > TestObject(id=93e267a0-d6f9-4303-a82b-51ef16f88b02, name=The Bodyguard, > artist=Whitney Houston, released=1992), > TestObject(id=29ebc993-d665-47ec-844f-c14a34dc0554, name=The Dark Side of the > Moon, artist=Pink Floyd, released=1973)]).onClick() > at > com.sun.faces.application.ActionListenerImpl.getNavigationOutcome(ActionListenerImpl.java:83) > at > com.sun.faces.application.ActionListenerImpl.processAction(ActionListenerImpl.java:62) > at > org.primefaces.application.DialogActionListener.processAction(DialogActionListener.java:54) > at jakarta.faces.component.UICommand.broadcast(UICommand.java:205) > at jakarta.faces.component.UIViewRoot.broadcastEvents(UIViewRoot.java:858) > at > jakarta.faces.component.UIViewRoot.processApplication(UIViewRoot.java:1332) > at > com.sun.faces.lifecycle.InvokeApplicationPhase.execute(InvokeApplicationPhase.java:56) > at com.sun.faces.lifecycle.Phase.doPhase(Phase.java:72) > at com.sun.faces.lifecycle.LifecycleImpl.execute(LifecycleImpl.java:159) > at > jakarta.faces.webapp.FacesServlet.executeLifecyle(FacesServlet.java:691) > at jakarta.faces.webapp.FacesServlet.service(FacesServlet.java:449) > at > org.eclipse.jetty.servlet.ServletHolder$NotAsync.service(ServletHolder.java:1410) > at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:764) > at > org.eclipse.jetty.servlet.ServletHandler$ChainEnd.doFilter(ServletHandler.java:1665) > at > org.eclipse.jetty.websocket.servlet.WebSocketUpgradeFilter.doFilter(WebSocketUpgradeFilter.java:170) > at org.eclipse.jetty.servlet.FilterHolder.doFilter(FilterHolder.java:202) > at > org.eclipse.jetty.servlet.ServletHandler$Chain.doFilter(ServletHandler.java:1635) > at > org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:527) > at > org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:131) > at > org.eclipse.jetty.security.SecurityHandler.handle(SecurityHandler.java:578) > at > org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:122) > at > org.eclipse.jetty.server.handler.ScopedHandler.nextHandle(ScopedHandler.java:223) > at > org.eclipse.jetty.server.session.SessionHandler.doHandle(SessionHandler.java:1570) > at > org.eclipse.jetty.server.handler.ScopedHandler.nextHandle(ScopedHandler.java:221) > at > org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1381) > at > org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:176) > at > org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:484) > at > org.eclipse.jetty.server.session.SessionHandler.doScope(SessionHandler.java:1543) > at > org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:174) > at > org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1303) > at > org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:129) > at > org.eclipse.jetty.server.handler.ContextHandlerCollection.handle(ContextHandlerCollection.java:149) > at > org.eclipse.jetty.server.handler.HandlerList.handle(HandlerList.java:51) > at > org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:122) > at org.eclipse.jetty.server.Server.handle(Server.java:563) > at > org.eclipse.jetty.server.HttpChannel.lambda$handle$0(HttpChannel.java:505) > at org.eclipse.jetty.server.HttpChannel.dispatch(HttpChannel.java:762) > at org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:497) > at > org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:282) > at > org.eclipse.jetty.io.AbstractConnection$ReadCallback.succeeded(AbstractConnection.java:314) > at org.eclipse.jetty.io.FillInterest.fillable(FillInterest.java:100) > at > org.eclipse.jetty.io.SelectableChannelEndPoint$1.run(SelectableChannelEndPoint.java:53) > at > org.eclipse.jetty.util.thread.strategy.AdaptiveExecutionStrategy.runTask(AdaptiveExecutionStrategy.java:416) > at > org.eclipse.jetty.util.thread.strategy.AdaptiveExecutionStrategy.consumeTask(AdaptiveExecutionStrategy.java:385) > at > org.eclipse.jetty.util.thread.strategy.AdaptiveExecutionStrategy.tryProduce(AdaptiveExecutionStrategy.java:272) > at > org.eclipse.jetty.util.thread.strategy.AdaptiveExecutionStrategy.produce(AdaptiveExecutionStrategy.java:194) > at > org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:969) > at > org.eclipse.jetty.util.thread.QueuedThreadPool$Runner.doRunJob(QueuedThreadPool.java:1194) > at > org.eclipse.jetty.util.thread.QueuedThreadPool$Runner.run(QueuedThreadPool.java:1149) > at java.base/java.lang.Thread.run(Thread.java:833) > Caused by: jakarta.el.MethodNotFoundException: /test.xhtml @17,60 > action="#{testView.onClick}": Method not found: TestView(string=Welcome to > PrimeFaces!!!, integer=null, decimal=null, localDateTime=null, > list=[TestObject(id=8e4d3985-c2f7-47dc-a5de-7e51acc41ec1, name=Thriller, > artist=Michael Jackson, released=1982), > TestObject(id=93adce20-fadb-4e8a-b16b-72eb9d03ae0e, name=Back in Black, > artist=AC/DC, released=1980), > TestObject(id=93e267a0-d6f9-4303-a82b-51ef16f88b02, name=The Bodyguard, > artist=Whitney Houston, released=1992), > TestObject(id=29ebc993-d665-47ec-844f-c14a34dc0554, name=The Dark Side of the > Moon, artist=Pink Floyd, released=1973)]).onClick() > at > com.sun.faces.facelets.el.TagMethodExpression.invoke(TagMethodExpression.java:74) > at > com.sun.faces.application.ActionListenerImpl.getNavigationOutcome(ActionListenerImpl.java:74) > ... 49 moreJuli 12, 2023 1:44:32 PM > com.sun.faces.context.AjaxExceptionHandlerImpl handlePartialResponseError > SCHWERWIEGEND: jakarta.el.MethodNotFoundException: /test.xhtml @17,60 > action="#{testView.onClick}": Method not found: TestView(string=Welcome to > PrimeFaces!!!, integer=null, decimal=null, localDateTime=null, > list=[TestObject(id=8e4d3985-c2f7-47dc-a5de-7e51acc41ec1, name=Thriller, > artist=Michael Jackson, released=1982), > TestObject(id=93adce20-fadb-4e8a-b16b-72eb9d03ae0e, name=Back in Black, > artist=AC/DC, released=1980), > TestObject(id=93e267a0-d6f9-4303-a82b-51ef16f88b02, name=The Bodyguard, > artist=Whitney Houston, released=1992), > TestObject(id=29ebc993-d665-47ec-844f-c14a34dc0554, name=The Dark Side of the > Moon, artist=Pink Floyd, released=1973)]).onClick() > jakarta.el.MethodNotFoundException: /test.xhtml @17,60 > action="#{testView.onClick}": Method not found: TestView(string=Welcome to > PrimeFaces!!!, integer=null, decimal=null, localDateTime=null, > list=[TestObject(id=8e4d3985-c2f7-47dc-a5de-7e51acc41ec1, name=Thriller, > artist=Michael Jackson, released=1982), > TestObject(id=93adce20-fadb-4e8a-b16b-72eb9d03ae0e, name=Back in Black, > artist=AC/DC, released=1980), > TestObject(id=93e267a0-d6f9-4303-a82b-51ef16f88b02, name=The Bodyguard, > artist=Whitney Houston, released=1992), > TestObject(id=29ebc993-d665-47ec-844f-c14a34dc0554, name=The Dark Side of the > Moon, artist=Pink Floyd, released=1973)]).onClick() > at > com.sun.faces.facelets.el.TagMethodExpression.invoke(TagMethodExpression.java:74) > at > com.sun.faces.application.ActionListenerImpl.getNavigationOutcome(ActionListenerImpl.java:74) > at > com.sun.faces.application.ActionListenerImpl.processAction(ActionListenerImpl.java:62) > at > org.primefaces.application.DialogActionListener.processAction(DialogActionListener.java:54) > at jakarta.faces.component.UICommand.broadcast(UICommand.java:205) > at jakarta.faces.component.UIViewRoot.broadcastEvents(UIViewRoot.java:858) > at > jakarta.faces.component.UIViewRoot.processApplication(UIViewRoot.java:1332) > at > com.sun.faces.lifecycle.InvokeApplicationPhase.execute(InvokeApplicationPhase.java:56) > at com.sun.faces.lifecycle.Phase.doPhase(Phase.java:72) > at com.sun.faces.lifecycle.LifecycleImpl.execute(LifecycleImpl.java:159) > at > jakarta.faces.webapp.FacesServlet.executeLifecyle(FacesServlet.java:691) > at jakarta.faces.webapp.FacesServlet.service(FacesServlet.java:449) > at > org.eclipse.jetty.servlet.ServletHolder$NotAsync.service(ServletHolder.java:1410) > at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:764) > at > org.eclipse.jetty.servlet.ServletHandler$ChainEnd.doFilter(ServletHandler.java:1665) > at > org.eclipse.jetty.websocket.servlet.WebSocketUpgradeFilter.doFilter(WebSocketUpgradeFilter.java:170) > at org.eclipse.jetty.servlet.FilterHolder.doFilter(FilterHolder.java:202) > at > org.eclipse.jetty.servlet.ServletHandler$Chain.doFilter(ServletHandler.java:1635) > at > org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:527) > at > org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:131) > at > org.eclipse.jetty.security.SecurityHandler.handle(SecurityHandler.java:578) > at > org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:122) > at > org.eclipse.jetty.server.handler.ScopedHandler.nextHandle(ScopedHandler.java:223) > at > org.eclipse.jetty.server.session.SessionHandler.doHandle(SessionHandler.java:1570) > at > org.eclipse.jetty.server.handler.ScopedHandler.nextHandle(ScopedHandler.java:221) > at > org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1381) > at > org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:176) > at > org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:484) > at > org.eclipse.jetty.server.session.SessionHandler.doScope(SessionHandler.java:1543) > at > org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:174) > at > org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1303) > at > org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:129) > at > org.eclipse.jetty.server.handler.ContextHandlerCollection.handle(ContextHandlerCollection.java:149) > at > org.eclipse.jetty.server.handler.HandlerList.handle(HandlerList.java:51) > at > org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:122) > at org.eclipse.jetty.server.Server.handle(Server.java:563) > at > org.eclipse.jetty.server.HttpChannel.lambda$handle$0(HttpChannel.java:505) > at org.eclipse.jetty.server.HttpChannel.dispatch(HttpChannel.java:762) > at org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:497) > at > org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:282) > at > org.eclipse.jetty.io.AbstractConnection$ReadCallback.succeeded(AbstractConnection.java:314) > at org.eclipse.jetty.io.FillInterest.fillable(FillInterest.java:100) > at > org.eclipse.jetty.io.SelectableChannelEndPoint$1.run(SelectableChannelEndPoint.java:53) > at > org.eclipse.jetty.util.thread.strategy.AdaptiveExecutionStrategy.runTask(AdaptiveExecutionStrategy.java:416) > at > org.eclipse.jetty.util.thread.strategy.AdaptiveExecutionStrategy.consumeTask(AdaptiveExecutionStrategy.java:385) > at > org.eclipse.jetty.util.thread.strategy.AdaptiveExecutionStrategy.tryProduce(AdaptiveExecutionStrategy.java:272) > at > org.eclipse.jetty.util.thread.strategy.AdaptiveExecutionStrategy.produce(AdaptiveExecutionStrategy.java:194) > at > org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:969) > at > org.eclipse.jetty.util.thread.QueuedThreadPool$Runner.doRunJob(QueuedThreadPool.java:1194) > at > org.eclipse.jetty.util.thread.QueuedThreadPool$Runner.run(QueuedThreadPool.java:1149) > at java.base/java.lang.Thread.run(Thread.java:833) {code} > What do you guys think? -- This message was sent by Atlassian Jira (v8.20.10#820010)