UPnP setting doesn't stick #815

issue gavinandresen opened this issue on February 9, 2012
  1. gavinandresen commented at 2:49 PM on February 9, 2012: contributor

    From https://bitcointalk.org/index.php?topic=63165.msg739328#msg739328

    Unchecking UPnP ports doesnt seem to save me selection.

  2. TheBlueMatt commented at 9:03 PM on February 9, 2012: contributor

    Thats not possible afaict, see OptionsModel::setData in qt/optionsmodel.cpp. In case MapPortUPnP:, there is no modification of successful, meaning it should always return true. Maybe Im missing some part of the way Qt works though.

  3. slothbag commented at 1:08 AM on February 10, 2012: none

    I'm not using any command line args or conf file, I tried it on a different PC and it still wont let me disable UPnP. Everytime I restart the application the UPnP is ticked again.

    I also tried with a brand new empty wallet file and its behaving the same way.

    I also got a few "Application has crashed blah blah C++" errors while shutting down and starting up the app, expected in a RC release I spose.

    I'm using windows 7.

  4. TheBlueMatt commented at 1:17 AM on February 10, 2012: contributor

    Oh, on reboot, well thats a completely different story.

  5. laanwj commented at 11:50 AM on February 11, 2012: member

    I can reproduce this.

  6. gavinandresen commented at 3:41 PM on February 16, 2012: contributor

    I introduced this when I modified the code to let command-line options override wallet settings.

    I'll fix...

  7. gavinandresen cross-referenced this on Feb 25, 2012 from issue Move options from wallet.dat to QT settings files by gavinandresen
  8. gavinandresen closed this on Feb 28, 2012

  9. lateminer referenced this in commit decee4bc8c on Apr 6, 2019
  10. bitcoin locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-05-19 06:55 UTC