After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 677608 - Fails to open message which is not yet downloaded
Fails to open message which is not yet downloaded
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
3.6.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
evolution[webkit]
Depends on:
Blocks:
 
 
Reported: 2012-06-07 07:45 UTC by Milan Crha
Modified: 2013-09-13 01:08 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
test message (3.49 KB, text/plain)
2012-06-07 07:45 UTC, Milan Crha
  Details
Patch (20.18 KB, patch)
2012-06-15 15:39 UTC, Dan Vrátil
committed Details | Review

Description Milan Crha 2012-06-07 07:45:25 UTC
Created attachment 215824 [details]
test message

My favourite IMAP account (not IMAP+). There I have message which I do not have downloaded yet in a folder which has disabled preview panel. I press Ctrl+U on the message which opens the Mail Browser window, but it stays empty. I've attached also the message. Once the message is downloaded I can see it as expected.

I see these on the console:

WRAPPING .message.mixed.1.plain_text.0 AS text/x-patch
WRAPPING .message.mixed.2.plain_text.0 AS text/x-patch
WRAPPING .message.mixed.1.plain_text.0 AS text/x-patch
WRAPPING .message.mixed.2.plain_text.0 AS text/x-patch

(evolution:23412): GLib-CRITICAL **: g_hash_table_destroy: assertion `hash_table != NULL' failed

(evolution:23412): GLib-CRITICAL **: g_hash_table_destroy: assertion `hash_table != NULL' failed

(evolution:23412): GLib-CRITICAL **: g_hash_table_destroy: assertion `hash_table != NULL' failed

(evolution:23412): GLib-CRITICAL **: g_str_has_suffix: assertion `str != NULL' failed

(evolution:23412): evolution-mail-CRITICAL **: handle_mail_request: assertion `part_list != NULL' failed
Comment 1 Dan Vrátil 2012-06-15 15:39:42 UTC
Created attachment 216529 [details] [review]
Patch

This patch removes the GHashTable in the global SoupSession that was used to store parsed EMailPartLists and replaces the entire mechanism by CamelObjectBag.
Comment 2 Milan Crha 2012-06-18 08:30:25 UTC
Thanks. It looks good, please commit.
Comment 3 Dan Vrátil 2012-06-18 12:01:36 UTC
Comment on attachment 216529 [details] [review]
Patch


Committed to master as fe59b10f04b794759f64d9

http://git.gnome.org/browse/evolution/commit/?id=fe59b10f04b794759f64d97281d907d7dce39972