Server debug log contains no information about network players' discarded tiles

Bug #684476 reported by Alex Elsayed
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
RMahjong
Fix Committed
Low
Unassigned

Bug Description

While annotating the log for https://bugs.launchpad.net/rmahjong/+bug/684457 , I originally had not realized that the bot had missed a Kan off of another bot's discard, and thought that it had only failed to Kan off of my discard. Accordingly, I looked in the server log for when I discarded the west wind, but while the bots had Target/Tile: lines specifying their new hand and their discard, there was no such line for my discards. I have implemented a quick patch which adds a logging.debug("%s: Discarded Tile: %s" % (self, tile) ) to the name = DROP path in NetworkPlayer's process_message method. Attached is a bzr merge directive of the change.

Related branches

Revision history for this message
Alex Elsayed (eternaleye) wrote :
Changed in rmahjong:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
Stanislav Böhm (spirali) wrote :

Thank you for your patch. The patch is commited.

Changed in rmahjong:
status: Confirmed → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.