Trinitycore: DB/Profession: Incorrect level and skill chains for professions

[NiganNi]

Description:
Most (if not all) recipes have incorrect values in the db. They haven't been updated in a while and aren't even separated into different expansions.

Already wrote in another topic, so the description in the screenshot.

Expected behaviour:
Give the opportunity to study tailoring up to 350

Steps to reproduce the problem:
1)Create a new character
2)Learn tailoring from a classic trainer (stormwind)
3)Raise it to level 300
4)Notice you can buy outland patterns from the same trainer, and their levelsi s incorrect

Branch(es):
master

TC rev. hash/commit:
TrinityCore rev. c9344c145ad1+ 2021-02-26 22:10:34 +0100 (master branch) (Win64, Release, Static) (worldserver-daemon)

Operating system:
Win10 x64

[MaxtorCoder]

The last Issue was clearly closed for a reason, there's also no clear description of what is wrong. Screenshot does not help either.

[matanshukry]

I'll clarify, and @NiganNi please change the topic and description according to it (or something like it)

• topic: DB/Profession: Incorrect level and skill chains for professions
• description:
  Most (if not all) recipes have incorrect values in the db. They haven't been updated in a while and aren't even separated into different expansions.

e.g. Bolt of Netherweave pattern shouldn't be available for "classic tailoring" level 300, but rather be available for "outland tailoring" level 1

Steps to reproduce:

1. Create a new character
2. Learn tailoring from a classic trainer (stormwind)
3. Raise it to level 300
4. Notice you can buy outland patterns from the same trainer, and their levelsi s incorrect

[mdX7]

https://github.com/TrinityCore/TrinityCore/issues/26364

^removed them from the trainers in (main) capitals as it should be. Still gotta get data from all the other trainers.

[matanshukry]

@mdX7 Making a list; correct me if I'm wrong/missing something:

| PR | Expansion | Cities | Merged |
| -- | -- | -- | -- |
| #26364 | Classic | Stormwind, Orgrimmar | |
| | Outland | | |
| | Northrend | | |
| #26364 | Cataclysm | Stormwind, Orgrimmar | |
| | Pandaria | | |
| | Draenor | | |
| | Legion | | |
| | Battle for Azeroth | Boralus, Dazar'alor | |
| | Shadowlands | Oribos | |

p.s.
The list suppose to be just for the basic trainers in the capital city relevant for that expansion - that is, the minimum amount allowing players to actually learn professions. After that we can make another issue with a list of trainers and do them one at a time.

Quick link to wowhead for list of all trainers for future references: https://www.wowhead.com/npcs?filter=28;1;0