Eevee only gets Celebrate from an Event, and thus Umbreon cannot have its HA or Egg Move (Wish) on a set that also has Celebrate.
Also fixes several items being left out of the item clause (Thanks Urkerab and DragonWhale)
Fixes a bug where Volcarona, Weavile, Whimsicott, Xurkitree and Zapdos never get rolled due to a misplaced }
Group Silvally, Ninetales, Rotom, and Thundurus formes to prevent them from being rolled more often than they should
Minor adjustments to some sets (guarantee Rock Tomb and replace Vacuum Wave with Bullet Punch on Sash Lucario)
Now that we've had time to playtest a little, we're making some
improvments to the sets:
* Fix legality issue with Kantonian Ninetales having Encore
* Change Frustration -> Return on Pinsir
* Move 4 SpD -> 4 Spe on TWave Togekiss
* Slash EQ with Gravity on Gigalith
Fixed a bug:
* Fix bug where Silvally-Normal could never get called
and added an Alias:
* Added "bssf" as an alias for "BSS Factory"
* Also added "bsd" as an alias for "Battle Spot Doubles" (unrelated but
still needed doing)
Errors in initial file caused some natures to be invalid. These were replaced with `Serious` in the most recent commit, which is a suboptimal nature for these Pokemon.
This is a surprisingly minor refactor considering how many files it
touches, but most of this is only renames.
In terms of file renames:
- `tools.js` is now `sim/dex.js`
- `battle-engine.js` is now `sim/index.js` and its three classes are
in `sim/battle.js`, `sim/side.js`, and `sim/pokemon.js`
- `prng.js` is now `sim/prng.js`
In terms of variable renames:
- `Tools` is now `Dex`
- `BattleEngine` is now `Sim`
- `BattleEngine.Battle` is now `Sim.Battle`
- `BattleEngine.BattleSide` is now `Sim.Side`
- `BattleEngine.BattlePokemon` is now `Sim.Pokemon`