Last modified: 2008-02-28 00:04:20 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T15112, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 13112 - alias for Special:RelatedChanges - patch included
alias for Special:RelatedChanges - patch included
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Special pages (Other open bugs)
1.12.x
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
: easy, patch, patch-need-review
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-02-22 18:14 UTC by Jack Schmidt
Modified: 2008-02-28 00:04 UTC (History)
1 user (show)

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


Attachments
Patch to implement (721 bytes, patch)
2008-02-22 18:14 UTC, Jack Schmidt
Details

Description Jack Schmidt 2008-02-22 18:14:25 UTC
Created attachment 4664 [details]
Patch to implement

Special:Recentchangeslinked makes awkward wikilinks and does not match the standard text in the toolbox, "Related Changes".  However, it is easy to add an alias for Special:RelatedChanges and this minor change is unlikely to break anything in the future.

A short discussion was (2008-02-19 to 2008-02-22) at:
http://en.wikipedia.org/wiki/Wikipedia:VP/T#Recent_changes_to_Main_Page_articles

I include a patch that does this.  It alters the english localization to add the alias.  According to the documentation, this should also provide the alias to most languages as well, using fallback.  I have tested the patch on svn-current r31195 (2008-02-22 11:20:15 -0500).
Comment 1 Charlotte Webb 2008-02-23 14:54:14 UTC
> -	'Recentchangeslinked'       => array( 'Recentchangeslinked' ),
> +	'Recentchangeslinked'       => array( 'Recentchangeslinked', 'RelatedChanges' ),

I'm pretty sure all "special:" pages are titled in lowercase except for the first letter. Of course, if the user requests [[Special:ReCeNtChaNGEsLInkEd]], the url will be coerced to "Recentchangeslinked", but I don't know whether that would work the other way in a case the internally defined name contains irregular capitalization. Even if it does work, probably better to keep them consistent.
Comment 2 Jack Schmidt 2008-02-23 18:32:35 UTC
(In reply to comment #1)
> > -	'Recentchangeslinked'       => array( 'Recentchangeslinked' ),
> > +	'Recentchangeslinked'       => array( 'Recentchangeslinked', 'RelatedChanges' ),
> 
> I'm pretty sure all "special:" pages are titled in lowercase except for the
> first letter. Of course, if the user requests [[Special:ReCeNtChaNGEsLInkEd]],
> the url will be coerced to "Recentchangeslinked", but I don't know whether that
> would work the other way in a case the internally defined name contains
> irregular capitalization. Even if it does work, probably better to keep them
> consistent.
> 

Hrm, the current file is a little inconsistent here.  Really most of the special pages names are lowercase as you say.  However, DoubleRedirects, BrokenRedirects, CreateAccount, MIMEsearch, and MergeHistory are mixed case.  Basically though it is only capitalization of the string on the left hand side of the => that matters, since even ReLAteDcHangEs will redirect to Recentchangeslinked (not RelatedChanges or anything).

I'm fine with either capitalization, and Relatedchanges seems consistent enough for me.
Comment 3 Brion Vibber 2008-02-28 00:04:20 UTC
Works for me. :)

Might consider switching the alias to the main name at some point, but no rush.

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


Navigation
Links