Bug 294905

Summary: Drag-and-dropping of two selection masks to the root layer crashes Krita
Product: [Applications] krita Reporter: Dmitry Kazakov <dimula73>
Component: GeneralAssignee: Dmitry Kazakov <dimula73>
Status: RESOLVED FIXED    
Severity: crash CC: halla
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Unlisted Binaries   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Dmitry Kazakov 2012-02-27 07:51:49 UTC
We must use KisNode::allowAsChild() mechanism in KisNodeManager instead of hardcoding checks in every slot.
Comment 1 Halla Rempt 2012-02-29 10:51:06 UTC
Bah, yes -- I can still reproduce this.
Comment 2 Dmitry Kazakov 2012-02-29 10:54:27 UTC
Git commit 839aac98b99644c91067e9280b79e6d071636ff0 by Dmitry Kazakov.
Committed on 29/02/2012 at 12:53.
Pushed by dkazakov into branch 'master'.

Fix dropping of Selection Masks onto the root layer

Now it is possible only in case there is no global selection present.

TODO: port the methods of KisLayerBox coupled with KisNodeManager to
check the possibility of moving a node through the allowAsChildMechanism()
instead of hardicong checks.

M  +4    -1    krita/image/kis_group_layer.cc
M  +13   -10   krita/ui/kis_node_model.cpp
M  +1    -1    krita/ui/kis_node_model.h
M  +5    -1    krita/ui/kis_selection_manager.cc

http://commits.kde.org/calligra/839aac98b99644c91067e9280b79e6d071636ff0
Comment 3 Halla Rempt 2019-05-10 06:49:48 UTC
Git commit 52f9b10aed1da0cb79b0b80764a936f17705fb97 by Boudewijn Rempt.
Committed on 10/05/2019 at 06:49.
Pushed by rempt into branch 'master'.

Fix KisGroupLayer::allowAsChild properly

M  +12   -4    libs/image/kis_group_layer.cc

https://invent.kde.org/kde/krita/commit/52f9b10aed1da0cb79b0b80764a936f17705fb97