Manually suppressed CouplingBetweenObjects warning in BMInterface
Refactored BMInterface->submit_die_values() to reduce cyclomatic complexity
Fixed BMInterface->submit_die_values() to correctly set error message
Refactored BMInterface->set_player_info() to reduce cyclomatic complexity
Refactored BMInterface->create_game() to reduce cyclomatic complexity
Refactored BMInterface->validate_player_info() to reduce cyclomatic complexity
Removed unused parameter from BMInterface->validate_game_info()
Started refactoring of BMInterface->load_game()
Refactored BMInterface->load_game_parameters()
Refactored BMInterface->load_die_attributes() to reduce cyclomatic complexity
Refactored BMInterface->save_game() to reduce cyclomatic complexity
Refactored BMInterface->assemble_search_filters() to reduce cyclomatic complexity
Fixed checkstyle errors
Removed UnusedLocalVariable warning from BMInterface->load_player_colors()
Refactored BMInterface->search_game_history() to reduce cyclomatic complexity
Refactored BMInterface->search_game_history() to reduce length
Fixed checkstyle errors
Merge remote-tracking branch 'upstream/master' into 417_pmd_BMInterface
Bug fixing and code cleanup in response to pull request remarks
Merge remote-tracking branch 'upstream/master' into 417_pmd_BMInterface

Manually resolved conflicts in:
	src/engine/BMInterface.php