* reduce duplicate data in dummy_responder loadGameData function
* removed duplicate SQL query code from BMInterface
* clean up copy/pasted get_all_completed_games function
* use longer variable name in utility function
* try PHPCS coding standard to see if we get fewer errors
* test checkstyle against Squiz ruleset
* try PSR1
* try PSR2
* define and use a custom BM coding standard (just make it equivalent
  to PSR2 for now)
* bah, failed to actually change the standard in the live check
* name coding standard relative to workspace, not to deploy/jenkins
* try modifying the opening-brace style we use in our standard
* try the exclude by a different name
* fix various braces which should be on same line at function open
* fix miscellaneous misindented closing braces (checkstyle "Indent" error)
* fix a couple of closing braces which are too far away (checkstyle CloseBraceAfterBody)
* use elseif consistently, not else if
* stop checking for missing namespaces
* also don't expect opening brace on new line for classes
* fix long lines in dummy_responder