remove minimum total fee option
This commit is contained in:
parent
94e52273f3
commit
091a9ae21c
5 changed files with 1 additions and 46 deletions
|
@ -513,8 +513,6 @@ void CoinControlDialog::updateLabels(WalletModel *model, QDialog* dialog)
|
|||
|
||||
// Fee
|
||||
nPayFee = CWallet::GetMinimumFee(nBytes, nTxConfirmTarget, ::mempool, ::feeEstimator);
|
||||
if (nPayFee > 0 && coinControl->nMinimumTotalFee > nPayFee)
|
||||
nPayFee = coinControl->nMinimumTotalFee;
|
||||
|
||||
if (nPayAmount > 0)
|
||||
{
|
||||
|
@ -573,7 +571,7 @@ void CoinControlDialog::updateLabels(WalletModel *model, QDialog* dialog)
|
|||
l5->setText(((nBytes > 0) ? ASYMP_UTF8 : "") + QString::number(nBytes)); // Bytes
|
||||
l7->setText(fDust ? tr("yes") : tr("no")); // Dust
|
||||
l8->setText(BitcoinUnits::formatWithUnit(nDisplayUnit, nChange)); // Change
|
||||
if (nPayFee > 0 && (coinControl->nMinimumTotalFee < nPayFee))
|
||||
if (nPayFee > 0)
|
||||
{
|
||||
l3->setText(ASYMP_UTF8 + l3->text());
|
||||
l4->setText(ASYMP_UTF8 + l4->text());
|
||||
|
|
|
@ -861,19 +861,6 @@
|
|||
</attribute>
|
||||
</widget>
|
||||
</item>
|
||||
<item>
|
||||
<widget class="QRadioButton" name="radioCustomAtLeast">
|
||||
<property name="toolTip">
|
||||
<string>If the custom fee is set to 1000 satoshis and the transaction is only 250 bytes, then "per kilobyte" only pays 250 satoshis in fee, while "total at least" pays 1000 satoshis. For transactions bigger than a kilobyte both pay by kilobyte.</string>
|
||||
</property>
|
||||
<property name="text">
|
||||
<string>total at least</string>
|
||||
</property>
|
||||
<attribute name="buttonGroup">
|
||||
<string notr="true">groupCustomFee</string>
|
||||
</attribute>
|
||||
</widget>
|
||||
</item>
|
||||
<item>
|
||||
<widget class="BitcoinAmountField" name="customFee"/>
|
||||
</item>
|
||||
|
|
|
@ -111,7 +111,6 @@ SendCoinsDialog::SendCoinsDialog(const PlatformStyle *_platformStyle, QWidget *p
|
|||
ui->groupFee->setId(ui->radioCustomFee, 1);
|
||||
ui->groupFee->button((int)std::max(0, std::min(1, settings.value("nFeeRadio").toInt())))->setChecked(true);
|
||||
ui->groupCustomFee->setId(ui->radioCustomPerKilobyte, 0);
|
||||
ui->groupCustomFee->setId(ui->radioCustomAtLeast, 1);
|
||||
ui->groupCustomFee->button((int)std::max(0, std::min(1, settings.value("nCustomFeeRadio").toInt())))->setChecked(true);
|
||||
ui->customFee->setValue(settings.value("nTransactionFee").toLongLong());
|
||||
ui->checkBoxMinimumFee->setChecked(settings.value("fPayOnlyMinFee").toBool());
|
||||
|
@ -608,7 +607,6 @@ void SendCoinsDialog::updateFeeSectionControls()
|
|||
ui->checkBoxMinimumFee ->setEnabled(ui->radioCustomFee->isChecked());
|
||||
ui->labelMinFeeWarning ->setEnabled(ui->radioCustomFee->isChecked());
|
||||
ui->radioCustomPerKilobyte ->setEnabled(ui->radioCustomFee->isChecked() && !ui->checkBoxMinimumFee->isChecked());
|
||||
ui->radioCustomAtLeast ->setEnabled(ui->radioCustomFee->isChecked() && !ui->checkBoxMinimumFee->isChecked() && CoinControlDialog::coinControl->HasSelected());
|
||||
ui->customFee ->setEnabled(ui->radioCustomFee->isChecked() && !ui->checkBoxMinimumFee->isChecked());
|
||||
}
|
||||
|
||||
|
@ -619,19 +617,12 @@ void SendCoinsDialog::updateGlobalFeeVariables()
|
|||
int nConfirmTarget = ui->sliderSmartFee->maximum() - ui->sliderSmartFee->value() + 2;
|
||||
payTxFee = CFeeRate(0);
|
||||
|
||||
// set nMinimumTotalFee to 0 to not accidentally pay a custom fee
|
||||
CoinControlDialog::coinControl->nMinimumTotalFee = 0;
|
||||
|
||||
// show the estimated required time for confirmation
|
||||
ui->confirmationTargetLabel->setText(GUIUtil::formatDurationStr(nConfirmTarget * Params().GetConsensus().nPowTargetSpacing) + " / " + tr("%n block(s)", "", nConfirmTarget));
|
||||
}
|
||||
else
|
||||
{
|
||||
payTxFee = CFeeRate(ui->customFee->value());
|
||||
|
||||
// if user has selected to set a minimum absolute fee, pass the value to coincontrol
|
||||
// set nMinimumTotalFee to 0 in case of user has selected that the fee is per KB
|
||||
CoinControlDialog::coinControl->nMinimumTotalFee = ui->radioCustomAtLeast->isChecked() ? ui->customFee->value() : 0;
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -830,21 +821,6 @@ void SendCoinsDialog::coinControlUpdateLabels()
|
|||
if (!model || !model->getOptionsModel())
|
||||
return;
|
||||
|
||||
if (model->getOptionsModel()->getCoinControlFeatures())
|
||||
{
|
||||
// enable minimum absolute fee UI controls
|
||||
ui->radioCustomAtLeast->setVisible(true);
|
||||
|
||||
// only enable the feature if inputs are selected
|
||||
ui->radioCustomAtLeast->setEnabled(ui->radioCustomFee->isChecked() && !ui->checkBoxMinimumFee->isChecked() &&CoinControlDialog::coinControl->HasSelected());
|
||||
}
|
||||
else
|
||||
{
|
||||
// in case coin control is disabled (=default), hide minimum absolute fee UI controls
|
||||
ui->radioCustomAtLeast->setVisible(false);
|
||||
return;
|
||||
}
|
||||
|
||||
// set pay amounts
|
||||
CoinControlDialog::payAmounts.clear();
|
||||
CoinControlDialog::fSubtractFeeFromAmount = false;
|
||||
|
|
|
@ -18,8 +18,6 @@ public:
|
|||
bool fAllowOtherInputs;
|
||||
//! Includes watch only addresses which match the ISMINE_WATCH_SOLVABLE criteria
|
||||
bool fAllowWatchOnly;
|
||||
//! Minimum absolute fee (not per kilobyte)
|
||||
CAmount nMinimumTotalFee;
|
||||
//! Override estimated feerate
|
||||
bool fOverrideFeeRate;
|
||||
//! Feerate to use if overrideFeeRate is true
|
||||
|
@ -40,7 +38,6 @@ public:
|
|||
fAllowOtherInputs = false;
|
||||
fAllowWatchOnly = false;
|
||||
setSelected.clear();
|
||||
nMinimumTotalFee = 0;
|
||||
nFeeRate = CFeeRate(0);
|
||||
fOverrideFeeRate = false;
|
||||
nConfirmTarget = 0;
|
||||
|
|
|
@ -2599,9 +2599,6 @@ bool CWallet::CreateTransaction(const std::vector<CRecipient>& vecSend, CWalletT
|
|||
currentConfirmationTarget = coinControl->nConfirmTarget;
|
||||
|
||||
CAmount nFeeNeeded = GetMinimumFee(nBytes, currentConfirmationTarget, ::mempool, ::feeEstimator);
|
||||
if (coinControl && nFeeNeeded > 0 && coinControl->nMinimumTotalFee > nFeeNeeded) {
|
||||
nFeeNeeded = coinControl->nMinimumTotalFee;
|
||||
}
|
||||
if (coinControl && coinControl->fOverrideFeeRate)
|
||||
nFeeNeeded = coinControl->nFeeRate.GetFee(nBytes);
|
||||
|
||||
|
|
Loading…
Reference in a new issue