ICEfaces
  1. ICEfaces
  2. ICE-1987

OutputProgressRenderer race condition

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6, 1.7DR#2
    • Fix Version/s: 1.6.1
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      JBoss 4.0.4 GA, Firefox 2.0.0.6

      Description

      Based on a forum posting, I started auditing OutputProgress and OutputProgressRenderer, and found that the Renderer was temporarily storing the text and fill state in the Component, which could explain some of the glitches we've been observing, including this forum posting about the two getting out of sync.

        Issue Links

          Activity

          Hide
          Mark Collette added a comment -

          I removed all the the fields and accessors in OutputProgress for holding the DOM elements pertaining to the text and bar, and rearranged OutputProgressRenderer so it would pass those to itself on the stack.

          Subversion 14644
          icefaces\component\src\com\icesoft\faces\component\outputprogress\OutputProgress.java
          icefaces\component\src\com\icesoft\faces\component\outputprogress\OutputProgressRenderer.java

          Show
          Mark Collette added a comment - I removed all the the fields and accessors in OutputProgress for holding the DOM elements pertaining to the text and bar, and rearranged OutputProgressRenderer so it would pass those to itself on the stack. Subversion 14644 icefaces\component\src\com\icesoft\faces\component\outputprogress\OutputProgress.java icefaces\component\src\com\icesoft\faces\component\outputprogress\OutputProgressRenderer.java
          Hide
          Mark Collette added a comment -

          Apparently this is still a problem in ICEfaces 1.7 DR2.

          Show
          Mark Collette added a comment - Apparently this is still a problem in ICEfaces 1.7 DR2.

            People

            • Assignee:
              Unassigned
              Reporter:
              Mark Collette
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: