wow/ouroloot: core.lua comparison

comparison core.lua @ 66:43913e02a1ef

Detect LFR loot as best we can, and bundle it into the same warning given for heroic loot formatted by name only. Less tedious method of bumping data revisions.

author	Farmbuyer of US-Kilrogg <farmbuyer@gmail.com>
date	Fri, 27 Apr 2012 10:11:56 +0000
parents	69fd720f853e
children	c01875b275ca

comparison

equal deleted inserted replaced

-:69fd720f853e
+:43913e02a1ef
 - disposition   offspec/gvault/shard; missing otherwise; can be set from
 the extratext field
 - count         e.g., "x3"; missing otherwise; can be set/removed from
 extratext; triggers only for a stack of items, not "the boss
 dropped double axes today"
-- is_heroic     true if item is heroic; missing otherwise
+- variant       1 = heroic item, 2 = LFR item; missing otherwise
 - cache_miss    if GetItemInfo failed; SHOULD be missing (changes other fields)
 - bcast_from    player's name if received rebroadcast from that player;
 missing otherwise; can be deleted as a result of in-game
 fiddling of loot data
 - extratext     text in Note column, including disposition and rebroadcasting
 OuroLootSV_log   = {}
 ------ Constants
 local option_defaults = {
-	['datarev'] = 17,    -- cheating, this isn't actually an option
+	['datarev'] = 18,    -- cheating, this isn't actually an option
 	['popup_on_join'] = true,
 	['register_slashloot'] = true,
 	['scroll_to_bottom'] = true,
 	['chatty_on_kill'] = false,
 	['no_tracking_wipes'] = false,
 		end,10,self)
 	else
 		virgin = nil
 	end
 	opts = OuroLootSV_opts
-	local stored_datarev = opts.datarev
+	local stored_datarev = opts.datarev or 14
 	for opt,default in pairs(option_defaults) do
 		if opts[opt] == nil then
 			opts[opt] = default
 		end
 	end
 	--OuroLootSV_hist = nil
 	-- Handle changes to the stored data format in stages from oldest to newest.
 	if OuroLootSV then
 		local dirty = false
-		if stored_datarev == nil then
+		local bumpers = {}
-			self:Print("Transitioning saved data format to 15..."); dirty = true
+		bumpers[14] = function()
 			for i,e in ipairs(OuroLootSV) do
 				if e.bosskill then
 					e.bossname, e.bosskill = e.bosskill, nil
 				end
 			end
-			stored_datarev = 15
+		end
-		end
-		if stored_datarev == 15 then
+		bumpers[15] = function()
-			self:Print("Transitioning saved data format to 16..."); dirty = true
 			for i,e in ipairs(OuroLootSV) do
 				if e.kind == 'boss' then
 					e.maxsize, e.raiderlist, e.raidersnap = 0, nil, {}
 				end
 			end
 			OuroLootSV.raiders = OuroLootSV.raiders or {}
 			for name,r in pairs(OuroLootSV.raiders) do
 				r.subgroup = 0
 			end
-			stored_datarev = 16
+		end
-		end
-		if stored_datarev == 16 then
+		bumpers[16] = function()
-			self:Print("Transitioning saved data format to 17..."); dirty = true
 			for i,e in ipairs(OuroLootSV) do
 				if e.kind == 'boss' then -- brown paper bag bugs
 					e.raidersnap = e.raidersnap or {}
 					e.maxsize = e.maxsize or 0
 				end
 			end
-			stored_datarev = 17
+		end
+		bumpers[17] = function()
+			for i,e in ipairs(OuroLootSV) do
+				if e.kind == 'loot' and e.is_heroic then
+					e.variant, e.is_heroic = 1, nil
+					-- Could try detecting any previous LFR loot here, but... gah
+				end
+			end
+		end
+		--[===[
+		local real = bumpers
+		bumpers = newproxy(true)
+		local mt = getmetatable(bumpers)
+		mt.__index = real
+		mt.__gc = function() print"whadda ya know, garbage collection works" end ]===]
+		while stored_datarev < opts.datarev do
+			self:Printf("Transitioning saved data format to %d...", stored_datarev+1)
+			dirty = true
+			bumpers[stored_datarev]()
+			stored_datarev = stored_datarev + 1
 		end
 		if dirty then self:Print("Saved data has been massaged into shape.") end
-		assert(stored_datarev==opts.datarev)
 	end
 	_init(self)
 	self.dprint('flow', "version strings:", revision_large, self.status_text)
-	self.OnInitialize = nil
+	self.OnInitialize = nil   -- free up ALL the things!
 end
 function addon:OnEnable()
 	self:RegisterEvent("PLAYER_LOGOUT")
 	self:RegisterEvent("RAID_ROSTER_UPDATE")
 					itexture	= itexture,
 					disposition	= (recipient == self.sharder) and 'shard' or nil,
 					count		= (count and count ~= "") and count or nil,
 					bcast_from	= from,
 					extratext	= extratext,
-					is_heroic	= self:is_heroic_item(ilink),
+					variant		= self:is_variant_item(ilink),
 				}
 				if local_override then
 					-- player is adding loot by hand, don't wait for network cache timeouts
 					-- keep this sync'd with prefer_local_loots above
 					if i.extratext == 'shard'
 			lefts[i] = L
 		end
 		tip.lefts = lefts
 		return tip
 	end
-	function addon:is_heroic_item(item)   -- returns true or *nil*
+	function addon:is_variant_item(item)   -- returns number or *nil*
 		itt = itt or create()
 		itt:SetOwner(UIParent,"ANCHOR_NONE")
 		itt:ClearLines()
 		itt:SetHyperlink(item)
 		local t = itt.lefts[2]:GetText()
 		itt:Hide()
-		return (t == ITEM_HEROIC) or nil
+		return (t == ITEM_HEROIC and 1)
+		       or (t == RAID_FINDER and 2)  -- no ITEM_ for this, apparently
+		       or nil
 	end
 end
 -- Called when first loading up, and then also when a 'clear' is being
 -- performed.  If SV's are present then g_restore_p will be true.

Mercurial > wow > ouroloot

comparison core.lua @ 66:43913e02a1ef