fix #84: Use sprite dragging API

Author: adazem009

src/renderedtarget.cpp

@@ -486,9 +486,11 @@ void RenderedTarget::mouseReleaseEvent(QMouseEvent *event)
     Q_ASSERT(m_mouseArea);
 
     // Stop dragging
-    if (m_mouseArea->draggedSprite() == this)
+    if (m_mouseArea->draggedSprite() == this) {
+        Sprite *sprite = m_spriteModel->sprite();
+        sprite->stopDragging();
         m_mouseArea->setDraggedSprite(nullptr);
-    else if (m_engine && m_spriteModel && m_spriteModel->sprite()->draggable()) {
+    } else if (m_engine && m_spriteModel && m_spriteModel->sprite()->draggable()) {
         // Notify libscratchcpp about the click
         m_engine->clickTarget(scratchTarget());
     }
@@ -503,6 +505,7 @@ void RenderedTarget::mouseMoveEvent(QMouseEvent *event)
     if (m_clicked && !m_mouseArea->draggedSprite() && m_spriteModel && m_spriteModel->sprite()->draggable()) {
         Q_ASSERT(m_engine);
         Sprite *sprite = m_spriteModel->sprite();
+        sprite->startDragging();
         m_dragDeltaX = m_engine->mouseX() - sprite->x();
         m_dragDeltaY = m_engine->mouseY() - sprite->y();
         m_mouseArea->setDraggedSprite(this);
@@ -758,8 +761,9 @@ void RenderedTarget::handleSceneMouseMove(qreal x, qreal y)
         Q_ASSERT(m_spriteModel && m_spriteModel->sprite());
         Q_ASSERT(m_engine);
         Sprite *sprite = m_spriteModel->sprite();
-        sprite->setX(x / m_stageScale - m_engine->stageWidth() / 2.0 - m_dragDeltaX);
-        sprite->setY(-y / m_stageScale + m_engine->stageHeight() / 2.0 - m_dragDeltaY);
+        const double dragX = x / m_stageScale - m_engine->stageWidth() / 2.0 - m_dragDeltaX;
+        const double dragY = -y / m_stageScale + m_engine->stageHeight() / 2.0 - m_dragDeltaY;
+        sprite->dragToPosition(dragX, dragY);
     }
 }
 

test/renderedtarget/renderedtarget_test.cpp

@@ -443,6 +443,7 @@ TEST_F(RenderedTargetTest, SpriteDragging)
     ASSERT_EQ(sprite.x(), 64.08);
     ASSERT_EQ(sprite.y(), -6.86);
     ASSERT_EQ(mouseArea.draggedSprite(), nullptr);
+    ASSERT_FALSE(sprite.dragging());
 
     // Try right mouse button (should not work)
     QMouseEvent moveEventRightButton(QEvent::MouseMove, QPointF(), QPointF(), Qt::RightButton, Qt::RightButton, Qt::NoModifier);
@@ -454,14 +455,17 @@ TEST_F(RenderedTargetTest, SpriteDragging)
     ASSERT_EQ(sprite.x(), 64.08);
     ASSERT_EQ(sprite.y(), -6.86);
     ASSERT_EQ(mouseArea.draggedSprite(), nullptr);
+    ASSERT_FALSE(sprite.dragging());
     EXPECT_CALL(engine, clickTarget).Times(0);
     QCoreApplication::sendEvent(&target, &releaseEventRightButton);
     ASSERT_EQ(mouseArea.draggedSprite(), nullptr);
+    ASSERT_FALSE(sprite.dragging());
 
     emit mouseArea.mouseMoved(1064, 651);
     ASSERT_EQ(sprite.x(), 64.08);
     ASSERT_EQ(sprite.y(), -6.86);
     ASSERT_EQ(mouseArea.draggedSprite(), nullptr);
+    ASSERT_FALSE(sprite.dragging());
 
     // Try right mouse button with "draggable" set to true (should not work)
     sprite.setDraggable(true);
@@ -471,14 +475,17 @@ TEST_F(RenderedTargetTest, SpriteDragging)
     ASSERT_EQ(sprite.x(), 64.08);
     ASSERT_EQ(sprite.y(), -6.86);
     ASSERT_EQ(mouseArea.draggedSprite(), nullptr);
+    ASSERT_FALSE(sprite.dragging());
     EXPECT_CALL(engine, clickTarget(&sprite));
     QCoreApplication::sendEvent(&target, &releaseEventRightButton);
     ASSERT_EQ(mouseArea.draggedSprite(), nullptr);
+    ASSERT_FALSE(sprite.dragging());
 
     emit mouseArea.mouseMoved(1064, 651);
     ASSERT_EQ(sprite.x(), 64.08);
     ASSERT_EQ(sprite.y(), -6.86);
     ASSERT_EQ(mouseArea.draggedSprite(), nullptr);
+    ASSERT_FALSE(sprite.dragging());
 
     // Try left mouse button (should not work with "draggable" set to false)
     sprite.setDraggable(false);
@@ -491,11 +498,13 @@ TEST_F(RenderedTargetTest, SpriteDragging)
     ASSERT_EQ(sprite.x(), 64.08);
     ASSERT_EQ(sprite.y(), -6.86);
     ASSERT_EQ(mouseArea.draggedSprite(), nullptr);
+    ASSERT_FALSE(sprite.dragging());
 
     emit mouseArea.mouseMoved(1064, 651);
     ASSERT_EQ(sprite.x(), 64.08);
     ASSERT_EQ(sprite.y(), -6.86);
     ASSERT_EQ(mouseArea.draggedSprite(), nullptr);
+    ASSERT_FALSE(sprite.dragging());
     EXPECT_CALL(engine, clickTarget).Times(0);
     QCoreApplication::sendEvent(&target, &releaseEvent);
 
@@ -510,6 +519,7 @@ TEST_F(RenderedTargetTest, SpriteDragging)
     ASSERT_EQ(sprite.x(), 64.08);
     ASSERT_EQ(sprite.y(), -6.86);
     ASSERT_EQ(mouseArea.draggedSprite(), &target);
+    ASSERT_TRUE(sprite.dragging());
 
     // Drag
     EXPECT_CALL(engine, stageWidth()).WillOnce(Return(480));
@@ -518,13 +528,15 @@ TEST_F(RenderedTargetTest, SpriteDragging)
     ASSERT_EQ(std::round(sprite.x() * 100) / 100, 61.22);
     ASSERT_EQ(std::round(sprite.y() * 100) / 100, -14.41);
     ASSERT_EQ(mouseArea.draggedSprite(), &target);
+    ASSERT_TRUE(sprite.dragging());
 
     EXPECT_CALL(engine, stageWidth()).WillOnce(Return(480));
     EXPECT_CALL(engine, stageHeight()).WillOnce(Return(360));
     emit mouseArea.mouseMoved(1092.47, 605.46);
     ASSERT_EQ(std::round(sprite.x() * 100) / 100, 68.26);
     ASSERT_EQ(std::round(sprite.y() * 100) / 100, -1.95);
     ASSERT_EQ(mouseArea.draggedSprite(), &target);
+    ASSERT_TRUE(sprite.dragging());
 
     // Create another sprite
     RenderedTarget anotherTarget;
@@ -540,12 +552,16 @@ TEST_F(RenderedTargetTest, SpriteDragging)
     anotherTarget.setStageScale(3.5);
     anotherTarget.setMouseArea(&mouseArea);
 
+    ASSERT_FALSE(anotherSprite.dragging());
+
     // Try to drag the second sprite while the first is being dragged
     sprite.setDraggable(true);
     EXPECT_CALL(engine, clickTarget).Times(0);
     QCoreApplication::sendEvent(&anotherTarget, &pressEvent);
     QCoreApplication::sendEvent(&anotherTarget, &moveEvent);
     ASSERT_EQ(mouseArea.draggedSprite(), &target);
+    ASSERT_TRUE(sprite.dragging());
+    ASSERT_FALSE(anotherSprite.dragging());
     EXPECT_CALL(engine, clickTarget(&sprite));
     QCoreApplication::sendEvent(&anotherTarget, &releaseEvent);
 
@@ -555,6 +571,8 @@ TEST_F(RenderedTargetTest, SpriteDragging)
     ASSERT_EQ(std::round(sprite.x() * 100) / 100, 68.26);
     ASSERT_EQ(std::round(sprite.y() * 100) / 100, -1.95);
     ASSERT_EQ(mouseArea.draggedSprite(), nullptr);
+    ASSERT_FALSE(sprite.dragging());
+    ASSERT_FALSE(anotherSprite.dragging());
 }
 
 TEST_F(RenderedTargetTest, Engine)