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: 03/Dec/08 11:00 PM  Due: 23/Feb/08
Component/s: Storage, Storage - Zone Tokens
Affects Version/s: 3.0.0, 3.0.1, 3.0.2, 3.0.3, 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   Transitions   FishEye   Crucible      Sort Order: Ascending order - Click to sort in descending order
Ed Barton (RockSlice) added a comment - 18/Feb/08 03:58 PM
put the corrent error in

Esamynn added a comment - 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 added a comment - 18/Feb/08 04:01 PM
This is a MUST FIX for next version of Gatherer.

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

Esamynn added a comment - 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) added a comment - 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 =)