Page MenuHomePhabricator

Document Special:PermanentLink and Special:Diff in the interface
Closed, ResolvedPublic

Description

As noted at https://en.wikipedia.org/w/index.php?title=Wikipedia:Village_pump_%28technical%29&oldid=539378665#How_to_edit.E2.80.94or_request_an_edit.E2.80.94of_Special:SpecialPages, Special:PermanentLink is not documented at Special:SpecialPages.


Version: 1.21.x
Severity: enhancement

Details

Reference
bz45221

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 1:13 AM
bzimport set Reference to bz45221.
bzimport added a subscriber: Unknown Object (MLST).

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).

(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?

(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.

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

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

Just wondering why a patch submitted for a code review has been apparently just sitting there, not reviewed for over a year now. Can someone explain the status of this. Is it normal for patches to wait over a year to be reviewed?
https://gerrit.wikimedia.org/r/#/c/129095/

Just wondering why a patch submitted for a code review has been apparently just sitting there, not reviewed for over a year now. Can someone explain the status of this. Is it normal for patches to wait over a year to be reviewed?
https://gerrit.wikimedia.org/r/#/c/129095/

@Wbm1058: Thanks for the ping! The patch got reviewed in October 2015 (Siebrand gave a +1 in Gerrit) but it needs rebasing. Would you fancy doing this in order to help moving forward?

In general: According to our statistics the median time to review is 0.6 days. (I don't think we have "time to merge" statistics though?) Unfortunately there are always some patches that fall through the cracks, for numerous reasons. We are working on finding ways to minimize this problem (cf. T78768: Agree on and implement actions to prioritize code review of patches submitted by volunteers).

Hej what's the current status of this ticket, we from the Documentation Table @ Wikimania Hackathon Montreal looking for it, maybe someone can come an help?

I rebased the patch. I had to rewrite it a little bit, since this code apparently changed a lot in the meantime.

matmarex claimed this task.

https://gerrit.wikimedia.org/r/#/c/129095/ is merged! Better late than never.

(Whoops, we forgot to fix the Bugzilla bug reference in the commit message, so this wasn't automatically linked.)