Bug 453226

Summary: Impossible to remove a cursor theme installed from KDE Store
Product: [Applications] systemsettings Reporter: Patrick Silva <bugseforuns>
Component: kcm_cursorthemeAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: alexander.lohnau, leinir, nate, nicolas.fella
Priority: NOR Keywords: regression
Version First Reported In: 5.24.90   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 5.25
Sentry Crash Report:
Attachments: screen recording

Description Patrick Silva 2022-04-30 11:36:56 UTC
Created attachment 148472 [details]
screen recording

SUMMARY
After click on inline 'Remove' button of  a cursor theme installed from KDE Store, apparently the cursor theme is removed. But the cursor theme will still be available for selection next time I open Cursors KCM.
Please watch the attached screen recording.

EXPECTED RESULT
Cursors KCM is able to remove cursor themes

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.24.80
KDE Frameworks Version: 5.94.0
Qt Version: 5.15.3
Graphics Platform: X11

ADDITIONAL INFORMATION
also it's impossible to remove a cursor theme via 'Get new cursors' window
Comment 1 Nate Graham 2022-05-03 14:53:18 UTC
Can reproduce.
Comment 2 Alexander Lohnau 2022-05-03 16:27:51 UTC
On the console I get:
file:///home/user/kde/usr/share/kpackage/kcms/kcm_cursortheme/contents/ui/main.qml: Writing to "pendingDeletion" broke the binding to the underlying model


Reverting f3293bc59e1d4998579ad64a908482a54f2ad5ae fixes the issue.
Comment 3 Bug Janitor Service 2022-05-21 08:46:21 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1753
Comment 4 Alexander Lohnau 2022-05-21 14:24:33 UTC
Git commit 90d36545eaa4deab38f70d127740af5373152006 by Alexander Lohnau.
Committed on 21/05/2022 at 14:15.
Pushed by alex into branch 'master'.

Revert "[kcms/cursor] Port to required properties"

We can not use the required properties, because it breaks the property binding.
https://github.com/qt/qtdeclarative/commit/ec79af7396fd695b25f8f7dba66e8048d54b953b apparently fixes that, however
it is not released as part of Qt6 or is part of the KDE patch collection.

This reverts commit f3293bc59e1d4998579ad64a908482a54f2ad5ae.

M  +9    -15   kcms/cursortheme/package/contents/ui/Delegate.qml

https://invent.kde.org/plasma/plasma-workspace/commit/90d36545eaa4deab38f70d127740af5373152006
Comment 5 Alexander Lohnau 2022-05-21 14:39:36 UTC
Git commit 628f7798395efcbd3382008addce036f4bbc4c19 by Alexander Lohnau.
Committed on 21/05/2022 at 14:39.
Pushed by alex into branch 'Plasma/5.25'.

Revert "[kcms/cursor] Port to required properties"

We can not use the required properties, because it breaks the property binding.
https://github.com/qt/qtdeclarative/commit/ec79af7396fd695b25f8f7dba66e8048d54b953b apparently fixes that, however
it is not released as part of Qt6 or is part of the KDE patch collection.

This reverts commit f3293bc59e1d4998579ad64a908482a54f2ad5ae.


(cherry picked from commit 90d36545eaa4deab38f70d127740af5373152006)

M  +9    -15   kcms/cursortheme/package/contents/ui/Delegate.qml

https://invent.kde.org/plasma/plasma-workspace/commit/628f7798395efcbd3382008addce036f4bbc4c19