attached patch converts role deletions to the deletion API. previously, role deletions happened without a confirmation screen, which i thought was kind of dangerous -- so they've been converted to use a confirm screen in this patch.
tested as working.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | dapi_roles_154266.patch | 3.3 KB | drewish |
| dapi_roles.patch | 3.22 KB | hunmonk |
Comments
Comment #1
drewish commentedi was getting an undefined index warning for the rid in user_admin_role_submit() when adding a new role. i reverted back to using $form_state['values']['name'] and $form_state['values']['rid']. otherwise it works great.
Comment #2
hunmonk commentedcode style is good. tested the patch, and it works perfectly.
note that you need to make sure that your menu cache has been cleared in order to test this patch, as it adds a new menu item...
Comment #3
jvandyk commentedConfirmed that the patch works as described.
Comment #4
hunmonk commentedComment #5
webchickDeletion API was rolled back.