Toward bug #13854: pokeravatar no longer needs to convert to a query string
for PacketPokerTablePicker(). Lots fewer tests and still 100% coverage!

(Browse SVN revision 5994)

The PokerService. getTableBestByCriteria() method does not account for the fact that PokerService.searchTables() might return a table in its list where player represented by 'serial' is already sitting at.

This means that if a player is already playing at a table and reissues PacketPokerTablePicker() via his client, the player may get a table on which the player is already joined or seated at.

I do not believe the behavior of PacketPokerTablePicker() is defined properly in this scenario, and thus weird results may occur.

Likely, getTablebestByCriteria() should eliminate from consideration tables which player is already seated at. However, it probably should handle gracefully the case where the player is already joined (i.e., watching) the game.