hosting_client module mail

hutch - June 26, 2009 - 17:59
Project:Hosting
Version:6.x-0.2-beta1
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs work
Description

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

AttachmentSize
hosting_client_2.patch2.21 KB

#1

hutch - June 27, 2009 - 11:19

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

anarcat - July 11, 2009 - 21:03
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.

#3

hutch - July 11, 2009 - 22:54

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

anarcat - July 13, 2009 - 18:27
Status:needs review» needs work

Help on the signup form would of course be welcome.

 
 

Drupal is a registered trademark of Dries Buytaert.