fixed amount part of URI in QR-Codes / removed (no label) string if we have NO label / coding style updates / removed an unused variable

This commit is contained in:
Philip Kaufmann 2012-04-11 14:21:15 +02:00
parent 460b66b14b
commit 9e0dba8c17
2 changed files with 23 additions and 19 deletions

View file

@ -310,16 +310,14 @@ void AddressBookPage::on_showQRCode_clicked()
QTableView *table = ui->tableView; QTableView *table = ui->tableView;
QModelIndexList indexes = table->selectionModel()->selectedRows(AddressTableModel::Address); QModelIndexList indexes = table->selectionModel()->selectedRows(AddressTableModel::Address);
QRCodeDialog *d;
foreach (QModelIndex index, indexes) foreach (QModelIndex index, indexes)
{ {
QString address = index.data().toString(), QString address = index.data().toString(),
label = index.sibling(index.row(), 0).data().toString(), label = index.sibling(index.row(), 0).data().toString(),
title = QString("%1 << %2 >>").arg(label).arg(address); title = QString("<< %1 >>").arg(address);
QRCodeDialog *d = new QRCodeDialog(title, address, label, tab == ReceivingTab, this); QRCodeDialog *dialog = new QRCodeDialog(title, address, label, tab == ReceivingTab, this);
d->show(); dialog->show();
} }
#endif #endif
} }

View file

@ -8,7 +8,7 @@
#include <qrencode.h> #include <qrencode.h>
#define EXPORT_IMAGE_SIZE 256 #define EXPORT_IMAGE_SIZE 256
QRCodeDialog::QRCodeDialog(const QString &title, const QString &addr, const QString &label, bool enableReq, QWidget *parent) : QRCodeDialog::QRCodeDialog(const QString &title, const QString &addr, const QString &label, bool enableReq, QWidget *parent) :
QDialog(parent), QDialog(parent),
@ -24,7 +24,9 @@ QRCodeDialog::QRCodeDialog(const QString &title, const QString &addr, const QStr
ui->lblAm1->setVisible(enableReq); ui->lblAm1->setVisible(enableReq);
ui->lblAm2->setVisible(enableReq); ui->lblAm2->setVisible(enableReq);
ui->lnLabel->setText(label); // don't display "(no label)" if there IS no label, as this is confusing in the QR dialog
if(label != tr("(no label)"))
ui->lnLabel->setText(label);
genCode(); genCode();
} }
@ -37,13 +39,14 @@ QRCodeDialog::~QRCodeDialog()
void QRCodeDialog::genCode() void QRCodeDialog::genCode()
{ {
QString uri = getURI(); QString uri = getURI();
//qDebug() << "Encoding:" << uri.toUtf8().constData();
QRcode *code = QRcode_encodeString(uri.toUtf8().constData(), 0, QR_ECLEVEL_L, QR_MODE_8, 1); QRcode *code = QRcode_encodeString(uri.toUtf8().constData(), 0, QR_ECLEVEL_L, QR_MODE_8, 1);
myImage = QImage(code->width + 8, code->width + 8, QImage::Format_RGB32); myImage = QImage(code->width + 8, code->width + 8, QImage::Format_RGB32);
myImage.fill(0xffffff); myImage.fill(0xffffff);
unsigned char *p = code->data; unsigned char *p = code->data;
for(int y = 0; y < code->width; y++) { for (int y = 0; y < code->width; y++)
for(int x = 0; x < code->width; x++) { {
for (int x = 0; x < code->width; x++)
{
myImage.setPixel(x + 4, y + 4, ((*p & 1) ? 0x0 : 0xffffff)); myImage.setPixel(x + 4, y + 4, ((*p & 1) ? 0x0 : 0xffffff));
p++; p++;
} }
@ -57,22 +60,26 @@ QString QRCodeDialog::getURI()
QString ret = QString("bitcoin:%1").arg(address); QString ret = QString("bitcoin:%1").arg(address);
int paramCount = 0; int paramCount = 0;
if(ui->chkReq->isChecked() && ui->lnReqAmount->text().isEmpty() == false) { if (ui->chkReq->isChecked() && !ui->lnReqAmount->text().isEmpty())
bool ok= false; {
double amount = ui->lnReqAmount->text().toDouble(&ok); bool ok = false;
if(ok) { ui->lnReqAmount->text().toDouble(&ok);
ret += QString("?amount=%1X8").arg(ui->lnReqAmount->text()); if (ok)
{
ret += QString("?amount=%1").arg(ui->lnReqAmount->text());
paramCount++; paramCount++;
} }
} }
if(ui->lnLabel->text().isEmpty() == false) { if (!ui->lnLabel->text().isEmpty())
{
QString lbl(QUrl::toPercentEncoding(ui->lnLabel->text())); QString lbl(QUrl::toPercentEncoding(ui->lnLabel->text()));
ret += QString("%1label=%2").arg(paramCount == 0 ? "?" : "&").arg(lbl); ret += QString("%1label=%2").arg(paramCount == 0 ? "?" : "&").arg(lbl);
paramCount++; paramCount++;
} }
if(ui->lnMessage->text().isEmpty() == false) { if (!ui->lnMessage->text().isEmpty())
{
QString msg(QUrl::toPercentEncoding(ui->lnMessage->text())); QString msg(QUrl::toPercentEncoding(ui->lnMessage->text()));
ret += QString("%1message=%2").arg(paramCount == 0 ? "?" : "&").arg(msg); ret += QString("%1message=%2").arg(paramCount == 0 ? "?" : "&").arg(msg);
paramCount++; paramCount++;
@ -99,9 +106,8 @@ void QRCodeDialog::on_lnMessage_textChanged(const QString &)
void QRCodeDialog::on_btnSaveAs_clicked() void QRCodeDialog::on_btnSaveAs_clicked()
{ {
QString fn = GUIUtil::getSaveFileName(this, tr("Save Image..."), QString(), tr("PNG Images (*.png)")); QString fn = GUIUtil::getSaveFileName(this, tr("Save Image..."), QString(), tr("PNG Images (*.png)"));
if(!fn.isEmpty()) { if (!fn.isEmpty())
myImage.scaled(EXPORT_IMAGE_SIZE, EXPORT_IMAGE_SIZE).save(fn); myImage.scaled(EXPORT_IMAGE_SIZE, EXPORT_IMAGE_SIZE).save(fn);
}
} }
void QRCodeDialog::on_chkReq_toggled(bool) void QRCodeDialog::on_chkReq_toggled(bool)