search

PirateNetwork / pirate

Pirate Chain (ARRR) - Untraceable, Anonymous, Private Cryptocurrency
https://piratechain.com
Other
112 stars 27 forks source link

Adjust CLI argument handling for z_getbalances #111

Closed scott-ftf closed 3 months ago

scott-ftf commented 4 months ago

This corrects a bug in client.cpp where the z_getbalances function was set to accept the incorrect argument due to the configuration only having the single line, { "z_getbalances", 1}. The z_getbalances command allows for an optional boolean argument, making it necessary to handle zero or one arguments. z_getbalances can now be correctly called with or without the optional Bool argument for including watchonly addresses

Corrected z_getbalances usage information. The error message has been corrected to show 'includeWatchonly' as the only expected parameter, where a second 'minconf' was also listed previously.

warelock2 commented 4 months ago

I tested this and confirm that it fixes the issue