WowAce

mike:
    - Removed the Vendors: Mavralyn and Valdaron because no longer in the game
    - Bye Bye Nandar Branson
    - Added "Simon Tanner" and "Stephen Ryback" ids to recipes they teach
    - Remove the vendor: Fradd Swiftgear ... he doesnt teach anything anymore
    - The Winterfall Totemic mob is in Winterspring not Felwood
    - Fixed Krugosh being in two different profession trainer databases and added some missing trainers and also added back a custom string
James D. Callahan III:
    - If a trainer is not found in the trainer list during a scan, that fact will be displayed in the output.
mike:
    - Datamined all the trainers on the Scryers tier in Shattrath
    - Don't need an obtain filter along with F.RETIRED anymore, just that will do
    - update jc database
    - got rid of unused locales
James D. Callahan III:
    - Added Runda's entries for Enchanting and Tailoring.
    - Moved "Runda" from the Alchemy trainers file to the re-instated always-loaded Trainer file since she apparently teaches recipes from every profession.
    - When scanning a trainer, any "extra" recipes will now show the item ID and name of the recipe's crafted item, so incorrect items will not result in a correct recipe being removed.
    - Revert "Removed the InitTrainer() call"
    This reverts commit 62e8fd34fc0f0f667285329e95347efefc1f6c2c.
    - Revert "Deleted Trainer.lua and removed some trainer entries"
    This reverts commit 9800574400640852a164b767fa27703e3c9f1a39.
mike:
    - Yay for legacy data!? .. oh and <3 Torhal, have my babies
    - Removed the InitTrainer() call
    - Deleted Trainer.lua and removed some trainer entries
James D. Callahan III:
    - Vendor scan fix.
    - While I was in Shattrath, I decided to scan everything I could find - ended up with several trainers added to recipes and several recipes updated to show limited quantity.
    - Moved recipe files to their own directory.
    - There were actually no recipes with the Cooking "trainer" on Scryer's Tier assigned to them, so the "trainer" did not dump initially. I went ahead and scanned all the trainers there and now it dumps just fine.
    - Split the trainers into by-profession files using the trainer dumps.
    - Fix for nil trainer and escape double-quotes in trainer names so they'll dump properly. Damn "\"Cookie\" McWeaksauce".
    - Added the ability to dump the trainers for a given profession via addon:DumpTrainers("tailoring") (for example).
    - Moved definition for the NO_ROLE_FLAG table to the end of the file because I was tired of fucking scrolling over it to get to code.
    - New file - Utilities.lua For now, all it has is ItemLinkToID (now a private method) and AddListEntry (also now a private method instead of an addon method). Various changes here and there.
John Pasula:
    - Add him as a vendor and add local strings.
    - Update Fizzix Blastbolt.
    - Update jc db: Oluros
    - Update enchanting db: Fael Morningsong.
    - Update tailoring db: Dustin Vail.
    - Update first aid db: Joseph Wilson
    - Update alchemy db: Cerie Larksong.
    - Update inscription db: Arhtur Denny.
    - Update leatherworking db: Kul'de.
    - Update engineering db: Binkie Brightgear.
    - Update blacksmithing db: .
    - Update tailoring db: Dustin Vail.
    - Update cooking db: Bethany Cromwell.
    - Update enchanting db: Fael Morningsong.
James D. Callahan III:
    - Seasons are now identified by a string instead of a number. Example: recipe:AddSeason(1) is now recipe:AddSeason("WINTER_VEIL")
    - Custom lookup entries are no longer numeric. Example: recipe:AddCustom(8) is now recipe:AddCustom("DEFAULT_RECIPE") <-- Knowing what the fuck is really going on is always a good thing.
    - Renamed addon:addLookupList() to addon:AddListEntry(). Converted all relevant files, and did some cleanup in the process.
    - Rolled addon:ScanSkillLevelData() into addon:ScanTrainerData() - There was much duplicated code and they were both ran simultaneously when scanning trainers, overwriting the output buffer of one with the other. Now trainer scans report which recipe levels were corrected, and which could not be because they could not be confirmed with an item ID.
    - Output from trainer scans will now show which items had a crafted item ID set so we can know when to do a new DB dump.
    - Trainer scan: Don't allow a match by name if the recipe has a crafted item ID - this means either the crafted ID is wrong or, in the case of two versions of Glyph of Death Coil, the trainer really doesn't teach it.
    - Updated coordinates for several trainers and vendors - by standing ON them, and not using fucking WoWHead. Yes, pompy, I'm looking at you.
    - Added a trainer to several smelting recipes.
    - Updated trainers for and set crafted item IDs on several Blacksmithing recipes.
    - During a trainer scan, after all of the extra/missing recipes have been determined: For a recipe which has no crafted item set, see if the recipe name matches the name of any scanned items. If so, set that item ID as the recipe's crafted item.
    - Added crafted item ID for "FIre Prism".
    - Corrected the crafted item ID for "Reckless Hessonite" - it was set to the ID for "Vivid Dream Emerald".
    - Set a crafted item ID on "The Perforator".
    - New Mining trainer - Gonto.
    - Updated the Cooking DB from a scan on Shazdar in Orgrimmar.
    - Vendor scans which find no recipe that matches the recipe item will attempt to find the correct recipe based on name and set the recipe item id on the recipe.
    - Trainer scans now make sure the recipe being looked at is of the same profession as the trainer - fixes issues such as the Smelting version of "Enchanted Thorium" being attributed to an Enchanting trainer.
mike:
    - Add a new trainer to the database: "Allison" and added her id to the recipes she teaches
    - Fix the crafted recipe id for "Stormy Chalcedony"
James D. Callahan III:
    - Removed the remnants of the thankfully-aborted attempt at a Retired acquire type.
    - Made the Retired filter be a soft filter and removed all VENDOR/QUEST/etc flags form the Alchemy recipes marked as Retired.
    - Removed DATAMINER_TRAINER_NOTTEACH and DATAMINER_TRAINER_TEACH - there was no reason for them to exist, and their text is no longer valid.
    - The scanner now checks to see if trainer-only recipes have a recipe item ID and warns about this. If a recipe item ID has a quality of 0 (junk), it warns that the recipe has probably been removed.
    - Logic cleanup and fixed string referring to SPELL_TO_RECIPE_MAP.
    - Cleanup.
    - Fixed scanner reputation check.
    - Future-proof the game version check against the number of entries in private.game_version_names instead of an error-prone exact version (which was WOTLK, by the way)
    - Expand the missing recipe item ID check to look for Instance, Raid, Mob Drop, and World Drop filters.
mike:
    - Improved "Vial of the Sands" acquire filter
    - "Glyph of Lay on Hands" was changed from Minor Inscription Research to Northrend Inscription Research a while back, fix that.
James D. Callahan III:
    - FIxed some limited quantity items in the Engineering DB.
    - Updated the Leatherworking DB since I added some regressions with the merge.
    - Updated coordinates and Engineering recipes for Roxxik in Orgrimmar.
    - When building RECIPE_ITEM_TO_SPELL_MAP, issue a debug print for recipes which have the Vendor filter set but do not have a recipe item ID set.
    - Cleanup.
    - Removed private.spell_to_recipe_map - it is no longer required since the recipe objects are now aware of their own recipe item ID.
    - Once again, converted database files to a new format. Removed now-unused parameters and code from addon:AddRecipe()
    - Re-factored variables and output strings in trainer scan to make more sense. Trainer scans will now attempt to match the recipe's crafted item ID and, failing that, the recipe name for more accuracy. The warning about goggles will only be displayed when actually scanning an Engineering trainer. General code cleanup.
    - Added crafted item ID to the "Strange Tarot" recipe.
    - Corrected the crafted item ID for Glyph of Typhoon - it was set to that of Glyph of Arcane Blast.
    - Whitespace.
    - Converted code for recipe dumps into the recipe:Dump() method, and cleaned it up a bit.
    - Changed the recipe dump format again - this time to reflect the addition of the new recipe methods, and soon-to-come changes to addon:AddRecipe()
    - Renamed recipe.item_id to recipe.crafted_item_id for clarity - other places in the code will need to be updated to reflect this, but using recipe:CraftedItemID() instead.
    - Always increment the profession recipe count when adding a new recipe - it will be decremented when a non-matching required faction is found.
    - Removed code dealing with spell_to_recipe_map from addon:AddRecipe()
    - Added recipe methods SetRequiredFaction() and RequiredFaction()
    - Added recipe methods SetSpecialty() and Specialty()
    - Added recipe methods SetSkillLevels() and SkillLevels()
    - Added recipe methods SetCraftedItemID() and CraftedItemID()
    - Added recipe methods SetRecipeItemID() and RecipeItemID()
    - Preliminary changes working toward getting rid of private.spell_to_recipe_map and allowing trainers to work on spell_ids rather than item name, since Blizzard has two Glyph of Death Coil and may do something similar in the future.
mike:
    - Added Enchantress Andiala's id to the acquire of the Enchanting recipes she teaches
    - Added Krugosh's id to recipes he teaches
    - Updated Jewelcrafting database..added two trainers acquire data .. and fixed skill levels of many jc recipes
    - Added Jo'mah's id to recipes that he teachers and fixed Glyph of Righteousness acquire - it's learned from Minor Inscription Research instead of trainers
    - Well shit, Heavy Knothide Leather hasn't been a pattern sold by a vendor in over a year.. removed that acquire from the recipe entry and removed a vendor from the vendor database
    - Update Leatherworking database - mainly fixes the skill levels of the Cataclysm trainable recipes introduced
James D. Callahan III:
    - May as well go ahead and detect if a vendor listed as limited supply actually has an unlimited supply and fix that, too.
    - Fixed two Alchemy recipes which were listed as unlimited-quantity via vendor-scan.
    - Detect if a vendor who is listed as selling a recipe in unlimited quantity is actually a limited-quantity vendor and fix the recipe.
    - More re-factoring. Vendor scans will now add missing vendors (limited-supply vendors, as well) and set the vendor flag on the recipe if it is missing.
    - Refactored for clarity, and added DATAMINER_VENDOR_INFO to compliment DATAMINER_TRAINER_INFO.
    - Re-wrote ItemLinkToID()
    - Don't try to get a trainer name and ID before we determine whether or not the NPC _is_ a trainer.
mike:
    - Update blacksmithing database
    - Update cooking database
    - Fixed up the skill levels for many Blacksmithing recipes introduced in Cataclysm and "Saronite Mindcrusher" is trainable now instead of removed from the game
    - Add them nil's
James D. Callahan III:
    - Removed an obsolete variable.
    - Added a specialty check to the recipe dump so it will properly output. Fixed missing commas in the output format.
    - Normalized all "AddRecipe" function wrappers to have the same parameters so that the few which actually use specialties will treat the required_faction parameter appropriately.
John Pasula:
    - Update docs.
    - Dev localizations so messed up rebased it.
    - Update ingame locales for new docs.
    - Update ingame text.
    - Update main page.  DB Dump does more than CSV.  Also mentioned Dev DB Dump.
James D. Callahan III:
    - Fixed ticked #1214.
    - Fixed the init func usage.
mike:
    - Fix up the embroideries
James D. Callahan III:
    - Revert "PROFESSION_INIT_FUNCS --> private.profession_init_funcs"
    This reverts commit 2dddab50eda131c78afe0da7d7a5162aad3dd1f9.
    - Fuck the retarded shit. Nuke the reference as soon as it's used.
    - Revert "After initializing the recipes, nil the profession entry from private.profession_init_funcs"
    This reverts commit 28624d6a4590a854e0c51cc8457b9e9cb2ee793f.
    - After initializing the recipes, nil the profession entry from private.profession_init_funcs
    - PROFESSION_INIT_FUNCS --> private.profession_init_funcs
    - Cleaned up code for clicking the Scan button, and added recipe dumps for the profession when holding the Control key.
    - Setting/updating the player's professions would, ideally, require a function call.
    - Moved the player reputation updates into a Player method. Also changed the profession skill-level updates while I was at it.
    - In the interest of actually being quasi-awesome, I added quotes around the faction string in recipe dumps.
    - Apparently, beer #8 and code are like oil with water.
    - Supply 11 parameters and try to print 10. Fucking cute.
    - Debug duplicate recipes instead of printing them to the ChatFrame.
mike:
    - Fixed!
James D. Callahan III:
    - Fix a Tailoring recipe.
mike:
    - Fix this shit.... seriously, enough with the copy/paste bullshit.. proofread your c/p if you are going to do it!!
    - For the "Forged Document" recipes, since they can only be obtained/used by a specific faction, make the entry only load for the given faction
James D. Callahan III:
    - Recipes can now be excluded from a specific faction via an optional parameter to AddRecipe(). Such recipes will never count toward the total recipes, or ever be known to exist. The recipe/profession dumping facility is fully aware of this change.
    - The return value for addon:InitializeProfession() was never used. Anywhere. Ever. Removed.
    - Variable re-naming.
    - Replaced Player["ProfessionLevel"] with private.current_profession_scanlevel - this is a more accurate designation, especially since the scanned trade skill may be from a link.
    - Removed wrapper functions for recipe methods since they're now unnecessary.
John Pasula:
    - Add the faction differentiation for the cooking recipes just to alleviate tickets.  We need a better solution overall.
    - Fixed some code changes that Torhal forgot (self:AddCustom -> recipe:AddCustom)
mike:
    - Converted the Runeforging, Blacksmithing, Engineering and Leatherworking databases to the new format
    - Added back some recipe entries that got lost in translation
James D. Callahan III:
    - Fix entry for "Durotar".
    - Added "Runda".
    - Added "Zarbo Porkpatty" and "The Chef".
    - Gracefully handle recipes which are taught by undefined trainers - the GUI shouldn't crash and burn.
    - Trainer updates for Alchemy, Cooking, and First Aid.
    - Trainer updates for Alchemy, Cooking, and First Aid.
John Pasula:
    - Update the headers to use git replacements.  I think some of these were still legacy from the svn days.
James D. Callahan III:
    - recipe:IsFlagged() --> recipe:HasFilter()
John Pasula:
    - Update org alchemy.
    - Update org enchanting.
James D. Callahan III:
    - Updated Inscription trainers/skill levels from Orgrimmar datamining.
    - Updated Tailoring trainers/skill levels from Orgrimmar datamining.
    - Updated cooking trainers from datamining Orgrimmar.
    - If the trainer datamine scan detects skills we don't show the trainer as knowing, fix our data. Also set the trainer filter on the recipe if necessary.
    - When detecting differences in skill levels between what ARL has and what is actually in-game, do a live update of the recipe.
    - Clarify messages in trainer scan output.
    - In recipe_prototype:AddFilters() - don't bxor a filter if it's already set.
John Pasula:
    - Update .docmeta
James D. Callahan III:
    - Converted Inscription recipes to the new format.
    - Converted Jewelcrafting recipes to the new format.
    - Converted Smelting recipes to the new format.
    - Converted cooking recipes to the new format.
    - Changed recipe_prototype:AddFlags() to recipe_prototype:AddFilters() for clarity.
    - LimitedVendor and Vendor entries actually dump as expected now.
    - Revert "Replaced Enchanting recipes with a new-format recipe dump."
    This reverts commit 9bf06fdaa252ad1d724e97aea52749d4834a1402.
    - Replaced Tailoring recipes with a new-format recipe dump. I realize that this removes the "Don't load this if you're not of this faction." code, but seriously - if people don't know how to use the filters by now then fuck 'em.
    - Wipe the sorted_data and reverse_map table after every flag iteration. Whoops.
    - Replaced Enchanting recipes with a new-format recipe dump.
    - Replaced Alchemy recipes with a new-format recipe dump.
    - Replaced First Aid recipes with a new-format recipe dump.
    - Mob, Vendor, and Trainer IDs are now sorted numerically in the recipe dump. World Drop strings are sorted as well.
    - Elixir of Minor Defense was missing the Horde filter flag...
    - Dump recipe flags in their numeric order for consistency.
    - Removed some dead code.
    - Re-wrote database dump to use the new recipe methods. Dumps will now properly handle seasonal, custom, achievement, and limited vendor acquire types. Cleaned up the upvalues, and all globals are now referenced from the local _G.
    - Somehow, Provisioner Nasela of the Mag'har was removed as a reputation vendor for Transmute: Primal Fire to Earth. Re-added.
    - Added methods to the recipe_prototype for adding acquire data. The old functions which do the same things are now wrappers for the methods.
    - Converted the First Aid recipes to use recipe:AddFlags() instead of addon:AddRecipeFlags().
    - Moved all of the recipe functions out of core.lua and into Recipe.lua - also changed recipes to use a metatable and assigned a prototype as the index so recipes can share methods.
John Pasula:
    - WTF was I doing with var2/arg3?
James D. Callahan III:
    - Code cleanup, and reference globals from our local _G.
    - Renamed PROFESSION_INITS to PROFESSION_INIT_FUNCS for clarity.
    - Moved player initialization code from core.lua to Player.lua
    - Moved definition of private.Player from core.lua to Player.lua
    - Cleanup and consistency changes.
    - Reference globals from our local _G.
    - Creating FilterPanel.value_map as an empty table and then creating it again as a populated table is just silly. Fixed.
    - Wrote general-purpose ToggleExpansionCheckBoxes(), which not only replaces four identical OnClick functions, but also has the added benefit of NOT toggling CheckBoxes/SavedVariables which are disabled.
    - Fixed a reference to a non-existent variable for folks who use Cartographer_Waypoints (do they still exist?).
    - Replaced a concatenation inside a for loop with a format() call.
    - Call some functions from our local reference to _G
    - Upvalued pairs() and select()
    - Removed unused constant and comment.
    - Future-proofing: Instead of relying on the hard-coded A_MAX, assume that private.acquire_strings will be updated simultaneously with private.acquire_types and simply check for its size instead.
mike:
    - Glyph of Rake will be Glyph of Pounce in 4.1
    - Updated the location of "Fyldan" and "Alegorn"
John Pasula:
    - We still refered to the dev versions of localizations as testenUS.lua this so that it they're now refered to as devel-enUS.lua
mike:
    - Removed the trainer: Borgus Steelhand and marked some Blacksmithing recipes as no longer in the game
    - Cleaned up the mob database and fixed acquire of some recipes in the Alchemy and Leatherworking databases
    - Removed some mobs that are no longer in the game
    - Removed the vendor: Heldan Galesong .. because he is no longer in the game
    - misc changes
    - In the Leatherworking database, marked "Quiver of a Thousand Feathers"  as no longer obtainable
    - Marked a bunch of Blacksmithing recipes as no longer obtainable
    - :O
John Pasula:
    - Formatting
mike:
    - fixed another ref
    - Fixed ref
    - Mark some Blacksmithing recipes as retired
John Pasula:
    - Update changelog.
    - Revert AceLocal change.  Mikk broke it.
    - Change log off.