Skip to content
This repository has been archived by the owner on Sep 15, 2023. It is now read-only.

Possible bug: __repr__ method of Run class overwrites .game and .category variables #13

Open
thecnoNSMB opened this issue Nov 14, 2018 · 3 comments

Comments

@thecnoNSMB
Copy link

.game and .category, before the object is printed, refer to ID strings. This behavior is inconsistent, especially when working between the interpreter and scripting.

@blha303
Copy link
Owner

blha303 commented Nov 15, 2018

You're right, the repr function shouldn't be modifying the data. I'll see what I can do, thanks

@thecnoNSMB
Copy link
Author

Just skimming over the code, it looks like the Leaderboard class has the same problem.

@blha303
Copy link
Owner

blha303 commented Nov 15, 2018

Yeah, I think I went with a method that solved a bug I had at the time, but turning data into a class instance definitely shouldn't be happening in repr.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants