CheckUser blocker should respect block log entry flag natural order
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	MarcoAurelio
	Feb 6 2016, 7:36 PM

Description

In T41213: Block interface after "Get users" check should allow talk page/send email blocking we were granted the ability to specify aditional block log settings when using the CU mass blocking interface. This however does not respect the order of blocklog flags:

19:30, 6 February 2016 MarcoAurelio (talk | contribs | block) blocked SOCKPUPPETNAME (talk | contribs) with an expiry time of infinite (email disabled, account creation disabled) (vandalism/abusing multiple accounts) (unblock | change block)

Regular order is: account creation disabled, email disabled, cannot edit own talk page

Thanks.

Related Changes in Gerrit:

	Subject	Repo	Branch	Lines +/-
	Add block log flags in the same order as SpecialBlock::blockLogFlags	mediawiki/extensions/CheckUser	master	+4 -4

Mentioned Here: T41213: Block interface after "Get users" check should allow talk page/send email blocking

MarcoAurelio raised the priority of this task from to Needs Triage.

MarcoAurelio updated the task description. (Show Details)

MarcoAurelio set Security to None.

'natural order', heh

Change 268909 had a related patch set uploaded (by Alex Monk):
Add block log flags in the same order as SpecialBlock::blockLogFlags

Change 268909 merged by jenkins-bot:
Add block log flags in the same order as SpecialBlock::blockLogFlags

Krenair closed this task as Resolved.Feb 8 2016, 12:58 AM