rnhmjoj/qutebrowser
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Ctrl-A only increments number in path segment

Browse Source 
This prevents a host like "myfoo42.bar" changing to "myfoo43.bar" when
pressing Ctrl-A. It further prevents increasing the port number, e.g.
going from "foo.bar:8080" to "foo.bar:8081".
This commit is contained in:
Daniel 2015-08-07 17:21:18 +02:00
parent 5f122759db
commit 0f3aa0bd8c
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
qutebrowser/browser/commands.py
View File

@ -472,9 +472,9 @@ class CommandDispatcher:
background: Open the link in a new background tab.
window: Open the link in a new window.
"""
encoded = bytes(url.toEncoded()).decode('ascii')
path = url.path()
# Get the last number in a string
match = re.match(r'(.*\D|^)(\d+)(.*)', encoded)
match = re.match(r'(.*\D|^)(\d+)(.*)', path)
if not match:
raise cmdexc.CommandError("No number found in URL!")
pre, number, post = match.groups()
@ -493,8 +493,10 @@ class CommandDispatcher:
val += 1
else:
raise ValueError("Invalid value {} for indec!".format(incdec))
urlstr = ''.join([pre, str(val), post]).encode('ascii')
new_url = QUrl.fromEncoded(urlstr)
new_path = ''.join([pre, str(val), post])
# Make a copy of the QUrl so we don't modify the original
new_url = QUrl(url)
new_url.setPath(new_path)
self._open(new_url, tab, background, window)
def _navigate_up(self, url, tab, background, window):