[#3075|Console] Fix config handling windows paths

The console config token parser was unable to handle windows paths
starting with 'C:\'.

Remove unneeded windows_checks.
This commit is contained in:
Calum Lind 2017-11-05 15:25:21 +00:00
parent 507c5df984
commit 9bcda41700
3 changed files with 6 additions and 4 deletions

View File

@ -16,7 +16,6 @@ import shlex
from twisted.internet import defer
from deluge.common import windows_check
from deluge.ui.client import client
from deluge.ui.console.parser import OptionParser, OptionParserError
from deluge.ui.console.utils.colors import strip_colors
@ -167,7 +166,6 @@ class BaseCommand(object):
return self.__doc__
def split(self, text):
if windows_check():
text = text.replace('\\', '\\\\')
result = shlex.split(text)
for i, s in enumerate(result):

View File

@ -55,6 +55,10 @@ def atom(src, token):
return False
elif token[0] is tokenize.STRING or token[1] == '/':
return token[-1].decode('string-escape')
elif token[1].isalpha():
# Parse Windows paths e.g. 'C:\\xyz' or 'C:/xyz'.
if next()[1] == ':' and next()[1] in '\/':
return token[-1].decode('string-escape')
raise SyntaxError('malformed expression (%s)' % token[1])

View File

@ -562,7 +562,7 @@ class EventLog(component.Component):
(state, t_name, torrent_id))
def on_torrent_finished_event(self, torrent_id):
if not deluge.common.windows_check() and component.get('TorrentList').config['ring_bell']:
if component.get('TorrentList').config['ring_bell']:
import curses.beep
curses.beep()
self.write('{!info!}Torrent Finished: %s ({!cyan!}%s{!info!})' %