depau/Purism-Calls
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/calls.git synced 2024-12-13 16:07:36 +00:00
Code Issues Releases Wiki Activity

ui-call-data: Allow contact provider to be NULL

Browse source 
The contact provider can be NULL in the ringer tests, because we're stubbing it.
Early return in that case and make sure the call is already set.
This commit is contained in:
Evangelos Ribeiro Tzaras 2022-01-31 16:18:07 +01:00
parent 6f9b071bbc
commit dd8cc08649
1 changed files with 22 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

34
src/calls-ui-call-data.c
View file

@ -77,7 +77,10 @@ calls_ui_call_data_get_display_name (CuiCall *call_data)
g_return_val_if_fail (CALLS_IS_UI_CALL_DATA (self), NULL);
g_return_val_if_fail (!!self->call, NULL);
return calls_best_match_get_name (self->best_match);
if (self->best_match)
return calls_best_match_get_name (self->best_match);
else
return calls_call_get_name (self->call);
}
static const char *
@ -282,9 +285,22 @@ set_call_data (CallsUiCallData *self,
g_return_if_fail (CALLS_IS_UI_CALL_DATA (self));
g_return_if_fail (CALLS_IS_CALL (call));
self->call = call;
g_signal_connect_object (self->call,
"notify::state",
G_CALLBACK (on_notify_state),
self,
G_CONNECT_SWAPPED);
on_notify_state (self);
manager = calls_manager_get_default ();
contacts_provider = calls_manager_get_contacts_provider (manager);
/* the contacts provider should only be NULL when running the test suite */
if (!contacts_provider)
return;
self->best_match =
calls_contacts_provider_lookup_id (contacts_provider,
calls_call_get_id (call));
@ -304,13 +320,7 @@ set_call_data (CallsUiCallData *self,
G_CALLBACK (on_notify_avatar),
self,
G_CONNECT_SWAPPED);
self->call = call;
g_signal_connect_object (self->call,
"notify::state",
G_CALLBACK (on_notify_state),
self,
G_CONNECT_SWAPPED);
}
static void
@ -383,17 +393,17 @@ calls_ui_call_data_get_property (GObject *object,
}
static void
calls_ui_call_data_finalize (GObject *object)
calls_ui_call_data_dispose (GObject *object)
{
CallsUiCallData *self = CALLS_UI_CALL_DATA (object);
g_object_unref (self->call);
g_object_unref (self->best_match);
g_clear_object (&self->call);
g_clear_object (&self->best_match);
g_clear_handle_id (&self->timer_id, g_source_remove);
g_clear_pointer (&self->timer, g_timer_destroy);
G_OBJECT_CLASS (calls_ui_call_data_parent_class)->finalize (object);
G_OBJECT_CLASS (calls_ui_call_data_parent_class)->dispose (object);
}
static void
@ -408,7 +418,7 @@ calls_ui_call_data_class_init (CallsUiCallDataClass *klass)
object_class->set_property = calls_ui_call_data_set_property;
object_class->get_property = calls_ui_call_data_get_property;
object_class->finalize = calls_ui_call_data_finalize;
object_class->dispose = calls_ui_call_data_dispose;
props[PROP_CALL] =
g_param_spec_object ("call",