ICEfaces
  1. ICEfaces
  2. ICE-8038

JavaScript error thrown when calling iceSubmitPartial on load of panelPopup in Chome 18

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-1.8.2.GA_P04
    • Fix Version/s: EE-1.8.2.GA_P05
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      Chome 18
    • Assignee Priority:
      P1

      Description

      A JavaScript error is thrown when calling the iceSubmitPartial() JaavScript API on load of a panelPopup. The error is thrown only in Chrom 18, all other browsers and older Chome versions do not show the error. The error prevents a valueChangeListener from being called.

        Activity

        Arran Mccullough created issue -
        Arran Mccullough made changes -
        Field Original Value New Value
        Attachment IceSubmitPartialError.jpg [ 14343 ]
        Arran Mccullough made changes -
        Salesforce Case [5007000000LI6A6]
        Arran Mccullough made changes -
        Attachment Case11198Example.zip [ 14344 ]
        Attachment Case11198Example.war [ 14345 ]
        Ken Fyten made changes -
        Fix Version/s EE-1.8.2.GA_P05 [ 10331 ]
        Assignee Priority P2
        Assignee Mircea Toma [ mircea.toma ]
        Ken Fyten made changes -
        Assignee Priority P2 P1
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #29817 Wed Jul 04 18:02:47 MDT 2012 mircea.toma ICE-8038 Test if the event to be decorated is truly a DOM event.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/bridge/lib/event.js
        Hide
        Mircea Toma added a comment - - edited

        The issue is caused by the global window.event variable that has its value set to a XMLHttpRequestProgressEvent during the second submit. When this non-DOM event is wrapped the code looking up the node's name invoked later on, fails.

        The applied fix tests if the event to be decorated is truly a DOM event, if not, a generic unknown event decorator is used.

        Show
        Mircea Toma added a comment - - edited The issue is caused by the global window.event variable that has its value set to a XMLHttpRequestProgressEvent during the second submit. When this non-DOM event is wrapped the code looking up the node's name invoked later on, fails. The applied fix tests if the event to be decorated is truly a DOM event, if not, a generic unknown event decorator is used.
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #31715 Wed Oct 24 08:33:51 MDT 2012 mircea.toma ICE-8038 Test for the existence of target element before trying to verify if the event is DOM related.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/bridge/lib/event.js
        Hide
        Mircea Toma added a comment - - edited

        The fix tests for the existence of target element (in Element.adaptToElement()) before trying to verify if the event is DOM related.

        Show
        Mircea Toma added a comment - - edited The fix tests for the existence of target element (in Element.adaptToElement()) before trying to verify if the event is DOM related.
        Hide
        Migration added a comment -

        The solution may solved the problem with Chrome, but caused a problem with rowSelector event on Firefox (tested on ver.15 and 16): Nothing happens, when clicking a table row, no request is generated..

        Show
        Migration added a comment - The solution may solved the problem with Chrome, but caused a problem with rowSelector event on Firefox (tested on ver.15 and 16): Nothing happens, when clicking a table row, no request is generated..
        Migration made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Migration made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Arran Mccullough
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: