Terminated ajax requests throw error even if terminated because of a form submit

Open on Drupal.org →

Created on 7 July 2017, over 7 years ago

Updated 2 November 2023, over 1 year ago

Problem/Motivation

There was a similar thread for D7 - #2808789: Fix "An AJAX HTTP request terminated abnormally" alert after user has navigated away from the page → .

The problem we are experiencing is mostly in JS functional tests having the autosave_form module enabled where ajax request are triggered at a specific interval but the form is submitted before an ajax request has finished, in which case the ajax request is terminated and drupal is throwing a JS error, at which point phantomjs will interrupt the test and exit with an error.

Proposed resolution

Do not throw the error if the response has the status code 0 and the unbeforeunload event has been triggered.

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report

Status

Needs work

Version

11.0 🔥

Component

Last updated about 20 hours ago

Maintained by
🇺🇸United States @effulgentsia
🇺🇸United States @tim.plunkett

Created by

🇩🇪Germany hchonov 🇪🇺🇩🇪🇧🇬

Live updates comments and jobs are added and updated live.

JavaScript

Contributed project blocker
It denotes an issue that prevents porting of a contributed project to the stable version of Drupal due to missing APIs, regressions, and so on.

Needs tests
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Needs subsystem maintainer review
It is used to alert the maintainer(s) of a particular core subsystem that an issue significantly impacts their subsystem, and their signoff is needed (see the governance policy draft for more information). Also, if you use this tag, make sure the issue component is set to the correct subsystem. If an issue significantly impacts more than one subsystem, use needs framework manager review instead.

Usability
Makes Drupal easier to use. Preferred over UX, D7UX, etc.

Needs steps to reproduce

Incomplete comments

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Comment about 2 years ago →
needs-review-queue-bot
The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
Comment over 1 year ago →
🇷🇴Romania radubutco
Adapted #16 for 11.x, also applies on 10.1.x.
Status changed to Needs review over 1 year ago11:17am 1 November 2023
Comment over 1 year ago →
🇷🇴Romania radubutco
Comment over 1 year ago →
🇺🇸United States tim.plunkett Philadelphia
Status changed to Needs work over 1 year ago3:16pm 2 November 2023
Comment over 1 year ago →
🇺🇸United States smustgrave
Moving to NW for the steps to reproduce.

Once those are there will need a failing test case to show the issue.

Thanks!
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MariaDB 10.3.22
last update about 1 year ago
30,486 pass

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024