Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update instance locations #22

Open
idiomatic opened this issue Oct 21, 2020 · 7 comments
Open

update instance locations #22

idiomatic opened this issue Oct 21, 2020 · 7 comments

Comments

@idiomatic
Copy link
Owner

surely more instances and map zones have been introduced during my hiatus.

@JanGalek
Copy link
Collaborator

JanGalek commented Oct 24, 2020

@idiomatic I can help, I see InstanceLocations, but I dont know how get coors X,Z and I'm not sure how get mapfile too

@idiomatic
Copy link
Owner Author

idiomatic commented Oct 24, 2020

Based on wowwiki, try going to an instance door and:

/script SetMapToCurrentZone() local x,y=GetPlayerMapPosition("player") DEFAULT_CHAT_FRAME:AddMessage(format("%s, %s: %.3f, %.3f",GetZoneText(),GetSubZoneText(),x,y))

Some reverse engineering of HandyNotes may be necessary for deep understanding of mapfile. Alternatively, experimentation with the map API could yield results.

@JanGalek
Copy link
Collaborator

Message: [string "SetMapToCurrentZone() local x,y=GetPlayerMa..."]:1: attempt to call global 'SetMapToCurrentZone' (a nil value)

@idiomatic
Copy link
Owner Author

idiomatic commented Oct 24, 2020

Or in WoW 8.0 or later:

/script print(C_Map.GetPlayerMapPosition(C_Map.GetBestMapForUnit("player"), "player"):GetXY())

@idiomatic
Copy link
Owner Author

idiomatic commented Oct 24, 2020

I'm contemplating looking for some old WowLua SavedVariables files for other small code snippets. However, they will be of limited use as they may be from before the 8.0 API changes.

@idiomatic idiomatic added this to the Expansion Data milestone Oct 24, 2020
@JanGalek
Copy link
Collaborator

I see HandyNotes using https://github.com/Nevcairiel/HereBeDragons but there is missing BFA mapFiles :/

@idiomatic
Copy link
Owner Author

I suggest delaying updating instance locations. The instance locations matter when #20 adds reference to instances.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants