Vlc path parsing fix

JBlackN · Sep 16, 2015 · a29f7a3 · a29f7a3
1 parent 1f61046
commit a29f7a3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/lib/budik/player.rb b/lib/budik/player.rb
@@ -51,8 +51,8 @@ def vlc(source)
     end
 
     def vlc_build_command(source)
-      vlc_path = @player_options['path']
-      vlc_path.gsub!(/^/, '"').gsub!(/$/, '"') if vlc_path =~ /\s/
+      vlc_path = Marshal.load(Marshal.dump(@player_options['path']))
+      vlc_path.gsub!(/(^|$)/, '"') if vlc_path =~ /\s/
 
       args = vlc_build_args
       files = vlc_cmd_add_items(source)

diff --git a/spec/lib/budik/player_spec.rb b/spec/lib/budik/player_spec.rb
@@ -37,7 +37,6 @@
 
     source = { path: %w(path1 path2) }
     cmd = player.vlc_build_command(source)
-    player.player_options['path'].gsub!(/^"/, '').gsub!(/"$/, '')
 
     expect(player).to receive(:spawn).with(cmd)
     allow(player).to receive(:vlc_volume_control) {}