ICEfaces
  1. ICEfaces
  2. ICE-6389

IceFaces OpenAjax integration is broken

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 1.8.2-EE-GA_P02
    • Fix Version/s: None
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      -

      Description

      When OpenAjax hub is enabled using the web.xml configuration parameter com.icesoft.faces.openAjaxHub = True, the browser fails to load openajax.js with HTTP 404 error as shown in the attached screenshot of the firebug console. On further investigation found that the path for src attribute for script tag being generated by icefaces differs in case of other js files such as icefaces-d2d.js and openajax.js

      For other files the path includes application startup time as shown below -
      /xmlhttp/1292322511113/icefaces-d2d.js
      /xmlhttp/1292322511113/ice-extras.js

      But, the path for openajax.js doesn't include this -
      /xmlhttp/openajax.js

      The code in question is line 499 in com.icesoft.faces.context.DOMResponseWriter class -

      private void appendContentReferences(Element container) {
      //load libraries
      Collection libs = new ArrayList();
      if (configuration.getAttributeAsBoolean("openAjaxHub", false)) {
      libs.add("/xmlhttp/openajax.js");
      }
      libs.add("/xmlhttp" + StartupTime.getStartupInc() + "icefaces-d2d.js");
      //todo: refactor how external libraries are loaded into the bridge; always include extra libraries for now
      libs.add("/xmlhttp" + StartupTime.getStartupInc() + "ice-extras.js");

      It's a trivial fix as demonstrated below -

      Remove Line -> libs.add("/xmlhttp/openajax.js");
      Replace With -> libs.add("/xmlhttp" + StartupTime.getStartupInc() + "openajax.js");

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #23567 Wed Dec 15 19:32:11 MST 2010 tjohnson Fix for ICE-6389
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/context/DOMResponseWriter.java
        Tyler Johnson created issue -
        Tyler Johnson made changes -
        Field Original Value New Value
        Salesforce Case [5007000000F4UZn]
        Tyler Johnson made changes -
        Assignee Tyler Johnson [ tyler.johnson ]
        Tyler Johnson made changes -
        Salesforce Case Reference 5007000000I0WhhAAF
        Tyler Johnson made changes -
        Salesforce Case Reference 5007000000I0WhhAAF
        Tyler Johnson made changes -
        Comment [ test ]
        Ken Fyten made changes -
        Assignee Tyler Johnson [ tyler.johnson ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Tyler Johnson
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated: