Posted by hutch on June 26, 2009 at 5:59pm
Jump to:
| Project: | Hosting |
| Version: | 6.x-0.2-beta1 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
mail sending in function hosting_client_register_user() was not working, this patch fixes it.
Only tested once though
| Attachment | Size |
|---|---|
| hosting_client_2.patch | 2.21 KB |
Comments
#1
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
#2
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.
#3
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
#4
Help on the signup form would of course be welcome.
#5
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.