ICEfaces
  1. ICEfaces
  2. ICE-883

AuctionMonitor: IE 6: Old bidding value pops up in one window if working in same session

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5
    • Fix Version/s: 1.5.1
    • Component/s: Sample Apps
    • Labels:
      None
    • Environment:
      Operating System: Windows XP
      Platform: PC

      Description

      Open auction monitor in IE 6 and then open a child window in same session
      by "NEW Window"
      Choose same bidding for both windows (parent and child, say Ice Car)
      Jut put a value $10 in child window (don't press ok) and a value $20 in parent
      window.
      Press Ok in child window and subsiquently press ok in parent. User will see
      the child's window value in parent (in this case i.e. $10).

      This is not frequently re-produced and requires a bit effort. (call me to help
      you producing this)

        Activity

        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Ken Fyten made changes -
        Affects Version/s 1.5 [ 10027 ]
        Affects Version/s 1.1 Alpha [ 10026 ]
        Icefaces Administrator made changes -
        Field Original Value New Value
        issue.field.bugzillaimportkey 901 12152
        Hide
        Patrick Corless added a comment -

        This has been in the head for some time with no more reported problems. Closing

        Show
        Patrick Corless added a comment - This has been in the head for some time with no more reported problems. Closing
        Hide
        Ted Goddard added a comment -

        Checked in a change that I fixes auctionMonitor from my testing. Handing back to Saifullah to verify
        fixed in the HEAD build.

        svn commit src/com/icesoft/faces/webapp/xmlhttp/BlockingResponseState.java -m "synchronizing list of
        DOM updates to fix concurrency problems during concurrentDOMViews (ICE-883)"
        Sending src/com/icesoft/faces/webapp/xmlhttp/BlockingResponseState.java
        Committed revision 10742.

        Show
        Ted Goddard added a comment - Checked in a change that I fixes auctionMonitor from my testing. Handing back to Saifullah to verify fixed in the HEAD build. svn commit src/com/icesoft/faces/webapp/xmlhttp/BlockingResponseState.java -m "synchronizing list of DOM updates to fix concurrency problems during concurrentDOMViews ( ICE-883 )" Sending src/com/icesoft/faces/webapp/xmlhttp/BlockingResponseState.java Committed revision 10742.
        Hide
        Ted Goddard added a comment -

        I'll take an initial look into this.

        Show
        Ted Goddard added a comment - I'll take an initial look into this.
        Hide
        Ted Goddard added a comment -

        This may be the root cause:

        java.util.ConcurrentModificationException
        at java.util.AbstractList$Itr.checkForComodification(AbstractList.java:449)
        at java.util.AbstractList$Itr.next(AbstractList.java:420)
        at com.icesoft.faces.webapp.xmlhttp.BlockingResponseState.serialize(BlockingResponseState.java:
        174)

        We have removed synchronization that was protecting us from simultaneous events under
        concurrentDOMViews. It was believed fixed in the other case because of the transient nature.

        Show
        Ted Goddard added a comment - This may be the root cause: java.util.ConcurrentModificationException at java.util.AbstractList$Itr.checkForComodification(AbstractList.java:449) at java.util.AbstractList$Itr.next(AbstractList.java:420) at com.icesoft.faces.webapp.xmlhttp.BlockingResponseState.serialize(BlockingResponseState.java: 174) We have removed synchronization that was protecting us from simultaneous events under concurrentDOMViews. It was believed fixed in the other case because of the transient nature.
        Hide
        Ted Goddard added a comment -

        It seems much less robust than the version on demo.icesoft.com.

        I also see frequent "session expired" with the current build. This is easily reproducible by
        entering a bid and then pressing the green "check" rather than pressing enter.

        Show
        Ted Goddard added a comment - It seems much less robust than the version on demo.icesoft.com. I also see frequent "session expired" with the current build. This is easily reproducible by entering a bid and then pressing the green "check" rather than pressing enter.
        Hide
        Steve Maryka added a comment -

        Concurrent DOMs again??

        Show
        Steve Maryka added a comment - Concurrent DOMs again??
        Saifullah Qazi created issue -

          People

          • Assignee:
            Saifullah Qazi
            Reporter:
            Saifullah Qazi
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: