Page MenuHomePhabricator
Create Task
Maniphest T22524

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

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

Related Objects
Search...

Event Timeline

• bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:54 PM
• bzimport added a project: MediaWiki-User-management.
• bzimport set Reference to bz20524.
Tobias created this task.Sep 7 2009, 1:11 AM
• bzimport added a comment.Oct 1 2009, 7:41 PM

mac.med02 wrote:

attempted patch

attachment New Text Document.TXT ignored as obsolete

• bzimport added a comment.Oct 2 2009, 3:14 AM

mac.med02 wrote:

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

• bzimport added a comment.Oct 2 2009, 3:20 AM

mac.med02 wrote:

Message Refinement and Rename

attachment New Text Document.TXT ignored as obsolete

• bzimport added a comment.Oct 2 2009, 3:50 AM

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

• bzimport added a comment.Oct 4 2009, 1:50 AM

mac.med02 wrote:

Sorry again

hopefully this really is the last one

attachment patch.patch ignored as obsolete

Tobias added a comment.Oct 4 2009, 6:56 PM

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.

• bzimport added a comment.Oct 5 2009, 2:28 AM

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.

• bzimport added a comment.Oct 5 2009, 3:42 AM

mac.med02 wrote:

FIX

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

Attached:

patch.patch1 KBDownload

Tobias added a comment.Oct 5 2009, 11:34 AM

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

Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits