Bug 440671

Summary: Performance mode still available for selection when inhibited
Product: [Plasma] plasmashell Reporter: Nate Graham <nate>
Component: Power and Battery widgetAssignee: David Redondo <kde>
Status: RESOLVED FIXED    
Severity: normal CC: plasma-bugs-null
Priority: NOR    
Version First Reported In: master   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 5.23
Sentry Crash Report:
Attachments: Performance Mode is marked as unavailable, yet selected

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