Would be lovely to integrate with inline registration.

CommentFileSizeAuthor
#4 email_registration-inline_registration-fix-1911574-4.patch599 bytesjdleonard

Comments

doublejosh’s picture

doublejosh commented

All that was necessary to make this work for me (with 6.x-1.3) was...

  // Inline Registration integration.
  if (in_array('inline_registration_submit', $form['#submit'])) {
    $form['register']['form']['account']['name']['#type'] = 'hidden';
    $form['register']['form']['account']['name']['#value'] = user_password();
    $form['register']['form']['account']['mail']['#title'] = t('E-mail');
  }

...added to the end of email_registration_form_alter()

This has not been tested with the latest dev branch.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Thanks for your feature request and for helping make this a better module. Could you provide this as a patch?

maked1sky’s picture

maked1sky commented

I'm sorry I do not understand where to put this code (I have not found ---email_registration_form_alter ()--- in inline_registration and email_registration modules

  // Inline Registration integration.
  if (in_array('inline_registration_submit', $form['#submit'])) {
    $form['register']['form']['account']['name']['#type'] = 'hidden';
    $form['register']['form']['account']['name']['#value'] = user_password();
    $form['register']['form']['account']['mail']['#title'] = t('E-mail');
  }

and if it works with Drupal 7?

jdleonard’s picture

jdleonard
he/him
Primary language English
Location Austin, Texas
commented
Version: 6.x-1.3 » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new email_registration-inline_registration-fix-1911574-4.patch599 bytes

Patch attached. This solves the problem for me when combined with an inline_registration patch (comment 18) from #272672: Getting it to work with Email Registration, Login Toboggan and Registration Toboggan modules. I don't see any negative consequences of applying this.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Needs review » Reviewed & tested by the community

This sure looks like it should be low impact. In fact, it's a better idea anyway.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Committed - thanks!

http://drupalcode.org/project/email_registration.git/commit/4cb4257

Status: Fixed » Closed (fixed)

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