Add deinitClone method to RenderedTarget

Author: adazem009

src/irenderedtarget.h

@@ -38,6 +38,8 @@ class IRenderedTarget : public QNanoQuickItem
 
         virtual void beforeRedraw() = 0;
 
+        virtual void deinitClone() = 0;
+
         virtual libscratchcpp::IEngine *engine() const = 0;
         virtual void setEngine(libscratchcpp::IEngine *newEngine) = 0;
 

src/renderedtarget.cpp

@@ -149,6 +149,16 @@ void RenderedTarget::beforeRedraw()
     setHeight(m_height);
 }
 
+void RenderedTarget::deinitClone()
+{
+    // Do not process mouse move events after the clone has been deleted
+    disconnect(m_mouseArea, &SceneMouseArea::mouseMoved, this, &RenderedTarget::handleSceneMouseMove);
+
+    // Release drag lock
+    if (m_mouseArea->draggedSprite() == this)
+        m_mouseArea->setDraggedSprite(nullptr);
+}
+
 IEngine *RenderedTarget::engine() const
 {
     return m_engine;

src/renderedtarget.h

@@ -43,6 +43,8 @@ class RenderedTarget : public IRenderedTarget
 
         void beforeRedraw() override;
 
+        void deinitClone() override;
+
         libscratchcpp::IEngine *engine() const override;
         void setEngine(libscratchcpp::IEngine *newEngine) override;
 

test/mocks/renderedtargetmock.h

@@ -24,6 +24,8 @@ class RenderedTargetMock : public IRenderedTarget
 
         MOCK_METHOD(void, loadCostume, (libscratchcpp::Costume *), (override));
 
+        MOCK_METHOD(void, deinitClone, (), (override));
+
         MOCK_METHOD(libscratchcpp::IEngine *, engine, (), (const, override));
         MOCK_METHOD(void, setEngine, (libscratchcpp::IEngine *), (override));
 

test/renderedtarget/renderedtarget_test.cpp

@@ -290,6 +290,21 @@ TEST_F(RenderedTargetTest, LoadSvgCostume)
     ASSERT_EQ(std::round(target.transformOriginPoint().y() * 100) / 100, 7236.27);
 }
 
+TEST_F(RenderedTargetTest, DeinitClone)
+{
+    RenderedTarget target1, target2;
+    SceneMouseArea mouseArea;
+    target1.setMouseArea(&mouseArea);
+
+    mouseArea.setDraggedSprite(&target2);
+    target1.deinitClone();
+    ASSERT_EQ(mouseArea.draggedSprite(), &target2);
+
+    mouseArea.setDraggedSprite(&target1);
+    target1.deinitClone();
+    ASSERT_EQ(mouseArea.draggedSprite(), nullptr);
+}
+
 TEST_F(RenderedTargetTest, PaintSvg)
 {
     std::string str = readFileStr("image.svg");