Error same place for at least a year

[kreelor]

This error keeps cropping up on many of my alt chars.
It has happened repeatedly for at least one year! I don't know why?
andorhal-to-western-plaguelands.jpg

[Cartman]

I just clicked through that guide to that position without the error.
The travel system seems to use both flight points, Chillwind Camp and Andorhal for Alliance without problems (German WoW client).

I occasionally saw this error in the past but not in the last one or two months.
Maybe the devs have an idea what's happening.

[cabby]

That's an error with the LibRover file. Are you by chance using the Ace3 library as a standalone addon? It would seem that what's happening is the LibRover library is being loaded by a different addon that hasn't been updated to fix the error and WoW won't load the same library twice, it will just use the first one that gets loaded.

[Cartman]

I have Ace3 as standalone addon running (version 1104) and no libRover errors at least in the last month.
It seems the Ace3 that comes with the guide is a bit old, inside the file I see version 1036 from August 2011.

libRover should not be the problem as it is a Zygor exclusive library.

[kreelor]

It happened again

Same thing again (Feb 2, 2014)
Here's the error message that I captured.

Unless I'm missing it, I see no reference to "Ace" in this.

Date: 2014-02-02 06:10:28
ID: 1
Error occured in: Global
Count: 1
Message: ...ZygorGuidesViewer\Libs\LibRover-1.0\LibRover-1.0.lua line 2646:
   {{...\AddOns\ZygorGuidesViewer\Libs\LibRover-1.0\Node.lua line 42:
   attempt to concatenate field 'num' (a nil value)}}
Debug:
   [C]: error()
   ...ZygorGuidesViewer\Libs\LibRover-1.0\LibRover-1.0.lua:2646: OnUpdate()
   ...ZygorGuidesViewer\Libs\LibRover-1.0\LibRover-1.0.lua:730:
      ...ZygorGuidesViewer\Libs\LibRover-1.0\LibRover-1.0.lua:730
Locals:
(*temporary) = "{{...\AddOns\ZygorGuidesViewer\Libs\LibRover-1.0\Node.lua:42: attempt to concatenate field 'num' (a nil value)}}"

AddOns:
  Swatter, v5.18.5433 (PassionatePhascogale)
  AucAdvanced, v5.18.5433 (PassionatePhascogale)
  AucFilterBasic, v5.18.5433 (PassionatePhascogale)
  AucFilterOutlier, v5.18.5433.5347(5.18/embedded)
  AucMatchUndercut, v5.18.5433.5364(5.18/embedded)
  AucScanData, v5.18.5433 (PassionatePhascogale)
  AucStatHistogram, v5.18.5433 (PassionatePhascogale)
  AucStatiLevel, v5.18.5433 (PassionatePhascogale)
  AucStatPurchased, v5.18.5433 (PassionatePhascogale)
  AucStatSales, v5.18.5433.5376(5.18/embedded)
  AucStatSimple, v5.18.5433 (PassionatePhascogale)
  AucStatStdDev, v5.18.5433 (PassionatePhascogale)
  AucStatWOWEcon, v5.18.5433.5323(5.18/embedded)
  AucUtilAHWindowControl, v5.18.5433.5347(5.18/embedded)
  AucUtilAppraiser, v5.18.5433.5427(5.18/embedded)
  AucUtilAskPrice, v5.18.5433.5347(5.18/embedded)
  AucUtilAutoMagic, v5.18.5433.5415(5.18/embedded)
  AucUtilCompactUI, v5.18.5433.5427(5.18/embedded)
  AucUtilEasyBuyout, v5.18.5433.5427(5.18/embedded)
  AucUtilFixAH, v5.18.5433 (PassionatePhascogale)
  AucUtilItemSuggest, v5.18.5433.5417(5.18/embedded)
  AucUtilPriceLevel, v5.18.5433.5427(5.18/embedded)
  AucUtilScanButton, v5.18.5433.5403(5.18/embedded)
  AucUtilScanFinish, v5.18.5433.5347(5.18/embedded)
  AucUtilScanProgress, v5.18.5433.4979(5.18/embedded)
  AucUtilScanStart, v5.18.5433.5347(5.18/embedded)
  AucUtilSearchUI, v5.18.5433.5373(5.18/embedded)
  AucUtilSimpleAuction, v5.18.5433.5415(5.18/embedded)
  AucUtilVendMarkup, v5.18.5433.4828(5.18/embedded)
  Babylonian, v5.1.DEV.332(/embedded)
  BeanCounter, v5.18.5433 (PassionatePhascogale)
  Carbonite, v5.4.0 alpha 2
  CarboniteInfo, v1.0
  CarboniteNodes, v1.02
  CarboniteNotes, v1.1
  CarboniteQuests, v1.0
  CarboniteSocial, v1.0
  CarboniteTransfer, v1.01
  CarboniteWarehouse, v1.0
  Configator, v5.1.DEV.344(/embedded)
  DebugLib, v5.1.DEV.337(/embedded)
  Informant, v5.18.5433 (PassionatePhascogale)
  LibExtraTip, v5.12.DEV.355(/embedded)
  Notebook, v5.3.0.12
  Stubby, v5.18.5433 (PassionatePhascogale)
  TipHelper, v5.12.DEV.351(/embedded)
  ZygorGuidesViewer, v4.0
  BlizRuntimeLib_enUS v5.4.2.50400 <us>
  (ck=838)

ERROR-Andorhal to Western Plaguelands.jpg

[cabby]

Ok, so from Chillwind to Anderhol. I'll see if I can reproduce it for the dev team. Did it happen with step 12 displayed in the viewer or a different step when you were flying?


Edit: Was able to reproduce it so going to post this with some additional info I have for the team to see if they can find out what's happening since it's just this one flight path.

[myrroddin]

It would seem that what's happening is the LibRover library is being loaded by a different addon that hasn't been updated to fix the error and WoW won't load the same library twice, it will just use the first one that gets loaded.

LibRover, as Cartman pointed out, is a Zygor exclusive, as is LibTaxi. Both are loaded by LibStub, which indeed loads only one copy into the game memory –– the latest version, no matter where from it is originally loaded. That is what LibStub does, and why it was designed.

Also, having looked at the code to verify LibRover being loaded by LibStub, I see some highly inefficient design choices, chiefly wrapping the if/then that checks the minor version as being latest inside a do/end loop. Why would you do that? The major and minors are also defined twice, under different names. I suggest looking at LibStub's API and correct LibRover accordingly. It won't break anything, but some minor rewriting is in order.

local MAJOR_VERSION = "LibRover-1.0"
local MINOR_VERSION = tonumber(("$Revision: 1 $"):match("%d+"))
local Lib = LibStub:NewLibrary(MAJOR_VERSION, MINOR_VERSION)
-- eliminate the do/end loop and simply exit if not the latest version
if not lib then return end

-- continue code here
local BZ = LibStub("LibBabble-SubZone-3.0")
local BZL,BZR = BZ:GetUnstrictLookupTable(),BZ:GetReverseLookupTable()

-- etc

[cabby]

Yeah, I don't do any of the coding so I wasn't aware that those libraries were specific to us.

[kreelor]

Cabby, are you going to add this to the 'infamous' list of things for the developers to look at? (big grin). I think it's about time it gets fixed (after years!).