Tapestry
  1. Tapestry
  2. TAPESTRY-2424

Validation occasionally triggers Stack Overflow Javascript Error in IE

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 5.0.12
    • Fix Version/s: 5.0.12
    • Component/s: JavaScript
    • Labels:
      None
    • Environment:
      IE Web Browser (6 or 7)

      Description

      In IE 6 or 7, form validation can trigger a Stack Overflow error in the browser.

      This issues appears to be caused by the isDeepVisible method call. Refactoring this code so that isDeepVisible is a function which takes an element as an argument, as opposed to adding the method to the Element object, should resolve the problem.

      IE has known compatibility issues with recursive calls to methods added to an object.

      The problem is most prevelent when validating components which are nested in several divs.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        2d 5h 15m 1 Howard M. Lewis Ship 22/May/08 00:27
        In Progress In Progress Closed Closed
        1h 38m 1 Howard M. Lewis Ship 22/May/08 02:06
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12568731 ] jira [ 12591755 ]
        Mark Thomas made changes -
        Workflow jira [ 12431373 ] Default workflow, editable Closed status [ 12568731 ]
        Howard M. Lewis Ship made changes -
        Status In Progress [ 3 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        Fix Version/s 5.0.12 [ 12313048 ]
        Hide
        Howard M. Lewis Ship added a comment -

        Hard to tell if this is really fixed, but the code was changed as advised, and the example application no longer seems to manifest the problem.

        Show
        Howard M. Lewis Ship added a comment - Hard to tell if this is really fixed, but the code was changed as advised, and the example application no longer seems to manifest the problem.
        Howard M. Lewis Ship made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Howard M. Lewis Ship made changes -
        Field Original Value New Value
        Assignee Howard M. Lewis Ship [ hlship ]
        Andrew Fortier created issue -

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            Andrew Fortier
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 2h
              2h
              Remaining:
              Remaining Estimate - 2h
              2h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development