Skip to content

Commit b40bdd6

Browse files
committed
qt: Show also value for options overridden on command line
Currently only the name of the option is shown for GUI options overridden by command line (or configuration file). This commit adds the value of the options as well, which is useful for troubleshooting.
1 parent 1e13f57 commit b40bdd6

File tree

2 files changed

+17
-9
lines changed

2 files changed

+17
-9
lines changed

src/qt/optionsmodel.cpp

Lines changed: 14 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,11 @@ OptionsModel::OptionsModel(QObject *parent) :
3030
Init();
3131
}
3232

33+
void OptionsModel::addOverriddenOption(const std::string &option)
34+
{
35+
strOverriddenByCommandLine += QString::fromStdString(option) + "=" + QString::fromStdString(mapArgs[option]) + " ";
36+
}
37+
3338
// Writes all missing QSettings with their default values
3439
void OptionsModel::Init()
3540
{
@@ -76,52 +81,52 @@ void OptionsModel::Init()
7681
settings.setValue("nTransactionFee", 0);
7782
nTransactionFee = settings.value("nTransactionFee").toLongLong(); // if -paytxfee is set, this will be overridden later in init.cpp
7883
if (mapArgs.count("-paytxfee"))
79-
strOverriddenByCommandLine += "-paytxfee ";
84+
addOverriddenOption("-paytxfee");
8085

8186
if (!settings.contains("bSpendZeroConfChange"))
8287
settings.setValue("bSpendZeroConfChange", true);
8388
if (!SoftSetBoolArg("-spendzeroconfchange", settings.value("bSpendZeroConfChange").toBool()))
84-
strOverriddenByCommandLine += "-spendzeroconfchange ";
89+
addOverriddenOption("-spendzeroconfchange");
8590
#endif
8691

8792
if (!settings.contains("nDatabaseCache"))
8893
settings.setValue("nDatabaseCache", (qint64)nDefaultDbCache);
8994
if (!SoftSetArg("-dbcache", settings.value("nDatabaseCache").toString().toStdString()))
90-
strOverriddenByCommandLine += "-dbcache ";
95+
addOverriddenOption("-dbcache");
9196

9297
if (!settings.contains("nThreadsScriptVerif"))
9398
settings.setValue("nThreadsScriptVerif", 0);
9499
if (!SoftSetArg("-par", settings.value("nThreadsScriptVerif").toString().toStdString()))
95-
strOverriddenByCommandLine += "-par ";
100+
addOverriddenOption("-par");
96101

97102
// Network
98103
if (!settings.contains("fUseUPnP"))
99104
#ifdef USE_UPNP
100105
settings.setValue("fUseUPnP", true);
101106
#else
102107
settings.setValue("fUseUPnP", false);
103-
#endif
108+
#endif
104109
if (!SoftSetBoolArg("-upnp", settings.value("fUseUPnP").toBool()))
105-
strOverriddenByCommandLine += "-upnp ";
110+
addOverriddenOption("-upnp");
106111

107112
if (!settings.contains("fUseProxy"))
108113
settings.setValue("fUseProxy", false);
109114
if (!settings.contains("addrProxy"))
110115
settings.setValue("addrProxy", "127.0.0.1:9050");
111116
// Only try to set -proxy, if user has enabled fUseProxy
112117
if (settings.value("fUseProxy").toBool() && !SoftSetArg("-proxy", settings.value("addrProxy").toString().toStdString()))
113-
strOverriddenByCommandLine += "-proxy ";
118+
addOverriddenOption("-proxy");
114119
if (!settings.contains("nSocksVersion"))
115120
settings.setValue("nSocksVersion", 5);
116121
// Only try to set -socks, if user has enabled fUseProxy
117122
if (settings.value("fUseProxy").toBool() && !SoftSetArg("-socks", settings.value("nSocksVersion").toString().toStdString()))
118-
strOverriddenByCommandLine += "-socks ";
123+
addOverriddenOption("-socks");
119124

120125
// Display
121126
if (!settings.contains("language"))
122127
settings.setValue("language", "");
123128
if (!SoftSetArg("-lang", settings.value("language").toString().toStdString()))
124-
strOverriddenByCommandLine += "-lang";
129+
addOverriddenOption("-lang");
125130

126131
language = settings.value("language").toString();
127132
}

src/qt/optionsmodel.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,9 @@ class OptionsModel : public QAbstractListModel
7575
/* settings that were overriden by command-line */
7676
QString strOverriddenByCommandLine;
7777

78+
/// Add option to list of GUI options overridden through command line/config file
79+
void addOverriddenOption(const std::string &option);
80+
7881
signals:
7982
void displayUnitChanged(int unit);
8083
void transactionFeeChanged(qint64);

0 commit comments

Comments
 (0)