From e4bf77f46267730d5c2e771b5dcef6a338eaf88e Mon Sep 17 00:00:00 2001 From: Jason Conti Date: Thu, 20 Aug 2020 09:32:00 +0200 Subject: Add blacklist hints for recent application names. --- src/indicator-notifications.c | 102 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 102 insertions(+) (limited to 'src/indicator-notifications.c') diff --git a/src/indicator-notifications.c b/src/indicator-notifications.c index 07ccd26..ab9e900 100644 --- a/src/indicator-notifications.c +++ b/src/indicator-notifications.c @@ -77,6 +77,8 @@ struct _IndicatorNotificationsPrivate { GHashTable *blacklist; + GList *blacklist_hints; + GSettings *settings; }; @@ -86,6 +88,8 @@ struct _IndicatorNotificationsPrivate { #define INDICATOR_ICON_READ "ayatana-indicator-notification-read" #define INDICATOR_ICON_UNREAD "ayatana-indicator-notification-unread" +#define HINT_MAX 10 + GType indicator_notifications_get_type(void); /* Indicator Class Functions */ @@ -111,6 +115,9 @@ static void set_unread(IndicatorNotifications *self, gboolean unread); static void update_blacklist(IndicatorNotifications *self); static void update_clear_item_markup(IndicatorNotifications *self); static void update_indicator_visibility(IndicatorNotifications *self); +static void load_blacklist_hints(IndicatorNotifications *self); +static void save_blacklist_hints(IndicatorNotifications *self); +static void update_blacklist_hints(IndicatorNotifications *self, Notification *notification); /* Callbacks */ static void clear_item_activated_cb(GtkMenuItem *menuitem, gpointer user_data); @@ -203,6 +210,10 @@ indicator_notifications_init(IndicatorNotifications *self) self->priv->max_items = g_settings_get_int(self->priv->settings, NOTIFICATIONS_KEY_MAX_ITEMS); update_blacklist(self); g_signal_connect(self->priv->settings, "changed", G_CALLBACK(setting_changed_cb), self); + + /* Set up blacklist hints */ + self->priv->blacklist_hints = NULL; + load_blacklist_hints(self); } static void @@ -245,6 +256,11 @@ indicator_notifications_dispose(GObject *object) self->priv->blacklist = NULL; } + if(self->priv->blacklist_hints != NULL) { + g_list_free_full(self->priv->blacklist_hints, g_free); + self->priv->blacklist_hints = NULL; + } + G_OBJECT_CLASS (indicator_notifications_parent_class)->dispose (object); return; } @@ -500,6 +516,88 @@ update_indicator_visibility(IndicatorNotifications *self) } } +/** + * load_blacklist_hints: + * @self: the indicator object + * + * Loads the blacklist hints from gsettings + **/ +static void +load_blacklist_hints(IndicatorNotifications *self) +{ + g_return_if_fail(IS_INDICATOR_NOTIFICATIONS(self)); + g_return_if_fail(self->priv->blacklist_hints == NULL); + + gchar **items = g_settings_get_strv(self->priv->settings, NOTIFICATIONS_KEY_BLACKLIST_HINTS); + int i; + + for (i = 0; items[i] != NULL; i++) { + self->priv->blacklist_hints = g_list_prepend(self->priv->blacklist_hints, items[i]); + } + + g_free(items); +} + +/** + * save_blacklist_hints: + * @self: the indicator object + * + * Saves the blacklist hints to gsettings + **/ +static void +save_blacklist_hints(IndicatorNotifications *self) +{ + g_return_if_fail(IS_INDICATOR_NOTIFICATIONS(self)); + + gchar *hints[HINT_MAX + 1]; + int i = 0; + + GList *l; + for (l = self->priv->blacklist_hints; (l != NULL) && (i < HINT_MAX); l = l->next, i++) { + hints[i] = (gchar *) l->data; + } + + hints[i] = NULL; + + g_settings_set_strv(self->priv->settings, NOTIFICATIONS_KEY_BLACKLIST_HINTS, (const gchar **) hints); +} + +/** + * update_blacklist_hints: + * @self: the indicator object + * + * Adds an application name to the hints + **/ +static void +update_blacklist_hints(IndicatorNotifications *self, Notification *notification) +{ + g_return_if_fail(IS_INDICATOR_NOTIFICATIONS(self)); + g_return_if_fail(IS_NOTIFICATION(notification)); + + const gchar *appname = notification_get_app_name(notification); + + /* Avoid duplicates */ + GList *l; + for (l = self->priv->blacklist_hints; l != NULL; l = l->next) { + if (g_strcmp0(appname, (const gchar *) l->data) == 0) + return; + } + + /* Add the appname */ + self->priv->blacklist_hints = g_list_prepend(self->priv->blacklist_hints, g_strdup(appname)); + + /* Keep only a reasonable number */ + while (g_list_length(self->priv->blacklist_hints) > HINT_MAX) { + GList *last = g_list_last(self->priv->blacklist_hints); + g_free(last->data); + self->priv->blacklist_hints = g_list_delete_link(self->priv->blacklist_hints, last); + } + + /* Save the hints */ + /* FIXME: maybe don't do this every update */ + save_blacklist_hints(self); +} + /** * clear_item_activated_cb: * @menuitem: the clear menuitem @@ -625,6 +723,10 @@ message_received_cb(DBusSpy *spy, Notification *note, gpointer user_data) return; } + /* Save a hint for the appname */ + update_blacklist_hints(self, note); + + /* Create the menuitem */ GtkWidget *item = notification_menuitem_new(); notification_menuitem_set_from_notification(NOTIFICATION_MENUITEM(item), note); g_signal_connect(item, NOTIFICATION_MENUITEM_SIGNAL_CLICKED, G_CALLBACK(notification_clicked_cb), self); -- cgit v1.2.3 From 82ca9185e9894121cc1d13bb06e7dc0e09dfd01b Mon Sep 17 00:00:00 2001 From: Jason Conti Date: Thu, 20 Aug 2020 09:35:44 +0200 Subject: Add our own do-not-disturb setting. * Remove the MATE specific code from indicator-notifications-settings. * When do-not-disturb is set the indicator will try to set do-not-disturb on all supported desktops (currently only MATE). --- src/indicator-notifications.c | 69 +++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 69 insertions(+) (limited to 'src/indicator-notifications.c') diff --git a/src/indicator-notifications.c b/src/indicator-notifications.c index ab9e900..ef8aff4 100644 --- a/src/indicator-notifications.c +++ b/src/indicator-notifications.c @@ -61,6 +61,7 @@ struct _IndicatorNotificationsPrivate { GList *hidden_items; gboolean clear_on_middle_click; + gboolean do_not_disturb; gboolean have_unread; gboolean hide_indicator; @@ -118,6 +119,8 @@ static void update_indicator_visibility(IndicatorNotifications *self); static void load_blacklist_hints(IndicatorNotifications *self); static void save_blacklist_hints(IndicatorNotifications *self); static void update_blacklist_hints(IndicatorNotifications *self, Notification *notification); +static void update_do_not_disturb(IndicatorNotifications *self); +static void settings_try_set_boolean(const gchar *schema, const gchar *key, gboolean value); /* Callbacks */ static void clear_item_activated_cb(GtkMenuItem *menuitem, gpointer user_data); @@ -206,9 +209,11 @@ indicator_notifications_init(IndicatorNotifications *self) /* Connect to GSettings */ self->priv->settings = g_settings_new(NOTIFICATIONS_SCHEMA); self->priv->clear_on_middle_click = g_settings_get_boolean(self->priv->settings, NOTIFICATIONS_KEY_CLEAR_MC); + self->priv->do_not_disturb = g_settings_get_boolean(self->priv->settings, NOTIFICATIONS_KEY_DND); self->priv->hide_indicator = g_settings_get_boolean(self->priv->settings, NOTIFICATIONS_KEY_HIDE_INDICATOR); self->priv->max_items = g_settings_get_int(self->priv->settings, NOTIFICATIONS_KEY_MAX_ITEMS); update_blacklist(self); + update_do_not_disturb(self); g_signal_connect(self->priv->settings, "changed", G_CALLBACK(setting_changed_cb), self); /* Set up blacklist hints */ @@ -598,6 +603,66 @@ update_blacklist_hints(IndicatorNotifications *self, Notification *notification) save_blacklist_hints(self); } +/** + * update_do_not_disturb: + * @self: the indicator object + * + * Updates the icon with the do-not-disturb version and sets do-not-disturb options + * on external notification daemons that are supported. + **/ +static void +update_do_not_disturb(IndicatorNotifications *self) +{ + g_return_if_fail(IS_INDICATOR_NOTIFICATIONS(self)); + + /* TODO: update icons here */ + + /* Mate do-not-disturb support */ + settings_try_set_boolean(MATE_SCHEMA, MATE_KEY_DND, self->priv->do_not_disturb); +} + +/** + * settings_try_set_boolean: + * @schema: the GSettings schema + * @key: the GSettings key + * @value: the boolean value + * + * Checks to see if the schema and key exist before setting the value. + */ +static void +settings_try_set_boolean(const gchar *schema, const gchar *key, gboolean value) +{ + /* Check if we can access the schema */ + GSettingsSchemaSource *source = g_settings_schema_source_get_default(); + if (source == NULL) { + return; + } + + /* Lookup the schema */ + GSettingsSchema *source_schema = g_settings_schema_source_lookup(source, schema, FALSE); + + /* Couldn't find the schema */ + if (source_schema == NULL) { + return; + } + + /* Found the schema, make sure we have the key */ + if (g_settings_schema_has_key(source_schema, key)) { + /* Make sure the key is of boolean type */ + GSettingsSchemaKey *source_key = g_settings_schema_get_key(source_schema, key); + + if (g_variant_type_equal(g_settings_schema_key_get_value_type(source_key), G_VARIANT_TYPE_BOOLEAN)) { + /* Set the value */ + GSettings *settings = g_settings_new(schema); + g_settings_set_boolean(settings, key, value); + g_object_unref(settings); + } + + g_settings_schema_key_unref(source_key); + } + g_settings_schema_unref(source_schema); +} + /** * clear_item_activated_cb: * @menuitem: the clear menuitem @@ -662,6 +727,10 @@ setting_changed_cb(GSettings *settings, gchar *key, gpointer user_data) self->priv->hide_indicator = g_settings_get_boolean(settings, NOTIFICATIONS_KEY_HIDE_INDICATOR); update_indicator_visibility(self); } + else if(g_strcmp0(key, NOTIFICATIONS_KEY_DND) == 0) { + self->priv->do_not_disturb = g_settings_get_boolean(settings, NOTIFICATIONS_KEY_DND); + update_do_not_disturb(self); + } else if(g_strcmp0(key, NOTIFICATIONS_KEY_CLEAR_MC) == 0) { self->priv->clear_on_middle_click = g_settings_get_boolean(self->priv->settings, NOTIFICATIONS_KEY_CLEAR_MC); } -- cgit v1.2.3 From 8f195248f8b7ca0cc6c87c02188c4ed360964143 Mon Sep 17 00:00:00 2001 From: Jason Conti Date: Thu, 20 Aug 2020 09:38:41 +0200 Subject: Add do-not-disturb mode read/unread icons. --- src/indicator-notifications.c | 46 +++++++++++++++++++++++++++++++++---------- 1 file changed, 36 insertions(+), 10 deletions(-) (limited to 'src/indicator-notifications.c') diff --git a/src/indicator-notifications.c b/src/indicator-notifications.c index ef8aff4..db53df7 100644 --- a/src/indicator-notifications.c +++ b/src/indicator-notifications.c @@ -86,8 +86,10 @@ struct _IndicatorNotificationsPrivate { #include "settings.h" #define INDICATOR_ICON_SIZE 22 -#define INDICATOR_ICON_READ "ayatana-indicator-notification-read" -#define INDICATOR_ICON_UNREAD "ayatana-indicator-notification-unread" +#define INDICATOR_ICON_READ "ayatana-indicator-notification-read" +#define INDICATOR_ICON_UNREAD "ayatana-indicator-notification-unread" +#define INDICATOR_ICON_READ_DND "ayatana-indicator-notification-read-dnd" +#define INDICATOR_ICON_UNREAD_DND "ayatana-indicator-notification-unread-dnd" #define HINT_MAX 10 @@ -113,6 +115,7 @@ static void clear_menuitems(IndicatorNotifications *self); static void insert_menuitem(IndicatorNotifications *self, GtkWidget *item); static void remove_menuitem(IndicatorNotifications *self, GtkWidget *item); static void set_unread(IndicatorNotifications *self, gboolean unread); +static void update_unread(IndicatorNotifications *self); static void update_blacklist(IndicatorNotifications *self); static void update_clear_item_markup(IndicatorNotifications *self); static void update_indicator_visibility(IndicatorNotifications *self); @@ -213,7 +216,6 @@ indicator_notifications_init(IndicatorNotifications *self) self->priv->hide_indicator = g_settings_get_boolean(self->priv->settings, NOTIFICATIONS_KEY_HIDE_INDICATOR); self->priv->max_items = g_settings_get_int(self->priv->settings, NOTIFICATIONS_KEY_MAX_ITEMS); update_blacklist(self); - update_do_not_disturb(self); g_signal_connect(self->priv->settings, "changed", G_CALLBACK(setting_changed_cb), self); /* Set up blacklist hints */ @@ -284,7 +286,8 @@ get_image(IndicatorObject *io) if(self->priv->image == NULL) { self->priv->image = GTK_IMAGE(gtk_image_new()); - set_unread(self, FALSE); + /* We have to wait until the image is created to update do-not-disturb the first time */ + update_do_not_disturb(self); update_indicator_visibility(self); } @@ -438,14 +441,37 @@ set_unread(IndicatorNotifications *self, gboolean unread) { g_return_if_fail(IS_INDICATOR_NOTIFICATIONS(self)); - if(unread) { - gtk_image_set_from_icon_name(self->priv->image, INDICATOR_ICON_UNREAD, GTK_ICON_SIZE_MENU); + self->priv->have_unread = unread; + update_unread(self); +} + +/** + * update_unread: + * @self: the indicator object + * + * Updates the indicator icons. + **/ +static void +update_unread(IndicatorNotifications *self) +{ + g_return_if_fail(IS_INDICATOR_NOTIFICATIONS(self)); + + if(self->priv->have_unread) { + if (self->priv->do_not_disturb) { + gtk_image_set_from_icon_name(self->priv->image, INDICATOR_ICON_UNREAD_DND, GTK_ICON_SIZE_MENU); + } + else { + gtk_image_set_from_icon_name(self->priv->image, INDICATOR_ICON_UNREAD, GTK_ICON_SIZE_MENU); + } } else { - gtk_image_set_from_icon_name(self->priv->image, INDICATOR_ICON_READ, GTK_ICON_SIZE_MENU); + if (self->priv->do_not_disturb) { + gtk_image_set_from_icon_name(self->priv->image, INDICATOR_ICON_READ_DND, GTK_ICON_SIZE_MENU); + } + else { + gtk_image_set_from_icon_name(self->priv->image, INDICATOR_ICON_READ, GTK_ICON_SIZE_MENU); + } } - - self->priv->have_unread = unread; } /** @@ -615,7 +641,7 @@ update_do_not_disturb(IndicatorNotifications *self) { g_return_if_fail(IS_INDICATOR_NOTIFICATIONS(self)); - /* TODO: update icons here */ + update_unread(self); /* Mate do-not-disturb support */ settings_try_set_boolean(MATE_SCHEMA, MATE_KEY_DND, self->priv->do_not_disturb); -- cgit v1.2.3 From d1b76b9aa5330cc27a0f3ecefc57becfbe8ea02e Mon Sep 17 00:00:00 2001 From: Jason Conti Date: Thu, 5 Sep 2019 09:23:54 -0400 Subject: Discard notifications when indicator is hidden. --- src/indicator-notifications.c | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'src/indicator-notifications.c') diff --git a/src/indicator-notifications.c b/src/indicator-notifications.c index db53df7..2ab4083 100644 --- a/src/indicator-notifications.c +++ b/src/indicator-notifications.c @@ -805,6 +805,12 @@ message_received_cb(DBusSpy *spy, Notification *note, gpointer user_data) g_return_if_fail(IS_INDICATOR_NOTIFICATIONS(user_data)); IndicatorNotifications *self = INDICATOR_NOTIFICATIONS(user_data); + /* Discard notifications if we are hidden */ + if(self->priv->hide_indicator) { + g_object_unref(note); + return; + } + /* Discard useless notifications */ if(notification_is_private(note) || notification_is_empty(note)) { g_object_unref(note); -- cgit v1.2.3 From 3fbc05dc4cbf564644f2500508e642eb80f07264 Mon Sep 17 00:00:00 2001 From: Tasos Sahanidis Date: Thu, 20 Aug 2020 09:41:52 +0200 Subject: Add option to swap Settings and Clear. --- src/indicator-notifications.c | 30 ++++++++++++++++++++++++++++++ 1 file changed, 30 insertions(+) (limited to 'src/indicator-notifications.c') diff --git a/src/indicator-notifications.c b/src/indicator-notifications.c index 2ab4083..67dcb96 100644 --- a/src/indicator-notifications.c +++ b/src/indicator-notifications.c @@ -64,6 +64,7 @@ struct _IndicatorNotificationsPrivate { gboolean do_not_disturb; gboolean have_unread; gboolean hide_indicator; + gboolean swap_clear_settings; gint max_items; @@ -124,6 +125,7 @@ static void save_blacklist_hints(IndicatorNotifications *self); static void update_blacklist_hints(IndicatorNotifications *self, Notification *notification); static void update_do_not_disturb(IndicatorNotifications *self); static void settings_try_set_boolean(const gchar *schema, const gchar *key, gboolean value); +static void swap_clear_settings_items(IndicatorNotifications *self); /* Callbacks */ static void clear_item_activated_cb(GtkMenuItem *menuitem, gpointer user_data); @@ -216,6 +218,9 @@ indicator_notifications_init(IndicatorNotifications *self) self->priv->hide_indicator = g_settings_get_boolean(self->priv->settings, NOTIFICATIONS_KEY_HIDE_INDICATOR); self->priv->max_items = g_settings_get_int(self->priv->settings, NOTIFICATIONS_KEY_MAX_ITEMS); update_blacklist(self); + self->priv->swap_clear_settings = g_settings_get_boolean(self->priv->settings, NOTIFICATIONS_KEY_SWAP_CLEAR_SETTINGS); + if(self->priv->swap_clear_settings) + swap_clear_settings_items(self); g_signal_connect(self->priv->settings, "changed", G_CALLBACK(setting_changed_cb), self); /* Set up blacklist hints */ @@ -689,6 +694,27 @@ settings_try_set_boolean(const gchar *schema, const gchar *key, gboolean value) g_settings_schema_unref(source_schema); } +/** + * swap_clear_settings_items: + * @self: the indicator object + * + * Swaps the position of the clear and settings items. + **/ +static void +swap_clear_settings_items(IndicatorNotifications *self) +{ + g_return_if_fail(IS_INDICATOR_NOTIFICATIONS(self)); + + // Pick which widget to move + GtkWidget *widget = self->priv->settings_item; + if(self->priv->swap_clear_settings) + widget = self->priv->clear_item; + + gtk_container_remove(GTK_CONTAINER(self->priv->menu), g_object_ref(widget)); + gtk_menu_shell_append(GTK_MENU_SHELL(self->priv->menu), widget); + g_object_unref(widget); +} + /** * clear_item_activated_cb: * @menuitem: the clear menuitem @@ -763,6 +789,10 @@ setting_changed_cb(GSettings *settings, gchar *key, gpointer user_data) else if(g_strcmp0(key, NOTIFICATIONS_KEY_BLACKLIST) == 0) { update_blacklist(self); } + else if(g_strcmp0(key, NOTIFICATIONS_KEY_SWAP_CLEAR_SETTINGS) == 0) { + self->priv->swap_clear_settings = g_settings_get_boolean(self->priv->settings, NOTIFICATIONS_KEY_SWAP_CLEAR_SETTINGS); + swap_clear_settings_items(self); + } /* TODO: Trim or extend the notifications list based on "max-items" key * (Currently requires a restart) */ } -- cgit v1.2.3 From aca92b39f32f0d850f5a74b79591a587273c2168 Mon Sep 17 00:00:00 2001 From: Jason Conti Date: Thu, 20 Aug 2020 09:45:37 +0200 Subject: Rename blacklist to filter-list. --- src/indicator-notifications.c | 98 +++++++++++++++++++++---------------------- 1 file changed, 49 insertions(+), 49 deletions(-) (limited to 'src/indicator-notifications.c') diff --git a/src/indicator-notifications.c b/src/indicator-notifications.c index 67dcb96..e5f2951 100644 --- a/src/indicator-notifications.c +++ b/src/indicator-notifications.c @@ -77,9 +77,9 @@ struct _IndicatorNotificationsPrivate { DBusSpy *spy; - GHashTable *blacklist; + GHashTable *filter_list; - GList *blacklist_hints; + GList *filter_list_hints; GSettings *settings; }; @@ -117,12 +117,12 @@ static void insert_menuitem(IndicatorNotifications *self, GtkWidget *item); static void remove_menuitem(IndicatorNotifications *self, GtkWidget *item); static void set_unread(IndicatorNotifications *self, gboolean unread); static void update_unread(IndicatorNotifications *self); -static void update_blacklist(IndicatorNotifications *self); +static void update_filter_list(IndicatorNotifications *self); static void update_clear_item_markup(IndicatorNotifications *self); static void update_indicator_visibility(IndicatorNotifications *self); -static void load_blacklist_hints(IndicatorNotifications *self); -static void save_blacklist_hints(IndicatorNotifications *self); -static void update_blacklist_hints(IndicatorNotifications *self, Notification *notification); +static void load_filter_list_hints(IndicatorNotifications *self); +static void save_filter_list_hints(IndicatorNotifications *self); +static void update_filter_list_hints(IndicatorNotifications *self, Notification *notification); static void update_do_not_disturb(IndicatorNotifications *self); static void settings_try_set_boolean(const gchar *schema, const gchar *key, gboolean value); static void swap_clear_settings_items(IndicatorNotifications *self); @@ -208,8 +208,8 @@ indicator_notifications_init(IndicatorNotifications *self) self->priv->spy = dbus_spy_new(); g_signal_connect(self->priv->spy, DBUS_SPY_SIGNAL_MESSAGE_RECEIVED, G_CALLBACK(message_received_cb), self); - /* Initialize an empty blacklist */ - self->priv->blacklist = g_hash_table_new_full(g_str_hash, g_str_equal, g_free, NULL); + /* Initialize an empty filter list */ + self->priv->filter_list = g_hash_table_new_full(g_str_hash, g_str_equal, g_free, NULL); /* Connect to GSettings */ self->priv->settings = g_settings_new(NOTIFICATIONS_SCHEMA); @@ -217,15 +217,15 @@ indicator_notifications_init(IndicatorNotifications *self) self->priv->do_not_disturb = g_settings_get_boolean(self->priv->settings, NOTIFICATIONS_KEY_DND); self->priv->hide_indicator = g_settings_get_boolean(self->priv->settings, NOTIFICATIONS_KEY_HIDE_INDICATOR); self->priv->max_items = g_settings_get_int(self->priv->settings, NOTIFICATIONS_KEY_MAX_ITEMS); - update_blacklist(self); + update_filter_list(self); self->priv->swap_clear_settings = g_settings_get_boolean(self->priv->settings, NOTIFICATIONS_KEY_SWAP_CLEAR_SETTINGS); if(self->priv->swap_clear_settings) swap_clear_settings_items(self); g_signal_connect(self->priv->settings, "changed", G_CALLBACK(setting_changed_cb), self); - /* Set up blacklist hints */ - self->priv->blacklist_hints = NULL; - load_blacklist_hints(self); + /* Set up filter-list hints */ + self->priv->filter_list_hints = NULL; + load_filter_list_hints(self); } static void @@ -263,14 +263,14 @@ indicator_notifications_dispose(GObject *object) self->priv->settings = NULL; } - if(self->priv->blacklist != NULL) { - g_hash_table_unref(self->priv->blacklist); - self->priv->blacklist = NULL; + if(self->priv->filter_list != NULL) { + g_hash_table_unref(self->priv->filter_list); + self->priv->filter_list = NULL; } - if(self->priv->blacklist_hints != NULL) { - g_list_free_full(self->priv->blacklist_hints, g_free); - self->priv->blacklist_hints = NULL; + if(self->priv->filter_list_hints != NULL) { + g_list_free_full(self->priv->filter_list_hints, g_free); + self->priv->filter_list_hints = NULL; } G_OBJECT_CLASS (indicator_notifications_parent_class)->dispose (object); @@ -480,24 +480,24 @@ update_unread(IndicatorNotifications *self) } /** - * update_blacklist: + * update_filter_list: * @self: the indicator object * - * Updates the blacklist from GSettings. This currently does not filter already + * Updates the filter list from GSettings. This currently does not filter already * allowed messages. It only applies to messages received in the future. **/ static void -update_blacklist(IndicatorNotifications *self) +update_filter_list(IndicatorNotifications *self) { g_return_if_fail(IS_INDICATOR_NOTIFICATIONS(self)); - g_return_if_fail(self->priv->blacklist != NULL); + g_return_if_fail(self->priv->filter_list != NULL); - g_hash_table_remove_all(self->priv->blacklist); - gchar **items = g_settings_get_strv(self->priv->settings, NOTIFICATIONS_KEY_BLACKLIST); + g_hash_table_remove_all(self->priv->filter_list); + gchar **items = g_settings_get_strv(self->priv->settings, NOTIFICATIONS_KEY_FILTER_LIST); int i; for(i = 0; items[i] != NULL; i++) { - g_hash_table_insert(self->priv->blacklist, g_strdup(items[i]), NULL); + g_hash_table_insert(self->priv->filter_list, g_strdup(items[i]), NULL); } g_strfreev(items); @@ -553,35 +553,35 @@ update_indicator_visibility(IndicatorNotifications *self) } /** - * load_blacklist_hints: + * load_filter_list_hints: * @self: the indicator object * - * Loads the blacklist hints from gsettings + * Loads the filter list hints from gsettings **/ static void -load_blacklist_hints(IndicatorNotifications *self) +load_filter_list_hints(IndicatorNotifications *self) { g_return_if_fail(IS_INDICATOR_NOTIFICATIONS(self)); - g_return_if_fail(self->priv->blacklist_hints == NULL); + g_return_if_fail(self->priv->filter_list_hints == NULL); - gchar **items = g_settings_get_strv(self->priv->settings, NOTIFICATIONS_KEY_BLACKLIST_HINTS); + gchar **items = g_settings_get_strv(self->priv->settings, NOTIFICATIONS_KEY_FILTER_LIST_HINTS); int i; for (i = 0; items[i] != NULL; i++) { - self->priv->blacklist_hints = g_list_prepend(self->priv->blacklist_hints, items[i]); + self->priv->filter_list_hints = g_list_prepend(self->priv->filter_list_hints, items[i]); } g_free(items); } /** - * save_blacklist_hints: + * save_filter_list_hints: * @self: the indicator object * - * Saves the blacklist hints to gsettings + * Saves the filter list hints to gsettings **/ static void -save_blacklist_hints(IndicatorNotifications *self) +save_filter_list_hints(IndicatorNotifications *self) { g_return_if_fail(IS_INDICATOR_NOTIFICATIONS(self)); @@ -589,23 +589,23 @@ save_blacklist_hints(IndicatorNotifications *self) int i = 0; GList *l; - for (l = self->priv->blacklist_hints; (l != NULL) && (i < HINT_MAX); l = l->next, i++) { + for (l = self->priv->filter_list_hints; (l != NULL) && (i < HINT_MAX); l = l->next, i++) { hints[i] = (gchar *) l->data; } hints[i] = NULL; - g_settings_set_strv(self->priv->settings, NOTIFICATIONS_KEY_BLACKLIST_HINTS, (const gchar **) hints); + g_settings_set_strv(self->priv->settings, NOTIFICATIONS_KEY_FILTER_LIST_HINTS, (const gchar **) hints); } /** - * update_blacklist_hints: + * update_filter_list_hints: * @self: the indicator object * * Adds an application name to the hints **/ static void -update_blacklist_hints(IndicatorNotifications *self, Notification *notification) +update_filter_list_hints(IndicatorNotifications *self, Notification *notification) { g_return_if_fail(IS_INDICATOR_NOTIFICATIONS(self)); g_return_if_fail(IS_NOTIFICATION(notification)); @@ -614,24 +614,24 @@ update_blacklist_hints(IndicatorNotifications *self, Notification *notification) /* Avoid duplicates */ GList *l; - for (l = self->priv->blacklist_hints; l != NULL; l = l->next) { + for (l = self->priv->filter_list_hints; l != NULL; l = l->next) { if (g_strcmp0(appname, (const gchar *) l->data) == 0) return; } /* Add the appname */ - self->priv->blacklist_hints = g_list_prepend(self->priv->blacklist_hints, g_strdup(appname)); + self->priv->filter_list_hints = g_list_prepend(self->priv->filter_list_hints, g_strdup(appname)); /* Keep only a reasonable number */ - while (g_list_length(self->priv->blacklist_hints) > HINT_MAX) { - GList *last = g_list_last(self->priv->blacklist_hints); + while (g_list_length(self->priv->filter_list_hints) > HINT_MAX) { + GList *last = g_list_last(self->priv->filter_list_hints); g_free(last->data); - self->priv->blacklist_hints = g_list_delete_link(self->priv->blacklist_hints, last); + self->priv->filter_list_hints = g_list_delete_link(self->priv->filter_list_hints, last); } /* Save the hints */ /* FIXME: maybe don't do this every update */ - save_blacklist_hints(self); + save_filter_list_hints(self); } /** @@ -786,8 +786,8 @@ setting_changed_cb(GSettings *settings, gchar *key, gpointer user_data) else if(g_strcmp0(key, NOTIFICATIONS_KEY_CLEAR_MC) == 0) { self->priv->clear_on_middle_click = g_settings_get_boolean(self->priv->settings, NOTIFICATIONS_KEY_CLEAR_MC); } - else if(g_strcmp0(key, NOTIFICATIONS_KEY_BLACKLIST) == 0) { - update_blacklist(self); + else if(g_strcmp0(key, NOTIFICATIONS_KEY_FILTER_LIST) == 0) { + update_filter_list(self); } else if(g_strcmp0(key, NOTIFICATIONS_KEY_SWAP_CLEAR_SETTINGS) == 0) { self->priv->swap_clear_settings = g_settings_get_boolean(self->priv->settings, NOTIFICATIONS_KEY_SWAP_CLEAR_SETTINGS); @@ -847,15 +847,15 @@ message_received_cb(DBusSpy *spy, Notification *note, gpointer user_data) return; } - /* Discard notifications on the blacklist */ - if(self->priv->blacklist != NULL && g_hash_table_contains(self->priv->blacklist, + /* Discard notifications on the filter list */ + if(self->priv->filter_list != NULL && g_hash_table_contains(self->priv->filter_list, notification_get_app_name(note))) { g_object_unref(note); return; } /* Save a hint for the appname */ - update_blacklist_hints(self, note); + update_filter_list_hints(self, note); /* Create the menuitem */ GtkWidget *item = notification_menuitem_new(); -- cgit v1.2.3 From ab788386e4d79b0199741388e55d8a4032f6e9cc Mon Sep 17 00:00:00 2001 From: Jason Conti Date: Fri, 21 Aug 2020 14:58:52 +0200 Subject: src/indicator-notifications: settings_try_set_boolean(): Enable the recursive schema lookup so we have a better chance of finding it. --- src/indicator-notifications.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/indicator-notifications.c') diff --git a/src/indicator-notifications.c b/src/indicator-notifications.c index e5f2951..56ddd7a 100644 --- a/src/indicator-notifications.c +++ b/src/indicator-notifications.c @@ -670,7 +670,7 @@ settings_try_set_boolean(const gchar *schema, const gchar *key, gboolean value) } /* Lookup the schema */ - GSettingsSchema *source_schema = g_settings_schema_source_lookup(source, schema, FALSE); + GSettingsSchema *source_schema = g_settings_schema_source_lookup(source, schema, TRUE); /* Couldn't find the schema */ if (source_schema == NULL) { -- cgit v1.2.3