Refactor: Rename reset() -> resetForNewGame() on map data classes
Code reformat: Ctor arguments should be one arg per line

* Vertical list makes it easier to version control additions and removals of single parameters
Refactor: rename functions for saving PredefinedMap data
Always deny monster movement on mapchange,keyarea,rest - regardless of isActive
Rename file Reward.java -> ScriptEffect.java
Refactor: Rename PhraseReward -> ScriptEffect

* This commit does not contain changes to json field names when parsing phrases though.
Merge branch 'rename-phrase-reward-to-script-effect'