Tapestry 5
  1. Tapestry 5
  2. TAP5-1192

IE 8 throws an exception while purging JavaScript event handlers from elements as they are destroyed

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 5.2.0
    • Fix Version/s: 5.2.1
    • Component/s: tapestry-core
    • Labels:
      None

      Description

      The fix for TAP5-1147 causes an exception when it finds an element with a function for an attribute value under IE 8. In the purge function in tapestry.js starting at line 640, the count of attributes is kept in a variable, then the attributes are processed starting at the beginning. When an attribute is found where the value is a function, the attribute is removed and the count of attributes changes. When the loop gets to the end, an exception is thrown because attrs[i] is undefined.

        Activity

        Hide
        Hudson added a comment -

        Integrated in tapestry-5.2-freestyle #184 (See https://hudson.apache.org/hudson/job/tapestry-5.2-freestyle/184/)
        TAP5-1192: IE 8 throws an exception while purging JavaScript event handlers from elements as they are destroyed

        Show
        Hudson added a comment - Integrated in tapestry-5.2-freestyle #184 (See https://hudson.apache.org/hudson/job/tapestry-5.2-freestyle/184/ ) TAP5-1192 : IE 8 throws an exception while purging JavaScript event handlers from elements as they are destroyed
        Hide
        Howard M. Lewis Ship added a comment -

        Iterating backwards through the names should resolve the problem.

        Show
        Howard M. Lewis Ship added a comment - Iterating backwards through the names should resolve the problem.
        Hide
        Brian Heston added a comment -

        This is a nice quick one to solve and would really help a lot so I don't have to figure out a hack until it's resolved correctly. Please take a look.

        Show
        Brian Heston added a comment - This is a nice quick one to solve and would really help a lot so I don't have to figure out a hack until it's resolved correctly. Please take a look.

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            Brian Heston
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development