This is kind of a blocker as i can't use this module with services 3.
when i call the user/register built in service the services function builds a form data object and tries to submit it, in this process the following code is running:

// register a new user
  $form_state['values'] = $account;
  $form_state['values']['pass'] = array(
    'pass1' => $account['pass'],
    'pass2' => $account['pass'],
  );
  $form_state['values']['op'] = variable_get('services_user_create_button_resource_create', t('Create new account'));

  // execute the register form
  drupal_form_submit('user_register_form', $form_state);

in the process of validating the user password, logintoboggan calls the function logintoboggan_validate_pass and runs the following code:
if (!strlen($pass)) return t('You must enter a password.');

as you can see using this module i will never be able to get over this.
logintoboggan needs to be aware of this behavior and support it.

Thanks

Comments

basik.drupal’s picture

and here is the patch to fix this (attached and tested)

cwithout’s picture

Status: Active » Reviewed & tested by the community
StatusFileSize
new694 bytes

Had the same problem. The patch fixed it, but patch itself has the full path from Drupal root instead of from within the logintoboggin directory. Just changed the file path.

azielinsky’s picture

Although the patch does resolve the "You must enter a password" issue. The pass is not stored correctly in the database and the user cannot login. I don't kn ow how to make a patch file so i'm putting my solution here. In the function logintoboggan_user_register_submit, you must assing the $pass variable the right value in order to be stored correctly.

damienmckenna’s picture

Version: 7.x-1.3 » 7.x-1.x-dev
Issue summary: View changes
dooug’s picture

Status: Reviewed & tested by the community » Needs work

It sounds like this needs work, or more reviews.

@azielinsky, please see the documentation to create patches in order to share your patch suggestion.

dooug’s picture

dooug’s picture

Priority: Critical » Normal