Sign in to follow this  
Followers 0
Ragethreat

[Declined][Battle for Azeroth]: Outland Mining Trainers

4 posts in this topic

Title: Outland Mining Trainers
Bugtracker Link:


Patch Content:
Hurnak Grimmord - Outland Mining - 10G - Mining - mining 275 req - lvl 58 req
INSERT INTO 'npc_trainer' ('ID', 'SpellID', 'MoneyCost', 'ReqSkillLine', 'ReqSkillRank', 'ReqLevel') VALUES ('18779', '29354', '100000', 'Mining', '275', '58')

Sorry if this is poorly written or plain incorrect. First time submitting a patch. Once/If this gets confirmed I'll look into adding the entry level Outland recipes to the various trainers in Hellfire Peninsula.

Share this post


Link to post
Share on other sites
DELETE FROM `npc_trainer` WHERE `ID`='18779' AND `SpellID`='29354';
INSERT INTO `npc_trainer` (`ID`, `SpellID`, `MoneyCost`, `ReqSkillLine`, `ReqSkillRank`, `ReqLevel`) VALUES ('18779', '29354', '100000', 'Mining', '275', '58');

 

Share this post


Link to post
Share on other sites

Updated for other missing recipes. In order: Outland Mining, Smelt Fel Iron, Earth Shatter, Fire Sunder, Smelt Adamantite, Smelt Eternium, Smelt Felsteel, Smelt Khorium, Smelt Hardened Adamantite.

Hurnak Grimmord

DELETE FROM `npc_trainer` WHERE `ID`='18779' AND `SpellID`='29354';
DELETE FROM `npc_trainer` WHERE `ID`='18779' AND `SpellID`='29356';
DELETE FROM `npc_trainer` WHERE `ID`='18779' AND `SpellID`='34750';
DELETE FROM `npc_trainer` WHERE `ID`='18779' AND `SpellID`='35751';
DELETE FROM `npc_trainer` WHERE `ID`='18779' AND `SpellID`='29358';
DELETE FROM `npc_trainer` WHERE `ID`='18779' AND `SpellID`='29359';
DELETE FROM `npc_trainer` WHERE `ID`='18779' AND `SpellID`='29360';
DELETE FROM `npc_trainer` WHERE `ID`='18779' AND `SpellID`='29361';
DELETE FROM `npc_trainer` WHERE `ID`='18779' AND `SpellID`='29686';
INSERT INTO `npc_trainer` (`ID`, `SpellID`, `MoneyCost`, `ReqSkillLine`, `ReqSkillRank`, `ReqLevel`) VALUES ('18779', '29354', '100000', 'Mining', '275', '58'),
('18779', '29356', '40000', 'Mining', '275', '58'),
('18779', '34750', '10000', 'Mining', '300', '58'),
('18779', '35751', '10000', 'Mining', '300', '58'),
('18779', '29358', '40000', 'Mining', '325', '58'),
('18779', '29359', '40000', 'Mining', '350', '58'),
('18779', '29360', '40000', 'Mining', '350', '58'),
('18779', '29361', '100000', 'Mining', '375', '58'),
('18779', '29686', '100000', 'Mining', '375', '58');

 

Also for Krugosh for Horde:

DELETE FROM `npc_trainer` WHERE `ID`='18747' AND `SpellID`='29354';
DELETE FROM `npc_trainer` WHERE `ID`='18747' AND `SpellID`='29356';
DELETE FROM `npc_trainer` WHERE `ID`='18747' AND `SpellID`='34750';
DELETE FROM `npc_trainer` WHERE `ID`='18747' AND `SpellID`='35751';
DELETE FROM `npc_trainer` WHERE `ID`='18747' AND `SpellID`='29358';
DELETE FROM `npc_trainer` WHERE `ID`='18747' AND `SpellID`='29359';
DELETE FROM `npc_trainer` WHERE `ID`='18747' AND `SpellID`='29360';
DELETE FROM `npc_trainer` WHERE `ID`='18747' AND `SpellID`='29361';
DELETE FROM `npc_trainer` WHERE `ID`='18747' AND `SpellID`='29686';
INSERT INTO `npc_trainer` (`ID`, `SpellID`, `MoneyCost`, `ReqSkillLine`, `ReqSkillRank`, `ReqLevel`) VALUES ('18747', '29354', '100000', 'Mining', '275', '58'),
('18747', '29356', '40000', 'Mining', '275', '58'),
('18747', '34750', '10000', 'Mining', '300', '58'),
('18747', '35751', '10000', 'Mining', '300', '58'),
('18747', '29358', '40000', 'Mining', '325', '58'),
('18747', '29359', '40000', 'Mining', '350', '58'),
('18747', '29360', '40000', 'Mining', '350', '58'),
('18747', '29361', '100000', 'Mining', '375', '58'),
('18747', '29686', '100000', 'Mining', '375', '58');

 

Wasn't sure if the DELETE command could be put into an array or not so kept it simple. Same for mixing both trainers into one statement.

Share this post


Link to post
Share on other sites

Greetings @Ragethreat,

Your patch is declined, we can't accept custom fixes or handmade fixes on that table, it needs to come from a sniff. Thank you for contributing to Firestorm anyway.

Regards, Thyvene.

Share this post


Link to post
Share on other sites
Guest
This topic is now closed to further replies.
Sign in to follow this  
Followers 0