GUI: Receive: Remove option to reuse a previous address
This was justified by the need to "resent" an invoice, but now that we have the request history, that need should be gone.
This commit is contained in:
parent
3cd836c1d8
commit
927f4ff5a2
2 changed files with 4 additions and 40 deletions
|
@ -28,23 +28,6 @@
|
|||
<layout class="QVBoxLayout" name="verticalLayout_3">
|
||||
<item>
|
||||
<layout class="QGridLayout" name="gridLayout">
|
||||
<item row="7" column="2">
|
||||
<widget class="QCheckBox" name="reuseAddress">
|
||||
<property name="toolTip">
|
||||
<string>Reuse one of the previously used receiving addresses. Reusing addresses has security and privacy issues. Do not use this unless re-generating a payment request made before.</string>
|
||||
</property>
|
||||
<property name="text">
|
||||
<string>R&euse an existing receiving address (not recommended)</string>
|
||||
</property>
|
||||
</widget>
|
||||
</item>
|
||||
<item row="7" column="0">
|
||||
<widget class="QLabel" name="label_4">
|
||||
<property name="text">
|
||||
<string/>
|
||||
</property>
|
||||
</widget>
|
||||
</item>
|
||||
<item row="6" column="0">
|
||||
<widget class="QLabel" name="label_3">
|
||||
<property name="toolTip">
|
||||
|
@ -127,7 +110,7 @@
|
|||
</property>
|
||||
</widget>
|
||||
</item>
|
||||
<item row="8" column="2">
|
||||
<item row="7" column="2">
|
||||
<layout class="QHBoxLayout" name="horizontalLayout">
|
||||
<item>
|
||||
<widget class="QPushButton" name="receiveButton">
|
||||
|
@ -184,7 +167,7 @@
|
|||
</item>
|
||||
</layout>
|
||||
</item>
|
||||
<item row="8" column="0">
|
||||
<item row="7" column="0">
|
||||
<widget class="QLabel" name="label_7">
|
||||
<property name="text">
|
||||
<string/>
|
||||
|
@ -324,7 +307,6 @@
|
|||
<tabstop>reqLabel</tabstop>
|
||||
<tabstop>reqAmount</tabstop>
|
||||
<tabstop>reqMessage</tabstop>
|
||||
<tabstop>reuseAddress</tabstop>
|
||||
<tabstop>receiveButton</tabstop>
|
||||
<tabstop>clearButton</tabstop>
|
||||
<tabstop>recentRequestsView</tabstop>
|
||||
|
|
|
@ -101,7 +101,6 @@ void ReceiveCoinsDialog::clear()
|
|||
ui->reqAmount->clear();
|
||||
ui->reqLabel->setText("");
|
||||
ui->reqMessage->setText("");
|
||||
ui->reuseAddress->setChecked(false);
|
||||
updateDisplayUnit();
|
||||
}
|
||||
|
||||
|
@ -130,25 +129,8 @@ void ReceiveCoinsDialog::on_receiveButton_clicked()
|
|||
|
||||
QString address;
|
||||
QString label = ui->reqLabel->text();
|
||||
if(ui->reuseAddress->isChecked())
|
||||
{
|
||||
/* Choose existing receiving address */
|
||||
AddressBookPage dlg(platformStyle, AddressBookPage::ForSelection, AddressBookPage::ReceivingTab, this);
|
||||
dlg.setModel(model->getAddressTableModel());
|
||||
if(dlg.exec())
|
||||
{
|
||||
address = dlg.getReturnValue();
|
||||
if(label.isEmpty()) /* If no label provided, use the previously used label */
|
||||
{
|
||||
label = model->getAddressTableModel()->labelForAddress(address);
|
||||
}
|
||||
} else {
|
||||
return;
|
||||
}
|
||||
} else {
|
||||
/* Generate new receiving address */
|
||||
address = model->getAddressTableModel()->addRow(AddressTableModel::Receive, label, "");
|
||||
}
|
||||
/* Generate new receiving address */
|
||||
address = model->getAddressTableModel()->addRow(AddressTableModel::Receive, label, "");
|
||||
SendCoinsRecipient info(address, label,
|
||||
ui->reqAmount->value(), ui->reqMessage->text());
|
||||
ReceiveRequestDialog *dialog = new ReceiveRequestDialog(this);
|
||||
|
|
Loading…
Reference in a new issue