phpBMS

Ticket #225 (assigned defect)

Opened 4 years ago

Last modified 7 months ago

Euro Symbol displaying as "€" when emailing invoice

Reported by: noelmul Owned by: brieb
Priority: major Milestone: 1.0
Component: phpbms Version: 0.51
Keywords: fixed Cc: dsterry/weex

Description

Euro Symbol displaying as "€" when emailing an invoice. Euro symbol displays fine when printing the exact same invoice. From trunk at revision 370.

Change History

Changed 4 years ago by noelmul

I looked at the code and noticed that when I change this line in include/session.php from: if(!isset($sqlEncoding))

$sqlEncoding = "utf8";

to: if(!isset($sqlEncoding))

$sqlEncoding = "latin1";

then the pdf invoices that phpBMS email are correctly encoded - showing the euro symbol correctly everytime. This has the knock-on effect of the euro symbols not displaying correctly in any of the phpbms html pages i.e. anything other than the reports.

Changed 4 years ago by anonymous

maybe phpbms can use tcpdf.sf.net to replace fpdf. fpdf lack support for utf-8

Changed 4 years ago by brieb

tcpdf is LGPL, and is not compatible with our modified BSD license.

Changed 3 years ago by worufu

  • version changed from trunk to 0.96

I experience the same problem (Euro symbol is broken in PDFs which are mailed only). PDFs generated via Print-Button are displaying the correct € symbol. Tested on invoices and quotes only.

I went through the source code for hours but I cannot figure out the difference between emailing and generating PDFs via the print-button. Basically emailing reuses the same class. Could it be that base64 cannot handle the character or something like that?

Changed 2 years ago by brieb

  • status changed from new to assigned
  • milestone changed from unknown to 1.0

It's possible. Can someone confirm this in v0.98?

We may need to make sure that the sql encoding is getting properly set when emailing

Changed 2 years ago by brieb

  • keywords needsReview added

Changed 15 months ago by dsterry/weex

  • cc dsterry/weex added
  • keywords fixed added; needsReview removed
  • version 0.96 deleted
  • priority changed from minor to major

A fix for this was posted in the forums at  http://phpbms.org/forum/viewtopic.php?f=30&t=699

I applied this patch to my repo with the 2 commits ending with  https://github.com/weex/phpBMS/commit/6d81c78d72043f4d90549d9ca8904f57cefda0e7 and when tested, it worked!

Changed 7 months ago by mipalmer

  • summary changed from Euro Symbol displaying as "Ã?¢â??Ã?¬" when emailing invoice to Euro Symbol displaying as "€" when emailing invoice
Note: See TracTickets for help on using tickets.
Scanned by Orvant Copyright © 2010 Kreotek, LLC. All Rights reserved.