Details
Description
The BridgeFacesContext has this around line 148:
String keepViewRoot = externalContext.getInitParameter("com.icesoft.faces.retainViewRoot");
if (keepViewRoot != null) {
retainViewRoot = Boolean.parseBoolean(keepViewRoot);
}
The parseBoolean method is only available in Java 5+. We don't compile using 1.4 for various reasons and testing is probably not catching this due to the conditional requirement for the context parameter. Should be changed to:
String keepViewRoot = externalContext.getInitParameter("com.icesoft.faces.retainViewRoot");
if (keepViewRoot != null) {
retainViewRoot = Boolean.valueOf(keepViewRoot).booleanValue();
}
String keepViewRoot = externalContext.getInitParameter("com.icesoft.faces.retainViewRoot");
if (keepViewRoot != null) {
retainViewRoot = Boolean.parseBoolean(keepViewRoot);
}
The parseBoolean method is only available in Java 5+. We don't compile using 1.4 for various reasons and testing is probably not catching this due to the conditional requirement for the context parameter. Should be changed to:
String keepViewRoot = externalContext.getInitParameter("com.icesoft.faces.retainViewRoot");
if (keepViewRoot != null) {
retainViewRoot = Boolean.valueOf(keepViewRoot).booleanValue();
}
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
Deryk Sinotte
created issue -
Deryk Sinotte
made changes -
Field | Original Value | New Value |
---|---|---|
Assignee | Deryk Sinotte [ deryk.sinotte ] |
Deryk Sinotte
made changes -
Status | Open [ 1 ] | Resolved [ 5 ] |
Fix Version/s | 1.8.2 [ 10190 ] | |
Resolution | Fixed [ 1 ] |
Ken Fyten
made changes -
Fix Version/s | 1.8.2-RC1 [ 10210 ] |
Ken Fyten
made changes -
Status | Resolved [ 5 ] | Closed [ 6 ] |