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 633780 - ESourceSelector - check paths when removing source/group
ESourceSelector - check paths when removing source/group
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Plugins
3.0.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-plugin-maintainers
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2010-11-02 08:46 UTC by Akhil Laddha
Modified: 2013-09-13 01:10 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
eds patch (1.02 KB, patch)
2010-11-22 11:58 UTC, Milan Crha 		committed Details | Review

Description Akhil Laddha 2010-11-02 08:46:00 UTC 
evolution 2.91.2 (gtk 2.24)

right click on email account -> disable account. I have tried with group wise and mapi accounts and got same warnings.

gdb traces of evolution

+ Trace 224437

  • #0 g_logv
    at gmessages.c line 439
  • #1 g_log
    at gmessages.c line 577
  • #2 g_return_if_fail_warning
  • #3 IA__gtk_tree_store_remove
    at gtktreestore.c line 1113
  • #4 rebuild_model
    at e-source-selector.c line 369
  • #5 on_idle_rebuild_model_callback
    at e-source-selector.c line 494
  • #6 g_idle_dispatch
    at gmain.c line 4378
  • #7 g_main_dispatch
    at gmain.c line 2229
  • #8 g_main_context_dispatch
    at gmain.c line 2786
  • #9 g_main_context_iterate
    at gmain.c line 2864
  • #10 g_main_loop_run
    at gmain.c line 3072
  • #11 IA__gtk_main
    at gtkmain.c line 1238
  • #12 main
    at main.c line 697
  • #0 g_logv
    at gmessages.c line 439
  • #1 g_log
    at gmessages.c line 577
  • #2 g_return_if_fail_warning
    at gmessages.c line 586
  • #3 IA__gtk_tree_model_get_iter
    at gtktreemodel.c line 1017
  • #4 rebuild_model
    at e-source-selector.c line 363
  • #5 on_idle_rebuild_model_callback
    at e-source-selector.c line 494
  • #6 g_idle_dispatch
    at gmain.c line 4378
  • #7 g_main_dispatch
    at gmain.c line 2229
  • #8 g_main_context_dispatch
    at gmain.c line 2786
  • #9 g_main_context_iterate
    at gmain.c line 2864
  • #10 g_main_loop_run
    at gmain.c line 3072
  • #11 IA__gtk_main
    at gtkmain.c line 1238
  • #12 main
    at main.c line 697 
Continuing.

(evolution:22338): Gtk-CRITICAL **: IA__gtk_tree_model_get_iter: assertion `path != NULL' failed
Comment 1 Milan Crha 2010-11-22 11:58:34 UTC 
Created attachment 175020 [details] [review]
eds patch

for evolution-data-server;

It does that also with eex account. The thing was either the group was removed first, thus this claimed for all sources under it, or the sources were removed, and when removing the last source then the group itself was also removed automaticly, thus when was trying to remove group by the tree reference, then it failed.

I tried to ensure that the ESource-s will be removed first, and then ESourceGroup-s, and also that the invalid references will be skipped in the process.

By the way, to reproduce this, one should also move to Contacts or Calendar view, otherwise the ESourceSelector is not initialized (maybe also Preferences would work these days).
Comment 2 Milan Crha 2010-11-22 12:00:48 UTC 
Created commit a92e363 in eds master (2.91.3+)