# HG changeset patch
# User MMOSimca <MMOSimca@gmail.com>
# Date 1419219894 18000
# Node ID da1daa9417775ce9d0860a0282f36fd2e9ea020b
# Parent  2fa787786d9a91741cec5e2135b3d1915c3992f6
ExtrapolatedCurrentActionFromLootData can now generate a proper current_action table for loot from fishing.

diff -r 2fa787786d9a -r da1daa941777 Main.lua
--- a/Main.lua	Sun Dec 21 22:43:43 2014 -0500
+++ b/Main.lua	Sun Dec 21 22:44:54 2014 -0500
@@ -2045,18 +2045,39 @@
 
 
     local function ExtrapolatedCurrentActionFromLootData(event_name)
+        local log_source = event_name .. "- ExtrapolatedCurrentActionFromLootData"
+        local previous_spell_label = current_action.spell_label
         local extrapolated_guid_registry = {}
         local num_guids = 0
+        table.wipe(current_action)
+
+        if _G.IsFishingLoot() then
+            -- Set up a proper 'fishing' current_action table
+            local zone_name, area_id, x, y, map_level, instance_token = CurrentLocationData()
+            if not (zone_name and area_id and x and y and map_level) then
+                Debug("%s: Missing current location data - %s, %d, %d, %d, %d.", log_source, zone_name, area_id, x, y, map_level)
+                return
+            end
+            current_action.instance_token = instance_token
+            current_action.map_level = map_level
+            current_action.x = x
+            current_action.y = y
+            current_action.zone_data = ("%s:%d"):format(zone_name, area_id)
+            current_action.spell_label = "FISHING"
+            current_action.loot_label = "fishing"
+            current_action.identifier = "FISHING:NONE"
+            current_action.target_type = AF.ZONE
+
+            Debug("%s: Fishing loot detected.", log_source)
+            return true
+        end
 
         -- Loot extrapolation cannot handle objects that need special spell labels (like HERBALISM or MINING) (MIND_CONTROL is okay)
-        if private.SPELL_FLAGS_BY_LABEL[current_action.spell_label] and not private.NON_LOOT_SPELL_LABELS[current_action.spell_label] then
+        if previous_spell_label and private.SPELL_FLAGS_BY_LABEL[previous_spell_label] and not private.NON_LOOT_SPELL_LABELS[previous_spell_label] then
             Debug("%s: Problematic spell label %s found. Loot extrapolation for this set of loot would have run an increased risk of introducing bad data into the system.", log_source, private.previous_spell_id)
-            table.wipe(current_action)
             return false
         end
 
-        table.wipe(current_action)
-
         for loot_slot = 1, _G.GetNumLootItems() do
             local loot_info = {
                 _G.GetLootSourceInfo(loot_slot)
@@ -2079,7 +2100,6 @@
                 end
             end
         end
-        local log_source = event_name .. "- ExtrapolatedCurrentActionFromLootData"
 
         if num_guids == 0 then
             Debug("%s: No GUIDs found in loot. Blank loot window?", log_source)