Page MenuHomePhabricator

GUI: Change the receive button to respond to keypool state changing

Authored by deadalnix on Mon, Jan 6, 13:46.


  • Check for more than private keys disabled to show receive button
  • Notify the GUI that the keypool has changed to set the receive button

Whenever the keypool changes (new keys generated, new seed set,
keypool runs out, etc.), notify the GUI that the keypool has changed. The
receive button can then be enabled and disabled as necessary.

This is a backport of Core PR15225

Test Plan
make check

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

deadalnix created this revision.Mon, Jan 6, 13:46
Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Jan 6, 13:46
Fabien requested changes to this revision.Mon, Jan 6, 16:17
Fabien added a subscriber: Fabien.

Is there any way to test this without PR15226 ?

436 ↗(On Diff #15175)

Use std::bind

This revision now requires changes to proceed.Mon, Jan 6, 16:17
deadalnix added inline comments.Mon, Jan 6, 16:24
436 ↗(On Diff #15175)

It's boost bind in core still. Not sure if it matters.

deadalnix added a comment.EditedMon, Jan 6, 16:25

Not sure about testing. PR15226 is indeed to next in my queue.

deadalnix updated this revision to Diff 15177.Mon, Jan 6, 16:32

Use std::bind

Fabien accepted this revision.Mon, Jan 6, 16:35
Fabien added inline comments.
436 ↗(On Diff #15175)

I just checked and you're right, it's still there... And it's the only occurrence in the codebase. That looks like a mistake to me, furthermore bind.hpp is never included.

This revision is now accepted and ready to land.Mon, Jan 6, 16:35