Bug 180576 - shutdown instead of logout when confirm logout unchecked
Summary: shutdown instead of logout when confirm logout unchecked
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
: 191085 (view as bug list)
Depends on:
Blocks:
 
Reported: 2009-01-13 20:25 UTC by Stijn Van Nieuwenhuyse
Modified: 2009-04-30 00:26 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Stijn Van Nieuwenhuyse 2009-01-13 20:25:26 UTC 
Version:            (using Devel)
OS:                Linux
Installed from:    Compiled sources

Choosing "Logout" in the kickoff menu does not logout but does a complete shutdown instead. This happens only when "Confirm logout" is deselected in System Settings > Session Manager.

If "Confirm logout" is checked "Logout" works like expected after confirming. 

KDE: 4.1.96
Comment 1 Stijn Van Nieuwenhuyse 2009-01-13 20:27:12 UTC 
I forgot to add:
This is using Arch Linux with KDE 4.1.96 from the testing repository
Comment 2 Aaron J. Seigo 2009-01-15 08:34:48 UTC 
SVN commit 911249 by aseigo:

don't shutdown when Logout is passed; happens when no confirmation is asked for, because the shutdown dialog was quitely changing type from Logout to None which obviously doesn't happen without confirmation ;) yet another not-a-plasma-bug-but-reported-against-plasma bug fixed ;)
BUG:180576


 M  +1 -1      kdisplaymanager.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=911249
Comment 3 Stijn Van Nieuwenhuyse 2009-01-15 10:29:34 UTC 
Thanks for the fix!

Will this also find its way to the 4.2 branch?
Comment 4 Aaron J. Seigo 2009-01-15 18:31:25 UTC 
it *might* .. i'll give it several days for people to test it in trunk to make sure the fix is completely safe and withou othert sideeffects, though.
Comment 5 Rex Dieter 2009-02-09 20:43:15 UTC 
Saw this too on 4.2, and can confirm the fix works like a charm, ok to backport now? (I can do it)
Comment 6 Kevin Kofler 2009-02-27 15:58:57 UTC 
This is still not in 4.2.1. :-(
Comment 7 Dario Andres 2009-04-30 00:26:45 UTC 
*** Bug 191085 has been marked as a duplicate of this bug. ***