Issue Details (XML | Word | Printable)

Key: JAVASERVERFACES-2738
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Critical Critical
Assignee: Manfred Riem
Reporter: Hanspeter Duennenberger
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
javaserverfaces

Facelet XHTML loading is broken - still worked with 2.2.0-m08

Created: 12/Feb/13 11:21 AM   Updated: 14/Feb/13 08:46 PM   Resolved: 14/Feb/13 08:37 PM
Component/s: None
Affects Version/s: 2.2.0-m09
Fix Version/s: 2.1.20, 2.2.0-m10

Time Tracking:
Not Specified

Issue Links:
Related
 

Tags:
Participants: Hanspeter Duennenberger and Manfred Riem


 Description  « Hide

To define the general layout we use Facelet tags implemented as XHTML using multiple levels of ui:composition/ui:decorate template="some/template.xhtml" and deliver that as a packaged jar to the applications.

Since JSF 2.2.0-m09 this does not work anymore, with 2.2.0-m08 it still works. The templates referenced from XHTML Facelet component packaged into a jar cannot be resolved anymore. I get the following exception report:

message /file:/C:/cs/ws4/mojarra_trunk/.metadata/.plugins/org.eclipse.wst.server.core/tmp0/wtpwebapps/webapp/WEB-INF/lib/component-lib-0.0.1-SNAPSHOT.jar!/facelet/tag/layout.xhtml @16,50 <ui:decorate template="template/webapp.xhtml"> Invalid path : template/webapp.xhtml

description The server encountered an internal error that prevented it from fulfilling this request.

To investigate and fix the problem I developed a small reproducer application which I will attach to this issue.



Manfred Riem made changes - 12/Feb/13 04:23 PM
Field Original Value New Value
Assignee Manfred Riem [ mriem ]
Manfred Riem made changes - 12/Feb/13 04:24 PM
Priority Major [ 3 ] Critical [ 2 ]
Manfred Riem added a comment - 14/Feb/13 08:31 PM

Applied to 2.2 trunk,

svn commit -m "Fixes http://java.net/jira/browse/JAVASERVERFACES-2738, r=rogerk, revert changes done for relative paths for resource library contracts since they cause a regression."
Sending jsf-ri\src\main\java\com\sun\faces\facelets\impl\DefaultFacelet.java
Adding test\agnostic\facelets\cc\nb-configuration.xml
Sending test\agnostic\facelets\cc\pom.xml
Adding test\agnostic\facelets\cc\src\main\webapp\ccInJar.xhtml
Adding test\agnostic\facelets\cc\src\main\webapp\templates
Adding test\agnostic\facelets\cc\src\main\webapp\templates\defaultLayout.xhtml
Adding test\agnostic\facelets\cc\src\main\webapp\templates\defaultPage.xhtml
Adding test\agnostic\facelets\cc\src\test\java\com\sun\faces\test\agnostic\facelets\composite\Issue2738IT.java
Adding test\agnostic\facelets\ccInJar
Adding test\agnostic\facelets\ccInJar\pom.xml
Adding test\agnostic\facelets\ccInJar\src
Adding test\agnostic\facelets\ccInJar\src\main
Adding test\agnostic\facelets\ccInJar\src\main\resources
Adding test\agnostic\facelets\ccInJar\src\main\resources\META-INF
Adding test\agnostic\facelets\ccInJar\src\main\resources\META-INF\ccInJar.taglib.xml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\layout.xhtml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template\layoutAll.xhtml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template\layoutContentBody.xhtml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template\layoutContentFooter.xhtml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template\layoutContentHead.xhtml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template\layoutContentLeft.xhtml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template\layoutHead.xhtml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template\webapp.xhtml
Sending test\agnostic\facelets\pom.xml
Sending test\agnostic\vdl\facelets\contracts\vhosts\src\test\java\com\sun\faces\test\agnostic\vdl\facelets\contracts\vhosts\Issue2511IT.java
Sending test\agnostic\vdl\facelets\contracts\vhosts\src\test\java\com\sun\faces\test\agnostic\vdl\facelets\contracts\vhosts\Issue2679IT.java
Transmitting file data ....................
Committed revision 11624.


Manfred Riem added a comment - 14/Feb/13 08:36 PM

Applied to 2.1 branch,

svn commit -m "Fixes http://java.net/jira/browse/JAVASERVERFACES-2738, r=rogerk, backport of tests that assert behavior related to relative paths and composite components."
Sending test\agnostic\facelets\cc\pom.xml
Adding test\agnostic\facelets\cc\src\main\webapp\ccInJar.xhtml
Adding test\agnostic\facelets\cc\src\main\webapp\templates
Adding test\agnostic\facelets\cc\src\main\webapp\templates\defaultLayout.xhtml
Adding test\agnostic\facelets\cc\src\main\webapp\templates\defaultPage.xhtml
Adding test\agnostic\facelets\cc\src\test\java\com\sun\faces\test\agnostic\facelets\composite\Issue2738IT.java
Adding test\agnostic\facelets\ccInJar
Adding test\agnostic\facelets\ccInJar\pom.xml
Adding test\agnostic\facelets\ccInJar\src
Adding test\agnostic\facelets\ccInJar\src\main
Adding test\agnostic\facelets\ccInJar\src\main\resources
Adding test\agnostic\facelets\ccInJar\src\main\resources\META-INF
Adding test\agnostic\facelets\ccInJar\src\main\resources\META-INF\ccInJar.taglib.xml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\layout.xhtml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template\layoutAll.xhtml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template\layoutContentBody.xhtml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template\layoutContentFooter.xhtml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template\layoutContentHead.xhtml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template\layoutContentLeft.xhtml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template\layoutHead.xhtml
Adding test\agnostic\facelets\ccInJar\src\main\resources\facelet\tag\template\webapp.xhtml
Sending test\agnostic\facelets\pom.xml
Transmitting file data ................
Committed revision 11625.


Manfred Riem made changes - 14/Feb/13 08:37 PM
Status Open [ 1 ] Closed [ 6 ]
Fix Version/s 2.1.20 [ 16327 ]
Fix Version/s 2.2.0-m10 [ 16142 ]
Resolution Fixed [ 1 ]
Manfred Riem made changes - 14/Feb/13 08:46 PM
Link This issue is related to JAVASERVERFACES-2679 [ JAVASERVERFACES-2679 ]