In EdsPlanner::on_source_enabled(), don't use ESources that don't have calendars or tasks.

1 files changed, 21 insertions, 8 deletions

diff --git a/src/planner-eds.cpp b/src/planner-eds.cpp
index 7d9416c..ee023a4 100644
--- a/src/planner-eds.cpp
+++ b/src/planner-eds.cpp
@@ -128,20 +128,33 @@ private:
     {
         auto self = static_cast<Impl*>(gself);
         ECalClientSourceType source_type;
+        bool client_wanted = false;
 
         if (e_source_has_extension(source, E_SOURCE_EXTENSION_CALENDAR))
+        {
             source_type = E_CAL_CLIENT_SOURCE_TYPE_EVENTS;
+            client_wanted = true;
+        }
         else if (e_source_has_extension(source, E_SOURCE_EXTENSION_TASK_LIST))
+        {
             source_type = E_CAL_CLIENT_SOURCE_TYPE_TASKS;
+            client_wanted = true;
+        }
+
+        const auto source_uid = e_source_get_uid(source);
+        if (client_wanted)
+        {
+            g_debug("%s connecting a client to source %s", G_STRFUNC, source_uid);
+            e_cal_client_connect(source,
+                                 source_type,
+                                 self->m_cancellable,
+                                 on_client_connected,
+                                 gself);
+        }
         else
-            g_assert_not_reached();
-
-        g_debug("connecting a client to source %s", e_source_get_uid(source));
-        e_cal_client_connect(source,
-                             source_type,
-                             self->m_cancellable,
-                             on_client_connected,
-                             gself);
+        {
+            g_debug("%s not using source %s -- no tasks/calendar", G_STRFUNC, source_uid);
+        }
     }
 
     static void on_client_connected(GObject* /*source*/, GAsyncResult * res, gpointer gself)