Page MenuHomePhabricator

wpReasonList-other should not be removed after selecting a reason from the dropdown on Special:MultiLock
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

What happens?:
The free-text box is smoothly animated away.

What should have happened instead?:
The box should remain and allow text to be entered in it.

Software version (on Special:Version page; skip for WMF-hosted wikis like Wikipedia):

Other information (browser name/version, screenshots, etc.):
This appears to have been caused by rECAU4807c178683e: Convert SpecialMultiLock to HTMLForm (@matmarex).

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change #1169126 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):

[mediawiki/extensions/CentralAuth@master] SpecialMultiLock: Change "Reason" field from 'selectorother' to 'selectandother'

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

Test plan

Before patch applied

  1. Visited the Special:MultiLock
  2. Search for a username using the form (then click "View information")
  3. Get a response with information about the account in (2) above.
  4. The reason field has an input field below the select that disappears once selected.
Screenshot 2025-07-15 at 1.27.05 PM.png (1×3 px, 455 KB)
Screenshot 2025-07-15 at 1.26.55 PM.png (1×2 px, 436 KB)

After the patch is applied

  1. Visited the Special:MultiLock
  2. Search for a username using the form (then click "View information")
  3. Get a response with information about the account in (2) above.
  4. The reason field has an input field below the select that stays once selected, which I think is the expected behavior.
Screenshot 2025-07-15 at 1.27.47 PM.png (1×3 px, 475 KB)
Screenshot 2025-07-15 at 1.28.03 PM.png (1×3 px, 476 KB)
NOTE: The input field for searching user names mentions in the text that the username should be prefixed. I assume it's something like User:Foobar (with the User: prefix), but when I tried that, the username wasn't found. But without the prefix, the search works. Maybe the text should be adjusted?

Screenshot 2025-07-15 at 1.36.49 PM.png (1×2 px, 83 KB)

@matmarex, does the test plan above make sense? If so, I can +2 the patch. :)

Yep, that looks right, thanks.

Change #1169126 merged by jenkins-bot:

[mediawiki/extensions/CentralAuth@master] SpecialMultiLock: Change "Reason" field from 'selectorother' to 'selectandother'

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