Emailing not setting mime-type

I found a solution for the reporting/includes/class.mail.inc when I got the issue with the same thing.

I have found the solutions by changing all the \n code in the file to \r\n and it works on both PHP7.4 and PHP8.0

Emails are now sending and displaying correctly when sent from FA.

\r\n is for a new line in windows as for \n in the new line in Linux.

Use \r\n instead of \n for your implode -- explicitly specifies CRLF as the line terminator.

With the 2.4.17 this issue is still there. It seems that the reporting/includes/class.mail.inc is still not fixed. With every update the new file is overwritten and we are back to the same issue.

When I replace the core file with the change information as indicated below it will send the emails and attachments as expected.

"I have found the solutions by changing all the \n code in the file to \r\n and it works on both PHP7.4 and PHP8.0

Emails are now sending and displaying correctly when sent from FA.

\r\n is for a new line in windows as for \n in the new line in Linux.

Use \r\n instead of \n for your implode -- explicitly specifies CRLF as the line terminator."

Why are we keep on having the same issue irrespective of the PHP it is set on?

Is no one else have the same issue?

This is what I receive in the email if I am lucky enough to get one without my changes to the file above:

--d6d94ed3a97c22d3af03070ba9b0c91c
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 8bit

JVBERi0xLjMKMS4wMCB3CjMgMCBvYmoKPDwvVHlwZSAvUGFnZQovUGFyZW50IDEgMCBSCi9N
ZWRpYUJveCBbMCAwIDU5NS4yOCA4NDEuODldCi9SZXNvdXJjZXMgMiAwIFIKL0NvbnRlbnRz
IDQgMCBSPj4KZW5kb2JqCjQgMCBvYmoKPDwvRmlsdGVyIC9GbGF0ZURlY29kZSAvTGVuZ3Ro
IDE1MDU+PgpzdHJlYW0KeJzVWltT2zgUfs+vOC87284QV5IlX3hqIKGTTqEUvO3utn0QsQLe
+gKyDU1//Up2LnISCJ1p7OkwdpBzLH/nO1cpxhZC8AAI3qrjP/j8VX2E6nijjuveUQCvTjBg
oqWCKYyCHv7ZG5DlIQrm+eJNj1az9JjD9D+O5+mPpEeRMYx7cDl/nJqEIQLmWU2Cf3oSz8Xq
dh85+kwYyOu5NPMrxIxUU/Vxda8UMF17yFwuad5mPmRN3qUN+Xr4hDxzGvL18LlKEPQ8JWq5
pHnbE6BQU2m0Q2lMGkrMh6Z8g72tytIm3YuxIUJYc5LF2BCxWXOWxdgQoc4aFGdjlnXTN01V
z2ISuU1v6tOmRD02RdaN0ySzErkDTD2LqThSf/VDaGVk168GkwRejTEMM/hQRyJpROLikr0z
mrdEG2x1vDsd9aAmnuNwK3hBCJ9fDO7KKI45DML7KM9kDpPJy68QvFWPhQ/6FqxR/FgCww1g
W7AuLvm/FL7lUtY4y1ofYlvMrxRy/VqhYPA3jM8+vh8fj3Yp4q+hxvCLQSM1hXmWphEcx7K9
GvM4ncRlGKXXsDBHIHmUqgsrFfYPiNEloPP3cJR9B+ywNgFQsgTwhst8mqWFiNIDOJeiyGTE
DwAhSgxEldXsVq1moxVJN1kqHvOx/QHBaAMJ8mzHdRBluH04c15UGVugOeHfu2PFwIE8Bxhm    etc.

With the changes implemented as indicated above, a pdf copy is correctly received as an attachment to the email and a proper email.

That happens when I updated FA to 2.4.17. Email an invoice to a client and that is what the client receives.

Then I will replace the standard class.mail.inc with the the one I have changed as indicated. by changing all the \n code in the file to \r\n and it works on both PHP7.4, HP8.0, 8.1 and 8.2.

Then it sends the invoice correctly as a PDF attachment.

@irronics, @joe: The said patch for reporting/includes/class.mail.inc will break for all versions of PHP < 5.4.
Refer the line:

var $headers = [];

It should be changed to:

var $headers = Array();

We need to take care of legacy users so as not to break their installs.

The PHP short array syntax does not work for older versions of PHP.

As of PHP 5.4 you can also use the short array syntax, which replaces array() with [].
https://stackoverflow.com/questions/40086788/short-array-syntax-in-php-5-6-doesnt-work

In fact there is a short array sniffer to check if needed but that is too bothersome.