Merge lp:~larsu/indicator-session/lp1024392 into lp:indicator-session/13.04

Proposed by Lars Karlitski
Status: Merged
Approved by: Charles Kerr
Approved revision: 370
Merged at revision: 370
Proposed branch: lp:~larsu/indicator-session/lp1024392
Merge into: lp:indicator-session/13.04
Diff against target: 28 lines (+8/-2)
1 file modified
src/session-menu-mgr.c (+8/-2)
To merge this branch: bzr merge lp:~larsu/indicator-session/lp1024392
Reviewer Review Type Date Requested Status
Charles Kerr (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+133075@code.launchpad.net

Commit message

Don't lock the session when clicking on the current user

Description of the change

Don't lock the session when clicking on the current user

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Charles Kerr (charlesk) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/session-menu-mgr.c'
2--- src/session-menu-mgr.c 2012-10-17 22:46:05 +0000
3+++ src/session-menu-mgr.c 2012-11-06 15:13:03 +0000
4@@ -775,6 +775,7 @@
5 /* set the is-current-user property */
6 const gboolean is_current_user =
7 !g_strcmp0 (g_get_user_name(), accounts_user_get_user_name(user));
8+
9 dbusmenu_menuitem_property_set_bool (mi,
10 USER_ITEM_PROP_IS_CURRENT_USER,
11 is_current_user);
12@@ -1077,9 +1078,14 @@
13 action_func_switch_to_user (AccountsUser * user)
14 {
15 SessionMenuMgr * mgr = user_get_mgr (user);
16+
17 g_return_if_fail (mgr != NULL);
18- action_func_lock (mgr);
19- users_service_dbus_activate_user_session (mgr->users_dbus_facade, user);
20+
21+ if (g_strcmp0 (g_get_user_name(), accounts_user_get_user_name(user)) != 0)
22+ {
23+ action_func_lock (mgr);
24+ users_service_dbus_activate_user_session (mgr->users_dbus_facade, user);
25+ }
26 }
27
28 static void

Subscribers

People subscribed via source and target branches