| Summary: | Discover shouldn't be able to remove Discover | ||
|---|---|---|---|
| Product: | [Applications] Discover | Reporter: | WS <ws.kde> |
| Component: | discover | Assignee: | Dan Leinir Turthra Jensen <leinir> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | aleixpol, nate |
| Priority: | NOR | Keywords: | usability |
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/discover/commit/bb064e8767970a0e8d32a2c531388f0d925b3d6c | Version Fixed In: | 5.24 |
| Sentry Crash Report: | |||
|
Description
WS
2022-01-27 18:39:23 UTC
Yeah, this is probably not the kind of thing that a user of Discover should ever have a reason to do. We should probably do two things here: 1. mark Discover as either "compulsory for desktop=kde" in its AppStream Metadata, or else manually add it to the critical packages list (I prefer the former) 2. Hide any critical packages from the UI, so you can't even see them at all. Or maybe for #2, we should simply disable the "Remove" button instead of hiding it entirely. That would be a bit more HIG-compliant. A possibly relevant merge request was started @ https://invent.kde.org/plasma/discover/-/merge_requests/240 I'd rather go for the second option for really essential stuff that would make Discover or the system unusable without the User knowing what they're doing, because if they are tinkering at that level, they shouldn't use Discover. Also, I'm not sure how Discover deals with critical packages, last I checked it was a warning, which I'm not a fan of, I think the "Remove" button shouldn't exist for critical packages, if they need to be shown. Yep, the merge request I'm about to submit will do just that. A possibly relevant merge request was started @ https://invent.kde.org/plasma/discover/-/merge_requests/241 Yeah, this is way better. The old way meant that if a user knew packageX was critical, they would see a "Remove" Button on Discover, wouldn't click on it, and wouldn't know Discover had a way to prevent critical packages from being uninstalled. They would just think Discover is dumb and unsafe. Great! Git commit d67837557bf68b2e801ac303184c7dfcefb4419c by Nate Graham. Committed on 29/01/2022 at 21:03. Pushed by ngraham into branch 'master'. Mark Discover compulsory_for_desktop with KDE That way anything within Discover that tries to uninstall Discover will fail, since Discover checks for this sort of thing before allowing uninstallation to proceed. FIXED-IN: 5.24 M +1 -0 discover/org.kde.discover.appdata.xml https://invent.kde.org/plasma/discover/commit/d67837557bf68b2e801ac303184c7dfcefb4419c Git commit bb064e8767970a0e8d32a2c531388f0d925b3d6c by Nate Graham. Committed on 29/01/2022 at 21:04. Pushed by ngraham into branch 'Plasma/5.24'. Mark Discover compulsory_for_desktop with KDE That way anything within Discover that tries to uninstall Discover will fail, since Discover checks for this sort of thing before allowing uninstallation to proceed. FIXED-IN: 5.24 (cherry picked from commit d67837557bf68b2e801ac303184c7dfcefb4419c) M +1 -0 discover/org.kde.discover.appdata.xml https://invent.kde.org/plasma/discover/commit/bb064e8767970a0e8d32a2c531388f0d925b3d6c Git commit 456b8c1eb4e5f14bed6fc2780e999b4fe04da681 by Nate Graham. Committed on 10/02/2022 at 18:04. Pushed by ngraham into branch 'master'. Don't display critical packages/apps in the UI Right now when the user clicks the Remove button for these, they'll get a big ugly error message. Instead, we should just not show them in the UI at all. M +3 -2 libdiscover/backends/PackageKitBackend/PackageKitBackend.cpp https://invent.kde.org/plasma/discover/commit/456b8c1eb4e5f14bed6fc2780e999b4fe04da681 Git commit 4e6748b1fc21bbd1f983adb78dcac95b7174706f by Nate Graham. Committed on 10/02/2022 at 18:08. Pushed by ngraham into branch 'Plasma/5.24'. Don't display critical packages/apps in the UI Right now when the user clicks the Remove button for these, they'll get a big ugly error message. Instead, we should just not show them in the UI at all. (cherry picked from commit 456b8c1eb4e5f14bed6fc2780e999b4fe04da681) M +3 -2 libdiscover/backends/PackageKitBackend/PackageKitBackend.cpp https://invent.kde.org/plasma/discover/commit/4e6748b1fc21bbd1f983adb78dcac95b7174706f |