Fix incorrect template replacement error message when chain not buildable

And no specific error returned

See: #419
This commit is contained in:
Jonathan G Rennison
2022-09-07 19:42:06 +01:00
parent 5a39734a1c
commit a97d5cb333

View File

@@ -7091,14 +7091,13 @@ CommandCost CmdTemplateReplaceVehicle(TileIndex tile, DoCommandFlag flags, uint3
} }
} else { } else {
CommandCost buyCost = TestBuyAllTemplateVehiclesInChain(tv, tile); CommandCost buyCost = TestBuyAllTemplateVehiclesInChain(tv, tile);
if (!buyCost.Succeeded() || !CheckCompanyHasMoney(buyCost)) { if (!buyCost.Succeeded()) {
if (leaveDepot) incoming->vehstatus &= ~VS_STOPPED; if (leaveDepot) incoming->vehstatus &= ~VS_STOPPED;
if (buyCost.GetErrorMessage() == INVALID_STRING_ID) return_cmd_error(STR_ERROR_CAN_T_BUY_TRAIN);
if (!buyCost.Succeeded() && buyCost.GetErrorMessage() != INVALID_STRING_ID) { return buyCost;
return buyCost; } else if (!CheckCompanyHasMoney(buyCost)) {
} else { if (leaveDepot) incoming->vehstatus &= ~VS_STOPPED;
return_cmd_error(STR_ERROR_NOT_ENOUGH_CASH_REQUIRES_CURRENCY); return_cmd_error(STR_ERROR_NOT_ENOUGH_CASH_REQUIRES_CURRENCY);
}
} }
} }