r18970 by jghali - Better fix for #12176 : r18967 cause a regression in path splitting functionality

scribus-commit scribus-commit at lists.scribus.net
Mon Mar 31 23:34:09 UTC 2014


Author: jghali
Date: Mon Mar 31 23:34:09 2014
New Revision: 18970

URL: http://scribus.net/websvn/listing.php?repname=Scribus&sc=1&rev=18970
Log:
Better fix for #12176 : r18967 cause a regression in path splitting functionality

Modified:
    trunk/Scribus/scribus/canvasmode_nodeedit.cpp

Modified: trunk/Scribus/scribus/canvasmode_nodeedit.cpp
URL: http://scribus.net/websvn/diff.php?repname=Scribus&rev=18970&path=/trunk/Scribus/scribus/canvasmode_nodeedit.cpp
==============================================================================
--- trunk/Scribus/scribus/canvasmode_nodeedit.cpp (original)
+++ trunk/Scribus/scribus/canvasmode_nodeedit.cpp Mon Mar 31 23:34:09 2014
@@ -527,7 +527,7 @@
 	for (int a=0; a < signed(Clip.size()); ++a)
 	{
 		if (((m_doc->nodeEdit.EdPoints) && (a % 2 != 0)) || ((!m_doc->nodeEdit.EdPoints) && (a % 2 == 0)))
-				continue;
+			continue;
 		QPointF npf = pm2.map(Clip.pointQF(a));
 		if (m_canvas->hitsCanvasPoint(m->globalPos(), npf))
 		{
@@ -581,11 +581,11 @@
 			}
 		}
 	}
-	if ((m_doc->nodeEdit.submode == NodeEditContext::SPLIT_PATH) && (m_doc->nodeEdit.ClRe2 != -1))
+	if (m_doc->nodeEdit.submode == NodeEditContext::SPLIT_PATH)
 	{
 		if (!m_doc->nodeEdit.EdPoints)
 			return;
-		if (!m_doc->nodeEdit.hasNodeSelected())	// We don't have a Point, try to add one onto the current curve segment
+		if ((!m_doc->nodeEdit.hasNodeSelected()) && (m_doc->nodeEdit.ClRe2 != -1)) // We don't have a Point, try to add one onto the current curve segment
 		{
 			bool foundP = false;
 			uint seg = 0;




More information about the scribus-commit mailing list