matteo
/
malcontent
1
0
Fork 0
Code Issues Pull Requests 2 Packages Projects Releases Wiki Activity

user-controls: Update when the ActUser changes 

When the user object changes one of its properties (for example, the
user’s locale might change if the administrator is editing a user in
g-c-c while viewing their parental controls in malcontent-control),
update the UI.

Signed-off-by: Philip Withnall <withnall@endlessm.com>
This commit is contained in:
Philip Withnall 2020-03-16 12:11:45 +00:00
parent 4698ad88a0
commit bf0eaea612
1 changed files with 25 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
libmalcontent-ui/user-controls.c
View File

@ -92,6 +92,7 @@ struct _MctUserControls
GSimpleActionGroup *action_group; /* (owned) */ GSimpleActionGroup *action_group; /* (owned) */
ActUser *user; /* (owned) (nullable) */ ActUser *user; /* (owned) (nullable) */
gulong user_changed_id;
GPermission *permission; /* (owned) (nullable) */ GPermission *permission; /* (owned) (nullable) */
gulong permission_allowed_id; gulong permission_allowed_id;
@ -727,6 +728,9 @@ mct_user_controls_finalize (GObject *object)
g_cancellable_cancel (self->cancellable); g_cancellable_cancel (self->cancellable);
g_clear_object (&self->action_group); g_clear_object (&self->action_group);
g_clear_object (&self->cancellable); g_clear_object (&self->cancellable);
if (self->user != NULL && self->user_changed_id != 0)
g_signal_handler_disconnect (self->user, self->user_changed_id);
self->user_changed_id = 0;
g_clear_object (&self->user); g_clear_object (&self->user);
g_clear_pointer (&self->user_locale, g_free); g_clear_pointer (&self->user_locale, g_free);
g_clear_pointer (&self->user_display_name, g_free); g_clear_pointer (&self->user_display_name, g_free);
@ -1054,6 +1058,17 @@ mct_user_controls_get_user (MctUserControls *self)
return self->user; return self->user;
} }
static void
user_changed_cb (ActUser *user,
gpointer user_data)
{
MctUserControls *self = MCT_USER_CONTROLS (user_data);
mct_user_controls_set_user_account_type (self, act_user_get_account_type (user));
mct_user_controls_set_user_locale (self, get_user_locale (user));
mct_user_controls_set_user_display_name (self, get_user_display_name (user));
}
/** /**
* mct_user_controls_set_user: * mct_user_controls_set_user:
* @self: an #MctUserControls * @self: an #MctUserControls
@ -1068,6 +1083,8 @@ void
mct_user_controls_set_user (MctUserControls *self, mct_user_controls_set_user (MctUserControls *self,
ActUser *user) ActUser *user)
{ {
g_autoptr(ActUser) old_user = NULL;
g_return_if_fail (MCT_IS_USER_CONTROLS (self)); g_return_if_fail (MCT_IS_USER_CONTROLS (self));
g_return_if_fail (user == NULL || ACT_IS_USER (user)); g_return_if_fail (user == NULL || ACT_IS_USER (user));
@ -1075,16 +1092,21 @@ mct_user_controls_set_user (MctUserControls *self,
* saved first. */ * saved first. */
flush_update_blacklisted_apps (self); flush_update_blacklisted_apps (self);
old_user = (self->user != NULL) ? g_object_ref (self->user) : NULL;
if (g_set_object (&self->user, user)) if (g_set_object (&self->user, user))
{ {
g_object_freeze_notify (G_OBJECT (self)); g_object_freeze_notify (G_OBJECT (self));
if (old_user != NULL)
g_signal_handler_disconnect (old_user, self->user_changed_id);
/* Update the starting widget state from the user. */ /* Update the starting widget state from the user. */
if (user != NULL) if (user != NULL)
{ {
mct_user_controls_set_user_account_type (self, act_user_get_account_type (user)); self->user_changed_id = g_signal_connect (user, "changed",
mct_user_controls_set_user_locale (self, get_user_locale (user)); (GCallback) user_changed_cb, self);
mct_user_controls_set_user_display_name (self, get_user_display_name (user)); user_changed_cb (user, self);
} }
update_app_filter_from_user (self); update_app_filter_from_user (self);