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)

CHANGELOG.md

@@ -1,5 +1,6 @@
 ### next
 - hjson: fix bad parsing on tab alignement
+- now check all args of externals are set, doesn't use the raw {arg}
 
 <a name="v1.6.2"></a>
 ### v1.6.2 - 2021-07-31

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

src/shell_install/bash.rs

@@ -17,7 +17,6 @@ use {
     },
     directories::UserDirs,
     lazy_regex::regex,
-    minimad::*,
     regex::Captures,
     std::{env, fs::OpenOptions, io::Write, path::PathBuf},
     termimad::mad_print_inline,

src/verb/verb.rs

@@ -245,7 +245,6 @@ impl Verb {
                     return "You can't focus without selection".to_string();
                 }
             }
-            // TODO check that before
         }
 
         let builder = || {