Undo current pattern piece.

--HG--
branch : develop

src/app/mainwindow.cpp

@@ -115,6 +115,7 @@ MainWindow::MainWindow(QWidget *parent)
     connect(doc, &VPattern::UndoCommand, this, &MainWindow::FullParseFile);
     connect(doc, &VPattern::SetEnabledGUI, this, &MainWindow::SetEnabledGUI);
     connect(doc, &VPattern::CheckLayout, this, &MainWindow::Layout);
+    connect(doc, &VPattern::SetCurrentPP, this, &MainWindow::GlobalChangePP);
     qApp->setCurrentDocument(doc);
 
     connect(qApp->getUndoStack(), &QUndoStack::cleanChanged, this, &MainWindow::PatternWasModified);
@@ -1484,7 +1485,23 @@ void MainWindow::FullParseFile()
     comboBoxDraws->blockSignals(false);
     ui->actionPattern_properties->setEnabled(true);
 
-    qint32 index = comboBoxDraws->findText(patternPiece);
+    GlobalChangePP(patternPiece);
+
+    if (comboBoxDraws->count() > 0)
+    {
+        SetEnableTool(true);
+    }
+    else
+    {
+        SetEnableTool(false);
+    }
+    SetEnableWidgets(true);
+}
+
+//---------------------------------------------------------------------------------------------------------------------
+void MainWindow::GlobalChangePP(const QString &patternPiece)
+{
+    const qint32 index = comboBoxDraws->findText(patternPiece);
     try
     {
         if ( index != -1 )
@@ -1508,19 +1525,8 @@ void MainWindow::FullParseFile()
         SetEnabledGUI(false);
         return;
     }
-
-    if (comboBoxDraws->count() > 0)
-    {
-        SetEnableTool(true);
-    }
-    else
-    {
-        SetEnableTool(false);
-    }
-    SetEnableWidgets(true);
 }
 
-
 //---------------------------------------------------------------------------------------------------------------------
 void MainWindow::SetEnabledGUI(bool enabled)
 {

src/app/mainwindow.h

@@ -127,6 +127,7 @@ public slots:
     void               ClickEndVisualization();
     void               Layout();
     void               UpdateGradation();
+    void               GlobalChangePP(const QString &patternPiece);
 signals:
     /**
      * @brief ModelChosen emit after calculation all details.

src/app/undocommands/addtocalc.cpp

@@ -48,7 +48,7 @@ AddToCalc::~AddToCalc()
 //---------------------------------------------------------------------------------------------------------------------
 void AddToCalc::undo()
 {
-    doc->ChangeActivPP(nameActivDraw);
+    doc->ChangeActivPP(nameActivDraw);//User will not see this change
     doc->setCursor(cursor);
 
     QDomElement calcElement;
@@ -80,12 +80,13 @@ void AddToCalc::undo()
     }
     emit NeedFullParsing();
     VAbstractTool::NewSceneRect(qApp->getCurrentScene(), qApp->getSceneView());
+    doc->SetCurrentPP(nameActivDraw);//Return current pattern piece after undo
 }
 
 //---------------------------------------------------------------------------------------------------------------------
 void AddToCalc::redo()
 {
-    doc->ChangeActivPP(nameActivDraw);
+    doc->ChangeActivPP(nameActivDraw);//User will not see this change
     doc->setCursor(cursor);
 
     QDomElement calcElement;
@@ -118,3 +119,14 @@ void AddToCalc::redo()
     RedoFullParsing();
     VAbstractTool::NewSceneRect(qApp->getCurrentScene(), qApp->getSceneView());
 }
+
+//---------------------------------------------------------------------------------------------------------------------
+void AddToCalc::RedoFullParsing()
+{
+    if (redoFlag)
+    {
+        emit NeedFullParsing();
+        doc->SetCurrentPP(nameActivDraw);//Return current pattern piece after undo
+    }
+    redoFlag = true;
+}

src/app/undocommands/addtocalc.h

@@ -39,6 +39,8 @@ public:
     virtual ~AddToCalc();
     virtual void undo();
     virtual void redo();
+protected:
+    virtual void RedoFullParsing();
 private:
     Q_DISABLE_COPY(AddToCalc)
     const QString     nameActivDraw;

src/app/undocommands/deltool.cpp

@@ -33,40 +33,14 @@
 
 //---------------------------------------------------------------------------------------------------------------------
 DelTool::DelTool(VPattern *doc, quint32 id, QUndoCommand *parent)
-    : VUndoCommand(QDomElement(), doc, parent), parentNode(QDomNode()), siblingId(NULL_ID)
+    : VUndoCommand(QDomElement(), doc, parent), parentNode(QDomNode()), siblingId(NULL_ID),
+      nameActivDraw(doc->GetNameActivPP())
 {
     setText(tr("Delete tool"));
     nodeId = id;
-
+    siblingId = doc->SiblingNodeId(nodeId);
-    QVector<VToolRecord> history = doc->getLocalHistory();
+    parentNode = doc->ParentNodeById(nodeId);
-    for (qint32 i = 0; i< history.size(); ++i)
+    xml = doc->CloneNodeById(nodeId);
-    {
-        const VToolRecord tool = history.at(i);
-        if (nodeId == tool.getId())
-        {
-            if (i == 0)
-            {
-                siblingId = NULL_ID;
-            }
-            else
-            {
-                const VToolRecord tool = history.at(i-1);
-                siblingId = tool.getId();
-            }
-        }
-    }
-
-    QDomElement domElement = doc->elementById(QString().setNum(id));
-    if (domElement.isElement())
-    {
-        xml = domElement.cloneNode().toElement();
-        parentNode = domElement.parentNode();
-    }
-    else
-    {
-        qDebug()<<"Can't get tool by id = "<<nodeId<<Q_FUNC_INFO;
-        return;
-    }
 }
 
 //---------------------------------------------------------------------------------------------------------------------
@@ -78,20 +52,14 @@ void DelTool::undo()
 {
     UndoDeleteAfterSibling(parentNode, siblingId);
     emit NeedFullParsing();
+    doc->SetCurrentPP(nameActivDraw);
 }
 
 //---------------------------------------------------------------------------------------------------------------------
 void DelTool::redo()
 {
-    QDomElement domElement = doc->elementById(QString().setNum(nodeId));
+    QDomElement domElement = doc->NodeById(nodeId);
-    if (domElement.isElement())
-    {
     parentNode.removeChild(domElement);
     emit NeedFullParsing();
-    }
+    doc->SetCurrentPP(nameActivDraw);
-    else
-    {
-        qDebug()<<"Can't get tool by id = "<<nodeId<<Q_FUNC_INFO;
-        return;
-    }
 }

src/app/undocommands/deltool.h

@@ -44,6 +44,7 @@ private:
     Q_DISABLE_COPY(DelTool)
     QDomNode      parentNode;
     quint32       siblingId;
+    const QString nameActivDraw;
 };
 
 #endif // DELTOOL_H

src/app/undocommands/vundocommand.cpp

@@ -59,14 +59,7 @@ void VUndoCommand::UndoDeleteAfterSibling(QDomNode &parentNode, const quint32 &s
     }
     else
     {
-        const QDomElement refElement = doc->elementById(QString().setNum(siblingId));
+        const QDomElement refElement = doc->NodeById(siblingId);
-        if (refElement.isElement())
-        {
         parentNode.insertAfter(xml, refElement);
     }
-        else
-        {
-            qDebug()<<"Can't find sibling node.";
-        }
-    }
 }

src/app/undocommands/vundocommand.h

@@ -64,7 +64,7 @@ protected:
     VPattern     *doc;
     quint32      nodeId;
     bool         redoFlag;
-    void        RedoFullParsing();
+    virtual void RedoFullParsing();
     void         UndoDeleteAfterSibling(QDomNode &parentNode, const quint32 &siblingId) const;
 private:
     Q_DISABLE_COPY(VUndoCommand)

src/app/version.h

@@ -37,7 +37,6 @@ extern const int DEBUG_VERSION;
 
 extern const QString APP_VERSION;
 
-// Don't forget change version number in manifest file /src/app/share/resources/valentina.exe.manifest.
 // Change version number in version.cpp also.
 
 #define VER_FILEVERSION             0,2,8,0

src/app/xml/vdomdocument.cpp

@@ -600,3 +600,28 @@ void VDomDocument::RemoveAllChild(QDomElement &domElement)
         }
     }
 }
+
+//---------------------------------------------------------------------------------------------------------------------
+QDomNode VDomDocument::ParentNodeById(const quint32 &nodeId)
+{
+    QDomElement domElement = NodeById(nodeId);
+    return domElement.parentNode();
+}
+
+//---------------------------------------------------------------------------------------------------------------------
+QDomElement VDomDocument::CloneNodeById(const quint32 &nodeId)
+{
+    QDomElement domElement = NodeById(nodeId);
+    return domElement.cloneNode().toElement();
+}
+
+//---------------------------------------------------------------------------------------------------------------------
+QDomElement VDomDocument::NodeById(const quint32 &nodeId)
+{
+    QDomElement domElement = elementById(QString().setNum(nodeId));
+    if (domElement.isNull() || domElement.isElement() == false)
+    {
+        throw VExceptionBadId(tr("Couldn't get node"), nodeId);
+    }
+    return domElement;
+}

src/app/xml/vdomdocument.h

@@ -109,6 +109,10 @@ public:
     QString        Minor() const;
     QString        Patch() const;
     static void    RemoveAllChild(QDomElement &domElement);
+
+    QDomNode       ParentNodeById(const quint32 &nodeId);
+    QDomElement    CloneNodeById(const quint32 &nodeId);
+    QDomElement    NodeById(const quint32 &nodeId);
 protected:
     /** @brief data container with data. */
     VContainer     *data;

src/app/xml/vpattern.cpp

@@ -2741,7 +2741,7 @@ QRectF VPattern::ToolBoundingRect(const QRectF &rec, const quint32 &id) const
 
 
 //---------------------------------------------------------------------------------------------------------------------
-QVector<VToolRecord> VPattern::getLocalHistory()
+QVector<VToolRecord> VPattern::getLocalHistory() const
 {
     QVector<VToolRecord> historyPP;
     for (qint32 i = 0; i< history.size(); ++i)
@@ -2755,3 +2755,45 @@ QVector<VToolRecord> VPattern::getLocalHistory()
     }
     return historyPP;
 }
+
+//---------------------------------------------------------------------------------------------------------------------
+quint32 VPattern::SiblingNodeId(const quint32 &nodeId) const
+{
+    quint32 siblingId = NULL_ID;
+
+    const QVector<VToolRecord> history = getLocalHistory();
+    for (qint32 i = 0; i < history.size(); ++i)
+    {
+        const VToolRecord tool = history.at(i);
+        if (nodeId == tool.getId())
+        {
+            if (i == 0)
+            {
+                siblingId = NULL_ID;
+            }
+            else
+            {
+                for (qint32 j = i; j > 0; --j)
+                {
+                    const VToolRecord tool = history.at(j-1);
+                    switch ( tool.getTypeTool() )
+                    {
+                        case Tool::Detail:
+                        case Tool::UnionDetails:
+                        case Tool::NodeArc:
+                        case Tool::NodePoint:
+                        case Tool::NodeSpline:
+                        case Tool::NodeSplinePath:
+                            continue;
+                            break;
+                        default:
+                            siblingId = tool.getId();
+                            j = 0;// break loop
+                            break;
+                    }
+                }
+            }
+        }
+    }
+    return siblingId;
+}

src/app/xml/vpattern.h

@@ -72,7 +72,7 @@ public:
     QHash<quint32, VDataTool*>* getTools();
     VDataTool*     getTool(const quint32 &id);
     QVector<VToolRecord> *getHistory();
-    QVector<VToolRecord> getLocalHistory();
+    QVector<VToolRecord> getLocalHistory() const;
     quint32        getCursor() const;
     void           setCursor(const quint32 &value);
     void           setCurrentData();
@@ -183,6 +183,8 @@ public:
     void                 SetVersion();
 
     QString              GenerateLabel(const LabelType &type)const;
+
+    quint32              SiblingNodeId(const quint32 &nodeId) const;
 signals:
     /**
      * @brief ChangedActivDraw change active pattern peace.
@@ -219,6 +221,7 @@ signals:
     void           UndoCommand();
     void           SetEnabledGUI(bool enabled);
     void           CheckLayout();
+    void           SetCurrentPP(const QString &patterPiece);
 public slots:
     void           LiteParseTree(const Document &parse);
     void           haveLiteChange();