BUG: parameter including_start is not correct
Closed this issue · 1 comments
Describe the bug
In der vault.py Klasse
ist bei der Methode
def listauctions(self, vaultId=None, height=None, including_start=False, limit=100): # 09
including_start als Default False gesetzt wie es auch in der Dokumentation steht
"including_start": bool, (boolean) If true, then iterate including starting position. False by default
Es wird jedoch immer dort ein JSON zusammengebaut.
DEBUG (MainThread) 21.05.2022 10:33:56 {"method": "listauctions", "params": [{"start": {}, "including_start": false, "limit": 100}], "jsonrpc": "2.0"}
Damit wird das Verhalten zum Standard Konsolenauruf von listauctions verändert und es kommt die erste Auktion nicht zurück.
Wenn man jedoch keine expliziten Parameter der Funktion übergibt würde ich erwarten das diese erstmal alles zurückliefert.
Dankeschön für den Report!
Dieser Bug hat sich sogar nicht nur auf die Funktion listauctions bezogen, sondern auf alle Methoden die den Parameter including_start beinhaltet haben:
Methoden die betroffen waren:
- Masternodes:
a. listmasternodes - Oracles:
a. listlatestrawprices
b. listoracles
c. listprices - Poolpair:
a. listpoolpairs
b. listpoolshares - Token:
a. listtokens - Vault:
a. listauctions
b. listvaults
Für den Endnutzer sollte sich an der Verwendung der Library nichts verändert haben, außer das jetzt die Ausgaben der gezeigten oberen Methoden nicht mehr unvollständig sind!
Thank you for the report!
This bug was not only related to the listauctions function, but to all methods that contained the including_start parameter:
Methods that were affected:
- Masternodes:
a. listmasternodes - Oracles:
a. listlatestrawprices
b. listoracles
c. listprices - Poolpairs:
a. listpoolpairs
b. listpoolshares - Tokens:
a. listtokens - Vault:
a. listauctions
b. listvaults