Bitcoin-Qt: fix known addressbook bugs
authorPhilip Kaufmann <phil.kaufmann@t-online.de>
Tue, 8 Jan 2013 07:17:58 +0000 (08:17 +0100)
committerCryptoManiac <balthazar@yandex.ru>
Sun, 23 Feb 2014 16:13:13 +0000 (20:13 +0400)
commit23ab7f1ea54ea1f030319a7c5b416acddbbef758
tree921f1a15a01f3a883dbb8a11f36a4ef688640f8a
parent2b72ed961b6dbab114197ad5cf0fc230ab92fadb
Bitcoin-Qt: fix known addressbook bugs

- add qSort() for cachedAddressTable, as qLowerBound() and qUpperBound()
  require the list to be in ascending order (see
  http://harmattan-dev.nokia.com/docs/library/html/qt4/qtalgorithms.html#qLowerBound)
- add a new check in AddressTableModel::setData() to just return, when no
  changes were made to a label or an address (prevents entry duplication
  issue)
- remove "rec->label = value.toString();" from
  AddressTableModel::setData() as the label gets updated by
  AddressTablePriv::updateEntry() anyway (seems @sipa added this line via
  https://github.com/bitcoin/bitcoin/commit/1025440184ef100a22d07c7bb543ee45cf169d64#L6R225)
- add another new check in AddressTableModel::setData() to just return, if
  a duplicate address was found (prevents address overwrite)
- add a new check to EditAddressDialog::setModel() to prevent setting an
  invalid model
- re-work the switch-case statement in AddressTableModel::accept() to
  always break (as return get's called anyway) and order the list to match
  the enum definition
- make accept() in editaddressdialog.h a public slot, which it should be
- misc small coding style changes
src/qt/addresstablemodel.cpp
src/qt/addresstablemodel.h
src/qt/editaddressdialog.cpp
src/qt/editaddressdialog.h