The Advanced User support described in the docs works fine when Ban and Unpublish is paired with advuser-6.x-2.x, but does not work with advuser-6.x-3.x.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | advuser.hook_user_operations.941066.7.patch | 1018 bytes | salvis |
The Advanced User support described in the docs works fine when Ban and Unpublish is paired with advuser-6.x-2.x, but does not work with advuser-6.x-3.x.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | advuser.hook_user_operations.941066.7.patch | 1018 bytes | salvis |
Comments
Comment #1
yelvington commentedI'll take a look.
If advuser 3.x isn't supporting bulk operations, I'd regard that as a showstopper regression in advuser. We use it on all our sites, and require bulk operations.
It's possible to replicate the functionality of advuser using Views Bulk Operations.
Comment #2
yelvington commentedComment #3
yelvington commentedForwarding this to the advuser project, noted as an undesirable feature regression.
It appears that advuser 3.x implements its own actions and ignores operations defined by http://api.drupal.org/api/function/hook_user_operations.
Comment #4
salvisI agree that using hook_user_operations() is a must-have feature!
Comment #5
Anonymous (not verified) commentedComment #6
Anonymous (not verified) commentedPushing to tasks to 7.x.
Comment #7
salvisThis patch is all that's needed.
Comment #8
salvisActually, there's one more thing: D7 core has a Cancel operation, but no Delete operation.
So, depending on your strategy, you either
unset($operations['cancel']), or you implement it instead of your Delete.Comment #9
salvisComment #10
Anonymous (not verified) commentedI need to review this more.
Comment #11
salvisAt the rate that advuser is progressing, it makes no sense to define hook_advuser_operations(). No third party will bother.
You can keep hook_advuser_operations() for your own use, but you must add support for core's hook_user_operations(). Can we agree on that?
Keeping both 'Cancel' and 'Delete' is really no big deal, certainly less so than locking out the modules that want to support the standard hook_user_operations(). So, please commit this.
Comment #12
Anonymous (not verified) commentedSalvis, I can agree to it if you take over lead on the project. I've no time for it this year.
Comment #13
salvisNo, sorry, there's too much functionality in advuser that I'll never use, and I'm already overcommitted.
Besides, I'm already maintaining Mail Editor — I'd go nuts maintaining this duplicate functionality...
Comment #14
natemow commentedhook_advuser_operations remains for now, but @salvis patch from https://drupal.org/node/941066#comment-6802656 now pushed to 7.x-3.x. Also removed duplicate block/unblock and add/remove role ops from Advuser, since core User mod can now just handle those directly.
Comment #15
salvisGreat, thank you!