num_races is a bad smell in the codebase

There are a bunch of places where num_races is used that aren't needed now that we have a proper datastore.  We should just ask for the next available player number (in enrol.cc) and load_race_data is just about precaching and we should push all the caching decisions into the database layer (with possibly hints to say "just load it all"