ICEfaces
  1. ICEfaces
  2. ICE-7543

JavaScript error in IE7 for commandLinks within ui:repeat

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.RC1
    • Fix Version/s: 3.0.RC2, 3.0
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      ICEfaces
    • Assignee Priority:
      P1

      Description

      When a commandLink within a ui:repeat is invoked on IE7 a JavaScript error occurs.

        Activity

        Ted Goddard created issue -
        Ted Goddard made changes -
        Field Original Value New Value
        Attachment ice7543.zip [ 13845 ]
        Hide
        Ted Goddard added a comment -

        Test case is on page.jsf.

        Show
        Ted Goddard added a comment - Test case is on page.jsf.
        Ken Fyten made changes -
        Salesforce Case []
        Assignee Priority P1
        Assignee Ted Goddard [ ted.goddard ]
        Ken Fyten made changes -
        Link This issue blocks ICE-7539 [ ICE-7539 ]
        Ken Fyten made changes -
        Link This issue blocks ICE-7534 [ ICE-7534 ]
        Ken Fyten made changes -
        Link This issue blocks ICE-7532 [ ICE-7532 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #26761 Tue Dec 06 13:49:32 MST 2011 ted.goddard move ul inside form (ICE-7543)
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/webapp/showcase.xhtml
        Hide
        Ted Goddard added a comment -

        The underlying problem was client-side DOM corruption due to a <ul> being used outside a <form>. This caused

        mojarra.dpf = function dpf(f) {
        var adp = f.adp;
        if (adp !== null) {
        for (var i = 0; i < adp.length; i++)

        { f.removeChild(adp[i]); }

        }
        };

        to fail because the hidden field added to the form did not have the form as a parent (this would seem to be impossible, but the behavior of IE7 appears to be completely undefined in this case).

        Show
        Ted Goddard added a comment - The underlying problem was client-side DOM corruption due to a <ul> being used outside a <form>. This caused mojarra.dpf = function dpf(f) { var adp = f.adp; if (adp !== null) { for (var i = 0; i < adp.length; i++) { f.removeChild(adp[i]); } } }; to fail because the hidden field added to the form did not have the form as a parent (this would seem to be impossible, but the behavior of IE7 appears to be completely undefined in this case).
        Hide
        Ted Goddard added a comment -

        showcase has been modified.

        Show
        Ted Goddard added a comment - showcase has been modified.
        Ted Goddard made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 3.0.RC2 [ 10313 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Ted Goddard
            Reporter:
            Ted Goddard
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: