mail sending in function hosting_client_register_user() was not working, this patch fixes it.
Only tested once though

CommentFileSizeAuthor
hosting_client_2.patch2.21 KBhutch

Comments

hutch’s picture

hutch commented

In the same function, line 246

  $edit['name'] = $node->email;

needs to be

  $edit['name'] = $node->client_name;

Drupal does not allow '@' in a username and folks would not be able to login

anarcat’s picture

anarcat commented
Status: Active » Needs review

The patch is a bit confusing because it changes the way the parsing works, but i guess it should be tested, once #516582: hosting signup form broken on d6 is resolved.

hutch’s picture

hutch commented

It's just a suggestion really, the additional function seems to be the way drupal_mail does things in D6. I'm going to have to re-install aegir on my devbox anyway, getting this now:
Fatal error: Unsupported operand types in /var/aegir/drupal-6.x-13/includes/common.inc on line 1552
oh well ;-(

I'll have a look at #516582, perhaps I can help there

anarcat’s picture

anarcat commented
Status: Needs review » Needs work

Help on the signup form would of course be welcome.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs work » Closed (fixed)

hosting_client_mail() got rolled in per #726756: Client welcome emails empty: no subject or body, fixing d5 code that hasn't worked for about a year, and the other issue seems closed so I'm closing this.