Bug 280801 - Evolution shows attachment icon even if there's isn't any
Summary: Evolution shows attachment icon even if there's isn't any
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: evolution
Version: 7
Hardware: All
OS: Linux
medium
low
Target Milestone: ---
Assignee: Matthew Barnes
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-09-06 15:43 UTC by Michal Babej
Modified: 2013-07-31 23:11 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-02-12 10:58:53 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

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


Note You need to log in before you can comment on or make changes to this bug.