suggested fix:

Index: D:/Documents and Settings/pbreau/workspace/ICEfaces/core/src/com/icesoft/faces/context/DOMResponseWriter.java
===================================================================
— D:/Documents and Settings/pbreau/workspace/ICEfaces/core/src/com/icesoft/faces/context/DOMResponseWriter.java (revision 14627)
+++ D:/Documents and Settings/pbreau/workspace/ICEfaces/core/src/com/icesoft/faces/context/DOMResponseWriter.java (working copy)
@@ -66,421 +66,459 @@
import java.util.Map;

/**

• * <p><strong>DOMResponseWriter</strong> is a DOM specific implementation of
  + * <p>
  + * <strong>DOMResponseWriter</strong> is a DOM specific implementation of

• <code>javax.faces.context.ResponseWriter</code>.
  */
  public class DOMResponseWriter extends ResponseWriter {

• private static final Log log = LogFactory.getLog(DOMResponseWriter.class);
• public static final String STREAM_WRITING = "com.icesoft.faces.streamWriting";
• //DOM and current node being written to for this ResponseWriter
• public static final String DOCTYPE_PUBLIC = "com.icesoft.doctype.public";
• public static final String DOCTYPE_SYSTEM = "com.icesoft.doctype.system";
• public static final String DOCTYPE_ROOT = "com.icesoft.doctype.root";
• public static final String DOCTYPE_OUTPUT = "com.icesoft.doctype.output";
• public static final String DOCTYPE_PRETTY_PRINTING =
• "com.icesoft.doctype.prettyprinting";
  + private static final Log log = LogFactory.getLog(DOMResponseWriter.class);
  + public static final String STREAM_WRITING = "com.icesoft.faces.streamWriting";
  + // DOM and current node being written to for this ResponseWriter
  + public static final String DOCTYPE_PUBLIC = "com.icesoft.doctype.public";
  + public static final String DOCTYPE_SYSTEM = "com.icesoft.doctype.system";
  + public static final String DOCTYPE_ROOT = "com.icesoft.doctype.root";
  + public static final String DOCTYPE_OUTPUT = "com.icesoft.doctype.output";
  + public static final String DOCTYPE_PRETTY_PRINTING = "com.icesoft.doctype.prettyprinting";

• public static final String RESPONSE_DOM = "com.icesoft.domResponseDocument";
• public static final String RESPONSE_DOM_ID =
• "com.icesoft.domResponseDocumentID";
• public static final String OLD_DOM = "com.icesoft.oldDocument";
• public static final String RESPONSE_VIEWROOT =
• "com.icesoft.domResponseViewRoot";
• //Hashtable of DOMContext objects associated with each component
• public static final String RESPONSE_CONTEXTS_TABLE =
• "com.icesoft.domResponseContexts";
• private static DocumentBuilder DOCUMENT_BUILDER;
  + public static final String RESPONSE_DOM = "com.icesoft.domResponseDocument";
  + public static final String RESPONSE_DOM_ID = "com.icesoft.domResponseDocumentID";
  + public static final String OLD_DOM = "com.icesoft.oldDocument";
  + public static final String RESPONSE_VIEWROOT = "com.icesoft.domResponseViewRoot";
  + // Hashtable of DOMContext objects associated with each component
  + public static final String RESPONSE_CONTEXTS_TABLE = "com.icesoft.domResponseContexts";
  + private static DocumentBuilder DOCUMENT_BUILDER;

• static {
• try { - DOCUMENT_BUILDER = - DocumentBuilderFactory.newInstance().newDocumentBuilder(); - }

  catch (ParserConfigurationException e)

  { - log.error("Cannot acquire a DocumentBuilder", e); - }
• }
  + static
  Unknown macro: {+ try { + DOCUMENT_BUILDER = DocumentBuilderFactory.newInstance() + .newDocumentBuilder(); + } catch (ParserConfigurationException e) { + log.error("Cannot acquire a DocumentBuilder", e); + }+ }

• private static boolean isStreamWritingFlag = false;
• private Document document;
• private Node cursor;
• private Map domResponseContexts;
• private Map contextServletTable;
• private BridgeFacesContext context;
• private DOMSerializer serializer;
• private Configuration configuration;
  + private static boolean isStreamWritingFlag = false;
  + private Document document;
  + private Node cursor;
  + private Map domResponseContexts;
  + private Map contextServletTable;
  + private BridgeFacesContext context;
  + private DOMSerializer serializer;
  + private Configuration configuration;

• public DOMResponseWriter(FacesContext context, DOMSerializer serializer, Configuration configuration) {
• this.serializer = serializer;
• this.configuration = configuration;
• try { - this.context = (BridgeFacesContext) context; - }

  catch (ClassCastException e)

  { - throw new IllegalStateException( - "ICEfaces requires the PersistentFacesServlet. " + - "Please check your web.xml servlet mappings"); - }
• this.initialize();
• }
  + public DOMResponseWriter(FacesContext context, DOMSerializer serializer,
  + Configuration configuration)
  Unknown macro: {+ this.serializer = serializer;+ this.configuration = configuration;+ try { + this.context = (BridgeFacesContext) context; + } catch (ClassCastException e) { + throw new IllegalStateException( + "ICEfaces requires the PersistentFacesServlet. " + + "Please check your web.xml servlet mappings"); + }+ this.initialize();+ }

• Map getDomResponseContexts() { - return domResponseContexts; - }

  + Map getDomResponseContexts()

  { + return domResponseContexts; + }

• public Node getCursorParent() { - return cursor; - }

  + public Node getCursorParent()

  { + return cursor; + }

• public Document getDocument() { - return document; - }

  + public Document getDocument()

  { + return document; + }

• public String getContentType() { - return "text/html; charset=UTF-8"; - }

  + public String getContentType()

  { + return "text/html; charset=UTF-8"; + }

• public String getCharacterEncoding() { - return "UTF-8"; - }

  + public String getCharacterEncoding()

  { + return "UTF-8"; + }

• public void startDocument() throws IOException { - }
  + public void startDocument() throws IOException { + }
  
  - private void initialize() {
  - contextServletTable = D2DViewHandler.getContextServletTable(context);
  - // contexts for each component
  - if (contextServletTable
  - .containsKey(DOMResponseWriter.RESPONSE_CONTEXTS_TABLE)) { - domResponseContexts = (Map) contextServletTable - .get(DOMResponseWriter.RESPONSE_CONTEXTS_TABLE); - }
  - if (null == domResponseContexts) { - domResponseContexts = new HashMap(); - contextServletTable.put(DOMResponseWriter.RESPONSE_CONTEXTS_TABLE, - domResponseContexts); - }
  - // viewroot, application
  - contextServletTable.put(DOMResponseWriter.RESPONSE_VIEWROOT,
  - context.getViewRoot());
  - cursor = document = DOCUMENT_BUILDER.newDocument();
  - contextServletTable.put(DOMResponseWriter.RESPONSE_DOM, document);
  - boolean streamWritingParam = "true".equalsIgnoreCase(
  - context.getExternalContext().getInitParameter(
  - DOMResponseWriter.STREAM_WRITING));
  - DOMResponseWriter.isStreamWritingFlag =
  - Beans.isDesignTime() || streamWritingParam;
  - }
  + private void initialize() {
  + contextServletTable = D2DViewHandler.getContextServletTable(context);
  + // contexts for each component
  + if (contextServletTable
  + .containsKey(DOMResponseWriter.RESPONSE_CONTEXTS_TABLE)) { + domResponseContexts = (Map) contextServletTable + .get(DOMResponseWriter.RESPONSE_CONTEXTS_TABLE); + }
  + if (null == domResponseContexts) { + domResponseContexts = new HashMap(); + contextServletTable.put(DOMResponseWriter.RESPONSE_CONTEXTS_TABLE, + domResponseContexts); + }
  + // viewroot, application
  + contextServletTable.put(DOMResponseWriter.RESPONSE_VIEWROOT, context
  + .getViewRoot());
  + cursor = document = DOCUMENT_BUILDER.newDocument();
  + contextServletTable.put(DOMResponseWriter.RESPONSE_DOM, document);
  + boolean streamWritingParam = "true".equalsIgnoreCase(context
  + .getExternalContext().getInitParameter(
  + DOMResponseWriter.STREAM_WRITING));
  + DOMResponseWriter.isStreamWritingFlag = Beans.isDesignTime()
  + || streamWritingParam;
  + }
  
  - public void endDocument() throws IOException {
  - if (!isStreamWriting()) { - enhanceAndFixDocument(); - serializer.serialize(document); - }
  - }
  + public void endDocument() throws IOException {
  + if (!isStreamWriting()) { + enhanceAndFixDocument(); + serializer.serialize(document); + }
  + }
  
  - public void flush() throws IOException { - }

  + public void flush() throws IOException

  { + }
  
  - public void startElement(String name, UIComponent componentForElement)
  - throws IOException {
  - Node oldCursor = cursor;
  - Element elem = document.createElement(name);
  - cursor = cursor.appendChild(elem);
  - if (log.isTraceEnabled()) { - log.trace("startElement() name: " + name + " elem: " + elem + - " oldCursor: " + oldCursor + " newCursor: " + cursor); - }
  - }
  + public void startElement(String name, UIComponent componentForElement)
  + throws IOException {
  + Node oldCursor = cursor;
  + Element elem = document.createElement(name);
  + if (log.isTraceEnabled()) { + log.trace("attempting startElement() name: " + name + " elem: " + + elem + " oldCursor: " + oldCursor); + }
  + cursor = cursor.appendChild(elem);
  + if (log.isTraceEnabled()) { + log.trace("startElement() newCursor: " + cursor); + }
  + }
  
  - public void endElement(String name) throws IOException {
  - Node oldCursor = cursor;
  - cursor = cursor.getParentNode();
  - if (log.isTraceEnabled()) { - log.trace("endElement() name: " + name + " oldCursor: " + - oldCursor + " newCursor: " + cursor); - }
  - }
  + public void endElement(String name) throws IOException {
  + Node oldCursor = cursor;
  + cursor = cursor.getParentNode();
  + if (log.isTraceEnabled()) { + log.trace("endElement() name: " + name + " oldCursor: " + + oldCursor + " newCursor: " + cursor); + }
  + }
  
  - public void writeAttribute(String name, Object value,
  - String componentPropertyName)
  - throws IOException { - //name.trim() because cardemo had a leading space in an attribute name - //which made the DOM processor choke - ((Element) cursor).setAttribute(name.trim(), String.valueOf(value)); - }
  + public void writeAttribute(String name, Object value,
  + String componentPropertyName) throws IOException { + // name.trim() because cardemo had a leading space in an attribute name + // which made the DOM processor choke + ((Element) cursor).setAttribute(name.trim(), String.valueOf(value)); + }
  
  - public void writeURIAttribute(String name, Object value,
  - String componentPropertyName)
  - throws IOException {
  - String stringValue = String.valueOf(value);
  - if (stringValue.startsWith("javascript:")) { - ((Element) cursor).setAttribute(name, stringValue); - } else { - ((Element) cursor) - .setAttribute(name, stringValue.replace(' ', '+')); - }
  - }
  + public void writeURIAttribute(String name, Object value,
  + String componentPropertyName) throws IOException {
  + String stringValue = String.valueOf(value);
  + if (stringValue.startsWith("javascript:")) { + ((Element) cursor).setAttribute(name, stringValue); + } else { + ((Element) cursor) + .setAttribute(name, stringValue.replace(' ', '+')); + }
  + }
  
  - public void writeComment(Object comment) throws IOException {
  - if (log.isTraceEnabled()) { - log.trace("writeComment() comment: " + comment); - }
  - cursor.appendChild(document.createComment(String.valueOf(comment)));
  - }
  + public void writeComment(Object comment) throws IOException {
  + if (log.isTraceEnabled()) { + log.trace("writeComment() comment: " + comment); + }
  + cursor.appendChild(document.createComment(String.valueOf(comment)));
  + }
  
  - public void writeText(Object text, String componentPropertyName)
  - throws IOException {
  - if (log.isTraceEnabled()) { - log.trace("writeText(O,S) text: " + text); - }
  - cursor.appendChild(document.createTextNode(String.valueOf(text)));
  - }
  + public void writeText(Object text, String componentPropertyName)
  + throws IOException {
  + if (log.isTraceEnabled()) { + log.trace("writeText(O,S) text: " + text); + }
  + cursor.appendChild(document.createTextNode(String.valueOf(text)));
  + }
  
  - public void writeText(char text[], int off, int len) throws IOException {
  - if (log.isTraceEnabled()) { - log.trace("writeText(c[],i,i) text: " + - (new String(text, off, len))); - }
  - cursor.appendChild(document.createTextNode(new String(text, off, len)));
  - }
  + public void writeText(char text[], int off, int len) throws IOException {
  + if (log.isTraceEnabled()) { + log.trace("writeText(c[],i,i) text: " + + (new String(text, off, len))); + }
  + cursor.appendChild(document.createTextNode(new String(text, off, len)));
  + }
  
  - public ResponseWriter cloneWithWriter(Writer writer) {
  - //FIXME: This is a hack for DOM rendering but JSF currently clones the writer
  - //just as the components are complete
  - if (null != document) {
  - try { - endDocument(); - } catch (IOException e) { - throw new IllegalStateException(e.toString()); - }
  - }
  - try { - return new DOMResponseWriter(context, serializer, configuration); - } catch (FacesException e) { - throw new IllegalStateException(); - }
  - }
  + public ResponseWriter cloneWithWriter(Writer writer) {
  + // FIXME: This is a hack for DOM rendering but JSF currently clones the
  + // writer
  + // just as the components are complete
  + if (null != document) {
  + try { + endDocument(); + } catch (IOException e) { + throw new IllegalStateException(e.toString()); + }
  + }
  + try { + return new DOMResponseWriter(context, serializer, configuration); + } catch (FacesException e) { + throw new IllegalStateException(); + }
  + }
  
  - public void close() throws IOException { - }
  + public void close() throws IOException { + }

• public void write(char[] cbuf, int off, int len) throws IOException {
• if (log.isTraceEnabled()) { - log.trace("writeText(c[],i,i) str: " + - (new String(cbuf, off, len))); - }
• cursor.appendChild(document.createTextNode(new String(cbuf, off, len)));
• }
  + public void write(char[] cbuf, int off, int len) throws IOException
  Unknown macro: {+ if (log.isTraceEnabled()) { + log.trace("writeText(c[],i,i) str: " + + (new String(cbuf, off, len))); + }+ cursor.appendChild(document.createTextNode(new String(cbuf, off, len)));+ }

• public void write(int c) throws IOException {
• if (log.isTraceEnabled()) { - log.trace("write(i) hex: " + Integer.toHexString(c) + - " decimal: " + c); - }
• cursor.appendChild(document.createTextNode(String.valueOf((char) c)));
• }
  + public void write(int c) throws IOException
  Unknown macro: {+ if (log.isTraceEnabled()) { + log.trace("write(i) hex: " + Integer.toHexString(c) + + " decimal: " + c); + }+ cursor.appendChild(document.createTextNode(String.valueOf((char) c)));+ }

• public void write(String str) throws IOException {
• if (log.isTraceEnabled()) { - log.trace("write(S) str: " + str); - }
• cursor.appendChild(document.createTextNode(str));
• }
  + public void write(String str) throws IOException
  Unknown macro: {+ if (log.isTraceEnabled()) { + log.trace("write(S) str: " + str); + }+ cursor.appendChild(document.createTextNode(str));+ }

• public void write(String str, int off, int len) throws IOException {
• if (log.isTraceEnabled()) { - log.trace("write(S,i,i) str_sub: " + str.substring(off, len)); - }
• cursor.appendChild(document.createTextNode(str.substring(off, len)));
• }
  + public void write(String str, int off, int len) throws IOException
  Unknown macro: {+ if (log.isTraceEnabled()) { + log.trace("write(S,i,i) str_sub: " + str.substring(off, len)); + }+ cursor.appendChild(document.createTextNode(str.substring(off, len)));+ }

• private void enhanceAndFixDocument() {
• Element html = (Element) document.getDocumentElement();
• enhanceHtml(html = "html".equals(html.getTagName()) ? html : fixHtml());
  + private void enhanceAndFixDocument() { + Element html = (Element) document.getDocumentElement(); + enhanceHtml(html = "html".equals(html.getTagName()) ? html : fixHtml()); - Element head = (Element) document.getElementsByTagName("head").item(0); - enhanceHead(head == null ? fixHead() : head); + Element head = (Element) document.getElementsByTagName("head").item(0); + enhanceHead(head == null ? fixHead() : head); - Element body = (Element) document.getElementsByTagName("body").item(0); - enhanceBody(body == null ? fixBody() : body); - }

  + Element body = (Element) document.getElementsByTagName("body").item(0);
  + enhanceBody(body == null ? fixBody() : body);
  + }

• private void enhanceHtml(Element html) { - //add lang attribute - Locale locale = context.getApplication().getViewHandler().calculateLocale(context); - html.setAttribute("lang", locale.getLanguage()); - }

  + private void enhanceHtml(Element html)

  { + // add lang attribute + Locale locale = context.getApplication().getViewHandler() + .calculateLocale(context); + html.setAttribute("lang", locale.getLanguage()); + }

• private void enhanceBody(Element body) {
• //id required for forwarded (server-side) redirects
• body.setAttribute("id", "body");
• Element iframe = document.createElement("iframe");
• body.insertBefore(iframe, body.getFirstChild());
• iframe.setAttribute("id", "history-frame");
• Object request = context.getExternalContext().getRequest();
  + private void enhanceBody(Element body) {
  + // id required for forwarded (server-side) redirects
  + body.setAttribute("id", "body");
  + Element iframe = document.createElement("iframe");
  + body.insertBefore(iframe, body.getFirstChild());
  + iframe.setAttribute("id", "history-frame");
  + Object request = context.getExternalContext().getRequest();

• final String frameURI;
• //another "workaround" to resolve the iframe URI
• if (request instanceof HttpServletRequest) {
• HttpServletRequest httpRequest = (HttpServletRequest) request;
• if (httpRequest.getRequestURI() == null) { - frameURI = "about:blank"; - }

  else

  { - frameURI = CoreUtils.resolveResourceURL(FacesContext.getCurrentInstance(), - "/xmlhttp/blank.iface"); - }
• } else { - frameURI = "about:blank"; - }
• iframe.setAttribute("title", "Icefaces Redirect");
• iframe.setAttribute("src", frameURI);
• iframe.setAttribute("frameborder", "0");
• iframe.setAttribute("style",
• "z-index: 10000; visibility: hidden; width: 0; height: 0; position: absolute; opacity: 0.22; filter: alpha(opacity=22);");
  + final String frameURI;
  + // another "workaround" to resolve the iframe URI
  + if (request instanceof HttpServletRequest)
  Unknown macro: {+ HttpServletRequest httpRequest = (HttpServletRequest) request;+ if (httpRequest.getRequestURI() == null) { + frameURI = "about:blank"; + } else { + frameURI = CoreUtils.resolveResourceURL(FacesContext + .getCurrentInstance(), "/xmlhttp/blank.iface"); + }+ }

  else

  { + frameURI = "about:blank"; + }

  + iframe.setAttribute("title", "Icefaces Redirect");
  + iframe.setAttribute("src", frameURI);
  + iframe.setAttribute("frameborder", "0");
  + iframe
  + .setAttribute(
  + "style",
  + "z-index: 10000; visibility: hidden; width: 0; height: 0; position: absolute; opacity: 0.22; filter: alpha(opacity=22);");

• // TODO This is only meant to be a transitional focus retention(management) solution.
• String focusId = context.getFocusId();
• if (focusId != null && !focusId.equals("null")) { - JavascriptContext.focus(context, focusId); - }

  + // TODO This is only meant to be a transitional focus
  + // retention(management) solution.
  + String focusId = context.getFocusId();
  + if (focusId != null && !focusId.equals("null"))

  { + JavascriptContext.focus(context, focusId); + }

• Element script =
• (Element) body.appendChild(document.createElement("script"));
• script.setAttribute("id", JavascriptContext.DYNAMIC_CODE_ID);
• script.setAttribute("language", "javascript");
• String calls = JavascriptContext.getJavascriptCalls(context);
• script.appendChild(document.createTextNode(calls));
  + Element script = (Element) body.appendChild(document
  + .createElement("script"));
  + script.setAttribute("id", JavascriptContext.DYNAMIC_CODE_ID);
  + script.setAttribute("language", "javascript");
  + String calls = JavascriptContext.getJavascriptCalls(context);
  + script.appendChild(document.createTextNode(calls));

• Map session = context.getExternalContext().getSessionMap();
• ElementController.from(session).addInto(body);
  + Map session = context.getExternalContext().getSessionMap();
  + ElementController.from(session).addInto(body);

• String sessionIDScript = "window.session='" + context.getIceFacesId() + "';\n";
• //add viewIdentifier property to the container element ("body" for servlet env., any element for the portlet env.)
• String viewIDScript = "document.getElementById('configuration-script').parentNode.viewIdentifier=" + context.getViewNumber() + ";\n";
• String viewsIDScript = "if (!window.views) window.views = []; window.views.push(" + context.getViewNumber() + ");\n";
  + String sessionIDScript = "window.session='" + context.getIceFacesId()
  + + "';\n";
  + // add viewIdentifier property to the container element ("body" for
  + // servlet env., any element for the portlet env.)
  + String viewIDScript = "document.getElementById('configuration-script').parentNode.viewIdentifier="
  + + context.getViewNumber() + ";\n";
  + String viewsIDScript = "if (!window.views) window.views = []; window.views.push("
  + + context.getViewNumber() + ");\n";

• String configurationScript =
• "window.configuration = {" +
• "synchronous: " + configuration.getAttribute("synchronousUpdate", "false") + "," +
• "redirectURI: " + configuration.getAttribute("connectionLostRedirectURI", "null") + "," +
• "connection: {" +
• "context: '" + context.getApplication().getViewHandler().getResourceURL(context, "/") + "'," +
• "timeout: " + configuration.getAttributeAsLong("connectionTimeout", 30000) + "," +
• "heartbeat: {" + - "interval: " + configuration.getAttributeAsLong("heartbeatInterval", 20000) + "," + - "timeout: " + configuration.getAttributeAsLong("heartbeatTimeout", 3000) + "," + - "retries: " + configuration.getAttributeAsLong("heartbeatRetries", 3) + - "}

  " +

• "}" +
• "};\n";
  + String configurationScript = "window.configuration = {"
  + + "synchronous: "
  + + configuration.getAttribute("synchronousUpdate", "false")
  + + ","
  + + "redirectURI: "
  + + configuration.getAttribute("connectionLostRedirectURI",
  + "null")
  + + ","
  + + "connection:
  Unknown macro: {"+ + "context}

  " + "};\n";

• Element configurationElement = (Element) body.appendChild(document.createElement("script"));
• configurationElement.setAttribute("id", "configuration-script");
• configurationElement.setAttribute("language", "javascript");
• configurationElement.appendChild(document.createTextNode(sessionIDScript + viewIDScript + viewsIDScript + configurationScript));
• body.appendChild(configurationElement);
• }
  + Element configurationElement = (Element) body.appendChild(document
  + .createElement("script"));
  + configurationElement.setAttribute("id", "configuration-script");
  + configurationElement.setAttribute("language", "javascript");
  + configurationElement.appendChild(document
  + .createTextNode(sessionIDScript + viewIDScript + viewsIDScript
  + + configurationScript));
  + body.appendChild(configurationElement);
  + }

• private void enhanceHead(Element head) {
• ViewHandler handler = context.getApplication().getViewHandler();
• //id required for forwarded (server-side) redirects
• head.setAttribute("id", "head");
• Element meta =
• (Element) head.appendChild(document.createElement("meta"));
• meta.setAttribute("name", "icefaces");
• meta.setAttribute("content", "Rendered by ICEFaces D2D");
  + private void addNoScriptIfNotAlreadyAdded(Element head)
  Unknown macro: {+ + NodeList list = head.getElementsByTagName("noscript");+ if( list.getLength() == 0 ){ + Element noscript = (Element) head.appendChild(document + .createElement("noscript")); + Element noscriptMeta = (Element) noscript.appendChild(document + .createElement("meta")); + noscriptMeta.setAttribute("http-equiv", "refresh"); + noscriptMeta.setAttribute("content", "0;url=" + + context.getApplication().getViewHandler().getResourceURL( + context, "/xmlhttp/javascript-blocked")); + }+ }

• Element noscript =
• (Element) head.appendChild(document.createElement("noscript"));
• Element noscriptMeta =
• (Element) noscript.appendChild(document.createElement("meta"));
• noscriptMeta.setAttribute("http-equiv", "refresh");
• noscriptMeta
• .setAttribute("content", "0;url=" + handler.getResourceURL(context, "/xmlhttp/javascript-blocked"));
  + private void enhanceHead(Element head) {
  + ViewHandler handler = context.getApplication().getViewHandler();
  + // id required for forwarded (server-side) redirects
  + head.setAttribute("id", "head");
  + Element meta = (Element) head.appendChild(document
  + .createElement("meta"));
  + meta.setAttribute("name", "icefaces");
  + meta.setAttribute("content", "Rendered by ICEFaces D2D");

• //load libraries
• Collection libs = new ArrayList();
• if (context.getExternalContext().getInitParameter(D2DViewHandler.INCLUDE_OPEN_AJAX_HUB) != null) { - libs.add("/xmlhttp/openajax.js"); - }
• libs.add("/xmlhttp" + StartupTime.getStartupInc() + "icefaces-d2d.js");
• //todo: refactor how extral libraries are loaded into the bridge; always include extra libraries for now
• libs.add("/xmlhttp" + StartupTime.getStartupInc() + "ice-extras.js");
• if (context.getExternalContext().getRequestMap().get(Constants.INC_SERVLET_PATH) == null) {
• String[] componentLibs = JavascriptContext.getIncludedLibs(context);
• for (int i = 0; i < componentLibs.length; i++) {
• String componentLib = componentLibs[i];
• if (!libs.contains(componentLib)) { - libs.add(componentLib); - }
• }
• }
  + addNoScriptIfNotAlreadyAdded(head);

• Iterator iterator = libs.iterator();
• while (iterator.hasNext()) { - String lib = (String) iterator.next(); - Element script = (Element) head - .appendChild(document.createElement("script")); - script.setAttribute("language", "javascript"); - script.setAttribute("src", handler.getResourceURL(context, lib)); - }

  + // load libraries
  + Collection libs = new ArrayList();
  + if (context.getExternalContext().getInitParameter(
  + D2DViewHandler.INCLUDE_OPEN_AJAX_HUB) != null)

  { + libs.add("/xmlhttp/openajax.js"); + }

  + libs.add("/xmlhttp" + StartupTime.getStartupInc() + "icefaces-d2d.js");
  + // todo: refactor how extral libraries are loaded into the bridge;
  + // always include extra libraries for now
  + libs.add("/xmlhttp" + StartupTime.getStartupInc() + "ice-extras.js");
  + if (context.getExternalContext().getRequestMap().get(
  + Constants.INC_SERVLET_PATH) == null) {
  + String[] componentLibs = JavascriptContext.getIncludedLibs(context);
  + for (int i = 0; i < componentLibs.length; i++)

  Unknown macro: {+ String componentLib = componentLibs[i];+ if (!libs.contains(componentLib)) { + libs.add(componentLib); + }+ }

  + }

• String sessionIdentifier = context.getIceFacesId();
• Element viewAndSessionScript = (Element) head.appendChild(document.createElement("script"));
• viewAndSessionScript.setAttribute("language", "javascript");
• viewAndSessionScript.appendChild(document.createTextNode(
• "window.session = '" + sessionIdentifier + "';"
• ));
• }
  + Iterator iterator = libs.iterator();
  + while (iterator.hasNext()) { + String lib = (String) iterator.next(); + Element script = (Element) head.appendChild(document + .createElement("script")); + script.setAttribute("language", "javascript"); + script.setAttribute("src", handler.getResourceURL(context, lib)); + }

• private Element fixHtml() { - Element root = document.getDocumentElement(); - Element html = document.createElement("html"); - document.replaceChild(html, root); - html.appendChild(root); + String sessionIdentifier = context.getIceFacesId(); + Element viewAndSessionScript = (Element) head.appendChild(document + .createElement("script")); + viewAndSessionScript.setAttribute("language", "javascript"); + viewAndSessionScript + .appendChild(document.createTextNode("window.session = '" + + sessionIdentifier + "';")); + }

• return html;
• }
  + private Element fixHtml() {
  + Element root = document.getDocumentElement();
  + Element html = document.createElement("html");
  + document.replaceChild(html, root);
  + html.appendChild(root);

• private Element fixBody() {
• Element html = document.getDocumentElement();
• Element body = document.createElement("body");
• NodeList children = html.getChildNodes();
• int length = children.getLength();
• Node[] nodes = new Node[length];
• //copy the children first, since NodeList is live
• for (int i = 0; i < nodes.length; i++) nodes[i] = children.item;
• for (int i = 0; i < nodes.length; i++) { - Node node = nodes[i]; - if (!(node instanceof Element && - "head".equals(((Element) node).getTagName()))) - body.appendChild(node); - }
• html.appendChild(body);
  + return html;
  + }

• return body;
• }
  + private Element fixBody() {
  + Element html = document.getDocumentElement();
  + Element body = document.createElement("body");
  + NodeList children = html.getChildNodes();
  + int length = children.getLength();
  + Node[] nodes = new Node[length];
  + // copy the children first, since NodeList is live
  + for (int i = 0; i < nodes.length; i++)
  + nodes[i] = children.item;
  + for (int i = 0; i < nodes.length; i++) { + Node node = nodes[i]; + if (!(node instanceof Element && "head".equals(((Element) node) + .getTagName()))) + body.appendChild(node); + }

  + html.appendChild(body);

• private Element fixHead() { - Element html = document.getDocumentElement(); - Element head = document.createElement("head"); - html.insertBefore(head, html.getFirstChild()); + return body; + }

• return head;
• }
  + private Element fixHead() {
  + Element html = document.getDocumentElement();
  + Element head = document.createElement("head");
  + html.insertBefore(head, html.getFirstChild());

• /**
• * This method sets the write cursor for DOM modifications. Subsequent DOM
• * modifications will take place below the cursor element.
• *
• * @param cursorParent parent node for subsequent modifications to the DOM
• */
• protected void setCursorParent(Node cursorParent) { - this.cursor = cursorParent; - }

  + return head;
  + }

• public static boolean isStreamWriting() { - return isStreamWritingFlag; - }

  + /**
  + * This method sets the write cursor for DOM modifications. Subsequent DOM
  + * modifications will take place below the cursor element.
  + *
  + * @param cursorParent
  + * parent node for subsequent modifications to the DOM
  + */
  + protected void setCursorParent(Node cursorParent)

  { + this.cursor = cursorParent; + }

+ public static boolean isStreamWriting()

}

without complicated diff:

private void addNoScriptIfNotAlreadyAdded(Element head) {

NodeList list = head.getElementsByTagName("noscript");
if( list.getLength() == 0 )

}

private void enhanceHead(Element head) {
ViewHandler handler = context.getApplication().getViewHandler();
// id required for forwarded (server-side) redirects
head.setAttribute("id", "head");
Element meta = (Element) head.appendChild(document
.createElement("meta"));
meta.setAttribute("name", "icefaces");
meta.setAttribute("content", "Rendered by ICEFaces D2D");

addNoScriptIfNotAlreadyAdded(head);

// load libraries
Collection libs = new ArrayList();
if (context.getExternalContext().getInitParameter(
D2DViewHandler.INCLUDE_OPEN_AJAX_HUB) != null)

libs.add("/xmlhttp" + StartupTime.getStartupInc() + "icefaces-d2d.js");
// todo: refactor how extral libraries are loaded into the bridge;
// always include extra libraries for now
libs.add("/xmlhttp" + StartupTime.getStartupInc() + "ice-extras.js");
if (context.getExternalContext().getRequestMap().get(
Constants.INC_SERVLET_PATH) == null) {
String[] componentLibs = JavascriptContext.getIncludedLibs(context);
for (int i = 0; i < componentLibs.length; i++) {
String componentLib = componentLibs[i];
if (!libs.contains(componentLib))

}
}

Iterator iterator = libs.iterator();
while (iterator.hasNext())

String sessionIdentifier = context.getIceFacesId();
Element viewAndSessionScript = (Element) head.appendChild(document
.createElement("script"));
viewAndSessionScript.setAttribute("language", "javascript");
viewAndSessionScript
.appendChild(document.createTextNode("window.session = '"
+ sessionIdentifier + "';"));
}

Marking as Closed / Invalid as part of legacy ICEfaces 1.x, 2.x JIRA cleanup.

Note: This issue may be resolved in a newer ICEfaces release, available here: http://www.icesoft.org/java/downloads/icefaces-downloads.jsf

If the issue persists with the current ICEfaces release, please create a new JIRA for it.