diff options
| author | frosch <frosch@openttd.org> | 2010-01-27 20:54:05 +0000 |
|---|---|---|
| committer | frosch <frosch@openttd.org> | 2010-01-27 20:54:05 +0000 |
| commit | a3168269f9920e764669d7d63abff9ec5fe332d0 (patch) | |
| tree | 6719dc9bf56f4b29ce383c943bddea7d6b91e631 /src/company_cmd.cpp | |
| parent | 7b237ee607ed12814e3a38f99d3dda33c34987ff (diff) | |
| download | openttd-a3168269f9920e764669d7d63abff9ec5fe332d0.tar.xz | |
(svn r18931) -Fix: Disabling autoreplace rules might count invalid engines.
Diffstat (limited to 'src/company_cmd.cpp')
| -rw-r--r-- | src/company_cmd.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/company_cmd.cpp b/src/company_cmd.cpp index e968108ea..dfe7478a6 100644 --- a/src/company_cmd.cpp +++ b/src/company_cmd.cpp @@ -663,7 +663,10 @@ CommandCost CmdSetAutoReplace(TileIndex tile, DoCommandFlag flags, uint32 p1, ui CommandCost cost; if (!Group::IsValidID(id_g) && !IsAllGroupID(id_g) && !IsDefaultGroupID(id_g)) return CMD_ERROR; + if (!Engine::IsValidID(old_engine_type)) return CMD_ERROR; + if (new_engine_type != INVALID_ENGINE) { + if (!Engine::IsValidID(new_engine_type)) return CMD_ERROR; if (!CheckAutoreplaceValidity(old_engine_type, new_engine_type, _current_company)) return CMD_ERROR; cost = AddEngineReplacementForCompany(c, old_engine_type, new_engine_type, id_g, flags); @@ -671,7 +674,7 @@ CommandCost CmdSetAutoReplace(TileIndex tile, DoCommandFlag flags, uint32 p1, ui cost = RemoveEngineReplacementForCompany(c, old_engine_type, id_g, flags); } - if (IsLocalCompany()) InvalidateAutoreplaceWindow(old_engine_type, id_g); + if ((flags & DC_EXEC) && IsLocalCompany()) InvalidateAutoreplaceWindow(old_engine_type, id_g); return cost; } |