Skip to content
Snippets Groups Projects
Commit 29cfea0e authored by Pregmodder's avatar Pregmodder
Browse files

Merge branch 'fixes' into 'pregmod-master'

HeroSlavesCleanup should finally work properly

Closes #259

See merge request pregmodfan/fc-pregmod!2535
parents 076d71a9 74bb9198
No related branches found
No related tags found
No related merge requests found
...@@ -1378,9 +1378,14 @@ extremely pale -5 ...@@ -1378,9 +1378,14 @@ extremely pale -5
s.nationality = hashChoice($nationalities); s.nationality = hashChoice($nationalities);
/* Check for a pre-set race and if the nationality fits, else regenerate */ /* Check for a pre-set race and if the nationality fits, else regenerate */
if(s.race && validRaces.includes(s.race)) { if(s.race && validRaces.includes(s.race)) {
while(setup.raceSelector[s.nationality] && !(s.race in setup.raceSelector[s.nationality])) { /* Maximum of 100 attempts */
for (var i = 0; setup.raceSelector[s.nationality] && !(s.race in setup.raceSelector[s.nationality]) && i < 100; i++) {
s.nationality = hashChoice($nationalities); s.nationality = hashChoice($nationalities);
} }
/* No success after 100 attempts, so just randomize according to race */
if (setup.raceSelector[s.nationality] && !(s.race in setup.raceSelector[s.nationality]) && i == 100) {
s.nationality = Object.keys(setup[s.race.toLowerCase().replace(/[ -]/g, '')+'Nationalities']).random();
}
} }
} }
if(!s.race || !validRaces.includes(s.race)) { if(!s.race || !validRaces.includes(s.race)) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment