let_there_be_flight icon indicating copy to clipboard operation
let_there_be_flight copied to clipboard

fix random activations on unmounted/npc vehicles

Open haywirephoenix opened this issue 2 years ago • 2 comments

add only check/activate for player not npcs and in non mounted vehicles add only run once on load instead of every mount helps prevent random activations

haywirephoenix avatar Jun 20 '22 08:06 haywirephoenix

I think I'd rather refine the activation height than disable it for NPCs - there might be a better place to check as well, where non-mounted vehicles could be activated.

This file is the result of running the tools/build.bat script, and gets overwritten during that process, so any change should really be made in the src/redscript/let_there_be_flight/* file.

jackhumbert avatar Jun 20 '22 10:06 jackhumbert

I think I'd rather refine the activation height than disable it for NPCs - there might be a better place to check as well, where non-mounted vehicles could be activated.

It is definately cool to see the odd NPC trying to fly but as you can imagine they can't fly well! I had trouble with empty cars activating when a vehicle is spawned in after a mission. There are cases where NPCs flight mode would activate on collision.

All of the above seems to be solved by these changes, plus it's not performing the check on every player entry plus NPCs.

This file is the result of running the tools/build.bat script, and gets overwritten during that process, so any change should really be made in the src/redscript/let_there_be_flight/* file.

Okay will do, thanks

haywirephoenix avatar Jun 20 '22 14:06 haywirephoenix