Last modified: 2014-09-23 23:32:27 UTC

Wikimedia Bugzilla is closed!

Wikimedia has migrated from Bugzilla to Phabricator. Bug reports should be created and updated in Wikimedia Phabricator instead. Please create an account in Phabricator and add your Bugzilla email address to it.
Wikimedia Bugzilla is read-only. If you try to edit or create any bug report in Bugzilla you will be shown an intentional error message.
In order to access the Phabricator task corresponding to a Bugzilla report, just remove "static-" from its URL.
You could still run searches in Bugzilla or access your list of votes but bug reports will obviously not be up-to-date in Bugzilla.
Bug 11400 - Special pages should follow a single standard in showing redlinks or hiding links
Special pages should follow a single standard in showing redlinks or hiding l...
Status: NEW
Product: MediaWiki
Classification: Unclassified
Special pages (Other open bugs)
All All
: Low minor (vote)
: ---
Assigned To: Nobody - You can work on this!
: 12679 (view as bug list)
Depends on:
  Show dependency treegraph
Reported: 2007-09-19 17:44 UTC by Huji
Modified: 2014-09-23 23:32 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---

Fixing patch (980 bytes, patch)
2007-09-23 19:05 UTC, Huji

Description Huji 2007-09-19 17:44:21 UTC
Many of the Special pages which list pages on the Wiki (for example, Special:Shortpages) have a useful feature: When a page listed on them is deleted, it is shown as a striked redlink. This even works when the contents of the Special page are cached, as is the case with Wikimedia wikis.

However, Special:Uncategorizedpages and other special pages of this family (Uncategorized... family) don't have this feature implemented in them. I think it is a good idea to add the feature to them as well, if possible.
Comment 1 Huji 2007-09-23 19:05:26 UTC
Created attachment 4144 [details]
Fixing patch

Special:Uncategorizedpages derives from a different class than those other Special pages which show deleted pages as striken text. The patch fixes the bug I mentined here.
Comment 2 Aryeh Gregor (not reading bugmail, please e-mail directly) 2007-11-09 21:15:28 UTC
Note that the striking-out generally indicates that the page is no longer subject to inclusion.  In this case, pages should be struck out if they have categories now.  Deleted pages should by that logic always be struck out, but I'm not sure it makes sense to strike out *only* deleted pages.  They should be red links, if applicable, but I would say that for consistency, striking-out shouldn't be used for them unless it's used for *all* pages that are no longer eligible for inclusion in the list.  If you see what I mean.

Also, the existence checks should be done using a LinkBatch here.  It's not necessarily critical, I know we have other places that just call Title::exists repeatedly, but really you shouldn't do that.  It results in lots of unnecessary small queries instead of one slightly larger one.  Other than that, I'd have to look closer to properly review the patch, which I don't have time for now.
Comment 3 Huji 2007-11-09 21:23:06 UTC
Actually, when I created this patch, I simply copied (and modified) lines of code from the other Special pages which had the strike out feature with them; I mean, if a change towards LinkBatch is needed, it should happen to those Special pages too (if not already done).
Comment 4 Huji 2008-01-19 07:58:00 UTC
*** Bug 12679 has been marked as a duplicate of this bug. ***
Comment 5 Huji 2008-01-19 08:00:08 UTC
With bug 12679 being marked as duplicate, I've updated the title of this bug too. Here is what I think: All special pages which show a list of pages should show the deleted pages as redlinks. Pages shouldn't be hidden from the list if they don't exist anymore. Any objections?
Comment 6 Sumana Harihareswara 2012-01-10 22:30:24 UTC
Huji, thank you for the patch.  I'm sorry, but in the time since you provided your patch, MediaWiki's codebase has changed enough that your patch no longer applies cleanly to trunk.  Therefore I'm marking it obsolete and removing the "need-review" and "patch" keywords.  If the issue is still one you're interested in fixing, please join us in IRC and talk with other developers before renewing and possibly revising your approach.  Thanks for your interest in improving MediaWiki!
Comment 7 Sumana Harihareswara 2012-01-10 22:30:50 UTC
Comment on attachment 4144 [details]
Fixing patch

This patch no longer applies to trunk per Rusty Burchfield's automated testing .

Note You need to log in before you can comment on or make changes to this bug.