MorgenGrauen / mg-mudlet

Mudlet-Skripte
2 stars 4 forks source link

Map wird mehrfach überschrieben #64

Closed Kebap closed 1 year ago

Kebap commented 1 year ago

Wenn man bereits eine Map hatte und sich zum Spiel verbindet, wird sie mit dem Startgebiet überschrieben. Meh. Man kann sich unter Mapper Einstellungen das Backup laden und hat dann wieder die frühere Karte. Mkay. Wenn man sich das nächste Mal verbindet, wird die Map schon wieder überschrieben! Argl.

Zukünftig soll es so sein, dass das Startgebiet gar nicht geladen wird, wenn man bereits eine Map hatte. Das sollte dann nur noch per "minit" erfolgen, wenn man denn unbedingt seine bisherige Karte ersetzen will. Derzeit lädt "minit" aber noch kein Startgebiet, nur eine leere Karte. Das soll es also erstmal lernen. Siehe "map" Branch / PR zum Vergleich.

Kebap commented 1 year ago

Das sollte dann nur noch per "minit" erfolgen, wenn man denn unbedingt seine bisherige Karte ersetzen will. Derzeit lädt "minit" aber noch kein Startgebiet, nur eine leere Karte. Das soll es also erstmal lernen.

Hier wurde minit() in eine eigene Funktion ausgelagert, die man per Lua aufrufen kann, aber es erstellt (wie bisher) eine leere Karte, anstatt das (inzwischen implementierte) Startgebiet zu zeigen: https://github.com/MorgenGrauen/mg-mudlet/pull/42

Hier wurden die Startgebiete implementiert: https://github.com/MorgenGrauen/mg-mudlet/pull/15

Kebap commented 1 year ago

Zukünftig soll es so sein, dass das Startgebiet gar nicht geladen wird, wenn man bereits eine Map hatte. Das sollte dann nur noch per "minit" erfolgen, wenn man denn unbedingt seine bisherige Karte ersetzen will.

Das Problem liegt in https://github.com/MorgenGrauen/mg-mudlet/blob/master/src/scripts/mapper/ladeStartgebiet.lua

    -- Ggf. Backup vorab?
    if not istKarteLeer() then
        echoM("Es war bereits eine Karte vorhanden. Speichere Backup, bevor sie überschrieben wird...")

Hier soll nicht ein Backup erstellt und neue Karte über gebügelt werden, sondern die existierende Karte beibehalten werden.

Kebap commented 1 year ago

Dies wurde durch #71 gelöst.