From bca58fa5555258facc404f550519e55740117893 Mon Sep 17 00:00:00 2001
From: blacksun
 <blacksun@r5936  blacksun  2019-06-24 19:44:37 +0200 (lun. 24 juin 2019)>
Date: Fri, 18 May 2018 19:36:27 +0000
Subject: [PATCH] Conductor is an herited class of QGraphicsObject, instead of
 QObject and QGraphicsPathItem

git-svn-id: svn+ssh://svn.tuxfamily.org/svnroot/qet/qet/trunk@5371 bfdf4180-ca20-0410-9c96-a3a8aa849046
---
 sources/qetgraphicsitem/conductor.cpp | 57 ++++++++++++++++++---------
 sources/qetgraphicsitem/conductor.h   |  5 ++-
 2 files changed, 43 insertions(+), 19 deletions(-)

diff --git a/sources/qetgraphicsitem/conductor.cpp b/sources/qetgraphicsitem/conductor.cpp
index 005a8e787..d8220b3fa 100644
--- a/sources/qetgraphicsitem/conductor.cpp
+++ b/sources/qetgraphicsitem/conductor.cpp
@@ -73,8 +73,6 @@ class ConductorXmlRetroCompatibility
  * @param p2 : second terminal of this conductor.
  */
 Conductor::Conductor(Terminal *p1, Terminal* p2) :
-	QObject(),
-	QGraphicsPathItem(nullptr),
 	terminal1(p1),
 	terminal2(p2),
 	m_mouse_over(false),
@@ -167,7 +165,7 @@ void Conductor::updatePath(const QRectF &rect) {
 	else
 		generateConductorPath(p1, terminal1 -> orientation(), p2, terminal2 -> orientation());
 	calculateTextItemPosition();
-	QGraphicsPathItem::update(rect);
+	QGraphicsObject::update(rect);
 }
 
 /**
@@ -608,7 +606,7 @@ void Conductor::mouseDoubleClickEvent(QGraphicsSceneMouseEvent *event) {
  */
 void Conductor::mousePressEvent(QGraphicsSceneMouseEvent *event)
 {
-	QGraphicsPathItem::mousePressEvent(event);
+	QGraphicsObject::mousePressEvent(event);
 	
 	if (event->modifiers() & Qt::ControlModifier)
 		setSelected(!isSelected());
@@ -621,7 +619,7 @@ void Conductor::mousePressEvent(QGraphicsSceneMouseEvent *event)
 void Conductor::mouseReleaseEvent(QGraphicsSceneMouseEvent *event)
 {
 	if (!(event -> modifiers() & Qt::ControlModifier))
-		QGraphicsPathItem::mouseReleaseEvent(event);
+		QGraphicsObject::mouseReleaseEvent(event);
 }
 
 /**
@@ -684,7 +682,7 @@ QVariant Conductor::itemChange(GraphicsItemChange change, const QVariant &value)
 		adjusteHandlerPos();
 	}
 	
-	return(QGraphicsPathItem::itemChange(change, value));
+	return(QGraphicsObject::itemChange(change, value));
 }
 
 /**
@@ -1443,6 +1441,21 @@ void Conductor::refreshText()
 	}
 }
 
+void Conductor::setPath(const QPainterPath &path)
+{
+	if(path == m_path)
+		return;
+	
+	prepareGeometryChange();
+	m_path = path;
+	update();
+}
+
+QPainterPath Conductor::path() const
+{
+	return m_path;
+}
+
 /**
  * @brief Conductor::setPropertiesToPotential
  * @param properties
@@ -1723,8 +1736,10 @@ QList<QPointF> Conductor::junctions() const {
 		// determine si le point est une bifurcation ou non
 		bool is_bend = false;
 		Qt::Corner current_bend_type = Qt::TopLeftCorner;
-		foreach(ConductorBend cb, bends_list) {
-			if (cb.first == point) {
+		foreach(ConductorBend cb, bends_list)
+		{
+			if (cb.first == point)
+			{
 				is_bend = true;
 				current_bend_type = cb.second;
 				break;
@@ -1735,22 +1750,28 @@ QList<QPointF> Conductor::junctions() const {
 		
 		bool is_junction = false;
 		QPointF scene_point = mapToScene(point);
-		foreach(Conductor *c, other_conductors) {
-			// exprime le point dans les coordonnees de l'autre conducteur
+		foreach(Conductor *c, other_conductors)
+		{
+				// exprime le point dans les coordonnees de l'autre conducteur
 			QPointF conductor_point = c -> mapFromScene(scene_point);
-			// recupere les segments de l'autre conducteur
+				// recupere les segments de l'autre conducteur
 			QList<ConductorSegment *> c_segments = c -> segmentsList();
-			if (c_segments.isEmpty()) continue;
-			// parcoure les segments a la recherche d'un point commun
-			for (int j = 0 ; j < c_segments.count() ; ++ j) {
+			if (c_segments.isEmpty())
+				continue;
+				// parcoure les segments a la recherche d'un point commun
+			for (int j = 0 ; j < c_segments.count() ; ++ j)
+			{
 				ConductorSegment *segment = c_segments[j];
-				// un point commun a ete trouve sur ce segment
-				if (isContained(conductor_point, segment -> firstPoint(), segment -> secondPoint())) {
+					// un point commun a ete trouve sur ce segment
+				if (isContained(conductor_point, segment -> firstPoint(), segment -> secondPoint()))
+				{
 					is_junction = true;
 					// ce point commun ne doit pas etre une bifurcation identique a celle-ci
 					QList<ConductorBend> other_conductor_bends = c -> bends();
-					foreach(ConductorBend cb, other_conductor_bends) {
-						if (cb.first == conductor_point && cb.second == current_bend_type) {
+					foreach(ConductorBend cb, other_conductor_bends)
+					{
+						if (cb.first == conductor_point && cb.second == current_bend_type)
+						{
 							is_junction = false;
 						}
 					}
diff --git a/sources/qetgraphicsitem/conductor.h b/sources/qetgraphicsitem/conductor.h
index 79ea3be68..b1fe3f0ae 100644
--- a/sources/qetgraphicsitem/conductor.h
+++ b/sources/qetgraphicsitem/conductor.h
@@ -39,7 +39,7 @@ typedef QHash<Qt::Corner, ConductorProfile> ConductorProfilesGroup;
 	This class represents a conductor, i.e. a wire between two element
 	terminals.
 */
-class Conductor : public QObject, public QGraphicsPathItem
+class Conductor : public QGraphicsObject
 {
 	Q_OBJECT
 
@@ -90,6 +90,8 @@ class Conductor : public QObject, public QGraphicsPathItem
 		ConductorSegment *middleSegment();
 		QPointF posForText(Qt::Orientations &flag);
 		void refreshText();
+		void setPath(const QPainterPath &path);
+		QPainterPath path() const;
 
 	public:
 		static bool valideXml (QDomElement &);
@@ -181,6 +183,7 @@ class Conductor : public QObject, public QGraphicsPathItem
 		static QPen conductor_pen;
 		static QBrush conductor_brush;
 		static bool pen_and_brush_initialized;
+		QPainterPath m_path;
 	
 	private:
 		void segmentsToPath();