search

smaitch / Grail

World of Warcraft addon to provide a database of quest information.
31 stars 20 forks source link

Ensure inherited faction requirements obey NPC-based faction requirements #146

Open smaitch opened 4 years ago

smaitch commented 4 years ago

In Classic, quest 593 is now marked fH because it seems the P:B592 does not convey quest 592's faction requirement which it gets from its A: NPC only being Horde. We should ensure this inheritance is passed along so the fH code can be removed.

Nerillida commented 4 years ago

I would be cautious here. There are some branching/merging chains. Some of the level 50 class quests, the ones involving Theldurin the Lost come to mind.

G[687]='K0400 L030 N255 AA:2786 AH:2787 T:2785 P:653,688 E31727'

comes to mind.

Unless I don't see what you mean.