I have a requirement for facilitating term selections from multiple vocabularies. I have modified the code accordingly and am attaching the patch for review & acceptance.

CommentFileSizeAuthor
user_terms.ravi_sam.patch4.98 KBravi_sam

Comments

todd nienkerk’s picture

todd nienkerk commented

ravi_sam: Thanks for your work. I have committed your patch and some other changes to CVS. There are a few improvements I'd like to make, though, before we release this as a new version.

  1. If vocab FOO is removed from User Terms settings as an available option, I think all terms from FOO should be prevented from being displayed in users' profiles. The function that builds this display should check to make sure that all vocabs are still selected. (I do not want to remove the term associations from the DB in case an admin accidentally unselects a vocab.)
  2. I think merging all terms across all vocabs into one section of the user's profile should be an option. I can foresee use cases for both "display each group of terms separately" and "display all terms together."

Would you be able to help make these changes?

Because this module will support multiple vocabs, I don't see a reason to include custom vocab title and description fields. However, if the "display all terms together" option is selected, these two fields will be displayed in the user's profile instead of each vocab's name and description.

todd nienkerk’s picture

todd nienkerk commented
Status: Patch (to be ported) » Needs review

Nevermind! I made the changes myself. Please let me know what you think when you have a moment. I've committed everything to CVS.

joachim’s picture

joachim commented
Status: Needs review » Fixed

Closing as fixed.

Status: Fixed » Closed (fixed)

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