search

FozLand / travelnet

network of teleporter-boxes that allow easy travelling to other boxes on the same network (Mod for Minetest)
0 stars 0 forks source link

Not deleting travelnet path on explosion. #2

Closed mootpoint closed 8 years ago

mootpoint commented 8 years ago

When tnt explodes a Travelnet, it leaves the station, but returns the travelnet. You can travel to the destination but not from it. It should be expected if travelnet no longer is there, then... delete that path.

mootpoint commented 8 years ago

I've taken a look and i believe this : on_blast = function() end will fix this problem. if you want ill add it to this and you can ask for it to be pulled upstream?

jhcole commented 8 years ago

After further review, it looks like this was already fixed upstream by Sokomine/travelnet@121f8f61f23d128cde497125bbbbfb1293b34b19. The upstream solution is to delete the path and then send the player back to the originating travelnet.

Regardless of the bug, do you think travelnets should be immune to tnt?

EDIT: This question also applies to elevators.

mootpoint commented 8 years ago

Yes, specifically have in mind the mining nets people place, I'd like to encourage mining with tnt, but not at the risk of replacing those travelnets every day or giving players free travelnets. Talked to hd, he is against tnt all together, but did say no one should be able to harm others stuff. So I would say make them indestructible.

On Sep 24, 2016 1:25 PM, "John Cole" notifications@github.com wrote:

After further review, it looks like this was already fixed upstream by Sokomine/travelnet@121f8f6. The upstream solution is to delete the path and then send the player back to the originating travelnet.

Regardless of the bug, do you think travelnets should be immune to tnt?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or mute the thread.