Details
Improvement
Major
Description
This JIRA is a spin-off of
This issue is a result of a deficiency in Mojarra JSF up to and including JSF 2.1.
This JIRA is to capture work done in ICEfaces EE to provide a solution to this issue that is additive to the ICEfaces 2 open-source library (will be delivered via a new icefaces-ee.jar).
Activity
| Field | Original Value | New Value |
|---|---|---|
| Issue Type | Bug [ 1 ] | Improvement [ 4 ] |
| Environment | IE browsers | ICEfaces EE 2, IE browsers |
| Salesforce Case | [] | |
| Component/s | Framework [ 10013 ] | |
| Affects | [Compatibility/Configuration] | [Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration] |
| Assignee Priority | P1 | |
| Affects Version/s | 2.0.0 [ 10230 ] | |
| Affects Version/s | 2.0-Beta1 [ 10231 ] | |
| Description | When a DOM update for entire document is received by the JSF bridge the code referenced in the 'head' element of the updated is ignored. |
When a DOM update for entire document is received by the JSF bridge the code referenced in the 'head' element of the updated is ignored. This JIRA is a spin-off of This issue is a result of a deficiency in Mojarra JSF up to and including JSF 2.1. This JIRA is to capture work done in ICEfaces EE to provide a solution to this issue that is additive to the ICEfaces 2 open-source library (will be delivered via a new icefaces-ee.jar). |
| Security | Private [ 10001 ] |
| Link | This issue blocks ICE-6479 [ ICE-6479 ] |
| Salesforce Case | [] | |
| Reporter | Mircea Toma [ mircea.toma ] | Ken Fyten [ ken.fyten ] |
| Repository | Revision | Date | User | Message |
| ICEsoft Public SVN Repository | #23824 | Thu Jan 20 09:56:09 MST 2011 | mircea.toma | |
| Files Changed | ||||
MODIFY
/icefaces2/trunk/icefaces/samples/core/test/build/common.xml
|
Added test application for verifying the fix in IE. Usage is described in the application pages.
The fix is effective only when the document found in the DOM update is not valid XML, commenting it out for now.
Inserted new component resource in the 'head' to run JS code that tests when 'head' update is applied as expected. The test is used after a javax.faces.ViewRoot update to determine if the JSF+IE fix needs to be executed.
The test case was enhanced to render invalid XML markup so that the JSF bug is triggered, thus exercising the current fix.
| Status | Open [ 1 ] | Resolved [ 5 ] |
| Resolution | Fixed [ 1 ] |
| Repository | Revision | Date | User | Message |
| ICEsoft Public SVN Repository | #23901 | Thu Feb 03 17:08:00 MST 2011 | deryk.sinotte | ICE-6513: revert common build file that was inadvertently checked in for |
| Files Changed | ||||
|
MODIFY
/icefaces2/trunk/icefaces/samples/core/test/build/common.xml
|
Looks like a the common.xml build file was checked in inadvertently instead of the test case. Perhaps when Mircea gets back he can take a look but it's low priority. I've reverted the common.xml file to correct the issue reported for ICE-6513.
icefaces-ee2 trunk revision# 26189
server: tomcat6
Browser: FF3.6 , IE6,7,8 & Chrome8
page B — updated by script contained in the "head" title is not displayed after navigating to PageB.
The problem can be reproduced on all browsers
| Resolution | Fixed [ 1 ] | |
| Status | Resolved [ 5 ] | Reopened [ 4 ] |
Fixed test case build to include icefaces-ee.jar and icefaces.jar in the resulting WAR file.
icefaces-ee/icefaces-ee/core and icefaces-ee/icefaces/core need to be build before generating the WAR file.
| Status | Reopened [ 4 ] | Resolved [ 5 ] |
| Resolution | Fixed [ 1 ] |
There's still a JS error in all the browsers with the exception of IE.
| Resolution | Fixed [ 1 ] | |
| Status | Resolved [ 5 ] | Reopened [ 4 ] |
It seems that even non-IE browsers are not evaluating the head if the HTML markup is XML invalid, so the original fix is enabled for all browsers. Added also a fix for non-IE browsers to avoid having the bridge configuration lost during 'body' element update.
| Status | Reopened [ 4 ] | Resolved [ 5 ] |
| Resolution | Fixed [ 1 ] |
icefaces-ee2 trunk revision# 26273
server: tomcat6
Browser: FF3.6 , IE6,7,8 & Chrome8
Problem still exists.
| Resolution | Fixed [ 1 ] | |
| Status | Resolved [ 5 ] | Reopened [ 4 ] |
I re-verified but cannot see any problems. The title of the document is changed after the navigation is executed. Tested with IE and FF.
icefaces-ee2 trunk revision# 26294
server: tomcat6
Browser: FF3.6 , IE6,7,8 & Chrome8
Tested successfully.
| Status | Reopened [ 4 ] | Closed [ 6 ] |
| Resolution | Fixed [ 1 ] |
| Resolution | Fixed [ 1 ] | |
| Status | Closed [ 6 ] | Reopened [ 4 ] |
| Assignee Priority | P1 | P2 |
| Salesforce Case | [] | |
| Fix Version/s | 2.0.1 [ 10255 ] | |
| Fix Version/s | 2.0.0-EE-Beta1 [ 10250 ] |
| Repository | Revision | Date | User | Message |
| ICEsoft Public SVN Repository | #24080 | Wed Mar 02 15:58:22 MST 2011 | mircea.toma | |
| Files Changed | ||||
|
MODIFY
/icefaces2/trunk/icefaces/core/src/main/javascript/application.js
MODIFY
/icefaces2/trunk/icefaces/core/build.javascript.xml
ADD
/icefaces2/trunk/icefaces/core/src/main/javascript/fixjsf.js
|
Moved JSF fix from icefaces-ee to icefaces2 since the fix addresses all the browsers. The fix resides in a separate file that's merged by the build into the final bridge code.
Moved also viewRootUpdate test application into ICEfaces 2 samples/core/test directory.
| Status | Reopened [ 4 ] | Resolved [ 5 ] |
| Resolution | Fixed [ 1 ] |
| Link | This issue blocks ICE-6035 [ ICE-6035 ] |
| Summary | CLONE -DOM update for entire document does not evaluate JS code referenced in <head> on IE browsers | DOM update for entire document does not evaluate JS code referenced in <head> on IE browsers |
| Salesforce Case | [] | |
| Security | Private [ 10001 ] |
| Assignee Priority | P2 |
| Status | Resolved [ 5 ] | Closed [ 6 ] |
Implemented http://java.net/jira/browse/JAVASERVERFACES-1790 fix in ICEfaces EE by using callbacks offered by the ICEfaces bridge. The implementation evaluates referenced or inlined only if it was not evaluated before.