Bug 336478

Summary:	Cloned array layer reset to original parent position after Converting to PaintLayer
Product:	[Applications] krita	Reporter:	vanyossi <ghevan>
Component:	Layer Stack	Assignee:	Dmitry Kazakov <dimula73>
Status:	RESOLVED FIXED
Severity:	normal	CC:	dimula73
Priority:	NOR
Version:	git master (please specify the git hash!)
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:	http://commits.kde.org/calligra/e5a116b5b423a3c22045a2851e219702d63d2ccf	Version Fixed In:
Attachments:	Cloned array test file

Description vanyossi 2014-06-19 20:03:26 UTC

Converting  cloned layers to PaintLayers, created with Clone arrays,  resets layer position to parent position

This happens even if I have manually move the cloned layer to another position.

To avoid this I have to get the layer inside a group layer and convert the group layer to paint layer instead.

Reproducible: Always

Steps to Reproduce:
1. Create a clone array
2. Select one clone and convert to PaintLayer
3. Layer will vanish as it is now below/above original source cloned layer.


Expected Results:  
Bake cloned layer in current position.

Comment 1 vanyossi 2014-06-19 20:04:47 UTC

Created attachment 87299 [details]
Cloned array test file

Attached test file for bug.

Comment 2 Dmitry Kazakov 2014-06-20 07:36:31 UTC

Confirmed

Comment 3 Dmitry Kazakov 2014-06-20 09:36:23 UTC

Git commit e5a116b5b423a3c22045a2851e219702d63d2ccf by Dmitry Kazakov.
Committed on 20/06/2014 at 09:35.
Pushed by dkazakov into branch 'master'.

Fix convertion of clone layers into paint layers and selection masks

Also add a unittest for that

M  +1    -0    krita/image/kis_mask.cc
M  +3    -3    krita/sdk/tests/ui_manager_test.h
M  +2    -2    krita/ui/kis_layer_manager.cc
M  +1    -1    krita/ui/kis_node_manager.cpp
A  +-    --    krita/ui/tests/data/node_manager_test/initial_with_selection_with_shape/initial_with_selection_with_shape_clone1_paintDevice.png
A  +7    -0    krita/ui/tests/data/node_manager_test/initial_with_selection_with_shape/readme.txt
A  +-    --    krita/ui/tests/data/node_manager_test/selection_from_blur_layer_Selection 1_paintDevice.png
A  +-    --    krita/ui/tests/data/node_manager_test/selection_from_clone_layer_Selection 1_paintDevice.png
M  +91   -0    krita/ui/tests/kis_node_manager_test.cpp
M  +5    -0    krita/ui/tests/kis_node_manager_test.h

http://commits.kde.org/calligra/e5a116b5b423a3c22045a2851e219702d63d2ccf