When Batch ID doesn't exist, Drupal should emit a 404

Created on 3 May 2013, about 11 years ago

Updated 7 February 2023, over 1 year ago

Currently if you try to go to /batch?id=123 you get redirected to the homepage with a message stating "No active batch".

Since the batch doesn't exist Drupal should just emit a 404 instead.
Display an error message to the user.

Error message displayed, "No batch with ID 20 exists."

Before

There is a redirect to the home page:

After

There is a 404 response instead of a redirect, and the error message is updated.

None

🐛 Bug report

Status

Needs work

Version

10.1 ✨

Component

Batch →

Last updated 7 days ago

Created by

🇺🇸United States timaholt

Live updates comments and jobs are added and updated live.

Documentation
Primarily changes documentation, not code. For Drupal core issues, select the Documentation component instead of using this tag. In general, component selection is preferred over tag selection.

Incomplete comments

Comments & Activities

Not all content is available!

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

Status changed to Needs review over 1 year ago10:00am 16 January 2023
Status changed to RTBC over 1 year ago3:01pm 20 January 2023
Status changed to Needs work over 1 year ago7:01am 3 February 2023
Comment over 1 year ago →
🇳🇿New Zealand quietone New Zealand
I agree with #193 that this needs a code review.
Comment over 1 year ago →
🇮🇳India Akhil Yadav
added patch against #196 in 10.1 version
Comment over 1 year ago →
🇧🇪Belgium BramDriesen Belgium 🇧🇪
Hiding patch #197 as it's incomplete and omitting important changes.

Adding to the list [# https://www.drupal.org/project/site_moderators/issues/3339883] →
Comment over 1 year ago →
🇳🇿New Zealand quietone New Zealand
@Akhil Yadav, It is expected that contributors read the issue before making comments etc. The MR is against 10.1.x. I am removing credit.

Production build 0.69.0 2024