Uploaded image for project: 'Tapestry 5'
  1. Tapestry 5
  2. TAP5-1533

File download breaks zone managers due to unload event

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 5.2.5
    • Fix Version/s: 5.3
    • Component/s: tapestry-core
    • Labels:
      None

      Description

      When a file is downloaded the unload event does some cleanup which prevents subsequent AJAX responses to fail. The request and response work fine, just the javascript handeling of the returned JSON fails. Once the page is refreshed all works fine again.

        Activity

        Hide
        joshcanfield Josh Canfield added a comment -

        This is related to TAP5-1115

        Show
        joshcanfield Josh Canfield added a comment - This is related to TAP5-1115
        Hide
        adamm Adam added a comment -

        Here's a patch. Although I haven't tried to generate the false success scenario, it works for ignoring download-based beforeunloads.

        Show
        adamm Adam added a comment - Here's a patch. Although I haven't tried to generate the false success scenario, it works for ignoring download-based beforeunloads.
        Hide
        hudson Hudson added a comment -

        Integrated in tapestry-trunk-freestyle #590 (See https://builds.apache.org/job/tapestry-trunk-freestyle/590/)
        TAP5-1533: File download breaks zone managers due to unload event

        hlship : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1186327
        Files :

        • /tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js
        Show
        hudson Hudson added a comment - Integrated in tapestry-trunk-freestyle #590 (See https://builds.apache.org/job/tapestry-trunk-freestyle/590/ ) TAP5-1533 : File download breaks zone managers due to unload event hlship : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1186327 Files : /tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js

          People

          • Assignee:
            hlship Howard M. Lewis Ship
            Reporter:
            joostschouten Joost Schouten
          • Votes:
            5 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development