WowAce

James D. Callahan III:
    - Moved the code for UpdateSite() directly into UpdateAllSites() since that's the only place it was ever called.
    - Logic cleanup.
    - Refactored UpdateSite() - variable renames for sanity, rearranged some things, and removed a completely useless call to GetArchaeologyRaceInfo().
    - Added data-mining code, courtesy of sidji.
    - Moved body of GetContinentSites() into UpdateSites() - no idea why they were separate functions, since the former was only ever used in the latter.
    - Renamed DIG_LOCATION_TEXTURE to DIG_LOCATION_TEXTURE_INDEX
    - Utilized the new releaseHandler parameter for LibQTip-1.0:SetAutoHideDelay() so Archy isn't clobbering another AddOn's tooltip.
    - Added IsInInstance check to PLAYER_ENTERING_WORLD; calls HideFrames if true, ShowFrames if false.
    - Added HideFrames and ShowFrames functions.
    - Renamed ShouldBeHidden() to FramesShouldBeHidden()
    - Changed PointsOfInterest to store only active POIs (as a key instead of a value), and removed the .active field from the POIs.
    - Renamed allPois to PointsOfInterest.
    - Combined ClearSitePOI() and ClearSurveyPOI() into ClearPOI()
    - Dead code removal.
    - Cleanup.
    - Moved the event handlers to the bottom of the file, pending relocation to their own file.
    - Comment  fix.
    - Moved initialization of CRATE_USE_STRING and digsitesTrackingID into :OnEnable() and greatly simplified the code required to do so.
    - Alphabetized RegesterEvent calls by event name for ease of skimming.
    - Do not perform tooltip scanning to get CRATE_USE_STRING if it's already set. Don't iterate through tracking types if digsitesTrackingID is already set...
    - Added LibTextDump-1.0 so the debug code will actually work.
    - Guard against nil artifactIndex.
    - Added RAD_135 and SQUARE_HALF constants. These should NOT have been constantly re-calculated in a damn OnUpdate.
    - Replaced convoluted StateDriver code with straightforward Show/Hide semantics for combat. Now it Just Works (™). Fixes ticket #494.
    - Added debug frame.
    - Changed subcommand handler into a table lookup, instead of an insanely long if-else chain.
    - Ensure that there is a value for distance, 0, in case Astrolabe does not return one. Fixes ticket #529
    - Don't just check if artifactSolved.raceId is greater than zero before announcing what was solved; make sure it matches the ID of the current race in the iteration so we announce the correct one...
    - Cleanup.
    - Corrected variable naming.
    - Replaced GetArtifactStats() with Race:GetArtifactCompletionDataByName() - uses name to index mapping table instead of iterating over indices to check for name matches.
    - Map artifact names to indices upon race creation, to do away with iterating the indices for stats lookups.
    - Dead code removal.
    - Severely reduced the number of table lookups in Archy:ScanBags()
    - Re-added assignment of keystone inventory value...whoops...
    - Logic simplification.
    - Removed artifact_data table in favor of each race entry containing its own artifact information, and replaced UpdateRaceArtifact() with Race:UpdateArtifact()
    - Don't check for the Races table being empty before initializing - it will be at this point.
    - Moved DB files into Database dir.
    - Removed useless/outdated comment.
    - Began revamp on race-handling: Initialize a race explicitly by ID when the AddOn is enabled, instead of upon an index attempt on the list table (initialization was happening on UpdatePlayerPosition, if the list table was empty, FFS), and moved the keystone cache-processing queue to Race.lua
    - Variable renames for clarification of purpose.
    - Cleanups and reformatting.