From 54e9be645961ea9838f809aaf3d8cef533c5d5ee Mon Sep 17 00:00:00 2001
From: Francesco Pio Gaglione <francesco.gaglione.p@gmail.com>
Date: Thu, 3 Oct 2024 18:23:50 +0200
Subject: [PATCH] permission dialog

---
 i18n/en/cosmic_files.ftl |  5 ++++
 src/app.rs               | 55 +++++++++++++++++++++++++++++++++++-----
 2 files changed, 53 insertions(+), 7 deletions(-)

diff --git a/i18n/en/cosmic_files.ftl b/i18n/en/cosmic_files.ftl
index e7579d1..e0f403b 100644
--- a/i18n/en/cosmic_files.ftl
+++ b/i18n/en/cosmic_files.ftl
@@ -66,6 +66,11 @@ apply-to-all = Apply to all
 keep-both = Keep both
 skip = Skip
 
+## Execution permission Dialog
+add-permission-title = Add execution permission
+add-permission-control = Add execution permission to {$path}
+add-permission = Add permission
+
 ## Metadata Dialog
 owner = Owner
 group = Group
diff --git a/src/app.rs b/src/app.rs
index 5bf7daa..45b42c8 100644
--- a/src/app.rs
+++ b/src/app.rs
@@ -256,6 +256,7 @@ impl MenuAction for NavMenuAction {
 pub enum Message {
     AddToSidebar(Option<Entity>),
     AppTheme(AppTheme),
+    AddExecutablePermission(PathBuf),
     CloseToast(widget::ToastId),
     Compress(Option<Entity>),
     Config(Config),
@@ -416,6 +417,10 @@ pub enum DialogPage {
         name: String,
         dir: bool,
     },
+    AddExecutablePermission {
+        file_path: PathBuf,
+        run: bool,
+    },
     Replace {
         from: tab::Item,
         to: tab::Item,
@@ -1546,12 +1551,31 @@ impl Application for App {
                             let to = parent.join(name);
                             self.operation(Operation::Rename { from, to });
                         }
+                        DialogPage::AddExecutablePermission { file_path, run } => {
+                            let mut perms = fs::metadata(&file_path)
+                                .expect("Failed to get metadata")
+                                .permissions();
+
+                            let current_mode = perms.mode();
+                            let new_mode = current_mode | 0o100;
+                            perms.set_mode(new_mode);
+                            fs::set_permissions(&file_path, perms)
+                                .expect("Failed to set permissions");
+
+                            if run {
+                                log::info!("running app: {:?}", file_path);
+                                let _ = std::process::Command::new(file_path).spawn();
+                            }
+                        }
                         DialogPage::Replace { .. } => {
                             log::warn!("replace dialog should be completed with replace result");
                         }
                     }
                 }
             }
+            Message::AddExecutablePermission(file_path) => {
+                return Command::batch([self.update(Message::AddExecutablePermission(file_path))]);
+            }
             Message::DialogPush(dialog_page) => {
                 self.dialog_pages.push_back(dialog_page);
             }
@@ -2357,17 +2381,16 @@ impl Application for App {
                                 let file_extension = path.extension();
                                 match file_extension {
                                     Some(ext) if ext == OsStr::new("AppImage") => {
-                                        // Set the executable permission to the file
                                         let mut perms = fs::metadata(&path)
                                             .expect("Failed to get metadata")
                                             .permissions();
-                                        // Set the executable permission to the file
-                                        perms.set_mode(0o755);
-                                        fs::set_permissions(&path, perms)
-                                            .expect("Failed to set permissions");
 
-                                        log::info!("running app: {:?}", ext);
-                                        let _ = std::process::Command::new(path).spawn();
+                                        self.dialog_pages.push_back(
+                                            DialogPage::AddExecutablePermission {
+                                                file_path: path.clone(),
+                                                run: true,
+                                            },
+                                        );
                                     }
                                     _ => match open::that_detached(&path) {
                                         Ok(()) => {
@@ -3109,6 +3132,24 @@ impl Application for App {
                         .spacing(space_xxs),
                     )
             }
+            DialogPage::AddExecutablePermission { file_path, run } => {
+                let mut dialog = widget::dialog(fl!("add-permission-title"))
+                    .primary_action(
+                        widget::button::text(fl!("add-permission"))
+                            .style(theme::Button::Suggested)
+                            .on_press(Message::DialogComplete),
+                    )
+                    .secondary_action(
+                        widget::button::text("cancell")
+                            .style(theme::Button::Destructive)
+                            .on_press(Message::DialogCancel),
+                    )
+                    .control(widget::column().push(widget::text::text(fl!(
+                        "add-permission-control",
+                        path = file_path.as_os_str().to_str()
+                    ))));
+                dialog
+            }
             DialogPage::RenameItem {
                 from,
                 parent,