# HG changeset patch # User frosch # Date 2010-05-21 20:51:00 # Node ID 23efec4c269dc153ad2514c10724f2c3520bbce2 # Parent bb931a96dab2bebecc10a3ffdd4b8a5cd64fe784 (svn r19877) -Change: Name invalid engines, cargos and industries 'invalid', if the player removed the supplying NewGRFs. diff --git a/src/lang/english.txt b/src/lang/english.txt --- a/src/lang/english.txt +++ b/src/lang/english.txt @@ -2433,6 +2433,13 @@ STR_NEWGRF_BUGGY STR_NEWGRF_BUGGY_ARTICULATED_CARGO :{WHITE}Cargo/refit information for '{1:ENGINE}' differs from purchase list after construction. This might cause autorenew/-replace to fail refitting correctly. STR_NEWGRF_BUGGY_ENDLESS_PRODUCTION_CALLBACK :{WHITE}'{1:STRING}' caused an endless loop in the production callback. +# 'User removed essential NewGRFs'-placeholders for stuff without specs. +STR_NEWGRF_INVALID_CARGO : +STR_NEWGRF_INVALID_CARGO_ABBREV :?? +STR_NEWGRF_INVALID_CARGO_QUANTITY :{COMMA} of +STR_NEWGRF_INVALID_ENGINE : +STR_NEWGRF_INVALID_INDUSTRYTYPE : + # Sign list window STR_SIGN_LIST_CAPTION :{WHITE}Sign List - {COMMA} Sign{P "" s} diff --git a/src/newgrf.cpp b/src/newgrf.cpp --- a/src/newgrf.cpp +++ b/src/newgrf.cpp @@ -6455,6 +6455,34 @@ static void CalculateRefitMasks() } } +/** Check for invalid engines */ +static void FinaliseEngineArray() +{ + Engine *e; + + FOR_ALL_ENGINES(e) { + if (e->grffile == NULL) { + const EngineIDMapping &eid = _engine_mngr[e->index]; + if (eid.grfid != INVALID_GRFID || eid.internal_id != eid.substitute_id) { + e->info.string_id = STR_NEWGRF_INVALID_ENGINE; + } + } + } +} + +/** Check for invalid cargos */ +static void FinaliseCargoArray() +{ + for (CargoID c = 0; c < NUM_CARGO; c++) { + CargoSpec *cs = CargoSpec::Get(c); + if (!cs->IsValid()) { + cs->name = cs->name_single = cs->units_volume = STR_NEWGRF_INVALID_CARGO; + cs->quantifier = STR_NEWGRF_INVALID_CARGO_QUANTITY; + cs->abbrev = STR_NEWGRF_INVALID_CARGO_ABBREV; + } + } +} + /** Add all new houses to the house array. House properties can be set at any * time in the GRF file, so we can only add a house spec to the house array * after the file has finished loading. We also need to check the dates, due to @@ -6583,6 +6611,9 @@ static void FinaliseIndustriesArray() indsp->conflicting[i] = MapNewGRFIndustryType(indsp->conflicting[i], indsp->grf_prop.grffile->grfid); } } + if (!indsp->enabled) { + indsp->name = STR_NEWGRF_INVALID_INDUSTRYTYPE; + } } } @@ -6963,9 +6994,15 @@ static void AfterLoadGRFs() } _grf_line_to_action6_sprite_override.clear(); + /* Polish cargos */ + FinaliseCargoArray(); + /* Pre-calculate all refit masks after loading GRF files. */ CalculateRefitMasks(); + /* Polish engines */ + FinaliseEngineArray(); + /* Set the block size in the depot windows based on vehicle sprite sizes */ InitDepotWindowBlockSizes(); diff --git a/src/strings.cpp b/src/strings.cpp --- a/src/strings.cpp +++ b/src/strings.cpp @@ -998,7 +998,7 @@ static char *FormatString(char *buff, co assert(e != NULL); - if (e->name != NULL) { + if (e->name != NULL && e->info.string_id != STR_NEWGRF_INVALID_ENGINE) { buff = strecpy(buff, e->name, last); } else { buff = GetStringWithArgs(buff, e->info.string_id, NULL, last);