ayatana-commits team mailing list archive

[Ted Gould <ted@xxxxxxxx>]

Ted Gould has proposed merging lp:~ted/indicator-session/no-check-pk into lp:indicator-session.

    Requested reviews:
    Indicator Applet Developers (indicator-applet-developers)


Removing the PK Check to see if the dialog needs to show.  CK bug :(
-- 
https://code.launchpad.net/~ted/indicator-session/no-check-pk/+merge/12362
Your team ayatana-commits is subscribed to branch lp:indicator-session.

=== modified file 'src/gtk-dialog/gtk-logout-helper.c'
--- src/gtk-dialog/gtk-logout-helper.c	2009-09-15 20:45:56 +0000
+++ src/gtk-dialog/gtk-logout-helper.c	2009-09-24 17:30:24 +0000
@@ -144,7 +144,12 @@
 	                                  INDICATOR_ICONS_DIR);
 
 	GtkWidget * dialog = NULL;
-	if (!pk_require_auth(type) && !supress_confirmations()) {
+	/* TODO: We're disabling PolicyKit checking here because there
+	   is a bug in ConsoleKit where the dialog doesn't come up until
+	   the session is entirely closed.  Stupid, but it's better than
+	   not getting a dialog at all. */
+	/* if (!pk_require_auth(type) && !supress_confirmations()) { */
+	if (!supress_confirmations()) {
 		dialog = logout_dialog_new(type);
 	}