Fix a crash when creating a text shape
eb61160966dd
Actions

Authored by dkazakov on Jun 7 2019, 10:45 AM.

Description

Fix a crash when creating a text shape

Every KoShapeController used to reset the global "shape document" in its
KoDocumentResourceManager. The problem is that in Krita we have multiple
KoShapeController-s, but the resource manager is the only one. It means
that every shape selection or shape layer used to reset the pointer to
the global "shape document" to its own one. The only reason why we didn't
have too many crashes is that almost noone uses this "shape document".

Ideally we should:

Remove KoDocumentResourceManager::GlobalShapeController resource. This resource is not global to the document, but is created per-layer. The only user of it atm is the legacy text tool, which should be deprecated.

Remove KoDocumentResourceManager at all. There is no much use of it anymore. Its main user is legacy text tool.

BUG:407554

Details

Committed

dkazakov

Jun 7 2019, 10:46 AM

Parents

R37:3dfad538fc94: Fix NN algorithm for certain percentages

Branches

Unknown

Tags

Unknown

References

refs/keep-around/eb61160966ddc794cfd02a9225e577f0844e51e0

dkazakov committed R37:eb61160966dd: Fix a crash when creating a text shape (authored by dkazakov).Jun 7 2019, 10:46 AM

Changes (11)

				Path	Packages
	M			libs/flake/KoDocumentResourceManager.cpp
	M			libs/flake/KoDocumentResourceManager.h
	M			libs/flake/KoShapeController.cpp
	M			libs/flake/KoShapeRegistry.cpp
	M			libs/flake/tests/TestKoShapeRegistry.cpp
	M			libs/flake/text/KoSvgTextShape.cpp
	M			libs/ui/KisDocument.cpp
	M			libs/ui/flake/kis_dummies_facade_base.h
	M			libs/ui/flake/kis_shape_controller.cpp
	M			libs/ui/flake/kis_shape_controller.h
	M			plugins/flake/textshape/TextShapeFactory.cpp