For filenames that get modified by the transliteration module the mail import fails because MailhandlerCommandsFiles::process() stores the file record with unsanitized filename but FeedsEnclosure will later try to copy that file to the final destination and sanitizes the filename before looking for it. That will cause an error and the attachment will later be missing.

I will submit a patch in a minute.

Comments

jurgenhaas’s picture

Status: Active » Needs review
StatusFileSize
new970 bytes

Status: Needs review » Needs work

The last submitted patch, 1: file_uri_problem_when-2534984-1.patch, failed testing.

Status: Needs work » Needs review
jurgenhaas’s picture

Status: Needs review » Active

I don't think the failure of the test is because of my patch, there must be a different reason. Can anyone check this please?

Status: Active » Needs work

The last submitted patch, 1: file_uri_problem_when-2534984-1.patch, failed testing.

k4v’s picture

StatusFileSize
new1.05 KB

This works for me :). For me, the mail import did not fail, but I had other problems with filenames that were not transliterated. With the module search_api_attachments I could not process them with the Tika engine.

Here's a reroll. Thanks for the patch, I was looking for some time for the right spot to fix this.

k4v’s picture

Status: Needs work » Closed (fixed)
k4v’s picture

Status: Closed (fixed) » Needs review