the "Log out" and "Shut down" menu show the same Log-Out dialog

Bug #1392187 reported by Huan Peng
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
unity8 (Ubuntu)
Fix Released
Medium
Christopher Townsend
unity8-desktop-session (Ubuntu)
Invalid
High
Unassigned

Bug Description

the "Log out" and "Shut down" menu show the same Log-Out dialog, and when confirmed to log out or shut down, they all just go back to greeter

Related branches

Revision history for this message
Christopher Townsend (townsend) wrote :

This is actually an issue in Unity 8. There is still more work done here.

Changed in unity8-desktop-session (Ubuntu):
status: New → Triaged
importance: Undecided → High
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu):
status: New → Triaged
importance: Undecided → Medium
tags: added: unity8-desktop
Revision history for this message
Christopher Townsend (townsend) wrote :

This looks more like an issue with indicator-session, or at least indicator-session getting confused about which Unity is running.

indicator-session is detecting Unity is running. It assumes then that gnome-session is running and tries to make some DBus calls to it. However, in Unity 8, gnome-session is not running, so the DBus calls fail. The callback checks the response from DBus, sees the failure, and upon failure, always signals to display the logout dialog regardless of what type of session request was done, ie, shutdown or reboot.

Marking indicator-session as affected and will work on some solution.

Revision history for this message
Christopher Townsend (townsend) wrote :

After some more research, trying to force indicator-session to work with what is in Unity 8 is probably not the best solution. I think a better solution is wrap the org.gnome.SessionManager.EndSessionDialog object withing the com.canonical.Unity bus. This way we don't have to hack indicator-session to work properly with Unity 8.

Revision history for this message
Christopher Townsend (townsend) wrote :

Also, removing the indicator-session task as it's not needed.

no longer affects: indicator-session (Ubuntu)
Changed in unity8 (Ubuntu):
status: Triaged → In Progress
assignee: nobody → Christopher Townsend (townsend)
Revision history for this message
Christopher Townsend (townsend) wrote :

Marking the unity8-desktop-session task as Invalid since the fix is in unity8 itself.

Changed in unity8-desktop-session (Ubuntu):
status: Triaged → Invalid
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.02+15.04.20150302-0ubuntu1

---------------
unity8 (8.02+15.04.20150302-0ubuntu1) vivid; urgency=medium

  [ Albert Astals ]
  * Don't precache items that are outisde the 1.5*height buffering area
    (LP: #1410131)

  [ Albert Astals Cid ]
  * Don't precache items that are outisde the 1.5*height buffering area
    (LP: #1410131)

  [ Chris Townsend ]
  * Add a wrapper for handling the
    /org/gnome/SessionManager/EndSessionDialog DBus object used by
    indicator-session. This allows indicator-session to properly talk to
    the Unity 8 session management functions. (LP: #1392187)
  * Allow for proper logout session management. This uses the
    EndSession() Upstart DBus method on the current session bus. (LP:
    #1360316)
  * Make the Unity 8 session dialogs behave in the same fashion as Unity
    7. From indicator-session: Clicking "Log Out" will now present a
    dialog to "Lock", "Log Out", or "Cancel". Clicking "Shut Down" will
    now present a dialog to "Reboot", "Shut Down", or "Cancel". (LP:
    #1416074)

  [ Daniel d'Andrada ]
  * Add NO_TESTS cmake option to skip tests and therefore speed up
    builds
  * Remove enums and properties that doesn't exist in the real
    ApplicationManager
  * Stabilize some Shell qml tests

  [ Josh Arenson ]
  * Create a PhysicalKeyMapper to handle all physical/hardware keys (LP:
    #1390393)

  [ Michael Terry ]
  * Make the draggable part of the right edge (during the tutorial) a
    constant size, instead of having it grow and shrink as the spread
    peeks out from the side.

  [ Michał Sawicz ]
  * Create a PhysicalKeyMapper to handle all physical/hardware keys (LP:
    #1390393)

  [ Robert Bruce Park ]
  * Launchpad automatic translations update.
 -- CI Train Bot <email address hidden> Mon, 02 Mar 2015 12:11:21 +0000

Changed in unity8 (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.