Files
@ r24498:e9114d9ab04a
Branch filter:
Location: cpp/openttd-patchpack/source/src/depot_cmd.cpp - annotation
r24498:e9114d9ab04a
2.4 KiB
text/x-c
Fix #6468: don't store version of AIs-started-via-console in name
You can do: "startai myai.3", which starts version 3 of "myai".
This is very useful for testing save/load code between different
versions of your AI.
However, when using this syntax, the AI got saved as "myai.3" as
name of the AI, instead of "myai". This caused several problems,
like indicating to the user the AI could not be found, but still
load the AI. But in all cases, the AI never got the chance to
load the saved data, making the whole reason this exists pointless.
By splitting the name and version already in the console command,
the code becomes simpler and AIs started this way now follow the
normal flow after initialization.
You can do: "startai myai.3", which starts version 3 of "myai".
This is very useful for testing save/load code between different
versions of your AI.
However, when using this syntax, the AI got saved as "myai.3" as
name of the AI, instead of "myai". This caused several problems,
like indicating to the user the AI could not be found, but still
load the AI. But in all cases, the AI never got the chance to
load the saved data, making the whole reason this exists pointless.
By splitting the name and version already in the console command,
the code becomes simpler and AIs started this way now follow the
normal flow after initialization.
r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r18845:66bf168f1100 r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r17263:a75c87f7e037 r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r16069:5049a8060a4d r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r21383:942c32fb8b0e r21383:942c32fb8b0e r17630:7d818445376d r17630:7d818445376d r17630:7d818445376d r17630:7d818445376d r17630:7d818445376d r15162:d47b50bb10ac r15162:d47b50bb10ac r23957:a76e956cb358 r24214:a65c412aafcc r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r23607:36c15679007d r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r16668:989c9aae97ec r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r24214:a65c412aafcc r15162:d47b50bb10ac r15162:d47b50bb10ac r24214:a65c412aafcc r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac r20821:e1ecbf7996ac r16069:5049a8060a4d r15162:d47b50bb10ac r15162:d47b50bb10ac r15162:d47b50bb10ac | /*
* This file is part of OpenTTD.
* OpenTTD is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation, version 2.
* OpenTTD is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
* See the GNU General Public License for more details. You should have received a copy of the GNU General Public License along with OpenTTD. If not, see <http://www.gnu.org/licenses/>.
*/
/** @file depot_cmd.cpp %Command Handling for depots. */
#include "stdafx.h"
#include "command_func.h"
#include "depot_base.h"
#include "company_func.h"
#include "string_func.h"
#include "town.h"
#include "vehicle_gui.h"
#include "vehiclelist.h"
#include "window_func.h"
#include "table/strings.h"
#include "safeguards.h"
/**
* Check whether the given name is globally unique amongst depots.
* @param name The name to check.
* @return True if there is no depot with the given name.
*/
static bool IsUniqueDepotName(const char *name)
{
for (const Depot *d : Depot::Iterate()) {
if (!d->name.empty() && d->name == name) return false;
}
return true;
}
/**
* Rename a depot.
* @param tile unused
* @param flags type of operation
* @param p1 id of depot
* @param p2 unused
* @param text the new name or an empty string when resetting to the default
* @return the cost of this operation or an error
*/
CommandCost CmdRenameDepot(TileIndex tile, DoCommandFlag flags, uint32 p1, uint32 p2, const char *text)
{
Depot *d = Depot::GetIfValid(p1);
if (d == nullptr) return CMD_ERROR;
CommandCost ret = CheckTileOwnership(d->xy);
if (ret.Failed()) return ret;
bool reset = StrEmpty(text);
if (!reset) {
if (Utf8StringLength(text) >= MAX_LENGTH_DEPOT_NAME_CHARS) return CMD_ERROR;
if (!IsUniqueDepotName(text)) return_cmd_error(STR_ERROR_NAME_MUST_BE_UNIQUE);
}
if (flags & DC_EXEC) {
if (reset) {
d->name.clear();
MakeDefaultName(d);
} else {
d->name = text;
}
/* Update the orders and depot */
SetWindowClassesDirty(WC_VEHICLE_ORDERS);
SetWindowDirty(WC_VEHICLE_DEPOT, d->xy);
/* Update the depot list */
VehicleType vt = GetDepotVehicleType(d->xy);
SetWindowDirty(GetWindowClassForVehicleType(vt), VehicleListIdentifier(VL_DEPOT_LIST, vt, GetTileOwner(d->xy), d->index).Pack());
}
return CommandCost();
}
|