Multiple choice Checkboxes don't save anything

Thanks for bringing this to my attention. I'm not sure how this slipped past me or why it wasn't discovered sooner.

I have identified the source of the problem and will have a new development snapshot available soon.

John

Hi Craig,

Thanks for the additional information.

The development version has a fix for the me aliases module. However your bug report revealed another problem that needs to be corrected as well. I'll be checking in a fix within the next couple of days and will update this issue at that time.

Thanks again,
John

Subscribing. :) I had the me module installed, as well. Thanks for rocking on this, John.

Shouldn't your uid lookup come from the account passed with the form?

      //$uid = (module_exists('me') && arg(1) == 'me') ? $user->uid : arg(1);
      // should be this?
      $uid = $form['_account']['#value']->uid;

Reason why I bring this up is if I call the form myself on w/e page I want, your code doesn't work as it's look towards the URL. For instance:

$form = drupal_get_form('user_profile_form', $account, 'Basics');
return $form;

Thoughts on this?

Also if an admin was editing a form, this wouldn't work either.

@iLLin,

I took your advice and made the indicated change--thanks!

@barber75,

The issue you originally reported has been fixed as well.

If you and anyone else could kindly test the newest dev snapshot. I believe this version is ready for release.

Regards,
John

Thanks for fixing this.
I installed the dev version and ran update.php.
It even picked up multi-choice values that were saved but not displayed.

Fixed in the 6.x-2.0 release.

after debugging i found that data from multicheckboxes save into users table but module logic based on that data save into profile_values table.
to fix it i've added implementation of hook_user to my module:

/**
 *  Implementation of hook_user().
 */
function mymodule_user($op, &$edit, &$account, $category = NULL) {
  switch ($op) {
    case 'update':
        return mymodule_save_profile($edit, $account);
    case 'insert':
      return mymodule_save_profile($edit, $account);
  }
}


/**
 *  Update/insert function for multi checkboxes.
 */
function mymodule_save_profile(&$edit, &$user) {
  $q = db_query("SELECT pf.name, pf.fid FROM {profile_checkboxes} pc 
                INNER JOIN {profile_fields} pf ON pf.fid = pc.fid
                WHERE pc.type = 'checkboxes'");
  while ($r = db_fetch_array($q)) {
    if (in_array($r['name'], array_keys($edit))) {
      db_query("DELETE FROM {profile_values} WHERE fid = %d AND uid = %d", $r['fid'], $user->uid);
      db_query("INSERT INTO {profile_values} (fid, uid, value) VALUES (%d, %d, '%s')", $r['fid'], $user->uid, $edit[$r['name']]);
    }
  }
}

so multiple checkboxes values save correctly now

Thanks for your contribution, but unless you're submitting a patch for Profile Checkboxes this issue is still closed.