[Qt] walletview: make backupWallet() use GUIUtil::getSaveFileName() #3160

pull Diapolo wants to merge 2 commits into bitcoin:master from Diapolo:walletview changing 1 files +16 −21

Diapolo commented at 5:33 PM on October 26, 2013: none

Related to #3157 and #3159 (same changes for addressbookpage and transactionview).

Goal: Harmonize user experience for export of addresses, transactions and backup of the wallet!

walletview: make backupWallet() use GUIUtil::getSaveFileName()

- this allows removal of several Qt headers and makes use of a
  standardized function in GUIUtil
- adds selected path to the UI output

0259c62f41

walletview: clarify 2 comments d127964a97
Diapolo commented at 8:22 PM on October 27, 2013: none

@laanwj This one should be save to merge, no "weird" changes to signals and slots :-D.
BitcoinPullTester commented at 9:48 PM on October 27, 2013: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/d127964a974ad83d01e80ecd82a666b90ffb6793 for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
laanwj referenced this in commit 033ffc4377 on Oct 30, 2013
laanwj merged this on Oct 30, 2013
laanwj closed this on Oct 30, 2013
Bushstar referenced this in commit 4db91c605b on Apr 8, 2020
DrahtBot locked this on Sep 8, 2021

Contributors