[Bug] Possible Missed Profession NPCs

[F-Lambda]

It seems like some profession trainers were missed. Just to give an example, in zone_durotar.sql, the first aid trainer. Rawrk, is changed from using the default spellID -202007 to the spellID edited in professions.sql, -350000. However, no such change is made to the first aid trainer in Dun Murogh, Thamner Pol, meaning that the changes made there don't affect him.

mod-individual-progression/sql/world/base/zone_durotar.sql

Lines 17 to 19 in cbb0679

	-- Rawrk <First Aid Trainer>
	DELETE FROM `npc_trainer` WHERE `ID`=5943;
	INSERT INTO `npc_trainer` (`ID`, `SpellID`) VALUES (5943, -350000);

[Grimfeather]

I looked into this. You're right. Thamner Pol should also get those changes.
Maybe he was missed because he's labeled as a Physician, not a First Aid Trainer.
I'll check the other first aid trainers as well. I'll fix this.

edit: that turned out to be quite some work. thanks for reporting this.

[F-Lambda]

It looks like cooking trainers might have the same issue (for likely the same reason, the vast variance in titles).

[Grimfeather]

It wasn't their title, it wasn't their faction, I don't know, some were just skipped.

I'll take a look at the cooking trainers as well then.
I think there may be many more. Darnassus had no changes yet.
If you notice anything else, please share it.

[Grimfeather]

I checked the cooking trainers. First I thought I noticed some mistakes as well, but I think they all are as they should be.
There have already been changes made to the cooking trainers.

the spiced bread recipe is set to 200 skill requirement because it's just too convenient for a vanilla recipe.
It's still available as a collection/completionist item.
and expert and artisan training was already removed, because of the cooking book.

Did you notice anything else wrong with the cooking trainers?