Bug 440671 - Performance mode still available for selection when inhibited
Summary: Performance mode still available for selection when inhibited
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Power and Battery widget (show other bugs)
Version: master
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: David Redondo
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-08-06 17:24 UTC by Nate Graham
Modified: 2021-08-10 14:43 UTC (History)
1 user (show)

See Also:
Latest Commit: https://invent.kde.org/plasma/plasma-workspace/commit/3df002db6e68f6688eaa983b21f78fff24b95e22
Version Fixed In: 5.23
Sentry Crash Report:

Attachments
Performance Mode is marked as unavailable, yet selected (661.18 KB, image/png)
2021-08-06 17:24 UTC, Nate Graham 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2021-08-06 17:24:08 UTC 
Created attachment 140565 [details]
Performance Mode is marked as unavailable, yet selected

Everything with git master. See attached screenshot; this probably shouldn't happen. :)
Comment 1 David Redondo 2021-08-09 08:13:28 UTC 
Can you please provide 

qdbus org.kde.Solid.PowerManagement /org/kde/Solid/PowerManagement/Actions/PowerProfile org.kde.Solid.PowerManagement.Actions.PowerProfile.currentProfile 

qdbus org.kde.Solid.PowerManagement /org/kde/Solid/PowerManagement/Actions/PowerProfile org.kde.Solid.PowerManagement.Actions.PowerProfile.performanceInhibitedReaso

qdbus --system net.hadess.PowerProfiles /net/hadess/PowerProfiles net.hadess.PowerProfiles.ActiveProfile

qdbus --system net.hadess.PowerProfiles /net/hadess/PowerProfiles net.hadess.PowerProfiles.PerformanceInhibited 

Thanks!
Comment 2 David Redondo 2021-08-09 08:14:07 UTC 
The second command should read

qdbus org.kde.Solid.PowerManagement /org/kde/Solid/PowerManagement/Actions/PowerProfile org.kde.Solid.PowerManagement.Actions.PowerProfile.performanceInhibitedReason

not the last 'n'
Comment 3 Nate Graham 2021-08-09 15:20:15 UTC 
> qdbus org.kde.Solid.PowerManagement /org/kde/Solid/PowerManagement/Actions/PowerProfile org.kde.Solid.PowerManagement.Actions.PowerProfile.currentProfile
balanced

> qdbus org.kde.Solid.PowerManagement /org/kde/Solid/PowerManagement/Actions/PowerProfile org.kde.Solid.PowerManagement.Actions.PowerProfile.performanceInhibitedReason
lap-detected

> qdbus --system net.hadess.PowerProfiles /net/hadess/PowerProfiles net.hadess.PowerProfiles.ActiveProfile
balanced

> qdbus --system net.hadess.PowerProfiles /net/hadess/PowerProfiles net.hadess.PowerProfiles.PerformanceInhibited 
lap-detected

So I guess it's actually doing the right thing on the backend, it's just that the UI still lets me move the slider over to performance mode inappropriately.
Comment 4 Bug Janitor Service 2021-08-10 09:09:38 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1002
Comment 5 Nate Graham 2021-08-10 14:33:08 UTC 
Git commit 3df002db6e68f6688eaa983b21f78fff24b95e22 by Nate Graham, on behalf of David Redondo.
Committed on 10/08/2021 at 14:32.
Pushed by ngraham into branch 'master'.

Do not allow dragging to performance if it's inhibited

M  +2    -0    applets/batterymonitor/package/contents/ui/PowerProfileItem.qml

https://invent.kde.org/plasma/plasma-workspace/commit/3df002db6e68f6688eaa983b21f78fff24b95e22