ICEfaces
  1. ICEfaces
  2. ICE-1496

Updates for multiple views on a page are not returned in proper XML format

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6DR#4
    • Fix Version/s: 1.6DR#4, 1.6
    • Component/s: None
    • Labels:
      None
    • Environment:
      Any

      Description

      When working on getting multiple ICEfaces portlets running on a page (although this issue could apply to multiple "includes" as well), there was a problem with AJAX push updates. The format of the updates looks like this:

      <updates><update address="_id19:_id20-2-0"><![CDATA[<td id="_id19:_id20-2-0">Fri, 15:21:20</td>]]></update><update address="_id19:_id20-2-1"><![CDATA[<td id="_id19:_id20-2-1">Fri, 15:21:20</td>]]></update></updates><updates><update address="_id19:_id20-2-0"><![CDATA[<td id="_id19:_id20-2-0">Fri, 15:21:20</td>]]></update><update address="_id19:_id20-2-1"><![CDATA[<td id="_id19:_id20-2-1">Fri, 15:21:20</td>]]></update></updates>

      It's hard to spot on first glance but this is invalid XML as it has two root elements ( <update></update> ). This causes the browsers parser to be unhappy and the updates cannot be applied to the views. Each browser handles invalid XML in different ways so the behaviour is dependent on the browser's implementation. Safari seems to accept the first updates document and throw away the second. Firefox balks completely. I did not try IE.

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #13660 Fri Apr 27 11:16:25 MDT 2007 deryk.sinotte ICE-1496 Added a command/handler to recognize and log "parsererror" responses in Firefox.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/command.js
        Deryk Sinotte created issue -
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #13662 Fri Apr 27 14:31:11 MDT 2007 mircea.toma Create a macro command if multiple commands are to be sent -- ICE-1496.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/http/core/SendUpdates.java
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/command/Macro.java
        Hide
        Deryk Sinotte added a comment -

        Assigning to Mircea

        Show
        Deryk Sinotte added a comment - Assigning to Mircea
        Deryk Sinotte made changes -
        Field Original Value New Value
        Assignee Mircea Toma [ mircea.toma ]
        Hide
        Mircea Toma added a comment -

        Send a macro command when multiple commands need to be sent to the browser.

        Show
        Mircea Toma added a comment - Send a macro command when multiple commands need to be sent to the browser.
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #13732 Wed May 02 17:58:08 MDT 2007 mircea.toma Cover also the case when no view identifiers are received -- ICE-1496.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/http/core/SendUpdates.java
        Ken Fyten made changes -
        Fix Version/s 1.6DR#4 [ 10060 ]
        Ken Fyten made changes -
        Fix Version/s 1.6 [ 10031 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Mircea Toma [ mircea.toma ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Deryk Sinotte
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: