Skip to content

Commit

Permalink
Help: only use male_name/female_name if the unit has male/female vari…
Browse files Browse the repository at this point in the history
…ants.
  • Loading branch information
jostephd authored and Vultraz committed Jul 8, 2018
1 parent a4b0de5 commit dcbdac8
Showing 1 changed file with 11 additions and 4 deletions.
15 changes: 11 additions & 4 deletions src/help/topic_text_generators.cpp
Expand Up @@ -462,10 +462,17 @@ std::string unit_topic_generator::generate() const
int must_have_nameless_traits = 0;

for(const config& trait : traits) {
std::string trait_name = trait["male_name"].str();
if (trait_name.empty()) trait_name = trait["female_name"].str();
if (trait_name.empty()) trait_name = trait["name"].str();
if (trait_name.empty()) continue; // Hidden trait
const std::string& male_name = trait["male_name"].str();
const std::string& female_name = trait["female_name"].str();
std::string trait_name;
if (type_.has_gender_variation(unit_race::MALE) && ! male_name.empty())
trait_name = male_name;
else if (type_.has_gender_variation(unit_race::FEMALE) && ! female_name.empty())
trait_name = female_name;
else if (! trait["name"].str().empty())
trait_name = trait["name"].str();
else
continue; // Hidden trait

std::string lang_trait_name = translation::gettext(trait_name.c_str());
if(lang_trait_name.empty() && trait["availability"].str() == "musthave") {
Expand Down

0 comments on commit dcbdac8

Please sign in to comment.