Thanks
Not sure why it got labelled low priority, it’s a bit of a major when every email you send has duplicated attachments (Well that’s just my opinion).
Thanks for getting back to me.
This task was marked as low priority and then changed to Medium, but i think this is a very critical issue. How does the nature of this task make it a medium priority. I attach 4 quotes in email and the client receives 8. It is a Blocking issue which stops us to use emails module al together. If there is any fix available please share it.
We tend to use the high priority label for issues that completely block basic functionality with no workaround. In this case as you are still able to send an email with an attachment, the medium priority label would more appropriate.
Sorry, the code above for is dirty fix that I applied to my own project the make attachment not duplicated. Somehow the $_FILES[“email_attachment”] is duplicated. I think this problems is caused by editor or javscript at client ( the EmailsComposeView.js ) But I can’t fix it thoroughly, so I apply that quick fix.
The code I applied is placed in Emails.php line 2395 above
for ($i = 0; $i < $max_files_upload; $i++) {
I placed a “debugger” above that line, but the code not run to there, I thought the Email.js is not have any relation here, we just have EmailsComposeView.js working here,
What do you think ?