MinoMino / minqlbot

An administration tool for the Quake Live client, extensible with plugins.
GNU General Public License v3.0
26 stars 10 forks source link

Player class should be able to represent arbitrary player not necessarily present #30

Open WalkerY opened 9 years ago

WalkerY commented 9 years ago

it raises exception: except NonexistentPlayerError: self.__invalidate("Tried to initialize a Player instance of a nonexistant player.")

but its needed to represent player that is not on server, for future-proof implementations for example all functions should pass Player instances instead of names. Behind the scenes, in methods like 'getelo' player instance could be used of the player that exists but is not necessarily on server and whose lets say steam id was found in some other way.

WalkerY commented 9 years ago

Ok, i guess dummyplayer can be used in this way.