smaitch / Grail

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

Wrong map pin location for "Fast Expansion" #113

Open stffdtiger opened 4 years ago

stffdtiger commented 4 years ago

The "Fast Expansion" quest given by Scout Valdez has a map pin close to the Twilight Glade flight master in Shadowmoon Valley in Draenor. Scout Valdez is actually located inside the Town Hall at Lunarfall, right next to the Garrison Missions table. This is where the quest starts.

smaitch commented 4 years ago

I am not able to get coordinates any more within garrisons, level 3 or 1. I know the coordinates for Scout Valdez in level 1 garrison are correct. However, the map pins do not show at all in the garrison any more. Blizzard map APIs are not doing what we would expect. Will need to investigate more for this.

yoshimo commented 4 years ago

Do we use UnitPosition or just GetPlayerMapPosition ?

smaitch commented 4 years ago

Actually I have found that Blizzard changed maps from 581 to 582 for garrison level 3 at least. Changing the NPC locations to 582 make them start showing up on map pins. However, I still cannot get their coordinates using API. The code uses UnitPosition() which does not return values in garrisons now.

smaitch commented 4 years ago

Nor does C_Map.GetPlayerMapPosition(582, "player") work.