Hello!

I've been experimenting with the module, especially with the auto create node on incoming mail feature. I've found that the attachments I send from mac mail comes up as 'noname' attachments in the nodes. When I send from web mail this is not a problem and I haven't been able to try on a windows machine.

I suppose there is something going on in the '_support_save_attachments' function in support.module since that seems to be where the 'noname' comes from if the code can't find a filename. It seems this is way above my head atm but I'll report it anyway to see if anyone else had any issues with this.

CommentFileSizeAuthor
#21 support.module.txt147.21 KBshevgeny
#8 mailtest.txt4.09 KBgigiozzz
#8 test_imap_attachment.txt7.9 KBgigiozzz
#8 mac.patch4.52 KBgigiozzz

Comments

span’s picture

span commented

Just thought I'd give some more information on this.

The attachments are saved as 'noname' without extension.

If there is no attachment to the mail however, the mail itself gets stored as an attachment called 'noname.html'. I assume this is expected behaviour.

When a mail with an attachment is being sent from mac mail only the attachment is saved (as 'noname'). The message itself is saved in the node as it should but there is no creation of an attachment of the mail itself as there is when a mail is being sent without attachment.

To my knowledge I'm using the default mac mail settings. I've checked the inbox on the server that recieves the mails and all _seems_ fine.

jeremy’s picture

jeremy commented
Priority: Critical » Normal
Status: Active » Postponed (maintainer needs more info)

When this happens, I would expect that you're essentially seeing two copies of the email. The first should show up in the ticket body, the second should show up as an attachment with the 'noname.html' name. This happens when an email includes both a text version and an HTML version. Or, are you saying that you're getting a blank ticket with only an html attachment?

span’s picture

span commented

The ticket content looks fine and I'm sure your right about the two versions of the mail causing one attachment to be saved as a .html file.

The issue I was trying to report though is if I attach a file to the mail sent, that file gets stored as an attachment with the name 'noname' without extension. So even if I attach 'example.pdf' it gets listed as 'noname'. When I send from webmail, this is not a problem.

jeremy’s picture

jeremy commented

This is happening for you for all messages attached from the Mac client?

span’s picture

span commented

Yes, any file I attach and send from the Mac client this happends to.

Valeratal’s picture

Valeratal commented

I have problem too
I set Mimemail to use the SMTP Authenticaiton module as the engine, the email went through, but it was a blank email with an attachment named "noname".

rsevero’s picture

rsevero commented

The patch available at #981622: File names with accents of attached files in emails for ticket creation got created with nontranslated filenames: not accessable might help with this issue.

Could some Mac owner test it?

gigiozzz’s picture

gigiozzz commented
StatusFileSize
new mac.patch4.52 KB
new test_imap_attachment.txt7.9 KB
new mailtest.txt4.09 KB

This don't resolve anything!!!

you didn't understand the real problem

problem:
1) apple MAIL program send mail in a very different way (see example attachment [1])
2) who wrote fetching attachment function misunderstood the SECTION param
"The part number. It is a string of integers delimited by period which index into a body part list as per the IMAP4 specification "
STRING like 2.1
because sometimes an attachment is in a "SUB MIXED SECTION" so you need recursive function

solution:
1) test the little cli program [2] with your mail from MAIL mac program
2) use patch [3]

i had the same problem and i solved it with this patch for stable version (i'm yet testing it)

you people should read better at docs (RTFM) ther'is always a solution (i arranged the patch in 1 hour more or less with function in the manual pages)
http://www.php.net/manual/en/function.imap-fetchstructure.php
http://www.php.net/manual/en/function.imap-fetchbody.php

please
i like support module
BUT
you guys should improve a lot the fetching mail stuff (because it's a really important feature)

rsevero’s picture

rsevero commented

@gigiozzz:
I'm sorry that my suggestion (trying the patch available at #981622: File names with accents of attached files in emails for ticket creation got created with nontranslated filenames: not accessable) didn't solve this issue. Please understand that the mentioned patch wasn't made to fix this issue at all.

I had this hunch that the mentioned patch could fix this issue also. It seems I was wrong.

It was made to fix the issue mentioned in issue # 981622. And as far as I can tell it does that.

I'm happy that you finally managed to fix this issue.

jeremy’s picture

jeremy commented
Status: Postponed (maintainer needs more info) » Active

Needs further testing and a resolution if still a problem, re-opening.

bdragon’s picture

bdragon commented
Assigned: Unassigned » bdragon

having a look-see.

Exploratus’s picture

Exploratus commented

I am having the same problem... When I email through a MAC, I get noname. Does the patch resolve this? Seems pretty old and a good % of people use MACs.

Exploratus’s picture

Exploratus commented

Patch seems to work. Can we get this commited?

xaa’s picture

xaa commented
Status: Active » Reviewed & tested by the community

hi, patch #8 kindly provided by gigiozzz is still valid for the 7.0-rc2 .
ps: we are on windows and ticket sent by outlook, gmail and Co generated the noname.html file (code source of the mail).

D7 follow-up : http://drupal.org/node/1876036

betz’s picture

betz commented

I can second this patch is applicable for D7.

I got a jpg as attachment, but when trying with a zip file, it is still a noname file.
But at least images work

jeremy’s picture

jeremy commented
Version: 6.x-1.3 » 6.x-1.x-dev

Here's a direct link to where this came from -- reviewing:
http://www.php.net/manual/en/function.imap-fetchbody.php#89002

jeremy’s picture

jeremy commented
Issue tags: +mail

Tagging to review when I'm set up to test mail.

Homotechsual’s picture

Homotechsual commented

Reviewed and tested on 7.x-1.x-dev and RC1. Works as intended.

bwoods’s picture

bwoods commented
Issue summary: View changes

For D7, I had to make one slight alteration to #8. I was seeing a notice for $obj->parts not being found, so I included an isset check before it. This is near the bottom of the add_part_to_array function. Thanks!

shevgeny’s picture

shevgeny
Primary language English
commented

#19, Hello Ben! Could you please send the code to your adjustment? I have the same problem arises. I'll be very grateful! (For D7)

shevgeny’s picture

shevgeny
Primary language English
commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Reviewed & tested by the community » Needs work
StatusFileSize
new support.module.txt147.21 KB

Error
Notice: Undefined property: stdClass :: $ nid function _support_subscribe_form_attach () (line 3043 file /sites/all/modules/support/support.module).
(Drupal 7)

purencool’s picture

purencool commented
purencool’s picture

purencool commented
purencool’s picture

purencool commented
Version: 7.x-1.x-dev » 8.x-1.x-dev