Skip to content
This repository has been archived by the owner on Nov 5, 2018. It is now read-only.

Show friendly decoy owner #699

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

softashell
Copy link

Implemented it because it's pretty useful to know who are you getting teamkills for when killing yourself with decoy.
Only problem I have noticed so far is that when player disconnects the name field will be empty and I'm not really sure what's the best way to check if owner is still in game.

@brcly
Copy link
Member

brcly commented Feb 16, 2017

I'll wait for a 2nd opinion on this.

But looks good imo.

@NoahFriendo
Copy link
Contributor

Nice feature for griefing. It would also be useful to have this on HE grenades and molotovs/incendiary grenades.

@softashell
Copy link
Author

I wanted to have this for mollies as well but I have no idea where to even start since the molotov gets destroyed when flames spawn

@Bobbls
Copy link

Bobbls commented Feb 16, 2017

If you're trying to track mollys, do an event hook for molotov_detonate it returns the userid, and the x, y and z coords of the molly.

Then listening for inferno_extinguish, or inferno_expire for when its gone

@softashell
Copy link
Author

Thanks, that's pretty useful to know. I will give it a try tomorrow assuming nobody else better than me is interested in doing that.

@McSwaggens McSwaggens force-pushed the master branch 2 times, most recently from 406a400 to 43fa785 Compare February 17, 2017 06:13
@softashell
Copy link
Author

20170217173131

I managed to get to this with molotov tracking code, but I can't figure out how to make entity id that's reported into something usable, otherwise I would have to hook molotov_detonate for owner and then check if it actually exploded later in inferno_startburn. I can't just use molotov_detonate because it gets called even if the flame doesn't get created so I would prefer just to do it inside inferno_startburn.
You can see the current mess I made here.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants