We have had a client request this, I suppose I will have to dig through the code to find where to override.

Comments

drupalninja99’s picture

Category: support » feature

I was able to accomplish this in a hook_form_alter via:

/**
   * Modify the user registration form.
   * We want to show the password field
   */
  if ($form_id == 'user_register') {
    $form['account']['pass']['#title'] = 'Password';
    $form['account']['pass']['#type'] = 'password';
    $form['account']['pass']['#description'] = t('Enter the password that accompanies your username.');
    unset($form['account']['pass']['#value']);
  }

But it would be better maybe if we had an admin setting to wrap this in a conditional in the gigya.module

// Set the password and hide the password field.
      $form_account['pass']['#type'] = 'hidden';
      $form_account['pass']['#value'] = user_password();

You could have a setting that says 'hide password' or 'show password' or whatever and use that as the basis for whether or not to hide/show the password.

And I know that the thought is - why make the user enter in a password - isn't it better to just use the social login? Well the thing is the gigya people are telling clients including my client that best practice is to show the password field. And so they are the reason we implemented the logic above. So even though I might prefer to hide the password, if the gigya people are telling customers to allow users to set the password, I think we should make it a settings option.

EvanDonovan’s picture

I would agree - this should be configurable, since there are many cases in which it's better UX to set the password. Goes for 7.x as well.

lirand’s picture

Thanks for the suggestion. We're currently evaluating adding such a feature to an upcoming 7.x version.

Gigya’s picture

Issue summary: View changes
Status: Active » Closed (won't fix)

Gigya's 6.x module is deprecated. We no longer support this version and we encourage upgrading to our latest module.