[MYFACES-4512] TCK: ViewDeclarationLanguage.createView NullPointerException

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.2.15, 2.3.10, 2.3-next-M7, 3.0.2, 4.0.0-RC2
• Fix Version/s: 2.3.11, 3.0.3, 2.3-next-M8, 2.2.16, 4.0.0-RC3
• Component/s: General
• Assignee: Paul Nicolucci
• Reporter: Paul Nicolucci
• Created: 04/Dec/22 16:49
• Updated: 16/Dec/22 21:38
• Resolved: 06/Dec/22 00:13

Description

TCK Test: https://github.com/jakartaee/faces/blob/4.0.1/tck/old-tck/source/src/com/sun/ts/tests/jsf/spec/navigation/TestServlet.java#L175

ViewDeclarationLanguage is null here: https://github.com/apache/myfaces/blob/main/impl/src/main/java/org/apache/myfaces/view/facelets/FaceletViewDeclarationLanguage.java#L1927

Exception:

Caused by: java.lang.NullPointerException
at org.apache.myfaces.view.facelets.FaceletViewDeclarationLanguage.createView(FaceletViewDeclarationLanguage.java:1935)
at org.apache.myfaces.application.ViewHandlerImpl.createView(ViewHandlerImpl.java:265)
at jakarta.faces.application.ViewHandlerWrapper.createView(ViewHandlerWrapper.java:116)
at com.sun.ts.tests.jsf.spec.navigation.TestServlet.setup(TestServlet.java:61)
 

This occurs because there is a TestViewHandler: https://github.com/jakartaee/faces/blob/4.0.1/tck/old-tck/source/src/com/sun/ts/tests/jsf/spec/navigation/TestViewHandler.java that is used and it does not override the default implementation of getViewDeclarationLanguage which just returns null:  https://github.com/apache/myfaces/blob/main/api/src/main/java/jakarta/faces/application/ViewHandler.java#L270

 

Issue Links

• links to: GitHub Pull Request #404
• links to: GitHub Pull Request #405
• links to: GitHub Pull Request #406
• links to: GitHub Pull Request #407
• links to: GitHub Pull Request #408