medigun drains health does not run "on hit" attributes

[KenanTheFab]

Initial troubleshooting checklist

• The attribute is loaded, or this bug report is about a plugin load failure. (Check sm plugins list on the server.)
• The attribute is the most recent revision.

Describe the bug
The "medigun drains health" attribute appears to not count as a "hit", causing "X on Hit" attributes to not work with it.

To Reproduce
Apply the attribute to a weapon, latch onto an enemy and something like "heal on hit" will not work.

Expected behavior
"medigun drains health" hitting the enemy, not being an invisible leech.

Server environment (please complete the following information):

• Operating System: [Unsure.]
• SourceMod version: [1.10 - build 6512]

[nosoop]

Thanks for the report; this one's not quite a bug, rather behavior that wasn't fleshed out since it was for a joke weapon demonstration.

I think this is due to the damage function call not receiving a weapon here:

SM-TFCustomAttributeStarterPack/scripting/joke_medigun_mod_drain_health.sp

Lines 305 to 306 in adf5e1d

	SDKHooks_TakeDamage(healTarget, medigun, healer, float(damageInflicted),
	DMG_PREVENT_PHYSICS_FORCE | DMG_ALWAYSGIB);

But I also vaguely remember the game crashing if the medigun was provided back when it was originally in development, so if that's the case then it won't be that simple of a fix.

[KenanTheFab]

Understandable. Could theoretically be caused by an excess amount of damage calls?

I'm not into tf2 modding or w/e, but if that is the case then theoretically you could make it only detect as damage on the initial attachment, make it check every X seconds for a connection, or simply doing a workaround such as..... adding a bleed effect when attached, then allowing the user to change the bleed frequency.

(On a side-note, would love to be able to change how much Uber is gained after a medigun kill.)