https://bugs.kde.org/show_bug.cgi?id=521106

Dmitry Kazakov <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
      Latest Commit|                            |https://invent.kde.org/grap
                   |                            |hics/krita/-/commit/eadecc8
                   |                            |81937c0e1c50f771e850e3938b4
                   |                            |f4734e
         Resolution|---                         |FIXED

--- Comment #7 from Dmitry Kazakov <[email protected]> ---
Git commit eadecc881937c0e1c50f771e850e3938b4f4734e by Dmitry Kazakov, on
behalf of Elena Sagalaeva.
Committed on 25/06/2026 at 07:34.
Pushed by dkazakov into branch 'master'.

Fix bug 521106. Remove combined shapes from a selected shapes list after
deletion.

Combined shapes sometimes remain in the selected shapes list and might be used,
leading to a crash. I added code, which removes deleted combined shapes from
the selected shapes list.

## Test Plan

Follow the repro steps in 521106. Krita doesn't crash anymore.

## Formalities Checklist

- [x] I confirmed this builds.
- [x] I confirmed Krita ran and the relevant functions work.
- [x] I tested the relevant unit tests and can confirm they are not broken. (If
not possible, don't hesitate to ask for help!)
- [x] I made sure my commits build individually and have good descriptions as
per [KDE guidelines](https://community.kde.org/Policies/Commit_Policy).
- [x] I made sure my code conforms to the standards set in the HACKING file.
- [x] I can confirm the code is licensed and attributed appropriately, and that
unattributed code is mine, as per [KDE Licensing
Policy](https://community.kde.org/Policies/Licensing_Policy).
- [ ] Does the patch add a user-visible feature? If yes, is there a
documentation MR ready for it at [Krita Documentation
Repository](https://invent.kde.org/documentation/docs-krita-org)?

**_Reminder: the reviewer is responsible for merging the patch, this is to
ensure at the least two people can build the patch. In case a patch breaks the
build, both the author and the reviewer should be contacted to fix the
build._** **_If this is not possible, the commits shall be reverted, and a
notification with the reasoning and any relevant logs shall be sent to the
mailing list, _****[email protected]_****_._**

M  +3    -2    plugins/tools/defaulttool/defaulttool/DefaultTool.cpp

https://invent.kde.org/graphics/krita/-/commit/eadecc881937c0e1c50f771e850e3938b4f4734e

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to