From e5742fd6b1bb84f0da1625c2300b25d369f4ba89 Mon Sep 17 00:00:00 2001
From: Frank LENORMAND <lenormf@gmail.com>
Date: Wed, 29 Nov 2017 20:17:45 +0300
Subject: [PATCH] - Fix: proper escaping of kak_{session, client}

---
 rc/extra/ranger.kak | 58 ++++++++++++++++++++++-----------------------
 1 file changed, 29 insertions(+), 29 deletions(-)

diff --git a/rc/extra/ranger.kak b/rc/extra/ranger.kak
index 48eb0c99..76e01453 100644
--- a/rc/extra/ranger.kak
+++ b/rc/extra/ranger.kak
@@ -4,31 +4,31 @@
 define-command ranger-open-on-edit-directory \
     -docstring 'Start the ranger file system explorer when trying to edit a directory' %{
         hook global RuntimeError "\d+:\d+: '\w+' (.*): is a directory" %{ %sh{
-          directory=$kak_hook_param_capture_1
-          echo ranger $directory
+            directory=$kak_hook_param_capture_1
+            echo ranger $directory
     }}
 }
 
 define-command \
-  -params .. -file-completion \
-  -docstring %{ranger [<arguments>]: open the file system explorer to select buffers to open
-All the optional arguments are forwarded to the ranger utility} \
-  ranger %{ %sh{
-  if [ -n "$TMUX" ]; then
-    tmux split-window -h \
-      ranger $@ --cmd " \
-        map <return> eval \
-          fm.execute_console('shell \
-            echo evaluate-commands -client $kak_client edit {file} | \
-            kak -p $kak_session; \
-            tmux select-pane -t $kak_client_env_TMUX_PANE'.format(file=fm.thisfile.path)) \
-          if fm.thisfile.is_file else fm.execute_console('move right=1')"
+    -params .. -file-completion \
+    -docstring %{ranger [<arguments>]: open the file system explorer to select buffers to open
+    All the optional arguments are forwarded to the ranger utility} \
+    ranger %{ %sh{
+    if [ -n "${TMUX}" ]; then
+        tmux split-window -h \
+            ranger $@ --cmd " \
+                map <return> eval \
+                    fm.execute_console('shell \
+                        echo evaluate-commands -client ' + ranger.ext.shell_escape.shell_escape('$kak_client') + ' edit {file} | \
+                        kak -p '.format(file=fm.thisfile.path) + ranger.ext.shell_escape.shell_escape('$kak_session') + '; \
+                        tmux select-pane -t $kak_client_env_TMUX_PANE') \
+                    if fm.thisfile.is_file else fm.execute_console('move right=1')"
 
-  elif [ -n "${STY}" ]; then
+    elif [ -n "${STY}" ]; then
 
-    script="/tmp/kak-ranger-${kak_client}-${kak_session}.sh"
-    selections="/tmp/kak-ranger-${kak_client}-${kak_session}.txt"
-    cat > "$script" << EOF
+        script="/tmp/kak-ranger-${kak_client}-${kak_session}.sh"
+        selections="/tmp/kak-ranger-${kak_client}-${kak_session}.txt"
+        cat > "$script" << EOF
 #! /usr/bin/env sh
 cd "$PWD"
 ranger --choosefiles="$selections" $@
@@ -46,14 +46,14 @@ EOF
             "screen sh '$script'" \
         < "/dev/$tty"
 
-  elif [ -n "$WINDOWID" ]; then
-    setsid $kak_opt_termcmd " \
-      ranger $@ --cmd "'"'" \
-        map <return> eval \
-          fm.execute_console('shell \
-            echo evaluate-commands -client $kak_client edit {file} | \
-            kak -p $kak_session; \
-            xdotool windowactivate $kak_client_env_WINDOWID'.format(file=fm.thisfile.path)) \
-          if fm.thisfile.is_file else fm.execute_console('move right=1')"'"' < /dev/null > /dev/null 2>&1 &
-  fi
+    elif [ -n "$WINDOWID" ]; then
+        setsid $kak_opt_termcmd " \
+            ranger $@ --cmd "'"'" \
+                map <return> eval \
+                    fm.execute_console('shell \
+                        echo evaluate-commands -client ' + ranger.ext.shell_escape.shell_escape('$kak_client') + ' edit {file} | \
+                        kak -p '.format(file=fm.thisfile.path) + ranger.ext.shell_escape.shell_escape('$kak_session') + '; \
+                        xdotool windowactivate $kak_client_env_WINDOWID') \
+                    if fm.thisfile.is_file else fm.execute_console('move right=1')"'"' < /dev/null > /dev/null 2>&1 &
+    fi
 }}