Page MenuHomePhabricator

Hideuser: Error when trying to block hidden user without hideuser right
Closed, ResolvedPublic

Description

Currently, it's a bit confusing for admins without hideuser right:

  1. Admin sees EvilUsername
  2. Oversight sees EvilUsername and is faster than the admin: He blocks EvilUsername with hideuser
  3. Admin goes to Special:Block/EvilUsername. There is a warning, that EvilUser is "Already blocked", but the block log is empty.
  4. Admin tries to block user and gets a strange error: "Error: could not submit form" and "<hookaborted>".

Expected behavior: Nice error message saying the admin is not allowed to change block settings as the user has been hidden.


Version: unspecified
Severity: normal

Details

Reference
bz20524

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:54 PM
bzimport set Reference to bz20524.

mac.med02 wrote:

attempted patch

attachment New Text Document.TXT ignored as obsolete

mac.med02 wrote:

Note: Tests on the above patch succeeded on my local install of MediaWiki.

mac.med02 wrote:

Message Refinement and Rename

attachment New Text Document.TXT ignored as obsolete

mac.med02 wrote:

Final Fix

Unrelated experimenal patch included in the last attachment. Sorry about all the emails.

attachment New Text Document.TXT ignored as obsolete

mac.med02 wrote:

Sorry again

hopefully this really is the last one

attachment patch.patch ignored as obsolete

Don't forget to add new messages to maintenance/language/messages.inc, please :)
Also, try to use tabulator instead of spaces in Special:Blockip.php. It doesn't really look nice atm.

mac.med02 wrote:

I've tried two different editors to try and fix that problem (Notepad2 and PSPad). What would you recommend to deal with the issue? (Ie that pressing tab just creates spaces.

mac.med02 wrote:

FIX

Hopefully this is the last one. Fixed messages.inc and changed spaces to tabs.

Attached:

Done in r57375. Minor tweaks (MessagesXX.php uses spaces, not tabulators :))