attached patch updates the module to 5.0. couple of things:

  • i moved the settings page to admin/user/roleassign, and lower-cased the title--this seemed to fit better with the new administration layout
  • i added a .info and .install file, which i'll attach in a minute here. the .install file isn't really necessary, but i threw it in with a hook_uninstall which would remove the roleassign variable from the variables table if the module is uninstalled.
  • the link to the help page from the settings page wasn't working properly for sites that aren't at the web root. proper usage of l() seems to have remedied that issue. you may want to backport that fix.
  • patch also removes unnecesary whitespace
CommentFileSizeAuthor
#2 roleassign.install139 byteshunmonk
#1 roleassign.info104 byteshunmonk
roleassign_0.patch5.25 KBhunmonk

Comments

hunmonk’s picture

hunmonk commented
StatusFileSize
new roleassign.info104 bytes

.info file...

hunmonk’s picture

hunmonk commented
StatusFileSize
new roleassign.install139 bytes

.install file...

hunmonk’s picture

hunmonk commented

btw, the patch has been tested on current HEAD and the module appears to be functioning perfectly.

TBarregren’s picture

TBarregren commented
Title: update to 5.0 » Update to 5.0
Status: Needs review » Fixed

Thanks hunmonk! The patch was great.

I have commited the patched file to HEAD. I have also updated the documentation and add some other minor improvements. So checkout the latest version from the repository.

ac’s picture

ac
Primary language English
Location Perth
commented

Is this going to get into core? It should be in there IMO. Is it just a matter of getting enough support or is it too late to squeeze it into 5.0?

TBarregren’s picture

TBarregren commented

I see the point to incorporate this kind of functionality with the User module. I will raise the question on the developer mailing list to see if there is support for the idea.

hunmonk’s picture

hunmonk commented

this will not go into 5.0--we are in an API freeze. better lobby for the next version--i'd like to see it in there... :)

TBarregren’s picture

TBarregren commented

You are of course right about the code freeze.

But I have thrown out a feeler on the developer mailing list to see if there is support for incorporating RoleAssign functionality in a future version of Drupal. The responses can be found here.

Anonymous’s picture

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