From b6047cdce8f8402d305936e0a3744f6b12072222 Mon Sep 17 00:00:00 2001
From: Kelsi <kelsihates2fa@gmail.com>
Date: Sun, 22 Mar 2026 16:18:52 -0700
Subject: [PATCH] feat: add world map navigation API for WorldMapFrame
 compatibility
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Implement the core map functions needed by WorldMapFrame.lua:
- SetMapToCurrentZone — sets map view from player's current mapId/zone
- GetCurrentMapContinent — returns continent (1=Kalimdor, 2=EK, etc.)
- GetCurrentMapZone — returns current zone ID
- SetMapZoom(continent, zone) — navigate map view
- GetMapContinents — returns continent name list
- GetMapZones(continent) — returns zone names per continent
- GetNumMapLandmarks — stub (returns 0)

Maps game mapId (0=EK, 1=Kalimdor, 530=Outland, 571=Northrend) to
WoW's continent numbering. Internal state tracks which continent/zone
the map UI is currently viewing.
---
 src/addons/lua_engine.cpp | 101 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 101 insertions(+)

diff --git a/src/addons/lua_engine.cpp b/src/addons/lua_engine.cpp
index e2505c4e..84146b84 100644
--- a/src/addons/lua_engine.cpp
+++ b/src/addons/lua_engine.cpp
@@ -1857,6 +1857,100 @@ static int lua_GetMinimapZoneText(lua_State* L) {
     return lua_GetZoneText(L);
 }
 
+// --- World Map Navigation API ---
+
+// Map ID → continent mapping
+static int mapIdToContinent(uint32_t mapId) {
+    switch (mapId) {
+        case 0:   return 2; // Eastern Kingdoms
+        case 1:   return 1; // Kalimdor
+        case 530: return 3; // Outland
+        case 571: return 4; // Northrend
+        default:  return 0; // Instance or unknown
+    }
+}
+
+// Internal tracked map state (which continent/zone the map UI is viewing)
+static int s_mapContinent = 0;
+static int s_mapZone = 0;
+
+// SetMapToCurrentZone() — sets map view to the player's current zone
+static int lua_SetMapToCurrentZone(lua_State* L) {
+    auto* gh = getGameHandler(L);
+    if (gh) {
+        s_mapContinent = mapIdToContinent(gh->getCurrentMapId());
+        s_mapZone = static_cast<int>(gh->getWorldStateZoneId());
+    }
+    return 0;
+}
+
+// GetCurrentMapContinent() → continentId (1=Kalimdor, 2=EK, 3=Outland, 4=Northrend)
+static int lua_GetCurrentMapContinent(lua_State* L) {
+    if (s_mapContinent == 0) {
+        auto* gh = getGameHandler(L);
+        if (gh) s_mapContinent = mapIdToContinent(gh->getCurrentMapId());
+    }
+    lua_pushnumber(L, s_mapContinent);
+    return 1;
+}
+
+// GetCurrentMapZone() → zoneId
+static int lua_GetCurrentMapZone(lua_State* L) {
+    if (s_mapZone == 0) {
+        auto* gh = getGameHandler(L);
+        if (gh) s_mapZone = static_cast<int>(gh->getWorldStateZoneId());
+    }
+    lua_pushnumber(L, s_mapZone);
+    return 1;
+}
+
+// SetMapZoom(continent [, zone]) — sets map view to continent/zone
+static int lua_SetMapZoom(lua_State* L) {
+    s_mapContinent = static_cast<int>(luaL_checknumber(L, 1));
+    s_mapZone = static_cast<int>(luaL_optnumber(L, 2, 0));
+    return 0;
+}
+
+// GetMapContinents() → "Kalimdor", "Eastern Kingdoms", ...
+static int lua_GetMapContinents(lua_State* L) {
+    lua_pushstring(L, "Kalimdor");
+    lua_pushstring(L, "Eastern Kingdoms");
+    lua_pushstring(L, "Outland");
+    lua_pushstring(L, "Northrend");
+    return 4;
+}
+
+// GetMapZones(continent) → zone names for that continent
+// Returns a basic list; addons mainly need this to not error
+static int lua_GetMapZones(lua_State* L) {
+    int cont = static_cast<int>(luaL_checknumber(L, 1));
+    // Return a minimal representative set per continent
+    switch (cont) {
+        case 1: // Kalimdor
+            lua_pushstring(L, "Durotar"); lua_pushstring(L, "Mulgore");
+            lua_pushstring(L, "The Barrens"); lua_pushstring(L, "Teldrassil");
+            return 4;
+        case 2: // Eastern Kingdoms
+            lua_pushstring(L, "Elwynn Forest"); lua_pushstring(L, "Westfall");
+            lua_pushstring(L, "Dun Morogh"); lua_pushstring(L, "Tirisfal Glades");
+            return 4;
+        case 3: // Outland
+            lua_pushstring(L, "Hellfire Peninsula"); lua_pushstring(L, "Zangarmarsh");
+            return 2;
+        case 4: // Northrend
+            lua_pushstring(L, "Borean Tundra"); lua_pushstring(L, "Howling Fjord");
+            return 2;
+        default:
+            return 0;
+    }
+}
+
+// GetNumMapLandmarks() → 0 (no landmark data exposed yet)
+static int lua_GetNumMapLandmarks(lua_State* L) {
+    lua_pushnumber(L, 0);
+    return 1;
+}
+
 // --- Player State API ---
 // These replace the hardcoded "return false" Lua stubs with real game state.
 
@@ -4167,6 +4261,13 @@ void LuaEngine::registerCoreAPI() {
         {"GetLocale",         lua_GetLocale},
         {"GetBuildInfo",      lua_GetBuildInfo},
         {"GetCurrentMapAreaID", lua_GetCurrentMapAreaID},
+        {"SetMapToCurrentZone", lua_SetMapToCurrentZone},
+        {"GetCurrentMapContinent", lua_GetCurrentMapContinent},
+        {"GetCurrentMapZone",   lua_GetCurrentMapZone},
+        {"SetMapZoom",          lua_SetMapZoom},
+        {"GetMapContinents",    lua_GetMapContinents},
+        {"GetMapZones",         lua_GetMapZones},
+        {"GetNumMapLandmarks",  lua_GetNumMapLandmarks},
         {"GetZoneText",          lua_GetZoneText},
         {"GetRealZoneText",      lua_GetZoneText},
         {"GetSubZoneText",       lua_GetSubZoneText},