[gnome-shell] appDisplay: Avoid double slash when creating default folders



commit c5f6482705797f33f6411e4ee57755fa170eadeb
Author: Adam Williamson <awilliam redhat com>
Date:   Tue Mar 15 13:25:16 2022 -0700

    appDisplay: Avoid double slash when creating default folders
    
    Creating these default folders still doesn't work. After some
    investigation I found that's because the template we use for
    the path when creating the `child` `Gio.Settings` instance
    results in a double slash - it comes out as e.g.
    /org/gnome/desktop/app-folders//folders/Utilities/ . dconf does
    not gracefully handle this as many other things that handle
    paths do, it considers it a programmer error. It results in
    error messages like:
    
    dconf_changeset_set: assertion 'dconf_is_path (path, NULL)' failed
    
    which is slightly confusing. Anyway, we fix it by removing a
    slash from the template.
    
    Signed-off-by: Adam Williamson <awilliam redhat com>
    Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/2242>

 js/ui/appDisplay.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
---
diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index 116e9931d8..4fa2a5f631 100644
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -1471,7 +1471,7 @@ class AppDisplay extends BaseAppView {
             const { name, categories, apps } = DEFAULT_FOLDERS[folder];
             const child = new Gio.Settings({
                 schema_id: 'org.gnome.desktop.app-folders.folder',
-                path: `${path}/folders/${folder}/`,
+                path: `${path}folders/${folder}/`,
             });
             child.set_string('name', name);
             child.set_boolean('translate', true);


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]