Bug 510135 - Does not warn when rebooting if other users are logged in
Summary: Does not warn when rebooting if other users are logged in
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Session Management (other bugs)
Version First Reported In: 6.4.5
Platform: Fedora RPMs Linux
: HI critical
Target Milestone: 1.0
Assignee: Nate Graham
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2025-10-01 17:34 UTC by Derek Enz
Modified: 2025-10-02 22:33 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.4.6
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Derek Enz 2025-10-01 17:34:15 UTC
***
If you're not sure this is actually a bug, instead post about it at https://discuss.kde.org

If you're reporting a crash, attach a backtrace with debug symbols; see https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports

Please remove this comment after reading and before submitting - thanks!
***

SUMMARY : When more that one user is logged in, there should be a warning when attempting to restart the system that user data for the other user might be lost.



STEPS TO REPRODUCE
1. Create at least two users.
2. Login with one user.
3. Through user switching login to the other.
4. Both users should then be logged in.
5. From either user attempt to restart the system.

OBSERVED RESULT
When restarting the system with both users logged in, no warning appears. 



EXPECTED RESULT
Should see a warning that the other user is logged in and might loose data.


SOFTWARE/OS VERSIONS
Windows: 
macOS: 
(available in the Info Center app, or by running `kinfo` in a terminal window)
Linux/KDE Plasma: Fedora 43
KDE Plasma Version: 6.4.5
KDE Frameworks Version: 6.18.0
Qt Version: 6.9.2

ADDITIONAL INFORMATION
Comment 1 TraceyC 2025-10-01 19:38:46 UTC
Changing to wishlist, since it's a feature request rather than a bug

This seems reasonable
Comment 2 Nate Graham 2025-10-02 15:00:28 UTC
Actually we do have this: https://invent.kde.org/plasma/plasma-workspace/-/blob/master/lookandfeel/org.kde.breeze/contents/logout/Logout.qml?ref_type=heads#L166

But it's broken. Probably by me, when I removed the "leave" option for Plasma 6.0. I'll fix it.

Raising to critical severity since this could cause data loss for another user.
Comment 3 Bug Janitor Service 2025-10-02 16:20:40 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/5882
Comment 4 Nate Graham 2025-10-02 22:24:16 UTC
Git commit 5fa05a00a642a36bee70539ba2043d4fbbef76b3 by Nate Graham.
Committed on 02/10/2025 at 21:02.
Pushed by ngraham into branch 'master'.

Logout: don't mistakenly think the model includes ourselves

It does not. Thus, we need to check for more than *zero* items in the
model to determine if there are any other users logged in.
FIXED-IN: 6.5.0

M  +3    -3    lookandfeel/org.kde.breeze/contents/logout/Logout.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/5fa05a00a642a36bee70539ba2043d4fbbef76b3
Comment 5 Nate Graham 2025-10-02 22:24:18 UTC
Git commit 5367ad5dd443629a1956cafd444dd370c8be8b1b by Nate Graham.
Committed on 02/10/2025 at 21:04.
Pushed by ngraham into branch 'master'.

Logout: be specific about which sessions to include in the model

Better to be specific here rather than leave it up to the default
settings, which could change in Plasma 7 or later.

M  +3    -0    lookandfeel/org.kde.breeze/contents/logout/Logout.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/5367ad5dd443629a1956cafd444dd370c8be8b1b
Comment 6 Nate Graham 2025-10-02 22:31:14 UTC
Git commit 2815281d214e41158dea5eb9ff5d6d465ec89588 by Nate Graham.
Committed on 02/10/2025 at 22:31.
Pushed by ngraham into branch 'Plasma/6.5'.

Logout: don't mistakenly think the model includes ourselves

It does not. Thus, we need to check for more than *zero* items in the
model to determine if there are any other users logged in.
FIXED-IN: 6.5.0


(cherry picked from commit 5fa05a00a642a36bee70539ba2043d4fbbef76b3)

Co-authored-by: Nate Graham <nate@kde.org>

M  +3    -3    lookandfeel/org.kde.breeze/contents/logout/Logout.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/2815281d214e41158dea5eb9ff5d6d465ec89588
Comment 7 Nate Graham 2025-10-02 22:32:56 UTC
Git commit f4aaf1de58b09778c3aa111fd3bdf3f32ab93659 by Nate Graham.
Committed on 02/10/2025 at 22:32.
Pushed by ngraham into branch 'Plasma/6.4'.

Logout: don't mistakenly think the model includes ourselves

It does not. Thus, we need to check for more than *zero* items in the
model to determine if there are any other users logged in.
FIXED-IN: 6.5.0


(cherry picked from commit 5fa05a00a642a36bee70539ba2043d4fbbef76b3)

Co-authored-by: Nate Graham <nate@kde.org>

M  +3    -3    lookandfeel/org.kde.breeze/contents/logout/Logout.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/f4aaf1de58b09778c3aa111fd3bdf3f32ab93659