ESOUI

ESOUI (https://www.esoui.com/forums/index.php)
-   Lua/XML Help (https://www.esoui.com/forums/forumdisplay.php?f=175)
-   -   EVENT_COMBAT_EVENT ActionResults (Wiki) (https://www.esoui.com/forums/showthread.php?t=6955)

ZOS_ChipHilseberg 05/16/17 01:06 PM

Quote:

Originally Posted by Kyoma (Post 30958)
It's this part that scares me:
It would break a large portion of Raid Notifier if you decide to strip the target unit id from the combat event.

Can you provide some specific examples?

Kyoma 05/16/17 02:19 PM

Quote:

Originally Posted by ZOS_ChipHilseberg (Post 30963)
Can you provide some specific examples?

Hmmm, well posting all the relevant code would be a bit messy but in short I use some code with EVENT_EFFECT_CHANGED to link a target unit id to a group unit tag and then in EVENT_COMBAT_EVENT I pair it back and display certain alerts based on conditions. The other target parameters are indeed empty so I suppose from a developer perspective you might actually have intended to NOT allow such things. I understand that in pvp this kind of stuff might allow unfair foresight but I do not believe it hurts anybody in pve.

ZOS_ChipHilseberg 05/17/17 08:01 AM

I've opened a dialogue with some of the game design teams about what we want to allow for combat addons like this. I'll post back when I hear something.

dorrino 05/18/17 12:53 AM

Quote:

Originally Posted by Kyoma (Post 30965)
Hmmm, well posting all the relevant code would be a bit messy but in short I use some code with EVENT_EFFECT_CHANGED to link a target unit id to a group unit tag and then in EVENT_COMBAT_EVENT I pair it back and display certain alerts based on conditions. The other target parameters are indeed empty so I suppose from a developer perspective you might actually have intended to NOT allow such things. I understand that in pvp this kind of stuff might allow unfair foresight but I do not believe it hurts anybody in pve.

I used pretty much this in my PvpAlerts before Homestead changes:)

Currently i wasn't able to find a way to link anonymous events by targetUnitId to EVENT_EFFECT_CHANGED ids to any benefit, since, at least in pvp the code seems only to send the later about allies.

I'm curious what is left in pve, that you built an addon feature out of it?

ZOS_ChipHilseberg 05/19/17 12:10 PM

After talking to the dungeon team they are willing to put in a buff for the light/dark state on the 2nd boss in Maw of Lorkahj which should allow tracking by addon. In the future we could open COMBAT_EVENT up a bit more but it would take a large audit of all the spells being cast by NPCs since there are a number that should not be known to a player client (this is also true of buffs and debuffs). This would be at a later date, but is something we want to clean up anyway.

Letho 05/19/17 03:56 PM

Thanks for your amazing work, chip, that's really good news! As a "rule of thumb" I feel that every effect that has a duration and is communicated by changing the visual appearance of your character (glowing hands, heads, etc.) should be communicated via EVENT_EFFECT_CHANGED for the sake of visibility in large scale combat situations. If you could add the fields sourceName and sourceUnitId or at least combatUnitType to EVENT_EFFECT_CHANGED it would make it absolutely perfect :)

Just out of curiosity: What type of buffs/debuffs/combat vvents shall not be visible to the client and why?

Solinur 06/12/17 07:17 AM

Quote:

Originally Posted by Letho (Post 30997)
Thanks for your amazing work, chip, that's really good news! As a "rule of thumb" I feel that every effect that has a duration and is communicated by changing the visual appearance of your character (glowing hands, heads, etc.) should be communicated via EVENT_EFFECT_CHANGED for the sake of visibility in large scale combat situations. If you could add the fields sourceName and sourceUnitId or at least combatUnitType to EVENT_EFFECT_CHANGED it would make it absolutely perfect :)

Just out of curiosity: What type of buffs/debuffs/combat vvents shall not be visible to the client and why?

sourceUnitType is already in EVENT_EFFECT_CHANGED.


All times are GMT -6. The time now is 11:08 PM.

vBulletin © 2024, Jelsoft Enterprises Ltd
© 2014 - 2022 MMOUI