Bug 460615

Summary: Cannot select multiple processes
Product: [Applications] plasma-systemmonitor Reporter: Lastique <andysem>
Component: generalAssignee: KSysGuard Developers <ksysguard-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: ahiemstra, nate, plasma-bugs-null
Priority: NOR    
Version First Reported In: 5.24.6   
Target Milestone: ---   
Platform: Kubuntu   
OS: Linux   
Latest Commit: https://invent.kde.org/plasma/plasma-systemmonitor/commit/ccf7c80315b7af568481053cdd06246e0af68708 Version Fixed/Implemented In: 5.27
Sentry Crash Report:

Description Lastique 2022-10-17 18:16:17 UTC 
SUMMARY

In the process list, multiple processes cannot be selected. This means it is not possible to send a signal to multiple processes.

This is a regression from KSysGuard.

STEPS TO REPRODUCE

1. In the list of processes, try selecting multiple processes - either by pressing Ctrl+Right Click, or by selecting a range of processes using Shift+Right Click, or by pressing Ctrl+A to select all listed processes.

OBSERVED RESULT

No process is selected.

EXPECTED RESULT

Multiple processes should be selected. Then, Right Clicking should open the context menu with an option so send a signal to all selected processes.

This should also work when search results are displayed in the process list.

SOFTWARE/OS VERSIONS

Operating System: Kubuntu 22.04
KDE Plasma Version: 5.24.6
KDE Frameworks Version: 5.92.0
Qt Version: 5.15.3
Kernel Version: 5.15.0-50-lowlatency (64-bit)
Graphics Platform: X11
Processors: 16 × 12th Gen Intel® Core™ i7-12700K
Memory: 31.1 GiB of RAM
Graphics Processor: NVIDIA GeForce RTX 2080 Ti/PCIe/SSE2
Comment 1 Bug Janitor Service 2023-01-13 10:30:40 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/200
Comment 2 Arjen Hiemstra 2023-01-13 11:49:55 UTC 
Git commit ccf7c80315b7af568481053cdd06246e0af68708 by Arjen Hiemstra.
Committed on 13/01/2023 at 10:30.
Pushed by ahiemstra into branch 'master'.

table: Don't clear the selection when setting current index

setCurrentIndex with ItemSelectionModel.ClearAndSelect will clear the
current index, then select and make the new index current. This breaks
the logic for multi selection that is later in this function. So make
sure to only update current index and leave actual selection to the rest
of the function.

M  +1    -1    src/table/BaseCellDelegate.qml

https://invent.kde.org/plasma/plasma-systemmonitor/commit/ccf7c80315b7af568481053cdd06246e0af68708