Moodle

Option to disable logout confirmation screen

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Duplicate
  • Affects Version/s: 1.7.2, 1.8.1, 1.9
  • Fix Version/s: None
  • Component/s: Authentication
  • Labels:
    None
  • Affected Branches:
    MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE

Description

I've just found the new logout confirmation screen that came in with login/logou.php around version 1.19 to add a sesskey and avoid unintended logouts.

I can't say I like it. Personally I find these "are you sure?" screens really annoying. I realise however that some institutions might like this, so is there a way we could set something in the config (perhaps in the auth plugin) to say whether to display the confirm screen or not and then I can choose to have it disabled?

Thanks

Issue Links

Activity

Hide
Petr Škoda (skodak) added a comment -

Hi,
this was already fixed in cvs, the confirmation is there only when
1/ going directly to logout page
2/ stale page - logged in as different user again

fixed cases:
1/ double clicking on logout link
2/ stale page - already logged out

Show
Petr Škoda (skodak) added a comment - Hi, this was already fixed in cvs, the confirmation is there only when 1/ going directly to logout page 2/ stale page - logged in as different user again fixed cases: 1/ double clicking on logout link 2/ stale page - already logged out
Hide
Jenny Gray added a comment -

Ah yes, sorry Petr. It turned out to be a problem with when we'd merged the user_login_string() function in weblib into the OU code base. I should have thought of that.

Show
Jenny Gray added a comment - Ah yes, sorry Petr. It turned out to be a problem with when we'd merged the user_login_string() function in weblib into the OU code base. I should have thought of that.
Hide
Petr Škoda (skodak) added a comment -

no problem, thanks for the report anyway

Show
Petr Škoda (skodak) added a comment - no problem, thanks for the report anyway

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: