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

Merge branch 'HolySmoke86-issue1987'

Browse Source
This commit is contained in:
Florian Bruhin 2016-10-03 06:45:23 +02:00
commit 7fae7257a9
3 changed files with 24 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
README.asciidoc
View File

@ -160,13 +160,13 @@ Contributors, sorted by the number of commits in descending order:
* Corentin Julé * Corentin Julé
* meles5 * meles5
* Philipp Hansch * Philipp Hansch
* Daniel Karbach
* Panagiotis Ktistakis * Panagiotis Ktistakis
* Kevin Velghe * Kevin Velghe
* Artur Shaik * Artur Shaik
* Nathan Isom * Nathan Isom
* Thorsten Wißmann * Thorsten Wißmann
* Austin Anderson * Austin Anderson
* Daniel Karbach
* Jimmy * Jimmy
* Niklas Haas * Niklas Haas
* Alexey "Averrin" Nabrodov * Alexey "Averrin" Nabrodov

6
qutebrowser/browser/commands.py
View File

@ -682,7 +682,11 @@ class CommandDispatcher:
else: else:
flags |= QUrl.FullyEncoded flags |= QUrl.FullyEncoded
url = QUrl(self._current_url()) url = QUrl(self._current_url())
url_query = QUrlQuery(url) url_query = QUrlQuery()
url_query_str = url.query()
if '&' not in url_query_str and ';' in url_query_str:
url_query.setQueryDelimiters('=', ';')
url_query.setQuery(url_query_str)
for key in dict(url_query.queryItems()): for key in dict(url_query.queryItems()):
if key in config.get('general', 'yank-ignored-url-parameters'): if key in config.get('general', 'yank-ignored-url-parameters'):
url_query.removeQueryItem(key) url_query.removeQueryItem(key)

18
tests/end2end/features/yankpaste.feature
View File

@ -57,6 +57,24 @@ Feature: Yanking and pasting.
Then the message "Yanked URL to clipboard: http://localhost:(port)/data/title with spaces.html" should be shown Then the message "Yanked URL to clipboard: http://localhost:(port)/data/title with spaces.html" should be shown
And the clipboard should contain "http://localhost:(port)/data/title with spaces.html" And the clipboard should contain "http://localhost:(port)/data/title with spaces.html"
Scenario: Yanking URL that has = and & in its query string
When I open data/title.html?a=b&c=d
And I run :yank
Then the message "Yanked URL to clipboard: http://localhost:(port)/data/title.html?a=b&c=d" should be shown
And the clipboard should contain "http://localhost:(port)/data/title.html?a=b&c=d"
Scenario: Yanking URL that has = and ; in its query string
When I open data/title.html?a=b;c=d
And I run :yank
Then the message "Yanked URL to clipboard: http://localhost:(port)/data/title.html?a=b;c=d" should be shown
And the clipboard should contain "http://localhost:(port)/data/title.html?a=b;c=d"
Scenario: Yanking URL with both & and ; in its query string
When I open data/title.html?a;b&c=d
And I run :yank
Then the message "Yanked URL to clipboard: http://localhost:(port)/data/title.html?a;b&c=d" should be shown
And the clipboard should contain "http://localhost:(port)/data/title.html?a;b&c=d"
#### {clipboard} and {primary} #### {clipboard} and {primary}
Scenario: Pasting a URL Scenario: Pasting a URL