WoWInterface

WoWInterface (https://www.wowinterface.com/forums/index.php)
-   Released AddOns (https://www.wowinterface.com/forums/forumdisplay.php?f=9)
-   -   LibResInfo - resurrection info without comms (https://www.wowinterface.com/forums/showthread.php?t=43933)

Phanx 08-05-16 09:28 PM

Re: One person casting multiple resurrects -- it's literally not possible to cast two spells at the same time, even two instant-cast spells. Once the first one is done casting, it moves in the pending state instead, where it's tracked by the recipient, not the caster.

I'm going to handle objects (totem, brazier, pylon) differently -- casting the summoning spell (eg. by right clicking the item) will register CLEU and listen for SPELL_SUMMON, which will insert the object's GUID into a new table that maps objectGUID=>casterGUID, and then listen for SPELL_RESURRECT events originating from the object (at which point the resurrection will be handled the same as any other), and UNIT_DIED to clear out the object GUID and unregister CLEU if there's nothing else going on.

p3lim 09-30-16 12:44 PM

You can put my oUF plugin under addons using LRI by the way, I intend to release it.

https://github.com/p3lim-wow/oUF_Resurrect

zivlaks 10-31-16 05:44 PM

hey can you please update the addon? Libresinfo makes an error pop up every time someone gets ressurected in combat

myrroddin 10-31-16 07:13 PM

Quote:

Originally Posted by zivlaks (Post 320532)
hey can you please update the addon? Libresinfo makes an error pop up every time someone gets ressurected in combat

What's the error? Phanx can't fix what she doesn't know.

doadin 11-17-16 01:01 AM

I dont know about others but im currently getting this error.

p3lim 11-17-16 04:25 AM

Quote:

Originally Posted by doadin (Post 320891)
I dont know about others but im currently getting this error.

A fix was applied 10 days ago, just no new release yet.

https://github.com/phanx-wow/LibResInfo/pull/12

And you would know this if you just looked at your Github notifications once in a while.

doadin 11-17-16 08:53 AM

Quote:

Originally Posted by p3lim (Post 320893)
A fix was applied 10 days ago, just no new release yet.

https://github.com/phanx-wow/LibResInfo/pull/12

And you would know this if you just looked at your Github notifications once in a while.

Actually i do and that fix is included in the Grid-7.1.0.1.zip on here which is also confirmed here which i have installed and im still getting this error. so...no?

Edit: also the errors dont even seem to be the same so did you even read it? maybe you miss read or followed a wrong link? Double checked my link and its right and its a different error.

not trying to sound mean or anything but if your gonna talk about me checking my info might wana check yours.

Anyways on topic.. it looks like there is a error which it seeing who is getting resed as also after that "fix" when people get mass resed its only showing one person as having a res pending.

p3lim 11-17-16 09:00 PM

Quote:

Originally Posted by doadin (Post 320896)
Actually i do and that fix is included in the Grid-7.1.0.1.zip on here which is also confirmed here which i have installed and im still getting this error. so...no?

Edit: also the errors dont even seem to be the same so did you even read it? maybe you miss read or followed a wrong link? Double checked my link and its right and its a different error.

not trying to sound mean or anything but if your gonna talk about me checking my info might wana check yours.

Anyways on topic.. it looks like there is a error which it seeing who is getting resed as also after that "fix" when people get mass resed its only showing one person as having a res pending.


You're right, here's a fix:
https://github.com/phanx-wow/LibResInfo/pull/13

doadin 11-17-16 10:08 PM

Quote:

Originally Posted by p3lim (Post 320909)

Nice! Thanks!

doadin 11-23-16 09:38 PM

Quote:

Originally Posted by p3lim (Post 320909)

Unfortunatly even after this fix there is still another error in ressing. Im thinking the ressing code as a whole needs to be revisited/looked over. This would be the fourth error produced. Anyways...here it is.

myrroddin 11-24-16 02:32 AM

Sadly, the most effective test bed AddOn, SmartRes2, has died with Legion. It requires a total rewrite, replacing several libraries, and coding some wrapper stuff around one or two newer libraries.

I'd love to update SR2, but I simply do not have the time. Hopefully your bug reports will help fix any issues.

p3lim 11-24-16 02:54 AM

Quote:

Originally Posted by doadin (Post 320983)
Unfortunatly even after this fix there is still another error in ressing. Im thinking the ressing code as a whole needs to be revisited/looked over. This would be the fourth error produced. Anyways...here it is.

Those lines don't match up with anything in LibResInfo, are you sure you've updated?

Edit: Seems like Grid hasn't updated its embedded version of LibResInfo. Try installing LRI separately.

doadin 11-24-16 11:15 AM

Quote:

Originally Posted by p3lim (Post 320989)
Those lines don't match up with anything in LibResInfo, are you sure you've updated?

Edit: Seems like Grid hasn't updated its embedded version of LibResInfo. Try installing LRI separately.

Thats basically what i did i noticed grid hasn't updated so i have the Grid 7.1.0.1 and removed/replaced the libres files. May be my fault on this one I am going to try and reinstall and see if its fixed because yes i see where it is different now. Ill let you know if i still get a error. My Apologies.

Phanx 05-04-18 01:23 PM

I've added WoW 8.x (BfA) compatibility, but can't test it since I don't have beta access, and don't expect to get a beta invite since my account doesn't have an active subscription.

If you do have beta access, and are the author or user of an addon that uses LibResInfo, grab the latest version from GitHub (https://github.com/phanx-wow/LibResInfo) and let me know whether it's working.

If it's not working, please open a ticket on GitHub and/or post in this thread with the error message. If there's no error message, run "/lri 3" to enable additional debugging (slash command only present with standalone version) and copy or screenshot the resulting messages. If you use ChatFrame3 for something already, use e.g. "/lri ChatFrame7" to move debugging to a different chat frame.


All times are GMT -6. The time now is 07:32 AM.

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