ICEfaces
  1. ICEfaces
  2. ICE-2973

multiple outputConnectionStatus components causing js errors.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7RC1
    • Fix Version/s: None
    • Component/s: Bridge, ICE-Components
    • Labels:
      None
    • Environment:
      ICEfaces component showcase enhanced.

      Description

      This was found by a client as well as by a recent enhancement to the component showcase. When using Facelets (not sure about plain JSF ) we have two connectionStatus components being used on two separate templates but never rendered at the same time.

      For example, assuming component showcase 2 facelets-enh.

      facelets-enh/web/WEB-INF/tabbed_conainer.jspx
      - template for all component pages, has a outputConnectionStatus at the top of the page


      facelets-enh/web/WEB-INF/plain_container.jspx
      - template for the single page splash screen
      - when the following component is added the application needs a hard browser refresh to get the application working
      <ice:outputConnectionStatus
                                  styleClass="navConnectionStatus"
                                  id="tabbedOutput"
                                  showPopupOnDisconnect="true"/>

      The templates are never used at the same time but they seem to cause some confusion in the bridge.

      Here is the error.

      this.elementID.asElement() has no properties
      initialize()icefaces-d2d.js (line 30)
      initialize(Object)icefaces-d2d.js (line 30)
      create()icefaces-d2d.js (line 2)
      each(Object)icefaces-d2d.js (line 4)
      each(function(), undefined)icefaces-d2d.js (line 4)
      initialize()icefaces-d2d.js (line 30)
      initialize()icefaces-d2d.js (line 30)
      initialize()icefaces-d2d.js (line 35)
      eachWithGuard(function())icefaces-d2d.js (line 14)
      each(function())icefaces-d2d.js (line 4)
      each(function(), undefined)icefaces-d2d.js (line 4)
      eachWithGuard(Object identifier=3400056 request=XMLHttpRequest)icefaces-d2d.js (line 14)
      initialize(Object identifier=3400056 request=XMLHttpRequest)icefaces-d2d.js (line 34)
      create()icefaces-d2d.js (line 2)
      initialize("/component-showcase/block/send-receive-updates", "ice.submit.partial=false&ice.event.target=navigationMenuForm%3Aj_id23%3A0%3Aj_id26&ice.event.capture...", function())icefaces-d2d.js (line 19)
      initialize("/component-showcase/block/send-receive-updates", "ice.submit.partial=false&ice.event.target=navigationMenuForm%3Aj_id23%3A0%3Aj_id26&ice.event.capture...", function())icefaces-d2d.js (line 19)
      initialize(Object parameters=[22])icefaces-d2d.js (line 34)
      initialize(Object logger=Object sendChannel=Object)icefaces-d2d.js (line 20)
      iceSubmit(form#navigationMenuForm.iceFrm iceSubmit(&#39;navig..., div.icePnlClpsblColpsdHdr, click clientX=0, clientY=0)icefaces-d2d.js (line 25)
      onclick(click clientX=0, clientY=0)showcase.iface# (line 1)
      [Break on this error] [Ice.Status=new Object].as(function(A){A.NOOPIndicator={on:Function.NOOP,off:Fun...

        Issue Links

          Activity

          Hide
          Mircea Toma added a comment -

          Re-create status indicators everytime connection status component is updated.
          Refactor fixes for ICE-2621.

          Show
          Mircea Toma added a comment - Re-create status indicators everytime connection status component is updated. Refactor fixes for ICE-2621 .
          Hide
          Mircea Toma added a comment -

          Revert changes for now.

          Show
          Mircea Toma added a comment - Revert changes for now.
          Hide
          Mircea Toma added a comment -

          Instantiate bridge at the begining of the container element.
          Refactor how custom status indicators are wired into the bridge.

          Show
          Mircea Toma added a comment - Instantiate bridge at the begining of the container element. Refactor how custom status indicators are wired into the bridge.
          Hide
          Ken Fyten added a comment -

          The last commits need to be made to the branches/icefaces-1.7 also.

          Show
          Ken Fyten added a comment - The last commits need to be made to the branches/icefaces-1.7 also.
          Hide
          Mircea Toma added a comment -

          The commit for issue ICE-3218 fixed the JS error that Philip mentioned.

          Show
          Mircea Toma added a comment - The commit for issue ICE-3218 fixed the JS error that Philip mentioned.

            People

            • Assignee:
              Unassigned
              Reporter:
              Patrick Corless
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: