Hello,
this is a really interesting and useful module. Are there any plans to port it to D7?

Thank you in advance

CommentFileSizeAuthor
#2 d7role_vocabulary-1315210-2.patch10.1 KBbulldozer2003

Comments

joelstein’s picture

I don't have any immediate plans to upgrade it. But I'll review patches, and I'm available for sponsored upgrades.

bulldozer2003’s picture

Status: Active » Needs work
StatusFileSize
new10.1 KB

Here is a patch for the work I've done to update this module to Drupal 7.

The install functions are NOT working, you will need to manually create a taxonomy, then SET that as your sync taxonomy, then use the sync function. Not sure why, will keep troubleshooting.

Also, of the role alter functions, ONLY saving roles syncs that role with the taxonomy. I'd like everything to work, but getting the correct hook_form_alter options is the problem.

Please leave comments and what not, this is a pre-alpha off the skin of my teeth draft version.

bulldozer2003’s picture

Actually, there still may be some errors in that patch, but nothing major, will update as I continue work.

bulldozer2003’s picture

Status: Needs work » Needs review

I've completed re-working this into D7 compatibility. All functions seem to working as they should. You can get the code from my sandbox here: https://drupal.org/sandbox/briankpeterson/1361734

joelstein’s picture

Thanks! I'll take a look at it.

joelstein’s picture

Status: Needs review » Fixed

Great work! I changed a couple things (like using hook_permission instead of hook_perm, and moving the Role Vocabulary settings and sync form to be sub-menus of Taxonomy, plus a few other minor code cleanups). Otherwise, it looks great.

Committed to new 7.x-1.x branch.
https://drupal.org/commitlog/commit/22598/56a7e45b58c88321765e4a520bf370...

Please try it out, and if I don't hear of any problems in a few days, I'll release a stable version.

bulldozer2003’s picture

Thanks. It was a learning experience :)

joelstein’s picture

Did you download the 7.x-1.x version and try it out?

bulldozer2003’s picture

Yes, it appears stable.

joelstein’s picture

Cool. I've released a stable 7.0 version.

bulldozer2003’s picture

Version: 6.x-1.0 » 7.x-1.0
Status: Fixed » Closed (fixed)

Thanks!