Rubidium opened the ticket and wrote:

town_cmd.c at about row 815 has the following statements:
...
// Only work with plain land that not already has a house with map5=0
if (ti.tileh == 0 && (ti.type != MP_HOUSE || ti.map5 != 0)) {
if (!CmdFailed(DoCommandByTile(tile, 0, 0, DC_AUTO, CMD_LANDSCAPE_CLEAR))) {
DoCommandByTile(tile, GenRandomRoadBits(), t->index, DC_EXEC | DC_AUTO, CMD_BUILD_ROAD);
...

The problem is that 'ti.map5 != 0' part of the if-statements makes no sense.
What it means is that it only allows destroying houses when they are in some, but not all, stages of construction and it allows destroying finished houses with a lift.

The original meaning probably was that it only may destroy houses when they are in development. On the other hand, http://svn.openttd.org/cgi-bin/trac.cgi/browser/branch/map completly removes the part.

So I've created two patches, one that changes the 'ti.map5 != 0' to 'building not finished', meaning that no finished house will be replaced by a road, and the other removes the complete part, meaning that no house will be replaced by a road.

Attachments

• town_cmd_change_statement.diff (0.76 KiB)
• town_cmd_remove_statement.diff (0.72 KiB)

Reported version: trunk
Operating system: All

This issue was imported from FlySpray: https://bugs.openttd.org/task/49