#223 Multiple edge/node/group selection dragging working...

Signed-off-by: cneben <[email protected]>
cneben · Feb 20, 2024 · baa5f3b · baa5f3b
1 parent 88941a1
commit baa5f3b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 10 deletions.
diff --git a/src/qanEdgeDraggableCtrl.cpp b/src/qanEdgeDraggableCtrl.cpp
@@ -162,7 +162,7 @@ void    EdgeDraggableCtrl::beginDragMove(const QPointF& sceneDragPos, bool dragS
 void    EdgeDraggableCtrl::dragMove(const QPointF& sceneDragPos, bool dragSelection,
                                     bool disableSnapToGrid, bool disableOrientation)
 {
-    qWarning() << "EdgeDraggableCtrl::dragMove(): target=" << getTargetItem() << " dragSelection=" << dragSelection;
+    //qWarning() << "EdgeDraggableCtrl::dragMove(): target=" << getTargetItem() << " dragSelection=" << dragSelection;
     Q_UNUSED(dragSelection)
     Q_UNUSED(disableSnapToGrid)
     Q_UNUSED(disableOrientation)
@@ -207,12 +207,6 @@ void    EdgeDraggableCtrl::dragMove(const QPointF& sceneDragPos, bool dragSelect
     const auto graph = getGraph();
     if (graph == nullptr)
         return;
-    qWarning() << "selected nodes=";
-    for (const auto& sn: graph->getSelectedNodes())
-        qWarning() << " " << sn;
-    qWarning() << "selected edges=";
-    for (const auto& se: graph->getSelectedEdges())
-        qWarning() << " " << se;
     if (dragSelection) {
         auto dragMoveSelected = [this, &sceneDragPos] (auto primitive) { // Call dragMove() on a given node, group or edge
             const auto primitiveIsNotSelf = static_cast<QQuickItem*>(primitive->getItem()) !=

diff --git a/src/qanEdgeItem.cpp b/src/qanEdgeItem.cpp
@@ -77,9 +77,8 @@ auto    EdgeItem::getEdge() noexcept -> qan::Edge* { return _edge.data(); }
 auto    EdgeItem::getEdge() const noexcept -> const qan::Edge* { return _edge.data(); }
 auto    EdgeItem::setEdge(qan::Edge* edge) noexcept -> void
 {
-    _edge = edge;
-    if (edge != nullptr &&
-        edge->getItem() != this) {
+    if (_edge != edge) {
+        _edge = edge;
         edge->setItem(this);
         const auto edgeDraggableCtrl = static_cast<EdgeDraggableCtrl*>(_draggableCtrl.get());
         edgeDraggableCtrl->setTarget(edge);