Document Special:PermanentLink and Special:Diff in the interface
OpenPublic

bzimport added a subscriber: Unknown Object (MLST).
bzimport set Reference to bz45221.
Mattflaschen created this task.Via LegacyFeb 21 2013, 7:49 AM
Nemo_bis added a comment.Via ConduitMay 27 2013, 8:55 PM

HSJ pointed to https://gerrit.wikimedia.org/r/gitweb?p=mediawiki/core.git;a=blob;f=includes/SpecialPageFactory.php;h=4d635536a75f42074314abbb51a9b49634d255d8;hb=refs/heads/master#l155

https://en.wikipedia.org/wiki/Special:PermanentLink gives "No such target page". Standard way to "document" such redirects is to use them in the interface, but the sidebar link includes the page title too.

Hazard-SJ added a comment.Via ConduitMay 27 2013, 8:56 PM

(Hazard-SJ) :P

matmarex added a comment.Via ConduitApr 5 2014, 4:54 PM

I would say this affects Special:Diff too.

Fixing this would basically require doing two things for each of the affected special pages:

  • Adding public function isListed() { return true; } (see e.g. SpecialCreateAccount for an example).
  • Making them generate a form with brief instructions instead of an exception when no parameters are provided (see e.g. SpecialComparePages for an example, I think).
PiRSquared17 added a comment.Via ConduitApr 22 2014, 6:03 PM

(In reply to Bartosz Dziewoński from comment #3)

I would say this affects Special:Diff too.

Fixing this would basically require doing two things for each of the
affected special pages:

  • Adding public function isListed() { return true; } (see e.g. SpecialCreateAccount for an example).
  • Making them generate a form with brief instructions instead of an exception when no parameters are provided (see e.g. SpecialComparePages for an example, I think).

It would be easy to accomplish, but generating a form would mean they would no longer extend RedirectSpecialPage. It's definitely feasible (e.g., mock-up http://pirsquared-dev.wmflabs.org/wiki/Special:PermanentLink), but is there any problem with changing the inheritance?

matmarex added a comment.Via ConduitApr 22 2014, 6:31 PM

(In reply to PiRSquared17 from comment #4)

It would be easy to accomplish, but generating a form would mean they would
no longer extend RedirectSpecialPage. It's definitely feasible (e.g.,
mock-up http://pirsquared-dev.wmflabs.org/wiki/Special:PermanentLink), but
is there any problem with changing the inheritance?

There isn't, the SpecialPage subclasses exist only for convenience.
If you are concerned, then you could just override the execute() method.

gerritbot added a comment.Via ConduitApr 23 2014, 3:38 AM

Change 129095 had a related patch set uploaded by Gerrit Patch Uploader:
Document Special:Diff and Special:PermanentLink

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

Add Comment