Skip to content

Commit 8392eb7

Browse files
committed
Add back the greeter guest account hint
1 parent 608e4e0 commit 8392eb7

File tree

5 files changed

+39
-7
lines changed

5 files changed

+39
-7
lines changed

src/seat.c

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -214,13 +214,6 @@ seat_get_allow_guest (Seat *seat)
214214
return seat_get_boolean_property (seat, "allow-guest") && guest_account_is_installed ();
215215
}
216216

217-
gboolean
218-
seat_get_greeter_allow_guest (Seat *seat)
219-
{
220-
g_return_val_if_fail (seat != NULL, FALSE);
221-
return seat_get_allow_guest (seat) && seat_get_boolean_property (seat, "greeter-allow-guest");
222-
}
223-
224217
static gboolean
225218
run_script (Seat *seat, DisplayServer *display_server, const gchar *script_name, User *user)
226219
{
@@ -976,6 +969,7 @@ create_greeter_session (Seat *seat)
976969
greeter_set_hint (greeter_session, "hide-users", seat_get_boolean_property (seat, "greeter-hide-users") ? "true" : "false");
977970
greeter_set_hint (greeter_session, "show-manual-login", seat_get_boolean_property (seat, "greeter-show-manual-login") ? "true" : "false");
978971
greeter_set_hint (greeter_session, "show-remote-login", seat_get_boolean_property (seat, "greeter-show-remote-login") ? "true" : "false");
972+
greeter_set_hint (greeter_session, "has-guest-account", seat_get_allow_guest (seat) && seat_get_boolean_property (seat, "greeter-allow-guest") ? "true" : "false");
979973

980974
return greeter_session;
981975
}

tests/Makefile.am

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ TESTS = \
77
test-greeter-xserver-crash \
88
test-greeter-wrapper \
99
test-greeter-default-session \
10+
test-greeter-allow-guest \
1011
test-greeter-hide-users \
1112
test-greeter-show-manual-login \
1213
test-greeter-show-remote-login \
@@ -290,6 +291,7 @@ EXTRA_DIST = \
290291
scripts/dbus.conf \
291292
scripts/denied.conf \
292293
scripts/expired.conf \
294+
scripts/greeter-allow-guest.conf \
293295
scripts/greeter-default-session.conf \
294296
scripts/greeter-fail-start.conf \
295297
scripts/greeter-hide-users.conf \
Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,32 @@
1+
#
2+
# Check greeter is informed if manual login should be showed (it defaults to show)
3+
#
4+
5+
[SeatDefaults]
6+
greeter-allow-guest=false
7+
8+
#?RUNNER DAEMON-START
9+
10+
# X server starts
11+
#?XSERVER-0 START VT=7
12+
13+
# Daemon connects when X server is ready
14+
#?*XSERVER-0 INDICATE-READY
15+
#?XSERVER-0 INDICATE-READY
16+
#?XSERVER-0 ACCEPT-CONNECT
17+
18+
# Greeter starts
19+
#?GREETER-X-0 START
20+
#?XSERVER-0 ACCEPT-CONNECT
21+
#?GREETER-X-0 CONNECT-XSERVER
22+
#?GREETER-X-0 CONNECT-TO-DAEMON
23+
#?GREETER-X-0 CONNECTED-TO-DAEMON
24+
25+
# Check has correct hint to allow guest
26+
#?GREETER-X-0 HAS-GUEST-ACCOUNT-HINT=FALSE
27+
28+
# Cleanup
29+
#?*STOP-DAEMON
30+
#?GREETER-X-0 TERMINATE SIGNAL=15
31+
#?XSERVER-0 TERMINATE SIGNAL=15
32+
#?RUNNER DAEMON-EXIT STATUS=0

tests/src/test-gobject-greeter.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -368,6 +368,8 @@ main (int argc, char **argv)
368368
status_notify ("%s SELECT-GUEST-HINT", greeter_id);
369369
if (lightdm_greeter_get_lock_hint (greeter))
370370
status_notify ("%s LOCK-HINT", greeter_id);
371+
if (!lightdm_greeter_get_has_guest_account_hint (greeter))
372+
status_notify ("%s HAS-GUEST-ACCOUNT-HINT=FALSE", greeter_id);
371373
if (lightdm_greeter_get_hide_users_hint (greeter))
372374
status_notify ("%s HIDE-USERS-HINT", greeter_id);
373375
if (lightdm_greeter_get_show_manual_login_hint (greeter))

tests/test-greeter-allow-guest

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
#!/bin/sh
2+
./src/dbus-env ./src/test-runner greeter-allow-guest test-gobject-greeter

0 commit comments

Comments
 (0)