Page MenuHomePhabricator
Create Task
Maniphest T326057

No warning on creating broken redirect to red link
Open, MediumPublicBUG REPORT
Actions

Description

Steps to replicate the issue (include links if applicable):

  • Create a new redirect to non-existent page (red link).

What happens?:

MediaWiki allows you to make a broken redirect like this, to a red link, no worries.

What should have happened instead?:

An error message should appear, similar to MediaWiki:Selfredirect, but specifically for this problem (creating a broken redirect).

Software version (skip for WMF-hosted wikis like Wikipedia): 1.35

Other information (browser name/version, screenshots, etc.):

Details

SubjectRepoBranchLines +/-
Add a warning when trying to create a broken redirectmediawiki/coremaster+210 -3
Customize query in gerrit

Event Timeline

Bugreporter2 created this task.Jan 1 2023, 12:41 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJan 1 2023, 12:41 PM
Aklapper changed the task status from Open to Stalled.Jan 10 2023, 9:32 PM

I don't see anything "broken". What is technically "broken" and why?

Bugreporter2 added a comment.Jan 10 2023, 10:57 PM

They are called "broken redirects" in Special:BrokenRedirects -- it's probably not the best term but it's the one you officially use.

Aklapper changed the task status from Stalled to Open.Jan 10 2023, 11:04 PM

Ah! Thanks.

Bugreporter2 triaged this task as Medium priority.Nov 14 2023, 3:05 AM
Bugreporter2 added a project: patch-welcome.
Bugreporter2 added a project: MediaWiki-Page-editing.Nov 15 2023, 11:13 AM
SomeRandomDeveloper claimed this task.Sun, Nov 10, 7:41 PM
gerritbot added a comment.Sun, Nov 10, 8:11 PM

Change #1089205 had a related patch set uploaded (by SomeRandomDeveloper; author: SomeRandomDeveloper):

[mediawiki/core@master] Add a warning when trying to create a broken redirect

https://gerrit.wikimedia.org/r/1089205

gerritbot added a project: Patch-For-Review.Sun, Nov 10, 8:11 PM
Pppery added a project: User-notice.Tue, Nov 12, 1:24 AM
UOzurumba added subscribers: Pppery, UOzurumba.Thu, Nov 14, 5:18 AM

Hello @Pppery, is this related to the entry in this Tech News? If it is unrelated, when will this be ready for Tech News, and what wording would you suggest as the content?

Pppery added a comment.Thu, Nov 14, 6:02 AM

Unrelated to that entry (which I'm not convinced even belongs in Tech News at all).

Timing: as soon as the patch above is merged (you can generally assume that when a task has both Patch-For-Review and User-notice then that's the timing)

Wording draft: Users will now be warned when creating a redirect to a page that doesn't exist.

Bugreporter2 awarded a token.Thu, Nov 14, 8:36 AM
matej_suchanek awarded a token.Thu, Nov 14, 11:59 AM
UOzurumba added a comment.Fri, Nov 15, 5:17 AM
In T326057#10321214, @Pppery wrote:

Unrelated to that entry (which I'm not convinced even belongs in Tech News at all).

Timing: as soon as the patch above is merged (you can generally assume that when a task has both Patch-For-Review and User-notice then that's the timing)

Wording draft: Users will now be warned when creating a redirect to a page that doesn't exist.

Thank you!

UOzurumba moved this task from To Triage to In current Tech/News draft on the User-notice board.Fri, Nov 15, 5:21 AM
Pppery moved this task from In current Tech/News draft to Already announced/Archive on the User-notice board.Thu, Nov 21, 7:27 PM

This shouldn't have been sent out yet since the patch still hasn't been merged,.

Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits