[Qt] small cleanup of coincontroldialog
- use a little more Qt-style - check for NULL pointers first and return in updateView() - small space and formating changes
This commit is contained in:
parent
d8dcfb9034
commit
d34b958406
1 changed files with 16 additions and 17 deletions
|
@ -434,7 +434,8 @@ void CoinControlDialog::updateLabelLocked()
|
|||
|
||||
void CoinControlDialog::updateLabels(WalletModel *model, QDialog* dialog)
|
||||
{
|
||||
if (!model) return;
|
||||
if (!model)
|
||||
return;
|
||||
|
||||
// nPayAmount
|
||||
qint64 nPayAmount = 0;
|
||||
|
@ -640,6 +641,9 @@ void CoinControlDialog::updateLabels(WalletModel *model, QDialog* dialog)
|
|||
|
||||
void CoinControlDialog::updateView()
|
||||
{
|
||||
if (!model || !model->getOptionsModel() || !model->getAddressTableModel())
|
||||
return;
|
||||
|
||||
bool treeMode = ui->radioTreeMode->isChecked();
|
||||
|
||||
ui->treeWidget->clear();
|
||||
|
@ -648,9 +652,7 @@ void CoinControlDialog::updateView()
|
|||
QFlags<Qt::ItemFlag> flgCheckbox = Qt::ItemIsSelectable | Qt::ItemIsEnabled | Qt::ItemIsUserCheckable;
|
||||
QFlags<Qt::ItemFlag> flgTristate = Qt::ItemIsSelectable | Qt::ItemIsEnabled | Qt::ItemIsUserCheckable | Qt::ItemIsTristate;
|
||||
|
||||
int nDisplayUnit = BitcoinUnits::BTC;
|
||||
if (model && model->getOptionsModel())
|
||||
nDisplayUnit = model->getOptionsModel()->getDisplayUnit();
|
||||
int nDisplayUnit = model->getOptionsModel()->getDisplayUnit();
|
||||
|
||||
map<QString, vector<COutput> > mapCoins;
|
||||
model->listCoins(mapCoins);
|
||||
|
@ -658,11 +660,10 @@ void CoinControlDialog::updateView()
|
|||
BOOST_FOREACH(PAIRTYPE(QString, vector<COutput>) coins, mapCoins)
|
||||
{
|
||||
QTreeWidgetItem *itemWalletAddress = new QTreeWidgetItem();
|
||||
itemWalletAddress->setCheckState(COLUMN_CHECKBOX, Qt::Unchecked);
|
||||
QString sWalletAddress = coins.first;
|
||||
QString sWalletLabel = "";
|
||||
if (model->getAddressTableModel())
|
||||
sWalletLabel = model->getAddressTableModel()->labelForAddress(sWalletAddress);
|
||||
if (sWalletLabel.length() == 0)
|
||||
QString sWalletLabel = model->getAddressTableModel()->labelForAddress(sWalletAddress);
|
||||
if (sWalletLabel.isEmpty())
|
||||
sWalletLabel = tr("(no label)");
|
||||
|
||||
if (treeMode)
|
||||
|
@ -722,10 +723,8 @@ void CoinControlDialog::updateView()
|
|||
}
|
||||
else if (!treeMode)
|
||||
{
|
||||
QString sLabel = "";
|
||||
if (model->getAddressTableModel())
|
||||
sLabel = model->getAddressTableModel()->labelForAddress(sAddress);
|
||||
if (sLabel.length() == 0)
|
||||
QString sLabel = model->getAddressTableModel()->labelForAddress(sAddress);
|
||||
if (sLabel.isEmpty())
|
||||
sLabel = tr("(no label)");
|
||||
itemOutput->setText(COLUMN_LABEL, sLabel);
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue