Posted by Pasqualle on December 15, 2009 at 8:37pm
Jump to:
| Project: | Role Delegation |
| Version: | 7.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
improvements:
- removed code duplication
- added "Delegate roles" action
| Attachment | Size |
|---|---|
| D7_port_role_delegation.patch | 14.41 KB |
Comments
#1
[edit: download link removed]
#2
Excellent patch committed!
The new actions are a welcome addition.
#3
Automatically closed -- issue fixed for 2 weeks with no activity.
#4
Coming from http://www.bywombats.com/blog/01-23-2010/delegating-role-delegation#comm...
Sorry, but no :)
Looking at the current CVS HEAD... http://drupalcode.org/viewvc/drupal/contributions/modules/role_delegatio...
...the code is still using the workaround of a separate page.
In #118345: Revamp hook_user_form/_register/_validate/_submit/_insert/_update, I made sure that modules like Role Delegation can easily alter the user account form and apply the changes directly, instead of using a separate page.
#5
@sun: the user account form is altered and changes applied directly (in hook_user_presave), the separate page is used only when the user do not have the 'access user profiles' permission..
#6
The intended behavior works as described by Pasqualle in #5.
#7
Automatically closed -- issue fixed for 2 weeks with no activity.