History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: GATH-38
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Blocker Blocker
Assignee: Esamynn
Reporter: Ed Barton (rockslice)
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Gatherer

Error on PTR with regards to new zone

Created: 18/Feb/08 03:55 PM   Updated: 27/Feb/08 01:49 PM  Due: 23/Feb/08
Component/s: Database
Affects Version/s: 3.0.2, 3.0.3, 3.0.0, 3.0.1, 3.0.4
Fix Version/s: 3.0.5

Environment: This occurs on PTR
Issue Links:
Reference
 


 Description  « Hide
Chat Output: Astrolabe is missing data for Isle of Quel'Danas

Also:

Date: 2008-02-18 14:57:59
ID: 13
Error occured in: Global
Count: 1
Message: ..\AddOns\Gatherer\GatherZoneTokens.lua line 125:
   table index is nil
Debug:
   [C]: ?
   Gatherer\GatherZoneTokens.lua:125: in main chunk
AddOns:
  AucAdvanced, v5.0.DEV
  AucDb, v<%version%>
  AucFilterBasic, v<%version%> (<%codename%>)
  AucFilterOutlier, v<%version%> (<%codename%>)
  AucMatchBeanCount, v<%version%> (<%codename%>)
  AucMatchUndercut, v<%version%> (<%codename%>)
  AucStatClassic, v<%version%> (<%codename%>)
  AucStatDebug, v<%version%> (<%codename%>)
  AucStatPurchased, v<%version%> (<%codename%>)
  AucStatSales, v<%version%> (<%codename%>)
  AucStatSimple, v<%version%> (<%codename%>)
  AucStatStdDev, v<%version%> (<%codename%>)
  AucStatWOWEcon, v<%version%> (<%codename%>)
  AucUtilAppraiser, v<%version%> (<%codename%>)
  AucUtilAskPrice, v<%version%> (<%codename%>)
  AucUtilAutoMagic, v<%version%> (<%codename%>)
  AucUtilCompactUI, v<%version%> (<%codename%>)
  AucUtilMover, v<%version%> (<%codename%>)
  AucUtilPriceLevel, v<%version%> (<%codename%>)
  AucUtilScanButton, v<%version%> (<%codename%>)
  AucUtilScanFinish, v<%version%> (<%codename%>)
  AucUtilScanProgress, v<%version%> (<%codename%>)
  AucUtilSearchUI, v<%version%> (<%codename%>)
  AucUtilVendMarkup, v<%version%> (<%codename%>)
  Auctioneer, v4.1.DEV
  Babylonian, v<%version%> (<%codename%>)
  BeanCounter, v<%version%> (<%codename%>)
  BlizzardFeedbackUI, v2.3.3
  Configator, v<%version%> (<%codename%>)
  DebugLib, v<%version%> (<%codename%>)
  Enchantrix, v4.0.DEV
  EnchantrixBarker, v<%version%> (<%codename%>)
  EnhTooltip, v4.9.DEV
  Gatherer, v3.0-DEV
  Informant, v4.1.DEV
  nLog, v4.9.DEV
  SlideBar, v<%version%> (<%codename%>)
  Stubby, v52
  Swatter, v4.1.DEV



 All   Comments   Change History   FishEye      Sort Order:
Ed Barton (rockslice) - 18/Feb/08 03:58 PM
put the corrent error in

Esamynn - 18/Feb/08 03:58 PM
Gatherer should handle new zones and zones it doesn't recognize because of a map file name change gracefully.

By graceful, I mean that it should use the new map file name as a token in the database, and treat the zone as essentially new, and then when updated, move the data saved under the temp token into table for the real token.

Esamynn - 18/Feb/08 04:01 PM
This is a MUST FIX for next version of Gatherer.

Esamynn - 26/Feb/08 01:20 AM
Fixed the code bug, and added a notification to the user when a zone is not recognized.

Esamynn - 26/Feb/08 05:57 PM
For the record, the current notification text (with highlight info) is: "Gatherer was unable to identify the following zones: "..HIGHLIGHT_FONT_COLOR_CODE.."%1|r. \nIf these are new zones, then this is not a problem, and you can continue as normal. \nIf these are not new zones, then remain calm, "..HIGHLIGHT_FONT_COLOR_CODE.."your data IS NOT LOST!|r Your data for these zones is still intact, but you will need to update Gatherer in order to access it. Until then you can continue as normal and any new data you collect will be merged with your old data once you upgrade. \n\n"..HIGHLIGHT_FONT_COLOR_CODE.."Please upgrade Gatherer when convenient. |r"

Nikk (Nechckn) - 26/Feb/08 06:04 PM
As for the warning itself, it may be helpful to simply state that "Gatherer has detected a new zone <name>." Rather than saying that it can't find any data, that may lessen the "fear factor". Then continue with your text as you have it.

Vile, copy-> paste of comment =)