Hi,

Patch attached has two small enchancements, feel free to apply if you like:
- remove anymous & authenticad rid (1 & 2) from the options on the settings page.
- hide role selection when you have administer users permissions: IMHO not needed when you create users as an admin

Haven't looked at the D6 version, I'll investigate that in a week or two when I have user_types (http://drupal.org/project/user_types) ready for D5 and integration with the autoassign module.

(note, it applies but with -1 offset, it originally also removed the drupal_set_message, but I noticed this allready has been fixed in cvs)

CommentFileSizeAuthor
role_selection.patch1.54 KBswentel

Comments

swentel’s picture

swentel commented
Title: Left some debug code + two other small fixes » Two small enchancements

(changing title as drupal_set_message doesn't apply anymore)

cyberswat’s picture

cyberswat commented
Assigned: Unassigned » cyberswat
Status: Needs review » Fixed

There's a bit of a different implementation already in place for the d6 version but this will work nicely for d5 ... thanks

http://drupal.org/cvs?commit=143030

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.