Tradeskill code fix
The current tradeskills code is not working like live code. The code is determining a tradeskill for the container type and as a result allows combines to be done in the wrong containers (ie, pottery has 2 combiners: Pottery Wheel (21) and Kiln (22). These are both used for pottery combines so current code lets either container do all pottery combines.) On live, some recipes require one and some the other.
Also on live, there are recipes that are done in a container that have nothing to do with the tradeskill that the container is normally used for. For example, the Toolbox is normally used by gnomes for Tinkering but can also be used by anybody to recharge tinkered items such as Anizok's Gauze Press. The current code prohibits other characters from doing this combine because of needing the Toolbox. With the new code, the recipes for recharging could be changed to tradeskill 75 and no fail and still use the toolbox.
The new code I am proposing does away with declaring combiners to only do certain tradeskills and eliminates being able to do all tradeskill combines in any container of that tradeskill type. (Example research or cultural recipes).
The following 5 posts show the changes made to the server code to support live like tradeskills. This code is working on my server.
Last edited by Darkonig; 03-04-2007 at 01:11 PM..
Reason: needed more room to fit the code
|