Closed (fixed)
Project:
Drupal.org site moderators
Component:
Site organization
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
25 Aug 2009 at 11:48 UTC
Updated:
3 Jan 2014 at 00:29 UTC
Jump to comment: Most recent
Comments
Comment #1
killes@www.drop.org commentedWe want to keep a "paper trail" for all block actions does this module provide this? (I guess it doesn't)
We should have a way to auto--create a webmaster issue for each blocked used.
Otherwise there isn't much gain in simplicity in this.
Comment #2
damien tournoud commentedfasttoggle does have a nice hook we can use to implement that.
Comment #3
killes@www.drop.org commentedCool! Then we need to find a volunteer. :)
Comment #4
avpadernoWould the hook be implemented in Drupal.org customizations?
Comment #5
killes@www.drop.org commentedIt would, yes.
Comment #6
avpadernoIf the code to add is simply the implementation of a hook that makes a call to
watchdog(), I can try to make a patch for drupalorg.module.Comment #7
gerhard killesreiter commentedNo, I don't think a watchdog entry is sufficient.
It should redirect the user to node/add/project-issue and set the appropriate parameters for such an issue.
Comment #8
damien tournoud commented@killes: I would go further: it should create an issue unconditionally. We can comment on it to add more info after it has been created.
Comment #9
avpadernoI still don't understand why a user administrator would be allowed to block a user without an issue report being automatically created, and a site maintainer could not block a user without the automatically created issue.
The issue report can also be automatically created, but then needs to be edited, and the user would need to then click on the submission button. If a user cannot be trusted about blocking a user, then he should not be trusted about deleting posts.
Correct me if I am wrong, but the way the feature is proposed to be implemented would just have the effect to have site maintainers asking to be granted the role of user administrators, which is what happens now without Fasttoggle. The only site maintainers who would not ask such role are who block users very rarely.
Comment #10
dave reidI can work on the action when a user is blocked to file a webmasters issue. I actually have most of it implemented in custom code.
Comment #11
damien tournoud commented@Dave: that would be awesome, thanks :)
Comment #12
damien tournoud commented@kiamlaluno: the idea is to reduce the number of person that needs the "User administrator" role, which comes with the very heavy "administer users" permission.
I don't see who have the need to block an user without creating an issue.
Comment #13
avpadernoI understand the reason of deploying fasttoggle.module on Drupal.org, and I understand that is preferable to use that module, rather than giving a role; what I was wondering is:
I personally create a report when I block a user, and a spam report was not already created by somebody else; I don't see why site maintainers should be forced to create such reports.
Comment #14
silverwing commentedinvestigating
Comment #15
kingandyFWIW: I've found FastToggle on the user profile page to be problematic in the past, as it only takes a single click for an administrator to block themselves.
For the most part I've experienced this on sites where the user is brought to their own profile page on login, so it's less likely to be a problem on d.o, but even so it's something that I'd be very wary of unless FastToggle has advanced to the point where it's not possible to block oneself unintentionally...
Comment #16
gerhard killesreiter commentedAfter re-reading this: I think the paper-trail idea still has merit but is not related to fasttoggle. We don#t have a paper trail for simply user blocks atm.
I guess this is good to go if somebody does the legwork.
Comment #17
gregglesOnce deployed I propose we configure the module to:
* Allow user administrators to block/unblock users
* Allow node admins to publish/unpublish nodes on full node view and on teasers
I just updated our vendor branch and merged the code to d.o. It just needs deployment to d.o.
Comment #18
drummDeployed.
Comment #19
drumm(untag)