I took version 1.1 out of CVS just now, and installed it on my site. When I went to save a User Term (or set of User Terms) using HS, nothing was saving. I opened up the code, and there is a small check missing that causes the data to never be processed. I took the check out of user_terms.module, and applied it here, and voila, it works!

I made a patch, although I'm no expert patcher (this is my first submitted patch, I believe). Here is the code I added to hs_user_terms_form_profile():

<?php
  // Set a hidden value on the form so that we can check in user_terms_save_profile()
  // that we've actually come from the account edit form.
  $form['user_terms_vids'] = array(
    '#type' => 'value', 
    '#value' => $vids,
  );
?>

It works!

CommentFileSizeAuthor
hs_user_terms.module.patch693 bytesphacts

Comments

cpliakas’s picture

Status: Patch (to be ported) » Reviewed & tested by the community

Hi phacts.

Congrats on your first patch! Very exciting. I tested it as well, it works just as you said. Thanks for the bug hunting.

~Chris

cpliakas’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: +6.x-1.0-beta1

Patch committed at #373990. The fix will be reflected in the 6.x-1.0-beta1 release.

Status: Fixed » Closed (fixed)
Issue tags: -6.x-1.0-beta1

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