Editor: Implement universal undo stack

- Almost everything can now be undone/redone

Author: juzzlin

src/common/mapbase.cpp

@@ -16,14 +16,12 @@
 #include "mapbase.hpp"
 
 #include "tracktilebase.hpp"
-#include "trackdatabase.hpp"
 
 #include <QPoint>
 #include <QPointF>
 
-MapBase::MapBase(TrackDataBase & trackData, unsigned int cols, unsigned int rows)
-    : m_trackData(trackData)
-    , m_cols(cols)
+MapBase::MapBase(unsigned int cols, unsigned int rows)
+    : m_cols(cols)
     , m_rows(rows)
     , m_map(rows, TrackTileRow(m_cols, nullptr))
 {}
@@ -125,11 +123,6 @@ std::vector<TrackTileBasePtr> MapBase::deleteRow(unsigned int at)
     return deleted;
 }
 
-TrackDataBase & MapBase::trackData()
-{
-    return m_trackData;
-}
-
 MapBase::~MapBase()
 {
 }

src/common/mapbase.hpp

@@ -19,18 +19,17 @@
 #include <vector>
 #include "tracktilebase.hpp"
 
-class TrackDataBase;
-
 //! Base class for the tile matrix used by TrackData.
 class MapBase
 {
 public:
 
     //! Constructor.
-    MapBase(TrackDataBase & trackData, unsigned int cols, unsigned int rows);
+    MapBase(unsigned int cols, unsigned int rows);
+
+    MapBase(const MapBase & other) = delete;
 
-    MapBase(MapBase & other) = delete;
-    MapBase & operator= (MapBase & other) = delete;
+    MapBase & operator= (const MapBase & other) = delete;
 
     enum class InsertDirection
     {
@@ -70,15 +69,8 @@ class MapBase
     //! Delete row at given index. Return deleted tiles.
     virtual std::vector<TrackTileBasePtr> deleteRow(unsigned int at);
 
-protected:
-
-    //! Get track data reference.
-    TrackDataBase & trackData();
-
 private:
 
-    TrackDataBase & m_trackData;
-
     unsigned int m_cols, m_rows;
 
     typedef std::vector<TrackTileBasePtr> TrackTileRow;

src/common/objectbase.hpp

@@ -30,6 +30,7 @@ class ObjectBase
     ObjectBase(QString category, QString role);
 
     ObjectBase(ObjectBase & other) = delete;
+
     ObjectBase & operator= (ObjectBase & other) = delete;
 
     //! Destructor.

src/common/objects.cpp

@@ -42,6 +42,11 @@ void Objects::remove(ObjectBase & object)
     }
 }
 
+void Objects::clear()
+{
+    m_objects.clear();
+}
+
 unsigned int Objects::count() const
 {
     return static_cast<int>(m_objects.size());

src/common/objects.hpp

@@ -31,15 +31,19 @@ class Objects
     //! Constructor.
     Objects();
 
-    Objects(Objects & other) = delete;
-    Objects & operator= (Objects & other) = delete;
+    Objects(const Objects & other) = delete;
+
+    Objects & operator= (const Objects & other) = delete;
 
     //! Add an object.
     void add(ObjectBasePtr object);
 
     //! Remove an object.
     void remove(ObjectBase & object);
 
+    //! Clear the container.
+    void clear();
+
     //! Return object count.
     unsigned int count() const;
 

src/common/route.hpp

@@ -32,8 +32,9 @@ class Route
     //! Constructor.
     Route();
 
-    Route(Route & other) = delete;
-    Route & operator= (Route & other) = delete;
+    Route(const Route & other) = delete;
+
+    Route & operator= (const Route & other) = delete;
 
     typedef std::vector<TargetNodeBasePtr> RouteVector;
 

src/common/targetnodebase.hpp

@@ -33,6 +33,7 @@ class TargetNodeBase
     TargetNodeBase();
 
     TargetNodeBase(TargetNodeBase & other) = delete;
+
     TargetNodeBase & operator= (TargetNodeBase & other) = delete;
 
     //! Destructor.

src/common/trackdatabase.cpp

@@ -16,9 +16,16 @@
 #include "trackdatabase.hpp"
 
 TrackDataBase::TrackDataBase(QString name, bool isUserTrack)
-: m_name(name)
-, m_index(999)
-, m_isUserTrack(isUserTrack)
+    : m_name(name)
+    , m_index(999)
+    , m_isUserTrack(isUserTrack)
+{
+}
+
+TrackDataBase::TrackDataBase(const TrackDataBase & other)
+    : m_name(other.m_name)
+    , m_index(other.m_index)
+    , m_isUserTrack(other.m_isUserTrack)
 {
 }
 

src/common/trackdatabase.hpp

@@ -180,8 +180,10 @@ class TrackDataBase
     //! Constructor.
     TrackDataBase(QString name, bool isUserTrack);
 
-    TrackDataBase(TrackDataBase & other) = delete;
-    TrackDataBase & operator= (TrackDataBase & other) = delete;
+    //! Copy constructor.
+    TrackDataBase(const TrackDataBase & other);
+
+    TrackDataBase & operator= (const TrackDataBase & other) = delete;
 
     //! Destructor.
     virtual ~TrackDataBase();

src/common/tracktilebase.cpp

@@ -14,17 +14,14 @@
 // along with Dust Racing 2D. If not, see <http://www.gnu.org/licenses/>.
 
 #include "tracktilebase.hpp"
-#include "trackdatabase.hpp"
 
 TrackTileBase::TrackTileBase(
-    TrackDataBase & trackData,
     QPointF location,
     QPoint matrixLocation,
     const QString & type)
 : m_tileType(type)
 , m_location(location)
 , m_matrixLocation(matrixLocation)
-, m_trackData(trackData)
 , m_computerHint(TrackTileBase::CH_NONE)
 , m_excludeFromMinimap(false)
 {
@@ -63,11 +60,6 @@ const QString & TrackTileBase::tileType() const
     return m_tileType;
 }
 
-TrackDataBase & TrackTileBase::trackData() const
-{
-    return m_trackData;
-}
-
 void TrackTileBase::setComputerHint(TrackTileBase::ComputerHint hint)
 {
     m_computerHint = hint;