Fixed the grainline tab icon in details dialog

--HG--
branch : feature

src/app/valentina/xml/vpattern.cpp

@@ -726,10 +726,18 @@ void VPattern::ParseDetailElement(const QDomElement &domElement, const Document
                     ptPos.setX(GetParametrDouble(element, AttrMx, "0"));
                     ptPos.setY(GetParametrDouble(element, AttrMy, "0"));
                     detail.GetGrainlineGeometry().SetPos(ptPos);
-                    QString qsLength = GetParametrString(element, AttrLength, "0");
+                    if (detail.GetGrainlineGeometry().IsVisible() == true)
-                    detail.GetGrainlineGeometry().SetLength(qsLength);
+                    {
-                    QString qsRot = GetParametrString(element, VToolDetail::AttrRotation, "0");
+                        QString qsLength = GetParametrString(element, AttrLength);
-                    detail.GetGrainlineGeometry().SetRotation(qsRot);
+                        detail.GetGrainlineGeometry().SetLength(qsLength);
+                        QString qsRot = GetParametrString(element, VToolDetail::AttrRotation);
+                        detail.GetGrainlineGeometry().SetRotation(qsRot);
+                    }
+                    else
+                    {
+                        detail.GetGrainlineGeometry().SetLength("");
+                        detail.GetGrainlineGeometry().SetRotation("");
+                    }
                 }
             }
         }

src/libs/vtools/dialogs/tools/dialogdetail.cpp

@@ -903,8 +903,7 @@ void DialogDetail::UpdateValues()
     }
     else
     {
-        QIcon icon;
+        ResetWarning();
-        ui.tabWidget->setTabIcon(2, icon);
     }
 }
 
@@ -974,6 +973,7 @@ void DialogDetail::EnableGrainlineRotation()
         ChangeColor(ui.labelEditLen, okColor);
         ChangeColor(ui.labelEditRot, okColor);
         bOk->setEnabled(true);
+        ResetWarning();
     }
 }
 
@@ -1024,3 +1024,11 @@ void DialogDetail::DeployLength()
 }
 
 //---------------------------------------------------------------------------------------------------------------------
+void DialogDetail::ResetWarning()
+{
+    QIcon icon;
+    ui.tabWidget->setTabIcon(2, icon);
+}
+
+//---------------------------------------------------------------------------------------------------------------------
+

src/libs/vtools/dialogs/tools/dialogdetail.h

@@ -135,6 +135,7 @@ private slots:
     void             EditFormula();
     void             DeployRotation();
     void             DeployLength();
+    void             ResetWarning();
 };
 
 //---------------------------------------------------------------------------------------------------------------------

src/libs/vtools/tools/vtooldetail.cpp

@@ -502,6 +502,7 @@ void VToolDetail::AddToFile()
     doc->SetAttribute(domData, AttrMy, glGeom.GetPos().y());
     doc->SetAttribute(domData, AttrLength, glGeom.GetLength());
     doc->SetAttribute(domData, AttrRotation, glGeom.GetRotation());
+    qDebug() << "XML ROTATION" << glGeom.GetRotation();
 
     // nodes
     for (int i = 0; i < detail.CountNode(); ++i)
@@ -580,6 +581,7 @@ void VToolDetail::RefreshDataInFile()
         doc->SetAttribute(domData, AttrMy, glGeom.GetPos().y());
         doc->SetAttribute(domData, AttrLength, glGeom.GetLength());
         doc->SetAttribute(domData, AttrRotation, glGeom.GetRotation());
+        qDebug() << "XML ROTATION2" << glGeom.GetRotation();
 
         // nodes
         for (int i = 0; i < det.CountNode(); ++i)
@@ -926,6 +928,7 @@ void VToolDetail::UpdateGrainline()
             QString qsFormula;
             qsFormula = geom.GetRotation().replace("\n", " ");
             qsFormula = qApp->TrVars()->FormulaFromUser(qsFormula, qApp->Settings()->GetOsSeparator());
+
             Calculator cal1;
             dRotation = cal1.EvalFormula(VDataTool::data.PlainVariables(), qsFormula);
 

src/libs/vtools/undocommands/savedetailoptions.cpp

@@ -208,8 +208,11 @@ void SaveDetailOptions::SaveGrainline(QDomElement &domElement, const VDetail &de
     doc->SetAttribute(domData, VAbstractPattern::AttrVisible, glGeom.IsVisible() == true? trueStr : falseStr);
     doc->SetAttribute(domData, AttrMx, glGeom.GetPos().x());
     doc->SetAttribute(domData, AttrMy, glGeom.GetPos().y());
-    doc->SetAttribute(domData, AttrLength, glGeom.GetLength());
+    if (glGeom.IsVisible() == true)
-    doc->SetAttribute(domData, VToolDetail::AttrRotation, glGeom.GetRotation());
+    {
+        doc->SetAttribute(domData, AttrLength, glGeom.GetLength());
+        doc->SetAttribute(domData, VToolDetail::AttrRotation, glGeom.GetRotation());
+    }
 
     domElement.appendChild(domData);
 }