Merge lp:~purejava/pantheon-mail/fix-1546062 into lp:~elementary-apps/pantheon-mail/trunk

Proposed by Ralph Plawetzki
Status: Merged
Approved by: Danielle Foré
Approved revision: 1989
Merged at revision: 1992
Proposed branch: lp:~purejava/pantheon-mail/fix-1546062
Merge into: lp:~elementary-apps/pantheon-mail/trunk
Diff against target: 41 lines (+18/-2)
2 files modified
src/client/conversation-viewer/conversation-viewer.vala (+11/-1)
src/engine/rfc822/rfc822-message.vala (+7/-1)
To merge this branch: bzr merge lp:~purejava/pantheon-mail/fix-1546062
Reviewer Review Type Date Requested Status
elementary Apps team Pending
Review via email: mp+287516@code.launchpad.net

Commit message

Workaround to display inline images that have a Content-Id ending with '.'

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/client/conversation-viewer/conversation-viewer.vala'
--- src/client/conversation-viewer/conversation-viewer.vala 2016-02-24 11:48:02 +0000
+++ src/client/conversation-viewer/conversation-viewer.vala 2016-02-29 19:21:27 +0000
@@ -2027,7 +2027,17 @@
2027 img.set_attribute("class", DATA_IMAGE_CLASS);2027 img.set_attribute("class", DATA_IMAGE_CLASS);
2028 if (!Geary.String.is_empty(filename))2028 if (!Geary.String.is_empty(filename))
2029 img.set_attribute("alt", filename);2029 img.set_attribute("alt", filename);
2030 img.set_attribute("replaced-id", replaced_images_index.get(content_id));2030
2031 // FIXME: bugzilla.gnome.org 762782
2032 // in case content_id has a trailing period it gets removed
2033 // this is necessary as g_mime_object_get_content_id removes it too
2034 if (content_id.has_suffix(".")) {
2035 string content_id_without_suffix;
2036 content_id_without_suffix = content_id.slice(0,content_id.length-1);
2037 img.set_attribute("replaced-id", replaced_images_index.get(content_id_without_suffix));
2038 } else {
2039 img.set_attribute("replaced-id", replaced_images_index.get(content_id));
2040 }
2031 2041
2032 // stash here so inlined image isn't listed as attachment (esp. if it has no2042 // stash here so inlined image isn't listed as attachment (esp. if it has no
2033 // Content-Disposition)2043 // Content-Disposition)
20342044
=== modified file 'src/engine/rfc822/rfc822-message.vala'
--- src/engine/rfc822/rfc822-message.vala 2016-02-16 16:37:14 +0000
+++ src/engine/rfc822/rfc822-message.vala 2016-02-29 19:21:27 +0000
@@ -714,7 +714,13 @@
714 return part.get_filename();714 return part.get_filename();
715 }715 }
716 716
717 private GMime.Part? find_mime_part_by_mime_id(GMime.Object root, string mime_id) {717 private GMime.Part? find_mime_part_by_mime_id(GMime.Object root, owned string mime_id) {
718 // FIXME: bugzilla.gnome.org 762782
719 // in case mime_id has a trailing period it gets removed
720 // this is necessary as g_mime_object_get_content_id removes it too
721 if (mime_id.has_suffix("."))
722 mime_id = mime_id.slice(0,mime_id.length-1);
723
718 // If this is a multipart container, check each of its children.724 // If this is a multipart container, check each of its children.
719 if (root is GMime.Multipart) {725 if (root is GMime.Multipart) {
720 GMime.Multipart multipart = root as GMime.Multipart;726 GMime.Multipart multipart = root as GMime.Multipart;

Subscribers

People subscribed via source and target branches