squidy5 / cargo_ships

factorio mod that adds cargo ships and other content to the game
25 stars 39 forks source link

landfill can't be placed over any kind of track, not just water ways #103

Open ChuckSavage opened 2 years ago

ChuckSavage commented 2 years ago

landfill can't be placed over track that can be placed on water, not just water ways. Such as the beautiful bridge railway mod.

robot256 commented 2 years ago

How big an issue is this? Wouldn't you want to replace the bridges with normal rails anyways? The landfill exclusion system is a little complicated, and does check for any collision with train-layer, so I'm hesitant to change it.

ChuckSavage commented 2 years ago

It's a big issue if you're in a ship - because you can't get out onto water.

robot256 commented 2 years ago

Even with the Shift-Enter command?

ChuckSavage commented 2 years ago

I'm not sure - I used the option to connect shipping ways to regular track then drove the boat over land to get out. At the time though I tried to use landfil, but couldn't put it under the shipping lanes or my bridge track from the beautiful bridge railways mod.

robot256 commented 2 years ago

Turns out, if you had gotten into the body of the ship instead of the engine, the shift-enter command doesn't work. [Edit: Actually, this only applied to entering the boat, not exiting.] (Pretty sure that code is so old, it was written when you could only ride in locomotives!) I'll fix that. If you get stuck in a boat again I'd like to see the save file!

robot256 commented 1 year ago

Resolved by df11f63c8698fc0c7b59376a6b2813ae862ce778 and release 0.1.19.