Thread Tools Display Modes
10-14-12, 07:08 AM   #41
p3lim
A Pyroguard Emberseer
 
p3lim's Avatar
AddOn Author - Click to view addons
Join Date: Feb 2007
Posts: 1,710
Holy and Protection Paladins also get Rebirth from Symbiosis
Code:
[113269] = GetSpellInfo(113269), -- Rebirth (prot/holy paladin symbiosis)
  Reply With Quote
10-15-12, 03:13 AM   #42
Phanx
Cat.
 
Phanx's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2006
Posts: 5,617
Added, thanks.
__________________
Retired author of too many addons.
Message me if you're interested in taking over one of my addons.
Don’t message me about addon bugs or programming questions.
  Reply With Quote
10-16-12, 05:47 AM   #43
Trelis
A Murloc Raider
Join Date: Jul 2012
Posts: 5
I reported this originally as part of a ticket to Grid:
http://www.wowace.com/addons/grid/ti...resurrect-lua/

It was marked fixed for Grid, but the bug is still present. I thought it would be a good idea to report it here, as the bug is in this library.

Grid-r1536\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:185: attempt to compare nil with number
Grid-r1536\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:185: in function "UnitHasIncomingRes"
Grid-r1536\Statuses\GridStatusResurrect.lua:117: in function "?"
libs\CallbackHandler-1.0\CallbackHandler-1.0-6.lua:147: in function <libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:147>
<string>:"safecall Dispatcher[6]":4: in function <string>:"safecall Dispatcher[6]":4
<in C code>
<string>:"safecall Dispatcher[6]":13: in function "?"
libs\CallbackHandler-1.0\CallbackHandler-1.0-6.lua:92: in function "Fire"
Grid-r1536\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:426: in function "?"
Grid-r1536\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:127: in function <Grid\Libs\LibResInfo-1.0\LibResInfo-1.0.lua:126>

The lua error can be removed with the following, although there is probably a better fix.

$ diff LibResInfo-1.0.lua.old LibResInfo-1.0.lua
185c185
< if not firstEnd or endTime < firstEnd then
---
> if endTime and (not firstEnd or endTime < firstEnd) then

I do not know why sometimes "local endTime = castEnd[casterGUID]" is nil.
  Reply With Quote
10-16-12, 03:42 PM   #44
Phanx
Cat.
 
Phanx's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2006
Posts: 5,617
That should have been resolved already. Can you be more specific about the circumstances under which you are seeing that error? How many players in the group are dead? How many res spells are being cast in the group? Are any of them Mass Res? etc.
__________________
Retired author of too many addons.
Message me if you're interested in taking over one of my addons.
Don’t message me about addon bugs or programming questions.
  Reply With Quote
10-17-12, 12:22 AM   #45
Trelis
A Murloc Raider
Join Date: Jul 2012
Posts: 5
Edit2: I have not had this issue in Grid-r1542, so it is looking like it has been fixed.

It is not resolved for me. It is still happening in Grid-r1540. I will try to turn my sound notification back on so that I can give more information on exactly when it is happening. From the timestamp of the first error it does seem to be from Mass Ressurect though. The timestamp of the error is from when the first Mass Ressurect finished and applied "Recently Mass Resurrected" to some people and a second Mass Ressurect cast started. Could the bug have something to do with multiple mass resurrects at the same time?

10/16 18:32:57.206 SPELL_CAST_START,0x03000000073E59B1,"Malarie",0x512,0x0,0x0000000000000000,nil,0x80000000,0x80000000,83968,"Mass Resurrection",0x2
...

10/16 18:33:06.939 SPELL_RESURRECT,0x03000000073E59B1,"Malarie",0x10512,0x0,0x03000000067FCB9D,"Pixel",0x512,0x0,83968,"Mass Resurrection",0x2
10/16 18:33:06.939 SPELL_RESURRECT,0x03000000073E59B1,"Malarie",0x10512,0x0,0x0300000006C89A16,"Yitzak",0x514,0x0,83968,"Mass Resurrection",0x2
10/16 18:33:06.939 SPELL_RESURRECT,0x03000000073E59B1,"Malarie",0x10512,0x0,0x03000000063F0DC3,"Ruined",0x514,0x0,83968,"Mass Resurrection",0x2
10/16 18:33:06.939 SPELL_RESURRECT,0x03000000073E59B1,"Malarie",0x10512,0x0,0x0300000006C8467F,"Treliss",0x511,0x0,83968,"Mass Resurrection",0x2

[18:33:08 is the timestamp of the lua error]
0/16 18:33:08.315 SPELL_AURA_APPLIED,0x03000000067FCB9D,"Pixel",0x512,0x0,0x03000000067FCB9D,"Pixel",0x512,0x0,95223,"Recently Mass Resurrected",0x1,DEBUFF
10/16 18:33:08.528 SPELL_AURA_APPLIED,0x03000000063F0DC3,"Ruined",0x514,0x0,0x03000000063F0DC3,"Ruined",0x514,0x0,95223,"Recently Mass Resurrected",0x1,DEBUFF
10/16 18:33:08.742
SPELL_CAST_START,0x030000000732C3A7,"Xebren",0x514,0x0,0x0000000000000000,nil,0x80000000,0x80000000,83968,"Mass Resurrection",0x2
[end of the second with the timestamp of the lua error]

10/16 18:33:09.669 SPELL_AURA_APPLIED,0x0300000006C8467F,"Treliss",0x511,0x0,0x0300000006C8467F,"Treliss",0x511,0x0,95223,"Recently Mass Resurrected",0x1,DEBUFF
10/16 18:33:14.188 SPELL_AURA_APPLIED,0x0300000006C89A16,"Yitzak",0x514,0x0,0x0300000006C89A16,"Yitzak",0x514,0x0,95223,"Recently Mass Resurrected",0x1,DEBUFF

2012/10/16 18:33:08
70x Grid-r1540\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:185: attempt to compare nil with number
Grid-r1540\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:185: in function "UnitHasIncomingRes"
Grid-r1540\Statuses\GridStatusResurrect.lua:117: in function "?"
libs\CallbackHandler-1.0\CallbackHandler-1.0-6.lua:147: in function <libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:147>
<string>:"safecall Dispatcher[6]":4: in function <string>:"safecall Dispatcher[6]":4
<in C code>
<string>:"safecall Dispatcher[6]":13: in function "?"
libs\CallbackHandler-1.0\CallbackHandler-1.0-6.lua:92: in function "Fire"
Grid-r1540\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:502: in function "?"
Grid-r1540\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:127: in function <Grid\Libs\LibResInfo-1.0\LibResInfo-1.0.lua:126>

Locals:
nil

Last edited by Trelis : 10-19-12 at 05:14 PM.
  Reply With Quote
10-23-12, 02:35 PM   #46
Goatbert
A Defias Bandit
Join Date: May 2006
Posts: 2
I was just looking up that same error - happened in LFR, while in combat (kept triggering, several times during a fight). 2-3 people dead. Will grab the error next time it pops up (if it pops up again) lost it when I restarted.
  Reply With Quote
10-23-12, 06:55 PM   #47
Phanx
Cat.
 
Phanx's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2006
Posts: 5,617
That issue was resolved on October 15th in LibResInfo r27. If you are using an addon which embeds LibResInfo, make sure you have a version of that addon released after October 15th, or you probably still have an old version of the library.

Also, while I appreciate that you wanted to be helpful, just saying "I got an error" without any other information -- like what version of what addon(s) you are using, what region and language you are playing WoW in, etc. -- does not actually give me anything I can use to track down the bug and fix it (or tell you why it's not a bug).
__________________
Retired author of too many addons.
Message me if you're interested in taking over one of my addons.
Don’t message me about addon bugs or programming questions.

Last edited by Phanx : 10-23-12 at 06:57 PM.
  Reply With Quote
10-30-12, 01:40 PM   #48
dracula
A Murloc Raider
AddOn Author - Click to view addons
Join Date: Aug 2006
Posts: 7
I replaced Grid's version of LibResInfo with the r27 you mention, but it spams this error during a mass-res, on 25-man even up to a point that the game complains about 'There are many addon errors in your UI'.

90x Grid-5.0.5.1539\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:185: attempt to compare nil with number
Grid-5.0.5.1539\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:185: in function "UnitHasIncomingRes"
Grid-5.0.5.1539\Statuses\GridStatusResurrect.lua:117: in function "?"
libs\CallbackHandler-1.0\CallbackHandler-1.0-6.lua:147: in function <libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:147>
<string>:"safecall Dispatcher[5]":4: in function <string>:"safecall Dispatcher[5]":4
<in C code>
<string>:"safecall Dispatcher[5]":13: in function "?"
libs\CallbackHandler-1.0\CallbackHandler-1.0-6.lua:92: in function "Fire"
Grid-5.0.5.1539\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:524: in function "?"
Grid-5.0.5.1539\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:127: in function <Grid\Libs\LibResInfo-1.0\LibResInfo-1.0.lua:126>

Locals:
nil
  Reply With Quote
10-31-12, 07:32 AM   #49
p3lim
A Pyroguard Emberseer
 
p3lim's Avatar
AddOn Author - Click to view addons
Join Date: Feb 2007
Posts: 1,710
5.1 changes protection paladin symbiosis again (they lose their rebirth).
Shouldn't affect this addon at all, but I thought I'd let you know.
  Reply With Quote
10-31-12, 07:44 AM   #50
myrroddin
A Pyroguard Emberseer
 
myrroddin's Avatar
AddOn Author - Click to view addons
Join Date: Oct 2008
Posts: 1,240
General question: I am getting kind of fed up with bugfixing SmartRes2, looking for a reason why you get cast bars for other people, but not your own; also, because your cast bars are tied to chat output, you don't spit out any messages. I am 99% convinced the problem stems from LibResComm, where I changed the event registration to RegisterUnitEvent from RegisterEvent and checking unit == player. That shouldn't have made any difference, but there you go.

That leaves me with the rewrite I was eventually planning, using LibResInfo instead. Based on the last few posts, I am wondering if this lib is ready to be used? Should I wait until 5.1, which will be at least a month or more away? Yes, LRI is "release", but with some bugs still being reported, and a possible breaking change to Paladins, I am just looking for confirmation.
  Reply With Quote
10-31-12, 07:05 PM   #51
Phanx
Cat.
 
Phanx's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2006
Posts: 5,617
I'm using it without any issues in both Grid and oUF_Phanx. I commited r28 weeks ago to add extra checks for the endTime error with Mass Res (even though I never saw the error myself), so any addons pulling from SVN should not have any problem. I guess I should post a new ZIP on the download page.

The removal of paladin Rebirth should not cause any problems, unless Blizzard breaks with 8 years of precedent and assigns a new spell to the spell ID previously assigned to the paladin Rebirth spell. Nonetheless, I will update the lib to remove the ID from the table of known res spells after the change goes live.
__________________
Retired author of too many addons.
Message me if you're interested in taking over one of my addons.
Don’t message me about addon bugs or programming questions.

Last edited by Phanx : 10-31-12 at 07:12 PM.
  Reply With Quote
10-31-12, 07:23 PM   #52
Phanx
Cat.
 
Phanx's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2006
Posts: 5,617
Originally Posted by dracula View Post
I replaced Grid's version of LibResInfo with the r27 you mention, but it spams this error during a mass-res, on 25-man even up to a point that the game complains about 'There are many addon errors in your UI'.
Yeah, don't do that. Grid is pulling newer versions of LRI directly from SVN. The ZIP on the download page mainly exists because it's not possible to have a "download" page with no download, but it's not really there for anyone to use directly. Developers should be embedding it in their addons from SVN, and users should be downloading it embedded inside of addons.

If you are getting errors from LRI while using any version of Grid posted on or after October 17th as-is, please post the actual error message(s) and a description of what was happening when the error occured.
__________________
Retired author of too many addons.
Message me if you're interested in taking over one of my addons.
Don’t message me about addon bugs or programming questions.
  Reply With Quote
11-01-12, 12:12 PM   #53
dracula
A Murloc Raider
AddOn Author - Click to view addons
Join Date: Aug 2006
Posts: 7
Yeah, I know what libraries are...

But you mentioned that addons updated after October 15th have a fixed version embedded, and I still get LibResInfo errors with Grid which was updated October 16th. So then I start trying things... but I'll await r28.
  Reply With Quote
11-01-12, 03:44 PM   #54
Phanx
Cat.
 
Phanx's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2006
Posts: 5,617
r28 was committed to SVN on October 16th. These versions of Grid all contain r28:
__________________
Retired author of too many addons.
Message me if you're interested in taking over one of my addons.
Don’t message me about addon bugs or programming questions.
  Reply With Quote
11-01-12, 06:13 PM   #55
dracula
A Murloc Raider
AddOn Author - Click to view addons
Join Date: Aug 2006
Posts: 7
Aye, just switched to Grid alpha version to get it.

This error triggered twice when I left the raid:

Code:
stack:
Grid-r1546\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:324: attempt to compare number with nil
Grid-r1546\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:324: in function "?"
Grid-r1546\Libs\LibResInfo-1.0\LibResInfo-1.0-4.lua:127: in function <Grid\Libs\LibResInfo-1.0\LibResInfo-1.0.lua:126>

locals:
self = <unnamed> {
	0 = <userdata>
	UNIT_SPELLCAST_SUCCEEDED = <func> @Grid\Libs\..\LibResInfo-1.0.lua:509
	callbacks = <table> {}
	UNIT_SPELLCAST_START = <func> @Grid\Libs\..\LibResInfo-1.0.lua:486
	INCOMING_RESURRECT_CHANGED = <func> @Grid\Libs\..\LibResInfo-1.0.lua:406
	RegisterCallback = <func> @TomTom\libs\..\CallbackHandler-1.0.lua:118
	GROUP_ROSTER_UPDATE = <func> @Grid\Libs\..\LibResInfo-1.0.lua:247
	UNIT_HEALTH = <func> @Grid\Libs\..\LibResInfo-1.0.lua:640
	UnregisterCallback = <func> @TomTom\libs\..\CallbackHandler-1.0.lua:181
	COMBAT_LOG_EVENT_UNFILTERED = <func> @Grid\Libs\..\LibResInfo-1.0.lua:580
	UNIT_SPELLCAST_INTERRUPTED = <func> @Grid\Libs\..\LibResInfo-1.0.lua:538
	UNIT_SPELLCAST_STOP = <func> @Grid\Libs\..\LibResInfo-1.0.lua:538
	UNIT_AURA = <func> @Grid\Libs\..\LibResInfo-1.0.lua:640
	UnregisterAllCallbacks = <func> @TomTom\libs\..\CallbackHandler-1.0.lua:202
}
event = "GROUP_ROSTER_UPDATE"
  Reply With Quote
11-01-12, 11:23 PM   #56
Phanx
Cat.
 
Phanx's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2006
Posts: 5,617
Okay, see, that's why I need the actual error message. That is NOT the same error that was reported before; it's in a completely different part of the code, and has nothing to do with Mass Res. >_<
__________________
Retired author of too many addons.
Message me if you're interested in taking over one of my addons.
Don’t message me about addon bugs or programming questions.
  Reply With Quote
11-04-12, 04:40 PM   #57
dracula
A Murloc Raider
AddOn Author - Click to view addons
Join Date: Aug 2006
Posts: 7
But.... I posted the exact error I got with r27, and I posted the exact error I got with r28.

It is occurring when me or another player leaves the raid, with everyone alive (ie. no resses going on).
  Reply With Quote
11-04-12, 05:46 PM   #58
Magicalhats27
A Fallenroot Satyr
 
Magicalhats27's Avatar
Join Date: Nov 2012
Posts: 24
Worth it

Originally Posted by Phanx View Post
[b]communication, so it is not necessary for anyone else in your group to have it installed.
Thats great. Actually this fact alone makes it worth the download. Lots of time saved - Thanks
  Reply With Quote
01-14-13, 05:59 AM   #59
myrroddin
A Pyroguard Emberseer
 
myrroddin's Avatar
AddOn Author - Click to view addons
Join Date: Oct 2008
Posts: 1,240
I'm finally at a point in the game where the daily quest grind is slowing down, and that means back to addon development. Phanx, do you prefer the pkgmeta from Curseforge/WoWAce to pull from the WoWInterface or Curseforge repository for this lib? I'm leaning to CF so you get author reward points, but wanted to check first.

On a side note, expect the WoWace forums' developer thread to get hit with questions, but not necessarily about LibResInfo
  Reply With Quote
01-14-13, 07:44 PM   #60
Phanx
Cat.
 
Phanx's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2006
Posts: 5,617
Originally Posted by myrroddin View Post
Phanx, do you prefer the pkgmeta from Curseforge/WoWAce to pull from the WoWInterface or Curseforge repository for this lib? I'm leaning to CF so you get author reward points, but wanted to check first.
There is no CurseForge repository. Files on CurseForge are manually uploaded. Your .pkgmeta will need to specify the WoWI repository URL. If you want me to get points, specify "libresinfo" (the CurseForge project name) in the "tools-used" section of your .pkgmeta, just like you do for LibDataBroker, but for reasons that actually make sense.
__________________
Retired author of too many addons.
Message me if you're interested in taking over one of my addons.
Don’t message me about addon bugs or programming questions.
  Reply With Quote

WoWInterface » AddOns, Compilations, Macros » Released AddOns » LibResInfo - resurrection info without comms

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off