Bug 280801

Summary: Evolution shows attachment icon even if there's isn't any
Product: [Fedora] Fedora Reporter: Michal Babej <mbabej>
Component: evolutionAssignee: Matthew Barnes <mbarnes>
Status: CLOSED UPSTREAM QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: medium    
Version: 7CC: desktop-bugs, mcrha, smohan
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-02-12 10:58:53 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Michal Babej 2007-09-06 15:43:12 UTC 
Description of problem:
Evolution shows attachment icon even if the email has no attachments. I guess
it's caused by the fact that the message in question has "Content-Type:
multipart/mixed;" but actually only contains 1 part (text/plain), properly
enclosed in boundaries. It's a bit misleading.

Version-Release number of selected component (if applicable):
evolution-2.10.3-2.fc7
evolution-data-server-1.10.3.1-2.fc7
evolution-webcal-2.10.0-1.fc7
evolution-data-server-devel-1.10.3.1-2.fc7

How reproducible:
always
Comment 1 Matthew Barnes 2007-09-06 16:37:23 UTC 
Can you post an mbox file containing such an email?
Comment 3 Matthew Barnes 2007-09-06 19:43:00 UTC 
Thanks Michal, easily reproducible.

I found the logic that decides whether to display the icon:

        ml_tree_value_at() (in message-list.c):

        case COL_ATTACHMENT:
                return GINT_TO_POINTER (
                        (camel_message_info_flags(msg_info) &
                         CAMEL_MESSAGE_ATTACHMENTS) != 0);

It's just checking a flag, so that's okay.  The bug, then, must be in Camel. 
I'll keep digging...
Comment 4 Matthew Barnes 2007-09-06 19:50:35 UTC 
You're right, Camel is just setting the CAMEL_MESSAGE_ATTACHMENTS flag based on
whether the MIME type is "multipart/mixed" (in camel-folder-summary.c).  I guess
it should also check whether the number of parts is > 1.  I need to consult with
upstream.
Comment 5 Milan Crha 2008-02-12 10:58:53 UTC 
Moving this upstream to bug [1]. We will track it there.

[1] http://bugzilla.gnome.org/show_bug.cgi?id=478239