Page MenuHomePhabricator

Auto-select reason per wpReasonList parameter value on Special:Movepage
Closed, ResolvedPublicFeature

Description

Under the current functionalities of Special:Movepage, the wpReasonList parameter to index.php, if any, is reflected in the reason textbox (instead of in the reason dropdown), given the coding in SpecialMovePage.php#199 and below.

I would like to request that the value of wpReasonList be selected in the dropdown, just like on action=delete regulated by DeleteAction.php#457.

Event Timeline

Which problem is this trying to solve?

Which problem is this trying to solve?

  1. For interface consistency, and

if MediaWiki:Movepage-reason-dropdown has a reason like [[Some policy]] and a given template has a link with a href like /w/index.php?title=Special:Movepage/PAGE&wpReasonList=[[Some policy]] (putting aside URI encoding),

  1. One could double-select the same reason in the dropdown, resulting in a moving reason like [[Some policy]]: [[Some policy]]: Blah, and
  2. [[Some policy]] in the textbox can be modified, meaning that one can "destroy" the fixed reason by accident; e.g. [[Some policy] (one of the brackets removed).

If we had the auto-select functionality, this kind of problems wouldn't occur. This is my main motivation for this proposal.

Umherirrender subscribed.

action=delete is using fields from the HTMLForm class, while Special:MovePage builds the html form directly from OOUI widgets, that makes the technical different in the code. These widgets does not take url parameter in to account by default.

To show the reason from the url there was extra code. In the case of the "confirm and delete" dialog (when moving to existing title as sysop) the reaon is shown as one reason field and not with the dropdown and the additional reason.

Change 1008565 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/core@master] specials: Allow pre-select from reason list on Special:MovePage

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

Change 1008565 merged by jenkins-bot:

[mediawiki/core@master] specials: Allow pre-select from reason list on Special:MovePage

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