Last modified: 2014-11-18 18:07:28 UTC
Sometimes, we block a user, he tries to edit a page and his IP address is
automatically blocked, then we unblock him and forget to unblock his IP address
- or we waste our time by searching the automatically-blocked addresses to
I suggest to automatically unblock the automatically-blocked IP address, when
unblocking a user.
Created attachment 1503 [details]
patch for unblocking autoblocked IPs when unblocking user
I guess this patch can help. Though it uses some hacks.
I think it would be more valuable having mediazilla showing at the unblocking a
This block produced the following automatically blocks:
where unblock is a link to automatically unblock that user/ip. Maybe with an
auto reason "Unblocking autoblock done due to <ip originally blocked>"
Created attachment 5036 [details]
Here's my proposed fix. It changes the meaning of ipb_user - now it is set to id of user who caused the autoblock instead of zero. I made removing autoblocks an option in unblock dialog, though in practice it should be used in 99% of cases so probably it won't hurt if that option is eliminated completely and autoblocks are always removed.
I need some advice regarding API format. I currently made it in form &removeautoblocks=1, and assume that autoblocks should be removed if that option is omitted, but this probably contradicts with the current API practice where it usually looks simply like &removeautoblocks. But I wanted this option to be active by default.
Maybe, it'll get noticed on Bug Monday?
(In reply to comment #3)
> Created an attachment (id=5036) [details]
> I need some advice regarding API format. I currently made it in form
> &removeautoblocks=1, and assume that autoblocks should be removed if that
> option is omitted, but this probably contradicts with the current API practice
> where it usually looks simply like &removeautoblocks. But I wanted this option
> to be active by default.
Why can't you have it enabled by default for api, and use &noremautoblocks to *not* remove them?
Massive time saver here. patch is a bit rotten, but can be saved..
Comment on attachment 5036 [details]
Marking this patch as obsolete. Unfortunately (Or fortunately, depending on your viewpoint =) ) the Block class has been refactored and no longer allows arbitrary setting of properties.
I propose to reuse the ipb_auto field and change it to an unsigned int: when it's 0, this block is manual; when it's not 0, it's an auto block created from a manual block whose number is ipb_auto.
Possible implementation submitted for review (Gerrit change #3841)
- New column instead of overloading ipb_auto for saner migration (per Tim)
- No option to not remove the autoblocks at the moment; honestly, I can't think of a scenario where this would make sense.
Merged. Should be released with 1.20.
CREDITS / RELEASE-NOTES update made with https://gerrit.wikimedia.org/r/3841
*** Bug 31893 has been marked as a duplicate of this bug. ***