Doesn't PLAYER_EQUIPMENT_CHANGED already take care of that?
I use it to update when we change specialization (change weapon) and it works fine for that at least.
As it was talked about, the problem for me with UNIT_INVENTORY_CHANGED is that is fires way too much and I don't think that's a good implementation. I know it won't really do anything harmful in an addon this size, but since I'm doing this to learn I don't think learning bad ways to do something is a good idea.
That's why I started to look for events that were more specific to what this addon does, so that the addon only updates when it's actually relevant.
I know I probably have too much now but that's just because I'm still testing them, I probably have that part different every day xD
|