Tapestry 5
  1. Tapestry 5
  2. TAP5-1533

File download breaks zone managers due to unload event

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major 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
        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 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
        Hide
        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
        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
        Josh Canfield added a comment -

        This is related to TAP5-1115

        Show
        Josh Canfield added a comment - This is related to TAP5-1115

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development