• From: Netherlands
  • Registered: 10/29/2009
  • Posts: 64

Topic: customer name contains comma causing e-mail to fail

I split customer name with a comma by:
lastname, firstname

also for the short name (as I did an import from my previous ledger with the FA cvs import module).

The result is that it tries to e-mail to:

To: lastname@serverdomein.nl, firstname <validemail@validdomain.nl>

I took this line from the mail it tries to send. FA 2.2.6 returns an error that it could not send the mail (I send it in the page just after submitting the sales invoice).

It looks like it tries to send it to two addresses:
lastname@serverdomein.nl
firstname <validemail@validdomain.nl>

  • tom
  • Senior Member
  • Offline
  • From: USA, FL, Tampa
  • Registered: 03/24/2009
  • Posts: 326

Re: customer name contains comma causing e-mail to fail

That is not surprising.

Maybe we should add checking to the add/modify customers to check for ",;' and any other specials that are not valid in names.

Granted we could attempt to escape it which may be as simple as "lastname, firstname" but that could look odd to most people.

Maybe YOU can try entering the name that way, the email should accept it, I am not sure how it will look elsewhere in FA

tom

tom's Website

  • joe
  • Administrator
  • Offline
  • Registered: 03/15/2007
  • Posts: 5,107

Re: customer name contains comma causing e-mail to fail

This has been fixed in the from/to email. The commas in from/to name has been been replaced with a ''.
The CVS Main trunk updated. Affected file: /reporting/includes/pdf_report.inc.
You can download the file here:



/Joe