Added ce_hitevents_healing_crossbow_only
#109
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When broadcasting, showing all healing events in combattext doesn't look
particularly good.
ce_hitevents_healing_crossbow_only
stops the combattext from showing
player_healed
events, but instead shows allcrossbow_heal
events as if they wereplayer_healed
events.Because we are hooking the
FireGameEvent
method on theCDamageAccountPanel, we wouldn't normally receive
crossbow_heal
events. We utilize our hook on
CDamageAccountPanel::ShouldDraw
tocatch the first invocation on a given
CDamageAccountPanel
instance andsubscribe to the event here.
We then add relevant checks and paths in our
FireGameEvent
hook. Whenwe encounter a
crossbow_heal
event, and crossbow-only mode is set, wecreate a new
player_healed
event with matching data and send this tothe original
CDamageAccountPanel::FireGameEvent
method.