diff options
author | Jimmy <jimmy@spalge.com> | 2018-01-02 14:13:13 +1300 |
---|---|---|
committer | Jimmy <jimmy@spalge.com> | 2018-03-03 13:14:49 +1300 |
commit | b91e2e32677ab296a752281d46a14b3f65542dab (patch) | |
tree | 7850778f67e198d6fc8a63249fc533eab99d18c9 /qutebrowser/browser/greasemonkey.py | |
parent | a7b74d8e8311fc60ab168736c7f408e23b395bfa (diff) | |
download | qutebrowser-b91e2e32677ab296a752281d46a14b3f65542dab.tar.gz qutebrowser-b91e2e32677ab296a752281d46a14b3f65542dab.zip |
Allow download manager to overwrite existing files unprompted.
This is to support the non-interactive use case of setting a
`FileDownloadTarget` and passing auto_remove and not caring if the target
file exists or not.
An alternative to adding the attribute to `FileDownloadTarget` and
having set_target pull it out would be to add a new param to `fetch()`
and `set_target()`. But it would only be used for one target type
anyway.
Diffstat (limited to 'qutebrowser/browser/greasemonkey.py')
-rw-r--r-- | qutebrowser/browser/greasemonkey.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/qutebrowser/browser/greasemonkey.py b/qutebrowser/browser/greasemonkey.py index 07e73acfa..bc6208e90 100644 --- a/qutebrowser/browser/greasemonkey.py +++ b/qutebrowser/browser/greasemonkey.py @@ -276,7 +276,8 @@ class GreasemonkeyManager(QObject): download_manager = objreg.get('qtnetwork-download-manager') for url, target_path in required_dls: - target = downloads.FileDownloadTarget(target_path) + target = downloads.FileDownloadTarget(target_path, + force_overwrite=True) download = download_manager.get(QUrl(url), target=target, auto_remove=True) download.requested_url = url |