check args are set on external execution

Previously it was possible to have {new_path} as
name of a new file because the argument wasn't filled.
Now the execution is prevented (as was the case in old
versions before a refactoring)

1 files changed, 8 insertions, 1 deletions

diff --git a/src/app/panel_state.rs b/src/app/panel_state.rs
index 63af80a..bae620f 100644
--- a/src/app/panel_state.rs
+++ b/src/app/panel_state.rs
@@ -461,9 +461,16 @@ pub trait PanelState {
         app_state: &mut AppState,
         cc: &CmdContext,
     ) -> Result<CmdResult, ProgramError> {
+        let sel_info = self.sel_info(app_state);
+        if let Some(invocation) = &invocation {
+            if let Some(error) = verb.check_args(&sel_info, &invocation, &app_state.other_panel_path) {
+                debug!("verb.check_args prevented execution: {:?}", &error);
+                return Ok(CmdResult::error(error));
+            }
+        }
         let exec_builder = ExecutionStringBuilder::with_invocation(
             &verb.invocation_parser,
-            self.sel_info(app_state),
+            sel_info,
             app_state,
             if let Some(inv) = invocation {
                 &inv.args