search

wowdev / WoWDBDefs

Client database definitions for World of Warcraft
Other
246 stars 95 forks source link

Name TrainerID in GossipNPCOption #115

Closed Shauren closed 2 years ago

Shauren commented 2 years ago

Values match TrainerID sent in SMSG_TRAINER_LIST (even for rows that have GossipNpcOption = Vendor, those are either mistakenly filled values or leftover from old data, first being a trainer then changed to vendor without clearing)

Marlamin commented 2 years ago

Good stuff, thanks!

bloerwald commented 2 years ago

even for rows that have GossipNpcOption = Vendor,

This table is an utter clusterfuck, this isn't the only type that has Vendor but isn't. There is a reason I have added all types, not just the majority in comments :D