Rewrote map editor so a single file contains all maps

Maps are viewed in a tree control on the left. They can be dragged and dropped. Maps are bolded when they are dirty. Saving saves expansion status and order of maps in tree.

1 files changed, 265 insertions, 56 deletions

@@ -5,6 +5,7 @@
 #include <wx/statline.h>
 #include "panel.h"
 #include <list>
+#include <exception>
 
 static std::list<wxWindow*> openWindows;
 
@@ -15,9 +16,18 @@ enum {
   MENU_FILE_OPEN,
   MENU_FILE_SAVE,
   MENU_FILE_CLOSE,
+  MENU_MAP_ADD_ROOT,
+  MENU_MAP_ADD_CHILD,
   TOOL_FILE_NEW,
   TOOL_FILE_OPEN,
-  TOOL_FILE_SAVE
+  TOOL_FILE_SAVE,
+  TOOL_MAP_ADD_ROOT,
+  TOOL_MAP_ADD_CHILD,
+  MAP_EDITOR_NOTEBOOK,
+  MAP_EDITOR_TREE,
+  MAP_TITLE_TEXTBOX,
+  ADD_ENTITY_BUTTON,
+  CANCEL_ENTITY_BUTTON
 };
 
 wxBEGIN_EVENT_TABLE(MapeditFrame, wxFrame)
@@ -27,22 +37,41 @@ wxBEGIN_EVENT_TABLE(MapeditFrame, wxFrame)
   EVT_MENU(MENU_FILE_NEW, MapeditFrame::OnNew)
   EVT_MENU(MENU_FILE_OPEN, MapeditFrame::OnOpen)
   EVT_MENU(MENU_FILE_SAVE, MapeditFrame::OnSave)
+  EVT_MENU(MENU_FILE_CLOSE, MapeditFrame::OnClose)
+  EVT_MENU(MENU_MAP_ADD_ROOT, MapeditFrame::OnAddRoot)
+  EVT_MENU(MENU_MAP_ADD_CHILD, MapeditFrame::OnAddChild)
   EVT_TOOL(TOOL_FILE_NEW, MapeditFrame::OnNew)
   EVT_TOOL(TOOL_FILE_OPEN, MapeditFrame::OnOpen)
   EVT_TOOL(TOOL_FILE_SAVE, MapeditFrame::OnSave)
-  EVT_MENU(MENU_FILE_CLOSE, MapeditFrame::OnClose)
+  EVT_TOOL(TOOL_MAP_ADD_ROOT, MapeditFrame::OnAddRoot)
+  EVT_TOOL(TOOL_MAP_ADD_CHILD, MapeditFrame::OnAddChild)
   EVT_CLOSE(MapeditFrame::OnExit)
+  EVT_NOTEBOOK_PAGE_CHANGED(MAP_EDITOR_NOTEBOOK, MapeditFrame::OnTabChange)
+  EVT_NOTEBOOK_PAGE_CHANGING(MAP_EDITOR_NOTEBOOK, MapeditFrame::OnTabChanging)
+  EVT_TREE_SEL_CHANGING(MAP_EDITOR_TREE, MapeditFrame::OnWillSelectMap)
+  EVT_TREE_SEL_CHANGED(MAP_EDITOR_TREE, MapeditFrame::OnDidSelectMap)
+  EVT_TREE_BEGIN_DRAG(MAP_EDITOR_TREE, MapeditFrame::OnWillDragMap)
+  EVT_TREE_END_DRAG(MAP_EDITOR_TREE, MapeditFrame::OnDidDragMap)
+  EVT_TEXT(MAP_TITLE_TEXTBOX, MapeditFrame::OnTitleChange)
+  EVT_BUTTON(ADD_ENTITY_BUTTON, MapeditFrame::OnAddEntity)
+  EVT_BUTTON(CANCEL_ENTITY_BUTTON, MapeditFrame::OnCancelAddEntity)
 wxEND_EVENT_TABLE()
 
-MapeditFrame::MapeditFrame(Map map, std::string filename) : wxFrame(NULL, wxID_ANY, "Map Editor", wxDefaultPosition, wxSize(GAME_WIDTH*3, GAME_HEIGHT*2)), map(map), filename(filename)
+MapeditFrame::MapeditFrame(std::unique_ptr<World> world) : wxFrame(NULL, wxID_ANY, "Map Editor", wxDefaultPosition, wxSize(GAME_WIDTH*2+TILE_WIDTH*6*6+10+10+150, GAME_HEIGHT*3))
 {
-  this->map.frame = this;
+  this->world = std::move(world);
+  this->world->setParent(this);
+  currentMap = this->world->getLastMap();
   
-  wxMenu* menuFile = new wxMenu;
+  menuFile = new wxMenu;
   menuFile->Append(MENU_FILE_NEW, "New\tCtrl-N");
   menuFile->Append(MENU_FILE_OPEN, "Open\tCtrl-O");
   menuFile->Append(MENU_FILE_SAVE, "Save\tCtrl-S");
   menuFile->Append(MENU_FILE_CLOSE, "Close\tCtrl-W");
+  menuFile->AppendSeparator();
+  menuFile->Append(MENU_MAP_ADD_ROOT, "New Map\tCtrl-Alt-N");
+  menuFile->Append(MENU_MAP_ADD_CHILD, "New Child Map\tCtrl-Alt-Shift-N");
+  menuFile->AppendSeparator();
   menuFile->Append(wxID_EXIT);
   
   wxMenu* menuView = new wxMenu;
@@ -59,22 +88,27 @@ MapeditFrame::MapeditFrame(Map map, std::string filename) : wxFrame(NULL, wxID_A
   // Layout 2: Non-splitter between layout 3 and notebook
   // Layout 3: Splitter between map editor and properties editor
   
-  wxSplitterWindow* layout3 = new wxSplitterWindow(this, wxID_ANY);
+  wxSplitterWindow* layout1 = new wxSplitterWindow(this, wxID_ANY);
+  mapTree = new wxTreeCtrl(layout1, MAP_EDITOR_TREE, wxDefaultPosition, wxSize(200, 0), wxTR_HIDE_ROOT | wxTR_HAS_BUTTONS);
+  wxTreeItemId mapTreeRoot = mapTree->AddRoot("root");
+  populateMapTree(mapTreeRoot, this->world->getRootMaps());
+  
+  wxPanel* layout2 = new wxPanel(layout1, wxID_ANY);
+  
+  wxSplitterWindow* layout3 = new wxSplitterWindow(layout2, wxID_ANY);
   layout3->SetSashGravity(1.0);
-  layout3->SetMinimumPaneSize(20);
   
-  notebook = new wxNotebook(this, wxID_ANY);
+  notebook = new wxNotebook(layout2, MAP_EDITOR_NOTEBOOK);
   
   tileEditor = new TileWidget(notebook, wxID_ANY, 6, 6, wxPoint(0,0), wxSize(TILE_WIDTH*6*6,TILE_HEIGHT*10*6));
-  notebook->AddPage(tileEditor, "Tile Chooser", true);
+  notebook->AddPage(tileEditor, "Tile Chooser", false);
   
-  mapEditor = new MapeditWidget(layout3, wxID_ANY, &this->map, tileEditor, wxPoint(0,0), wxSize(GAME_WIDTH*2, GAME_HEIGHT*2));
+  mapEditor = new MapeditWidget(layout3, wxID_ANY, currentMap, tileEditor, wxPoint(0,0), wxSize(GAME_WIDTH*2, GAME_HEIGHT*2));
   mapEditor->frame = this;
   
   // Set up property editor
   wxPanel* propertyEditor = new wxPanel(layout3, wxID_ANY);
-  titleBox = new wxTextCtrl(propertyEditor, wxID_ANY, map.getTitle());
-  titleBox->Bind(wxEVT_TEXT, &MapeditFrame::OnTitleChange, this);
+  titleBox = new wxTextCtrl(propertyEditor, MAP_TITLE_TEXTBOX, currentMap->getTitle());
   
   wxStaticText* titleLabel = new wxStaticText(propertyEditor, wxID_ANY, "Title:");
   
@@ -103,12 +137,9 @@ MapeditFrame::MapeditFrame(Map map, std::string filename) : wxFrame(NULL, wxID_A
     entityTypeBox->Append(entry.second->getType(), entry.second.get());
   }
   
-  addEntityButton = new wxButton(entityEditor, wxID_ANY, "Add Entity");
-  addEntityButton->Bind(wxEVT_BUTTON, &MapeditFrame::OnAddEntity, this);
-  
-  cancelEntityButton = new wxButton(entityEditor, wxID_ANY, "Cancel");
+  addEntityButton = new wxButton(entityEditor, ADD_ENTITY_BUTTON, "Add Entity");
+  cancelEntityButton = new wxButton(entityEditor, CANCEL_ENTITY_BUTTON, "Cancel");
   cancelEntityButton->Disable();
-  cancelEntityButton->Bind(wxEVT_BUTTON, &MapeditFrame::OnCancelAddEntity, this);
   
   wxStaticText* entityInfoLabel = new wxStaticText(entityEditor, wxID_ANY, "Click and drag an entity to move it.\nRight click an entity to delete it.");
   
@@ -129,51 +160,65 @@ MapeditFrame::MapeditFrame(Map map, std::string filename) : wxFrame(NULL, wxID_A
   
   // Finish setting up the layouts
   layout3->SplitHorizontally(mapEditor, propertyEditor);
+  layout1->SplitVertically(mapTree, layout2);
   
-  notebook->Bind(wxEVT_NOTEBOOK_PAGE_CHANGED, &MapeditFrame::OnTabChange, this);
-  notebook->Bind(wxEVT_NOTEBOOK_PAGE_CHANGING, &MapeditFrame::OnTabChanging, this);
+  wxBoxSizer* sizer1 = new wxBoxSizer(wxHORIZONTAL);
+  sizer1->Add(layout1, 1, wxEXPAND, 0);
+  sizer1->Add(mapTree, 0, wxALIGN_TOP | wxALIGN_LEFT, 0);
+  sizer1->Add(layout2, 1, wxEXPAND, 0);
+  layout1->SetSizer(sizer1);
+  sizer1->SetSizeHints(layout1);
   
   wxBoxSizer* sizer2 = new wxBoxSizer(wxHORIZONTAL);
   sizer2->Add(layout3, 1, wxEXPAND, 0);
   sizer2->Add(notebook, 0, wxALIGN_TOP | wxALIGN_CENTER_HORIZONTAL | wxLEFT | wxEXPAND, 2);
-  this->SetSizer(sizer2);
-  sizer2->SetSizeHints(this);
+  layout2->SetSizer(sizer2);
+  sizer2->SetSizeHints(layout2);
   
   wxBoxSizer* splitterSizer = new wxBoxSizer(wxVERTICAL);
   splitterSizer->Add(layout3, 1, wxEXPAND, 0);
   splitterSizer->Add(mapEditor, 1, wxEXPAND, 0);
-  splitterSizer->Add(propertyEditor, 0, wxALIGN_TOP, wxALIGN_LEFT, 0);
+  splitterSizer->Add(propertyEditor, 0, wxALIGN_TOP | wxALIGN_LEFT, 0);
   layout3->SetSizer(splitterSizer);
   splitterSizer->SetSizeHints(layout3);
   
   // Toolbar time!
   toolbar = CreateToolBar();
-  toolbar->AddTool(TOOL_FILE_NEW, "New", wxBitmap(wxImage("res/page_add.png")));
+  toolbar->AddTool(TOOL_FILE_NEW, "New", wxBitmap(wxImage("res/page.png")));
   toolbar->AddTool(TOOL_FILE_OPEN, "Open", wxBitmap(wxImage("res/folder_page.png")));
   toolbar->AddTool(TOOL_FILE_SAVE, "Save", wxBitmap(wxImage("res/disk.png")));
-  toolbar->EnableTool(TOOL_FILE_SAVE, this->map.getDirty());
+  toolbar->AddSeparator();
+  toolbar->AddTool(TOOL_MAP_ADD_ROOT, "Add Map", wxBitmap(wxImage("res/page_add.png")));
+  toolbar->AddTool(TOOL_MAP_ADD_CHILD, "Add Child Map", wxBitmap(wxImage("res/page_white_add.png")));
+  toolbar->EnableTool(TOOL_FILE_SAVE, this->world->getDirty());
   toolbar->Realize();
+  
+  mapTree->SetFocusedItem(currentMap->getTreeItemId());
+  SelectMap(currentMap);  
+  
+  Maximize(true);
 }
 
 void MapeditFrame::OnExit(wxCloseEvent& event)
 {
-  if (event.CanVeto() && map.hasUnsavedChanges())
+  if (event.CanVeto() && world->getDirty())
   {
-    switch (wxMessageBox("Current map has unsaved changes. Save before closing?", "Please confirm", wxICON_QUESTION|wxYES_NO|wxCANCEL, this))
+    switch (wxMessageBox("One or more maps have unsaved changes. Save before closing?", "Please confirm", wxICON_QUESTION|wxYES_NO|wxCANCEL, this))
     {
       case wxYES:
-        if (filename == "")
+        if (world->getFilename() == "")
         {
-          wxFileDialog saveFileDialog(this, "Save map", "", "", "XML files (*.xml)|*.xml", wxFD_SAVE);
+          wxFileDialog saveFileDialog(this, "Save world", "", "", "XML files (*.xml)|*.xml", wxFD_SAVE);
           if (saveFileDialog.ShowModal() == wxID_CANCEL)
           {
             return;
           }
     
-          filename = saveFileDialog.GetPath().ToStdString();
+          world->save(saveFileDialog.GetPath().ToStdString(), mapTree);
+        } else {
+          world->save(world->getFilename(), mapTree);
         }
   
-        map.save(filename);
         break;
         
       case wxCANCEL:
@@ -204,34 +249,34 @@ void MapeditFrame::ZoomOut(wxCommandEvent&)
 
 void MapeditFrame::OnNew(wxCommandEvent&)
 {
-  NewMap();
+  NewWorld();
 }
 
 void MapeditFrame::OnOpen(wxCommandEvent&)
 {
-  wxFileDialog openFileDialog(this, "Open map", "", "", "XML files (*.xml)|*.xml", wxFD_OPEN|wxFD_FILE_MUST_EXIST);
+  wxFileDialog openFileDialog(this, "Open world", "", "", "XML files (*.xml)|*.xml", wxFD_OPEN|wxFD_FILE_MUST_EXIST);
   if (openFileDialog.ShowModal() == wxID_CANCEL)
   {
     return;
   }
   
-  OpenMap(openFileDialog.GetPath().c_str());
+  OpenWorld(openFileDialog.GetPath().ToStdString());
 }
 
 void MapeditFrame::OnSave(wxCommandEvent&)
 {
-  if (filename == "")
+  if (world->getFilename() == "")
   {
-    wxFileDialog saveFileDialog(this, "Save map", "", "", "XML files (*.xml)|*.xml", wxFD_SAVE);
+    wxFileDialog saveFileDialog(this, "Save world", "", "", "XML files (*.xml)|*.xml", wxFD_SAVE);
     if (saveFileDialog.ShowModal() == wxID_CANCEL)
     {
       return;
     }
     
-    filename = saveFileDialog.GetPath().ToStdString();
+    world->save(saveFileDialog.GetPath().ToStdString(), mapTree);
+  } else {
+    world->save(world->getFilename(), mapTree);
   }
-  
-  map.save(filename);
 }
 
 void MapeditFrame::OnClose(wxCommandEvent&)
@@ -252,24 +297,8 @@ void MapeditFrame::OnQuit(wxCommandEvent&)
 
 void MapeditFrame::OnTitleChange(wxCommandEvent&)
 {
-  map.setTitle(titleBox->GetLineText(0).ToStdString());
-}
-
-void MapeditFrame::NewMap()
-{
-  LaunchWindow(Map(), "");
-}
-
-void MapeditFrame::OpenMap(const char* filename)
-{
-  LaunchWindow(Map(filename), filename);
-}
-
-void MapeditFrame::LaunchWindow(Map map, const char* filename)
-{
-  MapeditFrame* frame = new MapeditFrame(map, filename);
-  frame->closer = openWindows.insert(end(openWindows), frame);
-  frame->Show(true);
+  currentMap->setTitle(titleBox->GetValue().ToStdString());
+  mapTree->SetItemText(currentMap->getTreeItemId(), currentMap->getTitle());
 }
 
 void MapeditFrame::OnTabChange(wxBookCtrlEvent& event)
@@ -317,11 +346,113 @@ void MapeditFrame::OnCancelAddEntity(wxCommandEvent&)
   mapEditor->CancelAddingEntity();
 }
 
+void MapeditFrame::OnAddRoot(wxCommandEvent&)
+{
+  auto map = world->newMap();
+  wxTreeItemId node = mapTree->AppendItem(mapTree->GetItemParent(mapTree->GetSelection()), map->getTitle());
+  map->setTreeItemId(node);
+  mapTree->SetItemData(node, new MapPtrCtr(map.get()));
+  mapTree->SetFocusedItem(node);
+  SelectMap(map.get());
+}
+
+void MapeditFrame::OnAddChild(wxCommandEvent&)
+{
+  auto map = world->newMap();
+  wxTreeItemId node = mapTree->AppendItem(mapTree->GetSelection(), map->getTitle());
+  map->setTreeItemId(node);
+  mapTree->SetItemData(node, new MapPtrCtr(map.get()));
+  mapTree->SetFocusedItem(node);
+  mapTree->Expand(mapTree->GetSelection());
+  SelectMap(map.get());
+}
+
+void MapeditFrame::OnDidSelectMap(wxTreeEvent& event)
+{
+  MapPtrCtr* data = (MapPtrCtr*) mapTree->GetItemData(event.GetItem());
+  SelectMap(data->map);
+}
+
+void MapeditFrame::OnWillSelectMap(wxTreeEvent& event)
+{
+  if (addingEntity)
+  {
+    event.Veto();
+    return;
+  }
+  
+  event.Skip();
+}
+
+void MapeditFrame::OnWillDragMap(wxTreeEvent& event)
+{
+  if (!addingEntity)
+  {
+    event.Allow();
+    dragMap = event.GetItem();
+  }
+}
+
+void MapeditFrame::OnDidDragMap(wxTreeEvent& event)
+{
+  if (!dragMap.IsOk())
+  {
+    return;
+  }
+  
+  wxTreeItemId newParent = event.GetItem();
+  if (!newParent.IsOk())
+  {
+    newParent = mapTree->GetRootItem();
+  }
+  
+  wxTreeItemId newChild = MoveTreeNode(dragMap, newParent);
+  dragMap.Unset();
+  mapTree->SelectItem(newChild);
+}
+
+void MapeditFrame::NewWorld()
+{
+  LaunchWindow(std::unique_ptr<World>(new World()));
+}
+
+void MapeditFrame::OpenWorld(std::string filename)
+{
+  try
+  {
+    auto world = std::unique_ptr<World>(new World(filename));
+    
+    LaunchWindow(std::move(world));
+  } catch (std::exception& ex)
+  {
+    wxMessageBox(ex.what(), "Error loading world", wxOK | wxCENTRE | wxICON_ERROR);
+  }
+}
+
+void MapeditFrame::LaunchWindow(std::unique_ptr<World> world)
+{
+  MapeditFrame* frame = new MapeditFrame(std::move(world));
+  frame->closer = openWindows.insert(end(openWindows), frame);
+  frame->Show(true);
+}
+
 void MapeditFrame::StartAddingEntity()
 {
   addingEntity = true;
   addEntityButton->Disable();
   cancelEntityButton->Enable();
+  
+  toolbar->EnableTool(TOOL_FILE_NEW, false);
+  toolbar->EnableTool(TOOL_FILE_OPEN, false);
+  toolbar->EnableTool(TOOL_FILE_SAVE, false);
+  toolbar->EnableTool(TOOL_MAP_ADD_ROOT, false);
+  toolbar->EnableTool(TOOL_MAP_ADD_CHILD, false);
+  
+  menuFile->Enable(MENU_FILE_NEW, false);
+  menuFile->Enable(MENU_FILE_OPEN, false);
+  menuFile->Enable(MENU_FILE_SAVE, false);
+  menuFile->Enable(MENU_MAP_ADD_ROOT, false);
+  menuFile->Enable(MENU_MAP_ADD_CHILD, false);
 }
 
 void MapeditFrame::FinishAddingEntity()
@@ -329,9 +460,87 @@ void MapeditFrame::FinishAddingEntity()
   addingEntity = false;
   addEntityButton->Enable();
   cancelEntityButton->Disable();
+  toolbar->Enable();
+  
+  toolbar->EnableTool(TOOL_FILE_NEW, true);
+  toolbar->EnableTool(TOOL_FILE_OPEN, true);
+  toolbar->EnableTool(TOOL_FILE_SAVE, world->getDirty());
+  toolbar->EnableTool(TOOL_MAP_ADD_ROOT, true);
+  toolbar->EnableTool(TOOL_MAP_ADD_CHILD, true);
+  
+  menuFile->Enable(MENU_FILE_NEW, true);
+  menuFile->Enable(MENU_FILE_OPEN, true);
+  menuFile->Enable(MENU_FILE_SAVE, world->getDirty());
+  menuFile->Enable(MENU_MAP_ADD_ROOT, true);
+  menuFile->Enable(MENU_MAP_ADD_CHILD, true);
 }
 
 void MapeditFrame::MapDirtyDidChange(bool dirty)
 {
   toolbar->EnableTool(TOOL_FILE_SAVE, dirty);
+  menuFile->Enable(MENU_FILE_SAVE, dirty);
+  
+  if (dirty)
+  {
+    mapTree->SetItemBold(currentMap->getTreeItemId(), true);
+  } else {
+    for (auto map : world->getMaps())
+    {
+      mapTree->SetItemBold(map.second->getTreeItemId(), false);
+    }
+  }
+}
+
+void MapeditFrame::populateMapTree(wxTreeItemId node, std::list<std::shared_ptr<Map>> maps)
+{
+  for (auto map : maps)
+  {
+    wxTreeItemId childNode = mapTree->AppendItem(node, map->getTitle());
+    mapTree->SetItemData(childNode, new MapPtrCtr(map.get()));
+    map->setTreeItemId(childNode);
+    
+    populateMapTree(childNode, map->getChildren());
+    
+    if (map->getExpanded())
+    {
+      mapTree->Expand(childNode);
+    }
+  }
+}
+
+void MapeditFrame::SelectMap(Map* map)
+{
+  currentMap = map;
+  mapEditor->SetMap(map);
+  titleBox->ChangeValue(map->getTitle());
+}
+
+wxTreeItemId MapeditFrame::MoveTreeNode(wxTreeItemId toCopy, wxTreeItemId newParent)
+{
+  MapPtrCtr* ctl1 = (MapPtrCtr*) mapTree->GetItemData(toCopy);
+  MapPtrCtr* ctl2 = new MapPtrCtr(ctl1->map);
+  
+  wxTreeItemId copied = mapTree->AppendItem(newParent, mapTree->GetItemText(toCopy), -1, -1, ctl2);
+  if (mapTree->IsBold(toCopy))
+  {
+    mapTree->SetItemBold(toCopy, true);
+  }
+  
+  if (mapTree->ItemHasChildren(toCopy))
+  {
+    wxTreeItemIdValue cookie;
+    for (wxTreeItemId it = mapTree->GetFirstChild(toCopy, cookie); it.IsOk(); it = mapTree->GetNextChild(toCopy, cookie))
+    {
+      MoveTreeNode(it, copied);
+    }
+  }
+  
+  if (mapTree->IsExpanded(toCopy))
+  {
+    mapTree->Expand(copied);
+  }
+  
+  mapTree->Delete(toCopy);
+  
+  return copied;
 }