Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: find and send invoice attachment in sendEmailsRemindersOnInvoiceDueDate #31090

Open
wants to merge 1 commit into
base: 17.0
Choose a base branch
from

Conversation

FHenry
Copy link
Member

@FHenry FHenry commented Sep 23, 2024

Fix find correct last_main_doc in sendEmailsRemindersOnInvoiceDueDate

Actual dolibarr_cron.log :

CMailFile::sendfile: mail end error=Unable to open file for reading [/home/XXXX/dolibarr/documentsfacture/FA00001/FA00001.pdf]
Cronjob::run_jobs END result=1 error= : NOMCONTACT <emailcontact@xxxx.xx>

There is a missing "/" in path /home/XXXX/dolibarr/documentsfacture/
instead of /home/XXXX/dolibarr/documents/facture/

DOL_DATA_ROOT do not have last / by default, and last_main_doc not have first /

Every where in htdocs/core/tpl/card_presend.tpl.php where file is search there is
$diroutput.'/'.$ref

@FHenry FHenry changed the title fix: Fix find correct last_main_doc in sendEmailsRemindersOnInvoiceDueDate fix: find and send invoice attachment in sendEmailsRemindersOnInvoiceDueDate Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant