Bug 285006

Summary: Prompt to reboot or relogin if suggested by the installed update
Product: [Applications] apper Reporter: Will Stephenson <wstephenson>
Component: generalAssignee: Daniel Nicoletti <dantti12>
Status: RESOLVED FIXED    
Severity: wishlist CC: kevin.kofler, sven.burmeister
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Unlisted Binaries   
OS: Linux   
Latest Commit: Version Fixed In: 0.7.1
Attachments: missing reboot icon in notifier?

Description Will Stephenson 2011-10-26 14:04:51 UTC
If the update type is relogin suggested or reboot suggested, it should be possible for Apper to tell the session to reboot or logout.  Not sure of the api to do this but there should be something desktop neutral available via kwindowsystem?
Comment 1 Daniel Nicoletti 2011-10-26 14:18:10 UTC
Apper already has the functionality to tell the user he needs to restart or relogin. But it's the backend task to infor it. If zypper pm support this then the zypp backaend needs to be updated. (guessing you are running suse)
Comment 2 Will Stephenson 2011-10-26 21:09:21 UTC
Created attachment 64913 [details]
missing reboot icon in notifier?
Comment 3 Will Stephenson 2011-10-26 21:18:40 UTC
Thanks I see that now, in TransactionWatcher::logout(). I once saw the relogin icon in the panel but did not recognise it, and now the notifier is just blank and has an empty context menu, see attachment.

I checked the api and you should use the function KWorkSpace::requestShutDown() to perform the logout/reboot instead of hitting KSMServer directly:

http://api.kde.org/4.x-api/kde-workspace-apidocs/libs/kworkspace/html/namespaceKWorkSpace.html#a0d3e4e1632c4d0e57778faabf713b1d7
Comment 4 Daniel Nicoletti 2012-02-16 23:34:25 UTC
*** Bug 287318 has been marked as a duplicate of this bug. ***
Comment 5 Daniel Nicoletti 2012-02-16 23:36:02 UTC
Fix for the empty icon on the systray and make use of KWorkSpace::requestShutdown as Will Stephenson pointed out:
http://commits.kde.org/apper/c00e591a0a15906da4610b6d2750f80a32da5fd7