Fix UCA mistake and qvm-actions script

QubesOS · Dec 12, 2017 · c34a0a9 · c34a0a9
1 parent 6226531
commit c34a0a9
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 15 deletions.
diff --git a/misc/uca_qubes.xml b/misc/uca_qubes.xml
@@ -43,7 +43,7 @@
 	<icon>gtk-convert</icon>
 	<name>Convert in DisposableVM</name>
 	<unique-id>1507455488971315-6</unique-id>
-	<command>/usr/lib/qubes/qvm-actions.sh p %F</command>
+	<command>/usr/lib/qubes/qvm-actions.sh pdf %F</command>
 	<description></description>
 	<patterns>*.pdf</patterns>
 	<other-files/>

diff --git a/qubes-rpc/qvm-actions.sh b/qubes-rpc/qvm-actions.sh
@@ -1,50 +1,48 @@
 #!/bin/bash
 
-# Allow us to handle filenames with spaces using newline separator from Thunar
-IFS='
-'
-
 # Check if at least two arguments are provided: actions + file(s)
 if [ "$#" -le 1 ]; then
     echo "Not enough arguments provided. Aborting..."
 fi
 
-# File(s)
-files=${*:2}
+# Action
+action="$1"
+
+shift
 
 # copy and move handle a list of files where other actions don't
-case $1 in
+case "$action" in
     copy)
         #shellcheck disable=SC2016
-        qvm-copy-to-vm '$default' "$files" | zenity --notification --text="Copying files..." --timeout 3
+        qvm-copy-to-vm '$default' "$@" | zenity --notification --text="Copying files..." --timeout 3
         ;;
     move)
         #shellcheck disable=SC2016
-        qvm-move-to-vm '$default' "$files" | zenity --notification --text="Moving files..." --timeout 3
+        qvm-move-to-vm '$default' "$@" | zenity --notification --text="Moving files..." --timeout 3
         ;;
     img)
-        for file in $files
+        for file in "$@"
         do
             /usr/lib/qubes/qvm-convert-img.gnome "$file"
         done
         ;;
     pdf)
-        for file in $files
+        for file in "$@"
         do
             /usr/lib/qubes/qvm-convert-pdf.gnome "$file"
         done
         ;;
     openvm)
-        for file in $files
+        for file in "$@"
         do
             #shellcheck disable=SC2016
             qvm-open-in-vm '$default' "$file" | zenity --notification --text "Opening $file in VM..." --timeout 3 &
         done
         ;;
     opendvm)
-        for file in $files
+        for file in "$@"
         do
-            qvm-open-in-dvm "$files" | zenity --notification --text "Opening $file in DisposableVM..." --timeout 3 &
+            qvm-open-in-dvm "$file" | zenity --notification --text "Opening $file in DisposableVM..." --timeout 3 &
         done
         ;;
     *)