Bug 656267 – Checking for auth types doesn't work properly

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 656267 - Checking for auth types doesn't work properly


Summary:	Checking for auth types doesn't work properly


Status:	RESOLVED FIXED

Product:	evolution
Classification:	Applications
Component:	general
Version:	3.2.x (obsolete)
Hardware:	Other Linux

Importance:	Normal normal
Target Milestone:	---
Assigned To:	Evolution Shell Maintainers Team
QA Contact:	Evolution QA team

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2011-08-10 10:51 UTC by Akhil Laddha
Modified:	2013-09-13 01:05 UTC

See Also:
GNOME target:	---
GNOME version:	---

Attachments
eds patch (1.70 KB, patch) 2011-08-12 09:26 UTC, Milan Crha	committed	Details \| Review
evo patch (4.04 KB, patch) 2011-08-12 09:28 UTC, Milan Crha	committed	Details \| Review

Description Akhil Laddha 2011-08-10 10:51:37 UTC

Evolution 3.1.5

While configuring a new account (say IMAP/IMAP+), click on 'check for supported type' in receiving email

(gdb) bt

+ Trace 228058

#0 g_logv
at gmessages.c line 439
#1 g_log
at gmessages.c line 573
#2 g_return_if_fail_warning
#3 emae_check_authtype
at em-account-editor.c line 2009
#4 g_cclosure_marshal_VOID__VOID
at gmarshal.c line 79
#5 g_closure_invoke
at gclosure.c line 771
#6 signal_emit_unlocked_R
at gsignal.c line 3256
#7 g_signal_emit_valist
at gsignal.c line 2987
#8 g_signal_emit
at gsignal.c line 3044
#9 gtk_button_clicked
at gtkbutton.c line 1194
#10 gtk_real_button_released
at gtkbutton.c line 1827
#11 g_cclosure_marshal_VOID__VOID
at gmarshal.c line 79
#12 g_type_class_meta_marshal
at gclosure.c line 882
#13 g_closure_invoke
at gclosure.c line 771
#14 signal_emit_unlocked_R
at gsignal.c line 3186
#15 g_signal_emit_valist
at gsignal.c line 2987
#16 g_signal_emit
at gsignal.c line 3044
#17 gtk_button_released
at gtkbutton.c line 1180
#18 gtk_button_button_release
at gtkbutton.c line 1719
#19 _gtk_marshal_BOOLEAN__BOXED
at gtkmarshalers.c line 85
#20 g_type_class_meta_marshal
at gclosure.c line 882
#21 g_closure_invoke
at gclosure.c line 771
#22 signal_emit_unlocked_R
at gsignal.c line 3294
#23 g_signal_emit_valist
at gsignal.c line 2997
#24 g_signal_emit
at gsignal.c line 3044
#25 gtk_widget_event_internal
at gtkwidget.c line 6114
#26 gtk_widget_event
at gtkwidget.c line 5830
#27 gtk_propagate_event
at gtkmain.c line 2597
#28 gtk_main_do_event
at gtkmain.c line 1872
#29 _gdk_event_emit
at gdkevents.c line 71
#30 gdk_event_source_dispatch
at gdkeventsource.c line 318
#31 g_main_dispatch
at gmain.c line 2477
#32 g_main_context_dispatch
at gmain.c line 3050
#33 g_main_context_iterate
at gmain.c line 3128
#34 g_main_loop_run
at gmain.c line 3336
#35 gtk_main
at gtkmain.c line 1358
#36 main
at main.c line 691

Continuing.

(evolution:23602): evolution-mail-CRITICAL **: emae_check_authtype: assertion `CAMEL_IS_SERVICE (camel_service)' failed

Comment 1 Milan Crha 2011-08-12 09:26:21 UTC

Created attachment 193678 [details] [review]
eds patch

for evolution-data-server;

To be able to remove previously added service from a session.

Comment 2 Milan Crha 2011-08-12 09:28:45 UTC

Created attachment 193679 [details] [review]
evo patch

for evolution;

It doesn't work since move from uri to uid definitions of camel services. And of course this cannot work for newly creating accounts, they are not part of the session yet. And for those previously filled the actual behaviour is incorrect too, it would not use currently filled data in the editor, but those used when the service was added to the session.

Comment 3 Milan Crha 2011-08-12 09:31:49 UTC

Created commit f4bb87e in eds master (3.1.5+)
Created commit b7cfbb7 in evo master (3.1.5+)