Search options (Page 1 of 11)

After I updated the following bugs reported in this feed, https://frontaccounting.com/punbb/viewtopic.php?id=10451
the errors disappeared.

This issue on another installation is still there despite the updates.

@trafficpest I am getting the following error on only one site with the same setup as three other sites with the SMTP class.mail replacement file:

TAX INVOICE IN003/2024. Sending document by email failed. Email: (it is a working email address, and the SMTP setup was checked and rechecked, but no luck in sending the invoice.)

Not sure which other file might have an impact on the sending.

I have updated FA to 2.4.18.

It is a simular issue that happened in 2013 and 2016 by searching the phrase "Sending document by email called."

The Class.mail.inc at line 131 - 142 has the following code as downloaded for use with SMTP PHPMailer:

    public function send() {
        $this->phpmailerCharSet = $this->charset;

        //send the message, check for errors
        if (!$this->phpmailer->send()) {
            error_log('Mailer error:' . $this->phpmailer->ErrorInfo);
            //echo "Mailer Error: " . $this->phpmailer->ErrorInfo;
            return 0;
        } else {
            return 1;
        }
    }
}

-----------------
It is the same code on other installations and none of them has this particular problem.

On one of the installations of FA 2.4.17 get an age-old error. Do not know why on all the other FA installations I did not get this error: Sending document by email failed.

I have redone the PHPMail installation and replacement of all the files as per instruction but the error remains. The email address is correct and the SMTP mail sending setup is correct.

I have even deleted the mail account on the server and redo the setup, still have the same error.

Got this post for the same errors https://frontaccounting.com/punbb/viewtopic.php?id=6369

I am stuck again.

@apmuthu does not seem to have pulled through. Re-downloaded the file from your link, but that file has not changed same error.

@traficpest I installed the extension, took me a while but eventually got it.

It works great and just one issue, the old system have the attachment as the Invoice number, but on this mailer the attachment is just a long set of numbers.

Had a quick look at the code in class.mail.inc for both files:

Old File at line 68:

function attachment($file, $filename=null)
    {
        if (!isset($filename))
            $filename = basename($file);
        $this->attachment[$filename] = $file;
    }

New replacement file line 116:

public function attachment($file) {
        $this->phpmailer->addAttachment($file);
    }

How do we change that to use the invoice number instead of a long list of codes on the pdf attachment?

Get the following message after the above updates:

"This edit session has been abandoned by opening sales document in another browser tab. You cannot edit more than one sales document at once."

This happened after I loaded a new customer and wanted to issue a direct invoice. I was still in the same tab. Using Edge.

Thank you for the changes. I have updated it as indicated by you and it works, the errors have gone.

Sorry for not coming back earlier, I did not receive the notification.

I appreciate your assistance. I trust that this will be correct in future updates.

Another error on PHP 8.2 when you want to add a new direct invoice for a client:

Unhandled exception [0]: Attempt to assign property "reference" on null. in file: /home/mysite/public_html/accounts/sales/sales_order_entry.php at line 267

Has this issue been resolved?

It seems that The warning message I am seeing is related to the use of a feature in PHP called “dynamic properties”. Dynamic properties allow you to set and get object properties by using variable names, like you are doing in the __construct method of your database class. They indicate that this feature has been deprecated in PHP 8.2 and later versions.

How do we fix this in the frontaccounting.php?

I am using FA 2.4.17 and PHP 8.2

See the lines in the comment below.