first pvplog won't upload to wowroster...

second, 0.5.2 is generating errors in wow

Count: 15
Error: Interface\AddOns\PvPLog\PvPLog.lua:1404: attempt to index field `guilds' (a nil value)
--------------------------------------------------
Count: 1
Error: Interface\AddOns\PvPLog\PvPLog.lua:1243: attempt to compare number with nil
--------------------------------------------------
Count: 8
Error: Interface\AddOns\PvPLog\PvPLog.lua:1275: bad argument #1 to `foreach' (table expected, got nil)


those are the errors i'm getting... so far, there have been a few others.

anyone?

same problems here. I keep getting errors, I decided to downgrade.

downgrade to what?

dewce wrote:Error: Interface\AddOns\PvPLog\PvPLog.lua:1243: attempt to compare number with nil

dewce wrote:Error: Interface\AddOns\PvPLog\PvPLog.lua:1275: bad argument #1 to `foreach' (table expected, got nil)

dewce wrote:Error: Interface\AddOns\PvPLog\PvPLog.lua:1404: attempt to index field `guilds' (a nil value)

I'm surprised that more ppl don't report these. They have been in the code for a long time.

I think I have them all fixed. I'll have to test when I get back in game.

I found a fix for it.

The reason you get this error is because the array (or in this case known as a: table) doesn't contain any information.

Now, why wouldn't it contain any information? Well if you haven't used PvPlog before (or used an old version), you probably have a completely fresh logfile or whatever (with data: nil).

Run a:

/pl reset confirm
and it should fix itself.

For the coder:
You need to make sure that the array always contains information.

(one solution might be as simple as adding zeros to that array. Just make sure this isnt some auto-shit so old users logs get overwritten)

Common programming error I would say, and nothing to worry about

Right. I might be able to massage the data so that this never occurs but it seemed easier to handle the nil cases so that the error is never raised.

For example, this

Error: Interface\AddOns\PvPLog\PvPLog.lua:1404: attempt to index field `guilds' (a nil value)

is caused by an attempt to loop through all the data in an empty table. The fix is to only try to process the table if it is not empty.

Code: Select all

   if (table.getn(PvPLogData[realm][player].guilds) > 0) then

Has this been resolved?