Wrote an XML Schema describing maps file and also changed the spec a bit

5 files changed, 315 insertions, 492 deletions

@@ -190,22 +190,22 @@ MapeditFrame::MapeditFrame(World* world) : wxFrame(NULL, wxID_ANY, "Map Editor")
   wxStaticText* leftmapLabel = new wxStaticText(propertyEditor, wxID_ANY, "Leftmap Action:");
   wxChoice* leftmapChoice = new wxChoice(propertyEditor, LEFTMAP_TYPE_CHOICE);
   wxComboCtrl* leftmapCombo = new wxComboCtrl(propertyEditor, LEFTMAP_MAP_CHOICE, "", wxDefaultPosition, wxDefaultSize, wxCB_READONLY);
-  leftmapCombo->SetPopupControl(new MapSelectComboPopup(mapTree, currentMap->getLeftMoveMapID()));
+  leftmapCombo->SetPopupControl(new MapSelectComboPopup(mapTree, currentMap->getAdjacent(Map::MoveDir::Left).map));
   
   wxStaticText* rightmapLabel = new wxStaticText(propertyEditor, wxID_ANY, "Rightmap Action:");
   wxChoice* rightmapChoice = new wxChoice(propertyEditor, RIGHTMAP_TYPE_CHOICE);
   wxComboCtrl* rightmapCombo = new wxComboCtrl(propertyEditor, RIGHTMAP_MAP_CHOICE, "", wxDefaultPosition, wxDefaultSize, wxCB_READONLY);
-  rightmapCombo->SetPopupControl(new MapSelectComboPopup(mapTree, currentMap->getRightMoveMapID()));
+  rightmapCombo->SetPopupControl(new MapSelectComboPopup(mapTree, currentMap->getAdjacent(Map::MoveDir::Right).map));
   
   wxStaticText* upmapLabel = new wxStaticText(propertyEditor, wxID_ANY, "Upmap Action:");
   wxChoice* upmapChoice = new wxChoice(propertyEditor, UPMAP_TYPE_CHOICE);
   wxComboCtrl* upmapCombo = new wxComboCtrl(propertyEditor, UPMAP_MAP_CHOICE, "", wxDefaultPosition, wxDefaultSize, wxCB_READONLY);
-  upmapCombo->SetPopupControl(new MapSelectComboPopup(mapTree, currentMap->getUpMoveMapID()));
+  upmapCombo->SetPopupControl(new MapSelectComboPopup(mapTree, currentMap->getAdjacent(Map::MoveDir::Up).map));
   
   wxStaticText* downmapLabel = new wxStaticText(propertyEditor, wxID_ANY, "Downmap Action:");
   wxChoice* downmapChoice = new wxChoice(propertyEditor, DOWNMAP_TYPE_CHOICE);
   wxComboCtrl* downmapCombo = new wxComboCtrl(propertyEditor, DOWNMAP_MAP_CHOICE, "", wxDefaultPosition, wxDefaultSize, wxCB_READONLY);
-  downmapCombo->SetPopupControl(new MapSelectComboPopup(mapTree, currentMap->getDownMoveMapID()));
+  downmapCombo->SetPopupControl(new MapSelectComboPopup(mapTree, currentMap->getAdjacent(Map::MoveDir::Down).map));
   
   for (auto type : Map::listMoveTypes())
   {
@@ -674,157 +674,169 @@ void MapeditFrame::OnThreeMovingSash(wxSplitterEvent& event)
 
 void MapeditFrame::OnSetLeftmapType(wxCommandEvent&)
 {
-  wxChoice* leftmapChoice = (wxChoice*) wxWindow::FindWindowById(LEFTMAP_TYPE_CHOICE, this);
-  wxComboCtrl* leftmapCombo = (wxComboCtrl*) wxWindow::FindWindowById(LEFTMAP_MAP_CHOICE, this);
-  
-  Map::MoveType old = currentMap->getLeftMoveType();
-  Map::MoveType newt = ((MoveTypeCtr*) leftmapChoice->GetClientData(leftmapChoice->GetSelection()))->type;
+  wxChoice* choice = (wxChoice*) wxWindow::FindWindowById(LEFTMAP_TYPE_CHOICE, this);
+  wxComboCtrl* combo = (wxComboCtrl*) wxChoice::FindWindowById(LEFTMAP_MAP_CHOICE, this);
+  Map::MoveDir dir = Map::MoveDir::Left;
   
+  Map::Adjacent adjacent = currentMap->getAdjacent(dir);
+  Map::MoveType newt = ((MoveTypeCtr*) choice->GetClientData(choice->GetSelection()))->type;
   commitAction(std::make_shared<Undoable>("Set Leftmap Action", [=] () {
-    leftmapChoice->SetSelection(leftmapChoice->FindString(Map::stringForMoveType(newt)));
-    currentMap->setLeftMoveType(newt);
-    leftmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getLeftMoveType()));
+    choice->SetSelection(choice->FindString(Map::stringForMoveType(newt)));
+    currentMap->setAdjacent(dir, newt);
+    combo->Enable(Map::moveTypeTakesMap(newt));
   }, [=] () {
-    leftmapChoice->SetSelection(leftmapChoice->FindString(Map::stringForMoveType(old)));
-    currentMap->setLeftMoveType(old);
-    leftmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getLeftMoveType()));
+    choice->SetSelection(choice->FindString(Map::stringForMoveType(adjacent.type)));
+    currentMap->setAdjacent(dir, adjacent.type);
+    combo->Enable(Map::moveTypeTakesMap(adjacent.type));
   }));
 }
 
 void MapeditFrame::OnSetLeftmapMap(wxCommandEvent&)
 {
-  wxComboCtrl* leftmapCombo = (wxComboCtrl*) wxWindow::FindWindowById(LEFTMAP_MAP_CHOICE, this);
-  MapSelectComboPopup* popup = (MapSelectComboPopup*) leftmapCombo->GetPopupControl();
-  int old = currentMap->getLeftMoveMapID();
+  wxComboCtrl* combo = (wxComboCtrl*) wxWindow::FindWindowById(LEFTMAP_MAP_CHOICE, this);
+  Map::MoveDir dir = Map::MoveDir::Left;
+  
+  Map::Adjacent adjacent = currentMap->getAdjacent(dir);
+  MapSelectComboPopup* popup = (MapSelectComboPopup*) combo->GetPopupControl();
+  int old = adjacent.map;
   int newt = popup->GetSelectedMapID();
   
   if (old == newt) return;
   
   commitAction(std::make_shared<Undoable>("Set Leftmap Map", [=] () {
     popup->SetSelectedMapID(newt);
-    leftmapCombo->SetValue(world->getMap(newt)->getTitle());
-    currentMap->setLeftMoveMapID(newt);
+    combo->SetValue(world->getMap(newt)->getTitle());
+    currentMap->setAdjacent(dir, adjacent.type, newt);
   }, [=] () {
     popup->SetSelectedMapID(old);
-    leftmapCombo->SetValue(world->getMap(old)->getTitle());
-    currentMap->setLeftMoveMapID(old);
+    combo->SetValue(world->getMap(old)->getTitle());
+    currentMap->setAdjacent(dir, adjacent.type, old);
   }));
 }
 
 void MapeditFrame::OnSetRightmapType(wxCommandEvent&)
 {
-  wxChoice* rightmapChoice = (wxChoice*) wxWindow::FindWindowById(RIGHTMAP_TYPE_CHOICE, this);
-  wxComboCtrl* rightmapCombo = (wxComboCtrl*) wxWindow::FindWindowById(RIGHTMAP_MAP_CHOICE, this);
-  
-  Map::MoveType old = currentMap->getRightMoveType();
-  Map::MoveType newt = ((MoveTypeCtr*) rightmapChoice->GetClientData(rightmapChoice->GetSelection()))->type;
+  wxChoice* choice = (wxChoice*) wxWindow::FindWindowById(RIGHTMAP_TYPE_CHOICE, this);
+  wxComboCtrl* combo = (wxComboCtrl*) wxChoice::FindWindowById(RIGHTMAP_MAP_CHOICE, this);
+  Map::MoveDir dir = Map::MoveDir::Right;
   
+  Map::Adjacent adjacent = currentMap->getAdjacent(dir);
+  Map::MoveType newt = ((MoveTypeCtr*) choice->GetClientData(choice->GetSelection()))->type;
   commitAction(std::make_shared<Undoable>("Set Rightmap Action", [=] () {
-    rightmapChoice->SetSelection(rightmapChoice->FindString(Map::stringForMoveType(newt)));
-    currentMap->setRightMoveType(newt);
-    rightmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getRightMoveType()));
+    choice->SetSelection(choice->FindString(Map::stringForMoveType(newt)));
+    currentMap->setAdjacent(dir, newt);
+    combo->Enable(Map::moveTypeTakesMap(newt));
   }, [=] () {
-    rightmapChoice->SetSelection(rightmapChoice->FindString(Map::stringForMoveType(old)));
-    currentMap->setRightMoveType(old);
-    rightmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getRightMoveType()));
+    choice->SetSelection(choice->FindString(Map::stringForMoveType(adjacent.type)));
+    currentMap->setAdjacent(dir, adjacent.type);
+    combo->Enable(Map::moveTypeTakesMap(adjacent.type));
   }));
 }
 
 void MapeditFrame::OnSetRightmapMap(wxCommandEvent&)
 {
-  wxComboCtrl* rightmapCombo = (wxComboCtrl*) wxWindow::FindWindowById(RIGHTMAP_MAP_CHOICE, this);
-  MapSelectComboPopup* popup = (MapSelectComboPopup*) rightmapCombo->GetPopupControl();
-  int old = currentMap->getRightMoveMapID();
+  wxComboCtrl* combo = (wxComboCtrl*) wxWindow::FindWindowById(RIGHTMAP_MAP_CHOICE, this);
+  Map::MoveDir dir = Map::MoveDir::Right;
+  
+  Map::Adjacent adjacent = currentMap->getAdjacent(dir);
+  MapSelectComboPopup* popup = (MapSelectComboPopup*) combo->GetPopupControl();
+  int old = adjacent.map;
   int newt = popup->GetSelectedMapID();
   
   if (old == newt) return;
   
   commitAction(std::make_shared<Undoable>("Set Rightmap Map", [=] () {
     popup->SetSelectedMapID(newt);
-    rightmapCombo->SetValue(world->getMap(newt)->getTitle());
-    currentMap->setRightMoveMapID(newt);
+    combo->SetValue(world->getMap(newt)->getTitle());
+    currentMap->setAdjacent(dir, adjacent.type, newt);
   }, [=] () {
     popup->SetSelectedMapID(old);
-    rightmapCombo->SetValue(world->getMap(old)->getTitle());
-    currentMap->setRightMoveMapID(old);
+    combo->SetValue(world->getMap(old)->getTitle());
+    currentMap->setAdjacent(dir, adjacent.type, old);
   }));
 }
 
 void MapeditFrame::OnSetUpmapType(wxCommandEvent&)
 {
-  wxChoice* upmapChoice = (wxChoice*) wxWindow::FindWindowById(UPMAP_TYPE_CHOICE, this);
-  wxComboCtrl* upmapCombo = (wxComboCtrl*) wxWindow::FindWindowById(UPMAP_MAP_CHOICE, this);
-  
-  Map::MoveType old = currentMap->getUpMoveType();
-  Map::MoveType newt = ((MoveTypeCtr*) upmapChoice->GetClientData(upmapChoice->GetSelection()))->type;
+  wxChoice* choice = (wxChoice*) wxWindow::FindWindowById(UPMAP_TYPE_CHOICE, this);
+  wxComboCtrl* combo = (wxComboCtrl*) wxChoice::FindWindowById(UPMAP_MAP_CHOICE, this);
+  Map::MoveDir dir = Map::MoveDir::Up;
   
+  Map::Adjacent adjacent = currentMap->getAdjacent(dir);
+  Map::MoveType newt = ((MoveTypeCtr*) choice->GetClientData(choice->GetSelection()))->type;
   commitAction(std::make_shared<Undoable>("Set Upmap Action", [=] () {
-    upmapChoice->SetSelection(upmapChoice->FindString(Map::stringForMoveType(newt)));
-    currentMap->setUpMoveType(newt);
-    upmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getUpMoveType()));
+    choice->SetSelection(choice->FindString(Map::stringForMoveType(newt)));
+    currentMap->setAdjacent(dir, newt);
+    combo->Enable(Map::moveTypeTakesMap(newt));
   }, [=] () {
-    upmapChoice->SetSelection(upmapChoice->FindString(Map::stringForMoveType(old)));
-    currentMap->setUpMoveType(old);
-    upmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getUpMoveType()));
+    choice->SetSelection(choice->FindString(Map::stringForMoveType(adjacent.type)));
+    currentMap->setAdjacent(dir, adjacent.type);
+    combo->Enable(Map::moveTypeTakesMap(adjacent.type));
   }));
 }
 
 void MapeditFrame::OnSetUpmapMap(wxCommandEvent&)
 {
-  wxComboCtrl* upmapCombo = (wxComboCtrl*) wxWindow::FindWindowById(UPMAP_MAP_CHOICE, this);
-  MapSelectComboPopup* popup = (MapSelectComboPopup*) upmapCombo->GetPopupControl();
-  int old = currentMap->getUpMoveMapID();
+  wxComboCtrl* combo = (wxComboCtrl*) wxWindow::FindWindowById(UPMAP_MAP_CHOICE, this);
+  Map::MoveDir dir = Map::MoveDir::Up;
+  
+  Map::Adjacent adjacent = currentMap->getAdjacent(dir);
+  MapSelectComboPopup* popup = (MapSelectComboPopup*) combo->GetPopupControl();
+  int old = adjacent.map;
   int newt = popup->GetSelectedMapID();
   
   if (old == newt) return;
   
   commitAction(std::make_shared<Undoable>("Set Upmap Map", [=] () {
     popup->SetSelectedMapID(newt);
-    upmapCombo->SetValue(world->getMap(newt)->getTitle());
-    currentMap->setUpMoveMapID(newt);
+    combo->SetValue(world->getMap(newt)->getTitle());
+    currentMap->setAdjacent(dir, adjacent.type, newt);
   }, [=] () {
     popup->SetSelectedMapID(old);
-    upmapCombo->SetValue(world->getMap(old)->getTitle());
-    currentMap->setUpMoveMapID(old);
+    combo->SetValue(world->getMap(old)->getTitle());
+    currentMap->setAdjacent(dir, adjacent.type, old);
   }));
 }
 
 void MapeditFrame::OnSetDownmapType(wxCommandEvent&)
 {
-  wxChoice* downmapChoice = (wxChoice*) wxWindow::FindWindowById(DOWNMAP_TYPE_CHOICE, this);
-  wxComboCtrl* downmapCombo = (wxComboCtrl*) wxWindow::FindWindowById(DOWNMAP_MAP_CHOICE, this);
-  
-  Map::MoveType old = currentMap->getDownMoveType();
-  Map::MoveType newt = ((MoveTypeCtr*) downmapChoice->GetClientData(downmapChoice->GetSelection()))->type;
+  wxChoice* choice = (wxChoice*) wxWindow::FindWindowById(DOWNMAP_TYPE_CHOICE, this);
+  wxComboCtrl* combo = (wxComboCtrl*) wxChoice::FindWindowById(DOWNMAP_MAP_CHOICE, this);
+  Map::MoveDir dir = Map::MoveDir::Down;
   
+  Map::Adjacent adjacent = currentMap->getAdjacent(dir);
+  Map::MoveType newt = ((MoveTypeCtr*) choice->GetClientData(choice->GetSelection()))->type;
   commitAction(std::make_shared<Undoable>("Set Downmap Action", [=] () {
-    downmapChoice->SetSelection(downmapChoice->FindString(Map::stringForMoveType(newt)));
-    currentMap->setDownMoveType(newt);
-    downmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getDownMoveType()));
+    choice->SetSelection(choice->FindString(Map::stringForMoveType(newt)));
+    currentMap->setAdjacent(dir, newt);
+    combo->Enable(Map::moveTypeTakesMap(newt));
   }, [=] () {
-    downmapChoice->SetSelection(downmapChoice->FindString(Map::stringForMoveType(old)));
-    currentMap->setDownMoveType(old);
-    downmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getDownMoveType()));
+    choice->SetSelection(choice->FindString(Map::stringForMoveType(adjacent.type)));
+    currentMap->setAdjacent(dir, adjacent.type);
+    combo->Enable(Map::moveTypeTakesMap(adjacent.type));
   }));
 }
 
 void MapeditFrame::OnSetDownmapMap(wxCommandEvent&)
 {
-  wxComboCtrl* downmapCombo = (wxComboCtrl*) wxWindow::FindWindowById(DOWNMAP_MAP_CHOICE, this);
-  MapSelectComboPopup* popup = (MapSelectComboPopup*) downmapCombo->GetPopupControl();
-  int old = currentMap->getDownMoveMapID();
+  wxComboCtrl* combo = (wxComboCtrl*) wxWindow::FindWindowById(DOWNMAP_MAP_CHOICE, this);
+  Map::MoveDir dir = Map::MoveDir::Down;
+  
+  Map::Adjacent adjacent = currentMap->getAdjacent(dir);
+  MapSelectComboPopup* popup = (MapSelectComboPopup*) combo->GetPopupControl();
+  int old = adjacent.map;
   int newt = popup->GetSelectedMapID();
   
   if (old == newt) return;
   
   commitAction(std::make_shared<Undoable>("Set Downmap Map", [=] () {
     popup->SetSelectedMapID(newt);
-    downmapCombo->SetValue(world->getMap(newt)->getTitle());
-    currentMap->setDownMoveMapID(newt);
+    combo->SetValue(world->getMap(newt)->getTitle());
+    currentMap->setAdjacent(dir, adjacent.type, newt);
   }, [=] () {
     popup->SetSelectedMapID(old);
-    downmapCombo->SetValue(world->getMap(old)->getTitle());
-    currentMap->setDownMoveMapID(old);
+    combo->SetValue(world->getMap(old)->getTitle());
+    currentMap->setAdjacent(dir, adjacent.type, old);
   }));
 }
 
@@ -924,20 +936,20 @@ void MapeditFrame::SelectMap(Map* map)
   wxComboCtrl* upmapCombo = (wxComboCtrl*) wxWindow::FindWindowById(UPMAP_MAP_CHOICE, this);
   wxComboCtrl* downmapCombo = (wxComboCtrl*) wxWindow::FindWindowById(DOWNMAP_MAP_CHOICE, this);
   
-  leftmapChoice->SetSelection(leftmapChoice->FindString(Map::stringForMoveType(currentMap->getLeftMoveType())));
-  rightmapChoice->SetSelection(rightmapChoice->FindString(Map::stringForMoveType(currentMap->getRightMoveType())));
-  upmapChoice->SetSelection(upmapChoice->FindString(Map::stringForMoveType(currentMap->getUpMoveType())));
-  downmapChoice->SetSelection(downmapChoice->FindString(Map::stringForMoveType(currentMap->getDownMoveType())));
+  leftmapChoice->SetSelection(leftmapChoice->FindString(Map::stringForMoveType(currentMap->getAdjacent(Map::MoveDir::Left).type)));
+  rightmapChoice->SetSelection(rightmapChoice->FindString(Map::stringForMoveType(currentMap->getAdjacent(Map::MoveDir::Right).type)));
+  upmapChoice->SetSelection(upmapChoice->FindString(Map::stringForMoveType(currentMap->getAdjacent(Map::MoveDir::Up).type)));
+  downmapChoice->SetSelection(downmapChoice->FindString(Map::stringForMoveType(currentMap->getAdjacent(Map::MoveDir::Down).type)));
   
-  leftmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getLeftMoveType()));
-  rightmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getRightMoveType()));
-  upmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getUpMoveType()));
-  downmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getDownMoveType()));
+  leftmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getAdjacent(Map::MoveDir::Left).type));
+  rightmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getAdjacent(Map::MoveDir::Right).type));
+  upmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getAdjacent(Map::MoveDir::Up).type));
+  downmapCombo->Enable(Map::moveTypeTakesMap(currentMap->getAdjacent(Map::MoveDir::Down).type));
   
-  leftmapCombo->SetValue(world->getMap(currentMap->getLeftMoveMapID())->getTitle());
-  rightmapCombo->SetValue(world->getMap(currentMap->getRightMoveMapID())->getTitle());
-  upmapCombo->SetValue(world->getMap(currentMap->getUpMoveMapID())->getTitle());
-  downmapCombo->SetValue(world->getMap(currentMap->getDownMoveMapID())->getTitle());
+  leftmapCombo->SetValue(world->getMap(currentMap->getAdjacent(Map::MoveDir::Left).map)->getTitle());
+  rightmapCombo->SetValue(world->getMap(currentMap->getAdjacent(Map::MoveDir::Right).map)->getTitle());
+  upmapCombo->SetValue(world->getMap(currentMap->getAdjacent(Map::MoveDir::Up).map)->getTitle());
+  downmapCombo->SetValue(world->getMap(currentMap->getAdjacent(Map::MoveDir::Down).map)->getTitle());
 }
 
 wxTreeItemId MapeditFrame::MoveTreeNode(wxTreeItemId toCopy, wxTreeItemId newParent)
@@ -21,14 +21,7 @@ Map::Map(const Map& map)
   treeItemId = map.treeItemId;
   children = map.children;
   hidden = map.hidden;
-  leftType = map.leftType;
-  rightType = map.rightType;
-  upType = map.upType;
-  downType = map.downType;
-  leftMap = map.leftMap;
-  rightMap = map.rightMap;
-  downMap = map.downMap;
-  upMap = map.upMap;
+  adjacents = map.adjacents;
 }
 
 Map::Map(Map&& map) : Map(-1, map.world)
@@ -58,14 +51,7 @@ void swap(Map& first, Map& second)
   std::swap(first.treeItemId, second.treeItemId);
   std::swap(first.children, second.children);
   std::swap(first.hidden, second.hidden);
-  std::swap(first.leftType, second.leftType);
-  std::swap(first.rightType, second.rightType);
-  std::swap(first.upType, second.upType);
-  std::swap(first.downType, second.downType);
-  std::swap(first.leftMap, second.leftMap);
-  std::swap(first.rightMap, second.rightMap);
-  std::swap(first.downMap, second.downMap);
-  std::swap(first.upMap, second.upMap);
+  std::swap(first.adjacents, second.adjacents);
 }
 
 std::list<Map::MoveType> Map::listMoveTypes()
@@ -78,9 +64,9 @@ std::string Map::stringForMoveType(MoveType type)
   switch (type)
   {
     case MoveType::Wall: return "Wall";
-    case MoveType::Warp: return "Warp";
+    case MoveType::Warp: return "Teleport";
     case MoveType::Wrap: return "Wrap";
-    case MoveType::ReverseWarp: return "Reverse Warp";
+    case MoveType::ReverseWarp: return "Reverse Teleport";
   }
 }
 
@@ -106,6 +92,17 @@ std::string Map::shortForMoveType(MoveType type)
   }
 }
 
+std::string Map::shortForMoveDir(MoveDir dir)
+{
+  switch (dir)
+  {
+    case MoveDir::Left: return "left";
+    case MoveDir::Right: return "right";
+    case MoveDir::Up: return "up";
+    case MoveDir::Down: return "down";
+  }
+}
+
 Map::MoveType Map::moveTypeForShort(std::string str)
 {
   if (str == "wrap") return MoveType::Wrap;
@@ -115,6 +112,15 @@ Map::MoveType Map::moveTypeForShort(std::string str)
   return MoveType::Wall;
 }
 
+Map::MoveDir Map::moveDirForShort(std::string str)
+{
+  if (str == "right") return MoveDir::Right;
+  if (str == "up") return MoveDir::Up;
+  if (str == "down") return MoveDir::Down;
+  
+  return MoveDir::Left;
+}
+
 int Map::getID() const
 {
   return id;
@@ -167,47 +173,21 @@ bool Map::getHidden() const
   return hidden;
 }
 
-Map::MoveType Map::getLeftMoveType() const
-{
-  return leftType;
-}
-
-Map::MoveType Map::getRightMoveType() const
-{
-  return rightType;
-}
-
-Map::MoveType Map::getUpMoveType() const
+const std::map<Map::MoveDir, Map::Adjacent>& Map::getAdjacents() const
 {
-  return upType;
+  return adjacents;
 }
 
-Map::MoveType Map::getDownMoveType() const
+const Map::Adjacent& Map::getAdjacent(MoveDir direction) const
 {
-  return downType;
-}
-
-int Map::getLeftMoveMapID() const
-{
-  return leftMap;
-}
-
-int Map::getRightMoveMapID() const
-{
-  return rightMap;
-}
-
-int Map::getUpMoveMapID() const
-{
-  return upMap;
-}
-
-int Map::getDownMoveMapID() const
-{
-  return downMap;
+  if (adjacents.count(direction) > 0)
+  {
+    return adjacents.at(direction);
+  } else {
+    return defaultAdjacent;
+  }
 }
 
-
 void Map::setTitle(std::string title, bool dirty)
 {
   this->title = title;
@@ -279,79 +259,11 @@ void Map::setHidden(bool hid)
   hidden = hid;
 }
 
-void Map::setLeftMoveType(Map::MoveType move, bool dirty)
-{
-  leftType = move;
-  
-  if (dirty)
-  {
-    world->setDirty(true);
-  }
-}
-
-void Map::setRightMoveType(Map::MoveType move, bool dirty)
-{
-  rightType = move;
-  
-  if (dirty)
-  {
-    world->setDirty(true);
-  }
-}
-
-void Map::setUpMoveType(Map::MoveType move, bool dirty)
-{
-  upType = move;
-  
-  if (dirty)
-  {
-    world->setDirty(true);
-  }
-}
-
-void Map::setDownMoveType(Map::MoveType move, bool dirty)
-{
-  downType = move;
-  
-  if (dirty)
-  {
-    world->setDirty(true);
-  }
-}
-
-void Map::setLeftMoveMapID(int id, bool dirty)
-{
-  leftMap = id;
-  
-  if (dirty)
-  {
-    world->setDirty(true);
-  }
-}
-
-void Map::setRightMoveMapID(int id, bool dirty)
-{
-  rightMap = id;
-  
-  if (dirty)
-  {
-    world->setDirty(true);
-  }
-}
-
-void Map::setUpMoveMapID(int id, bool dirty)
-{
-  upMap = id;
-  
-  if (dirty)
-  {
-    world->setDirty(true);
-  }
-}
-
-void Map::setDownMoveMapID(int id, bool dirty)
+void Map::setAdjacent(MoveDir direction, MoveType type, int map, bool dirty)
 {
-  downMap = id;
+  Adjacent& cur = adjacents[direction];
+  cur.type = type;
+  if (map != -1) cur.map = map;
   
   if (dirty)
   {
@@ -7,6 +7,7 @@
 #include <list>
 #include <memory>
 #include <wx/treectrl.h>
+#include <map>
 
 class MapObject;
 class World;
@@ -64,6 +65,13 @@ class Map {
     Map& operator= (Map other);
     friend void swap(Map& first, Map& second);
     
+    enum class MoveDir {
+      Left,
+      Right,
+      Up,
+      Down
+    };
+    
     enum class MoveType {
       Wall,
       Wrap,
@@ -71,11 +79,18 @@ class Map {
       ReverseWarp
     };
     
+    struct Adjacent {
+      MoveType type = MoveType::Wall;
+      int map = 0;
+    };
+    
     static std::list<MoveType> listMoveTypes();
     static std::string stringForMoveType(MoveType type);
     static bool moveTypeTakesMap(MoveType type);
     static std::string shortForMoveType(MoveType type);
+    static std::string shortForMoveDir(MoveDir dir);
     static MoveType moveTypeForShort(std::string str);
+    static MoveDir moveDirForShort(std::string str);
     
     int getID() const;
     std::string getTitle() const;
@@ -86,14 +101,8 @@ class Map {
     bool getExpanded() const;
     World* getWorld() const;
     bool getHidden() const;
-    MoveType getLeftMoveType() const;
-    MoveType getRightMoveType() const;
-    MoveType getUpMoveType() const;
-    MoveType getDownMoveType() const;
-    int getLeftMoveMapID() const;
-    int getRightMoveMapID() const;
-    int getUpMoveMapID() const;
-    int getDownMoveMapID() const;
+    const std::map<MoveDir, Adjacent>& getAdjacents() const;
+    const Adjacent& getAdjacent(MoveDir direction) const;
     
     void setTitle(std::string title, bool dirty = true);
     void setTileAt(int x, int y, int tile, bool dirty = true);
@@ -104,14 +113,7 @@ class Map {
     void addChild(int id);
     void setExpanded(bool exp);
     void setHidden(bool hid);
-    void setLeftMoveType(MoveType move, bool dirty = true);
-    void setRightMoveType(MoveType move, bool dirty = true);
-    void setUpMoveType(MoveType move, bool dirty = true);
-    void setDownMoveType(MoveType move, bool dirty = true);
-    void setLeftMoveMapID(int id, bool dirty = true);
-    void setRightMoveMapID(int id, bool dirty = true);
-    void setUpMoveMapID(int id, bool dirty = true);
-    void setDownMoveMapID(int id, bool dirty = true);
+    void setAdjacent(MoveDir direction, MoveType type, int map = -1, bool dirty = true);
     
   private:
     int id;
@@ -123,14 +125,8 @@ class Map {
     wxTreeItemId treeItemId;
     bool expanded = false;
     bool hidden = false;
-    MoveType leftType = MoveType::Wall;
-    MoveType rightType = MoveType::Wall;
-    MoveType upType = MoveType::Wall;
-    MoveType downType = MoveType::Wall;
-    int leftMap = 0;
-    int rightMap = 0;
-    int upMap = 0;
-    int downMap = 0;
+    std::map<MoveDir, Adjacent> adjacents;
+    const Adjacent defaultAdjacent {};
 };
 
 class MapPtrCtr : public wxTreeItemData {
@@ -75,30 +75,30 @@ void MapeditWidget::OnPaint(wxPaintEvent&)
 
   RenderMap(map, dc, tiles_dc);
   
-  if (map->getLeftMoveType() == Map::MoveType::Warp)
+  if (map->getAdjacent(Map::MoveDir::Left).type == Map::MoveType::Warp)
   {
-    auto tomap = map->getWorld()->getMap(map->getLeftMoveMapID());
+    auto tomap = map->getWorld()->getMap(map->getAdjacent(Map::MoveDir::Left).map);
     
     RenderMap(tomap.get(), dc, tiles_dc, -EDITOR_SPACING_X, EDITOR_SPACING_Y, false);
   }
   
-  if (map->getRightMoveType() == Map::MoveType::Warp)
+  if (map->getAdjacent(Map::MoveDir::Right).type == Map::MoveType::Warp)
   {
-    auto tomap = map->getWorld()->getMap(map->getRightMoveMapID());
+    auto tomap = map->getWorld()->getMap(map->getAdjacent(Map::MoveDir::Right).map);
     
     RenderMap(tomap.get(), dc, tiles_dc, EDITOR_WIDTH-EDITOR_SPACING_X, EDITOR_SPACING_Y, false);
   }
   
-  if (map->getUpMoveType() == Map::MoveType::Warp)
+  if (map->getAdjacent(Map::MoveDir::Up).type == Map::MoveType::Warp)
   {
-    auto tomap = map->getWorld()->getMap(map->getUpMoveMapID());
+    auto tomap = map->getWorld()->getMap(map->getAdjacent(Map::MoveDir::Up).map);
     
     RenderMap(tomap.get(), dc, tiles_dc, EDITOR_SPACING_X, -EDITOR_SPACING_Y, false);
   }
   
-  if (map->getDownMoveType() == Map::MoveType::Warp)
+  if (map->getAdjacent(Map::MoveDir::Down).type == Map::MoveType::Warp)
   {
-    auto tomap = map->getWorld()->getMap(map->getDownMoveMapID());
+    auto tomap = map->getWorld()->getMap(map->getAdjacent(Map::MoveDir::Down).map);
     
     RenderMap(tomap.get(), dc, tiles_dc, EDITOR_SPACING_X, EDITOR_HEIGHT-EDITOR_SPACING_Y, false);
   }
@@ -36,43 +36,43 @@ World::World(std::string filename)
     throw MapLoadException(filename);
   }
   
+  xmlChar* nextmapKey = xmlGetProp(top, (xmlChar*) "nextmap");
+  if (nextmapKey != 0)
+  {
+    nextMapID = atoi((char*) nextmapKey);
+  }
+  xmlFree(nextmapKey);
+  
+  xmlChar* lastmapKey = xmlGetProp(top, (xmlChar*) "lastmap");
+  if (lastmapKey != 0)
+  {
+    lastmap = atoi((char*) lastmapKey);
+  }
+  xmlFree(lastmapKey);
+  
+  xmlChar* startxKey = xmlGetProp(top, (xmlChar*) "startx");
+  if (startxKey == 0) throw MapLoadException(filename);
+  startingPosition.first = atoi((char*) startxKey);
+  xmlFree(startxKey);
+  
+  xmlChar* startyKey = xmlGetProp(top, (xmlChar*) "starty");
+  if (startyKey == 0) throw MapLoadException(filename);
+  startingPosition.second = atoi((char*) startyKey);
+  xmlFree(startyKey);
+  
+  xmlChar* startmapKey = xmlGetProp(top, (xmlChar*) "startmap");
+  if (startxKey == 0) throw MapLoadException(filename);
+  startingMap = atoi((char*) startmapKey);
+  xmlFree(startmapKey);
+  
   for (xmlNodePtr node = top->xmlChildrenNode; node != NULL; node = node->next)
   {
-    if (!xmlStrcmp(node->name, (const xmlChar*) "nextmapid"))
-    {
-      xmlChar* key = xmlNodeListGetString(doc, node->xmlChildrenNode, 1);
-      if (key != 0)
-      {
-        nextMapID = atoi((char*) key);
-      }
-      xmlFree(key);
-    } else if (!xmlStrcmp(node->name, (const xmlChar*) "lastmap"))
+    if (!xmlStrcmp(node->name, (const xmlChar*) "root"))
     {
       xmlChar* key = xmlNodeListGetString(doc, node->xmlChildrenNode, 1);
-      if (key != 0)
-      {
-        lastmap = atoi((char*) key);
-      }
+      if (key == 0) throw MapLoadException(filename);
+      rootChildren.push_back(atoi((char*) key));
       xmlFree(key);
-    } else if (!xmlStrcmp(node->name, (const xmlChar*) "root"))
-    {
-      xmlChar* key = xmlNodeListGetString(doc, node->xmlChildrenNode, 1);
-      if (key != 0)
-      {
-        rootChildren.push_back(atoi((char*) key));
-      }
-      xmlFree(key);
-    } else if (!xmlStrcmp(node->name, (const xmlChar*) "startpos"))
-    {
-      xmlChar* idKey = xmlGetProp(node, (xmlChar*) "id");
-      if (idKey == 0) throw MapLoadException(filename);
-      startingMap = atoi((char*) idKey);
-      xmlFree(idKey);
-      
-      xmlChar* posKey = xmlGetProp(node, (xmlChar*) "pos");
-      if (posKey == 0) throw MapLoadException(filename);
-      sscanf((char*) posKey, "%d,%d", &startingPosition.first, &startingPosition.second);
-      xmlFree(posKey);
     } else if (!xmlStrcmp(node->name, (const xmlChar*) "map"))
     {
       xmlChar* idKey = xmlGetProp(node, (xmlChar*) "id");
@@ -82,18 +82,21 @@ World::World(std::string filename)
       
       auto map = std::make_shared<Map>(id, this);
       
+      xmlChar* expandKey = xmlGetProp(node, (xmlChar*) "expanded");
+      if ((expandKey != 0) && (!xmlStrcmp(expandKey, (const xmlChar*) "true")))
+      {
+        map->setExpanded(true);
+      }
+      xmlFree(expandKey);
+      
+      xmlChar* titleKey = xmlGetProp(node, (xmlChar*) "title");
+      if (titleKey == 0) throw MapLoadException(filename);
+      map->setTitle((char*) titleKey, false);
+      xmlFree(titleKey);
+      
       for (xmlNodePtr mapNode = node->xmlChildrenNode; mapNode != NULL; mapNode = mapNode->next)
       {
-        if (!xmlStrcmp(mapNode->name, (const xmlChar*) "name"))
-        {
-          xmlChar* key = xmlNodeListGetString(doc, mapNode->xmlChildrenNode, 1);
-          if (key != 0)
-          {
-            map->setTitle((char*) key, false);
-          }
-          
-          xmlFree(key);
-        } else if (!xmlStrcmp(mapNode->name, (const xmlChar*) "environment"))
+        if (!xmlStrcmp(mapNode->name, (const xmlChar*) "environment"))
         {
           xmlChar* key = xmlNodeListGetString(doc, mapNode->xmlChildrenNode, 1);
           int* mapdata = (int*) malloc(MAP_WIDTH*MAP_HEIGHT*sizeof(int));
@@ -104,88 +107,50 @@ World::World(std::string filename)
           }
           map->setMapdata(mapdata, false);
           xmlFree(key);
-        } else if (!xmlStrcmp(mapNode->name, (const xmlChar*) "leftmap"))
+        } else if (!xmlStrcmp(mapNode->name, (const xmlChar*) "entity"))
         {
-          xmlChar* typeKey = xmlGetProp(mapNode, (xmlChar*) "type");
-          if (typeKey == 0) throw MapLoadException(filename);
-          map->setLeftMoveType(Map::moveTypeForShort((char*) typeKey), false);
-          xmlFree(typeKey);
+          auto data = std::make_shared<MapObjectEntry>();
           
-          if (Map::moveTypeTakesMap(map->getLeftMoveType()))
-          {
-            xmlChar* idKey = xmlGetProp(mapNode, (xmlChar*) "map");
-            if (idKey == 0) throw MapLoadException(filename);
-            map->setLeftMoveMapID(atoi((char*) idKey), false);
-            xmlFree(idKey);
-          }
-        } else if (!xmlStrcmp(mapNode->name, (const xmlChar*) "rightmap"))
-        {
-          xmlChar* typeKey = xmlGetProp(mapNode, (xmlChar*) "type");
+          xmlChar* typeKey = xmlGetProp(mapNode, (const xmlChar*) "type");
           if (typeKey == 0) throw MapLoadException(filename);
-          map->setRightMoveType(Map::moveTypeForShort((char*) typeKey), false);
+          data->object = MapObject::getAllObjects().at((char*) typeKey).get();
           xmlFree(typeKey);
           
-          if (Map::moveTypeTakesMap(map->getRightMoveType()))
-          {
-            xmlChar* idKey = xmlGetProp(mapNode, (xmlChar*) "map");
-            if (idKey == 0) throw MapLoadException(filename);
-            map->setRightMoveMapID(atoi((char*) idKey), false);
-            xmlFree(idKey);
-          }
-        } else if (!xmlStrcmp(mapNode->name, (const xmlChar*) "upmap"))
-        {
-          xmlChar* typeKey = xmlGetProp(mapNode, (xmlChar*) "type");
-          if (typeKey == 0) throw MapLoadException(filename);
-          map->setUpMoveType(Map::moveTypeForShort((char*) typeKey), false);
-          xmlFree(typeKey);
+          xmlChar* xKey = xmlGetProp(mapNode, (const xmlChar*) "x");
+          if (xKey == 0) throw MapLoadException(filename);
+          data->position.first = atoi((char*) xKey);
+          xmlFree(xKey);
           
-          if (Map::moveTypeTakesMap(map->getUpMoveType()))
-          {
-            xmlChar* idKey = xmlGetProp(mapNode, (xmlChar*) "map");
-            if (idKey == 0) throw MapLoadException(filename);
-            map->setUpMoveMapID(atoi((char*) idKey), false);
-            xmlFree(idKey);
-          }
-        } else if (!xmlStrcmp(mapNode->name, (const xmlChar*) "downmap"))
+          xmlChar* yKey = xmlGetProp(mapNode, (const xmlChar*) "y");
+          if (yKey == 0) throw MapLoadException(filename);
+          data->position.second = atoi((char*) yKey);
+          xmlFree(yKey);
+          
+          map->addObject(data, false);
+        } else if (!xmlStrcmp(mapNode->name, (const xmlChar*) "adjacent"))
         {
-          xmlChar* typeKey = xmlGetProp(mapNode, (xmlChar*) "type");
+          Map::MoveDir direction;
+          Map::MoveType moveType;
+          int mapId = 0;
+          
+          xmlChar* dirKey = xmlGetProp(mapNode, (const xmlChar*) "dir");
+          if (dirKey == 0) throw MapLoadException(filename);
+          direction = Map::moveDirForShort((char*) dirKey);
+          xmlFree(dirKey);
+          
+          xmlChar* typeKey = xmlGetProp(mapNode, (const xmlChar*) "type");
           if (typeKey == 0) throw MapLoadException(filename);
-          map->setDownMoveType(Map::moveTypeForShort((char*) typeKey), false);
+          moveType = Map::moveTypeForShort((char*) typeKey);
           xmlFree(typeKey);
           
-          if (Map::moveTypeTakesMap(map->getDownMoveType()))
+          xmlChar* mapIdKey = xmlGetProp(mapNode, (const xmlChar*) "map");
+          if (mapIdKey != 0)
           {
-            xmlChar* idKey = xmlGetProp(mapNode, (xmlChar*) "map");
-            if (idKey == 0) throw MapLoadException(filename);
-            map->setDownMoveMapID(atoi((char*) idKey), false);
-            xmlFree(idKey);
+            mapId = atoi((char*) mapIdKey);
           }
-        } else if (!xmlStrcmp(mapNode->name, (const xmlChar*) "entities"))
-        {
-          for (xmlNodePtr entityNode = mapNode->xmlChildrenNode; entityNode != NULL; entityNode = entityNode->next)
-          {
-            if (!xmlStrcmp(entityNode->name, (const xmlChar*) "entity"))
-            {
-              auto data = std::make_shared<MapObjectEntry>();
+          xmlFree(mapIdKey);
           
-              for (xmlNodePtr entityDataNode = entityNode->xmlChildrenNode; entityDataNode != NULL; entityDataNode = entityDataNode->next)
-              {
-                if (!xmlStrcmp(entityDataNode->name, (const xmlChar*) "entity-type"))
-                {
-                  xmlChar* key = xmlNodeListGetString(doc, entityDataNode->xmlChildrenNode, 1);
-                  data->object = MapObject::getAllObjects().at((char*) key).get();
-                  xmlFree(key);
-                } else if (!xmlStrcmp(entityDataNode->name, (const xmlChar*) "entity-position"))
-                {
-                  xmlChar* key = xmlNodeListGetString(doc, entityDataNode->xmlChildrenNode, 1);
-                  sscanf((char*) key, "%d,%d", &data->position.first, &data->position.second);
-                  xmlFree(key);
-                }
-              }
-        
-              map->addObject(data, false);
-            }
-          }
+          map->setAdjacent(direction, moveType, mapId, false);
         } else if (!xmlStrcmp(mapNode->name, (const xmlChar*) "child"))
         {
           xmlChar* key = xmlNodeListGetString(doc, mapNode->xmlChildrenNode, 1);
@@ -194,13 +159,6 @@ World::World(std::string filename)
             map->addChild(atoi((char*) key));
           }
           xmlFree(key);
-        } else if (!xmlStrcmp(mapNode->name, (const xmlChar*) "expanded"))
-        {
-          xmlChar* key = xmlNodeListGetString(doc, mapNode->xmlChildrenNode, 1);
-          if ((key != 0) && ((char) key[0] == '1'))
-          {
-            map->setExpanded(true);
-          }
         }
       }
       
@@ -266,16 +224,24 @@ void World::save(std::string name, wxTreeCtrl* mapTree)
   rc = xmlTextWriterStartElement(writer, (xmlChar*) "world");
   if (rc < 0) throw MapWriteException(name);
   
-  //   <nextmapid/>
-  std::ostringstream nextMap_out;
-  nextMap_out << nextMapID;
-  rc = xmlTextWriterWriteElement(writer, (xmlChar*) "nextmapid", (xmlChar*) nextMap_out.str().c_str());
+  //   nextmap=
+  rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "nextmap", "%d", nextMapID);
+  if (rc < 0) throw MapWriteException(name);
+  
+  //   lastmap=
+  rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "lastmap", "%d", lastmap);
+  if (rc < 0) throw MapWriteException(name);
+  
+  //   startx=
+  rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "startx", "%d", startingPosition.first);
   if (rc < 0) throw MapWriteException(name);
   
-  //   <lastmap/>
-  std::ostringstream lastMap_out;
-  lastMap_out << lastmap;
-  rc = xmlTextWriterWriteElement(writer, (xmlChar*) "lastmap", (xmlChar*) lastMap_out.str().c_str());
+  //   starty=
+  rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "starty", "%d", startingPosition.second);
+  if (rc < 0) throw MapWriteException(name);
+  
+  //   startmap=
+  rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "startmap", "%d", startingMap);
   if (rc < 0) throw MapWriteException(name);
   
   // ASSUMPTION: There will always be at least one child of the invisible root element. i.e. you cannot delete to zero maps.
@@ -285,28 +251,10 @@ void World::save(std::string name, wxTreeCtrl* mapTree)
   {
     // <root>
     MapPtrCtr* ctl = (MapPtrCtr*) mapTree->GetItemData(it);
-    std::ostringstream rootid_out;
-    rootid_out << ctl->map->getID();
-    rc = xmlTextWriterWriteElement(writer, (xmlChar*) "root", (xmlChar*) rootid_out.str().c_str());
+    rc = xmlTextWriterWriteFormatElement(writer, (xmlChar*) "root", "%d", ctl->map->getID());
     if (rc < 0) throw MapWriteException(name);
   }
   
-  //   <startpos/>
-  rc = xmlTextWriterStartElement(writer, (xmlChar*) "startpos");
-  if (rc < 0) throw MapWriteException(name);
-  
-  //   id=
-  rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "id", "%d", startingMap);
-  if (rc < 0) throw MapWriteException(name);
-  
-  //   pos=
-  rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "pos", "%d,%d", startingPosition.first, startingPosition.second);
-  if (rc < 0) throw MapWriteException(name);
-  
-  //   </startpos>
-  rc = xmlTextWriterEndElement(writer);
-  if (rc < 0) throw MapWriteException(name);
-  
   for (auto mapPair : maps)
   {
     Map& map = *mapPair.second;
@@ -318,16 +266,33 @@ void World::save(std::string name, wxTreeCtrl* mapTree)
     if (rc < 0) throw MapWriteException(name);
     
     // id=
-    std::ostringstream id_out;
-    id_out << map.getID();
-    rc = xmlTextWriterWriteAttribute(writer, (xmlChar*) "id", (xmlChar*) id_out.str().c_str());
+    rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "id", "%d", map.getID());
     if (rc < 0) throw MapWriteException(name);
     
-    //   <name/>
-    rc = xmlTextWriterWriteElement(writer, (xmlChar*) "name", (xmlChar*) map.getTitle().c_str());
+    // expanded=
+    wxTreeItemId node = map.getTreeItemId();
+    if (mapTree->IsExpanded(node))
+    {
+      rc = xmlTextWriterWriteAttribute(writer, (xmlChar*) "expanded", (xmlChar*) "true");
+      if (rc < 0) throw MapWriteException(name);
+    } else {
+      rc = xmlTextWriterWriteAttribute(writer, (xmlChar*) "expanded", (xmlChar*) "false");
+      if (rc < 0) throw MapWriteException(name);
+    }
+    
+    // title=
+    rc = xmlTextWriterWriteAttribute(writer, (xmlChar*) "name", (xmlChar*) map.getTitle().c_str());
     if (rc < 0) throw MapWriteException(name);
   
-    //   <environment/>
+    //   <environment
+    rc = xmlTextWriterStartElement(writer, (xmlChar*) "environment");
+    if (rc < 0) throw MapWriteException(name);
+    
+    //   type=
+    rc = xmlTextWriterWriteAttribute(writer, (xmlChar*) "type", (xmlChar*) "0");
+    if (rc < 0) throw MapWriteException(name);
+    
+    //   >
     std::ostringstream mapdata_out;
     for (int y=0; y<MAP_HEIGHT; y++)
     {
@@ -341,113 +306,60 @@ void World::save(std::string name, wxTreeCtrl* mapTree)
   
     rc = xmlTextWriterWriteElement(writer, (xmlChar*) "environment", (xmlChar*) mapdata_out.str().c_str());
     if (rc < 0) throw MapWriteException(name);
-  
-    //   <leftmap/>
-    rc = xmlTextWriterStartElement(writer, (xmlChar*) "leftmap");
-    if (rc < 0) throw MapWriteException(name);
     
-    //   type=
-    rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "type", "%s", Map::shortForMoveType(map.getLeftMoveType()).c_str());
-    if (rc < 0) throw MapWriteException(name);
-    
-    if (Map::moveTypeTakesMap(map.getLeftMoveType()))
-    {
-      // map=
-      rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "map", "%d", map.getLeftMoveMapID());
-      if (rc < 0) throw MapWriteException(name);
-    }
-    
-    //   </leftmap>
+    //   </environment>
     rc = xmlTextWriterEndElement(writer);
     if (rc < 0) throw MapWriteException(name);
     
-    //   <rightmap/>
-    rc = xmlTextWriterStartElement(writer, (xmlChar*) "rightmap");
-    if (rc < 0) throw MapWriteException(name);
-    
-    //   type=
-    rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "type", "%s", Map::shortForMoveType(map.getRightMoveType()).c_str());
-    if (rc < 0) throw MapWriteException(name);
-    
-    if (Map::moveTypeTakesMap(map.getRightMoveType()))
+    for (auto object : map.getObjects())
     {
-      // map=
-      rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "map", "%d", map.getRightMoveMapID());
+      // <entity>
+      rc = xmlTextWriterStartElement(writer, (xmlChar*) "entity");
       if (rc < 0) throw MapWriteException(name);
-    }
-    
-    //   </rightmap>
-    rc = xmlTextWriterEndElement(writer);
-    if (rc < 0) throw MapWriteException(name);
     
-    //   <upmap/>
-    rc = xmlTextWriterStartElement(writer, (xmlChar*) "upmap");
-    if (rc < 0) throw MapWriteException(name);
-    
-    //   type=
-    rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "type", "%s", Map::shortForMoveType(map.getUpMoveType()).c_str());
-    if (rc < 0) throw MapWriteException(name);
-    
-    if (Map::moveTypeTakesMap(map.getUpMoveType()))
-    {
-      // map=
-      rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "map", "%d", map.getUpMoveMapID());
+      // type=
+      rc = xmlTextWriterWriteAttribute(writer, (xmlChar*) "type", (xmlChar*) object->object->getType().c_str());
       if (rc < 0) throw MapWriteException(name);
-    }
-    
-    //   </upmap>
-    rc = xmlTextWriterEndElement(writer);
-    if (rc < 0) throw MapWriteException(name);
     
-    //   <downmap/>
-    rc = xmlTextWriterStartElement(writer, (xmlChar*) "downmap");
-    if (rc < 0) throw MapWriteException(name);
-    
-    //   type=
-    rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "type", "%s", Map::shortForMoveType(map.getDownMoveType()).c_str());
-    if (rc < 0) throw MapWriteException(name);
+      // x=
+      rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "x", "%d", object->position.first);
+      if (rc < 0) throw MapWriteException(name);
+      
+      // y=
+      rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "y", "%d", object->position.second);
+      if (rc < 0) throw MapWriteException(name);
     
-    if (Map::moveTypeTakesMap(map.getDownMoveType()))
-    {
-      // map=
-      rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "map", "%d", map.getDownMoveMapID());
+      // </entity>
+      rc = xmlTextWriterEndElement(writer);
       if (rc < 0) throw MapWriteException(name);
     }
     
-    //   </downmap>
-    rc = xmlTextWriterEndElement(writer);
-    if (rc < 0) throw MapWriteException(name);
-  
-    //   <entities>
-    rc = xmlTextWriterStartElement(writer, (xmlChar*) "entities");
-    if (rc < 0) throw MapWriteException(name);
-  
-    for (auto object : map.getObjects())
+    for (auto adjacent : map.getAdjacents())
     {
-      //   <entity>
-      rc = xmlTextWriterStartElement(writer, (xmlChar*) "entity");
+      // <adjacent>
+      rc = xmlTextWriterStartElement(writer, (xmlChar*) "adjacent");
       if (rc < 0) throw MapWriteException(name);
-    
-      //     <entity-type/>
-      rc = xmlTextWriterWriteElement(writer, (xmlChar*) "entity-type", (xmlChar*) object->object->getType().c_str());
+      
+      // dir=
+      rc = xmlTextWriterWriteAttribute(writer, (xmlChar*) "dir", (xmlChar*) Map::shortForMoveDir(adjacent.first).c_str());
       if (rc < 0) throw MapWriteException(name);
-    
-      //     <entity-position/>
-      std::ostringstream entpos_out;
-      entpos_out << object->position.first << "," << object->position.second;
-      rc = xmlTextWriterWriteElement(writer, (xmlChar*) "entity-position", (xmlChar*) entpos_out.str().c_str());
+      
+      // type=
+      rc = xmlTextWriterWriteAttribute(writer, (xmlChar*) "type", (xmlChar*) Map::shortForMoveType(adjacent.second.type).c_str());
       if (rc < 0) throw MapWriteException(name);
-    
-      //   </entity>
+      
+      // map=
+      if (Map::moveTypeTakesMap(adjacent.second.type))
+      {
+        rc = xmlTextWriterWriteFormatAttribute(writer, (xmlChar*) "map", "%d", adjacent.second.map);
+        if (rc < 0) throw MapWriteException(name);
+      }
+      
+      // </adjacent>
       rc = xmlTextWriterEndElement(writer);
       if (rc < 0) throw MapWriteException(name);
     }
-  
-    //   </entities>
-    rc = xmlTextWriterEndElement(writer);
-    if (rc < 0) throw MapWriteException(name);
     
-    wxTreeItemId node = map.getTreeItemId();
     if (mapTree->ItemHasChildren(node))
     {
       wxTreeItemIdValue cookie2;
@@ -455,16 +367,7 @@ void World::save(std::string name, wxTreeCtrl* mapTree)
       {
       // <child/>
         MapPtrCtr* ctl = (MapPtrCtr*) mapTree->GetItemData(it);
-        std::ostringstream childid_out;
-        childid_out << ctl->map->getID();
-        rc = xmlTextWriterWriteElement(writer, (xmlChar*) "child", (xmlChar*) childid_out.str().c_str());
-        if (rc < 0) throw MapWriteException(name);
-      }
-      
-      if (mapTree->IsExpanded(node))
-      {
-      // <expanded/>
-        rc = xmlTextWriterWriteElement(writer, (xmlChar*) "expanded", (xmlChar*) "1");
+        rc = xmlTextWriterWriteFormatElement(writer, (xmlChar*) "child", "%d", ctl->map->getID());
         if (rc < 0) throw MapWriteException(name);
       }
     }