Looking at this now. I'm wondering if we should keep a copy of the old map and reload that if the restore goes to hell. The downside is we'll be slowing down the load more by making a deep copy of the map. One alternative is to just wipe the current map for loading in, and if it fails the user just has to select a good map to load in.
-- You received this bug notification because you are a member of Mudlet Makers, which is subscribed to Mudlet. https://bugs.launchpad.net/bugs/1124802 Title: Rooms are not properly removed on map load Status in Mudlet the MUD client: Confirmed Bug description: When you load a new map in, current rooms are not properly removed first. This leads to an issue where if a room was deleted in a new map and you load it, it will not get painted, but it will exist - and show up in searchRoom, and roomExists will say it still exists. For a short demo, import the "map with non-deleted room.dat" map, and then import the "map with deleted room.dat" map in that order. You'll see that room 4855 was deleted, however it still shows up in searchRooms (causing problems) and roomExists(4855) will say true for it. To manage notifications about this bug go to: https://bugs.launchpad.net/mudlet/+bug/1124802/+subscriptions _______________________________________________ Mailing list: https://launchpad.net/~mudlet-makers Post to : [email protected] Unsubscribe : https://launchpad.net/~mudlet-makers More help : https://help.launchpad.net/ListHelp
