faif/python-patterns — pull request #27 by slunk (2013-10-22 03:56:07+00:00)
Title: Several changes to mediator.py

First of all, I really enjoyed reading through these patterns today. Thanks for compiling this! The mediator example struck me as a little weak compared to the others in terms of code quality. Here's my proposed fixes to what struck me as the most egregious offenses:

1) Fixed several methods whose parameters were not being used (setTm in
TC, Reporter, and DB), and places where class members were being
assigned the value of global variables.
2) Moved an import to the top of the file, as suggested in pep8
(http://www.python.org/dev/peps/pep-0008/#imports).
3) Removed an unused local variable in TestManager's publishReport
method.