Adding functionality to set default terminal in cosmic_settings_config when changed

Author: ellieplayswow

cosmic-settings/src/pages/system/default_apps.rs

@@ -12,8 +12,11 @@ use cosmic::{
     widget::{self, dropdown, icon, settings},
     Apply, Element, Task,
 };
+use cosmic_config::{ConfigGet, ConfigSet};
+use cosmic_settings_config::shortcuts::action::System;
+use cosmic_settings_config::shortcuts::SystemActions;
 use cosmic_settings_page::{self as page, section, Section};
-use freedesktop_desktop_entry::{default_paths, DesktopEntry, Iter as DesktopEntryIter};
+use freedesktop_desktop_entry::{default_paths, get_languages_from_env, DesktopEntry, Iter as DesktopEntryIter};
 use mime_apps::App;
 use slotmap::SlotMap;
 use tokio::sync::mpsc;
@@ -81,6 +84,7 @@ pub struct AppMeta {
 pub struct Page {
     on_enter_handle: Option<cosmic::iced::task::Handle>,
     mime_apps: Option<CachedMimeApps>,
+    shortcuts_config: Option<cosmic_config::Config>
 }
 
 impl page::AutoBind<crate::pages::Message> for Page {}
@@ -107,6 +111,10 @@ impl page::Page<crate::pages::Message> for Page {
             handle.abort();
         }
 
+        if self.shortcuts_config.is_none() {
+            self.shortcuts_config = cosmic_settings_config::shortcuts::context().ok();
+        }
+
         let (task, on_enter_handle) = Task::future(async move {
             let mut list = mime_apps::List::default();
             list.load_from_paths(&mime_apps::list_paths());
@@ -239,6 +247,50 @@ impl Page {
                 if meta.selected != Some(id) {
                     meta.selected = Some(id);
                     let appid = &meta.app_ids[id];
+
+                    // if we're changing terminal, we want to update the COSMIC system actions shortcut
+                    // for terminal
+                    if category == Category::Terminal && self.shortcuts_config.is_some() {
+                        let shortcuts = self.shortcuts_config.as_ref().unwrap();
+                        let system_actions = shortcuts.get::<SystemActions>("system_actions");
+                        match system_actions {
+                            Err(e) => {
+                                // we don't have system_actions set?
+                                println!("{}", e);
+                            }
+                            Ok(mut actions) => {
+                                // get the current terminal option, which should default to cosmic-term
+                                let current_terminal = actions.get(&System::Terminal);
+                                if current_terminal.is_some() {
+                                    let default_paths = default_paths();
+                                    let mut resolved_path = None;
+
+                                    // loop through all FDE paths to try and find a valid .desktop file
+                                    for path in default_paths {
+                                        let full_path = path.join([appid, ".desktop"].concat());
+                                        if full_path.exists() && full_path.is_file() {
+                                            resolved_path = Some(full_path);
+                                        }
+                                    }
+
+                                    // if we find a valid .desktop file, we can grab its exec
+                                    if resolved_path.is_some() {
+                                        let desktop_entry = DesktopEntry::from_path(resolved_path.unwrap(), Some(&get_languages_from_env()));
+                                        if desktop_entry.is_ok() {
+                                            // grab the exec, add it to system_actions and commit
+                                            let exec = String::from(desktop_entry.unwrap().exec().unwrap());
+                                            actions.insert(System::Terminal, exec);
+
+                                            if let Err(e) = shortcuts.set("system_actions", actions) {
+                                                println!("{}", e);
+                                            }
+                                        }
+                                    }
+                                }
+                            }
+                        }
+                    }
+
                     for mime in mime_types {
                         if let Ok(mime) = mime.parse() {
                             mime_apps