Removed code in addon:Scan() which would set the is_known state for every recipe to false before setting the recipes known for that profession to true. Instead, when a recipe is initialized its is_known state will be set to false. Fixes tooltip issue with vendors selling items from two professions that the character knows.
James D. Callahan III [03-14-10 - 08:25]
Removed code in addon:Scan() which would set the is_known state for every recipe to false before setting the recipes known for that profession to true. Instead, when a recipe is initialized its is_known state will be set to false. Fixes tooltip issue with vendors selling items from two professions that the character knows.
diff --git a/ARL.lua b/ARL.lua
index 403fd77..968408c 100644
--- a/ARL.lua
+++ b/ARL.lua
@@ -875,6 +875,7 @@ function addon:AddRecipe(spell_id, skill_level, item_id, quality, profession, sp
["trivial_level"] = trivial_level or skill_level + 20,
["is_visible"] = true, -- Set to be displayed until the filtering occurs
["is_relevant"] = true, -- Set to be showing in the search results
+ ["is_known"] = false, -- Initially not known - will determine in addon:Scan()
}
if not recipe.name then
@@ -1527,13 +1528,6 @@ do
-- TODO: Figure out what this variable was supposed to be for - it isn't used anywhere. -Torhal
Player.totalRecipes = addon:InitializeRecipe(Player.current_prof)
- --- Set the known flag to false for every recipe in the database.
- local recipe_list = private.recipe_list
-
- for SpellID in pairs(recipe_list) do
- recipe_list[SpellID].is_known = false
- end
-
-------------------------------------------------------------------------------
-- Scan all recipes and mark the ones we know
-------------------------------------------------------------------------------
@@ -1567,6 +1561,7 @@ do
end
end
end
+ local recipe_list = private.recipe_list
local recipes_found = 0
for i = 1, GetNumTradeSkills() do