[#MDL-10023] Option to disable logout confirmation screen

Issue Details (XML | Word | Printable)

Key:	MDL-10023
Type:	Improvement
Status:	Resolved
Resolution:	Duplicate
Priority:	Minor
Assignee:	Petr Skoda
Reporter:	Jenny Gray
Votes:	0
Watchers:	1

Operations

Add/Edit UI Mockup to this issue

If you were logged in you would be able to see more operations.

Moodle

Option to disable logout confirmation screen

Created: 05/Jun/07 10:48 PM Updated: 06/Jun/07 05:08 PM

Return to search

Component/s:

Authentication

Affects Version/s:

1.7.2, 1.8.1, 1.9

Fix Version/s:

None

Issue Links:

Duplicate

This issue duplicates:

~~MDL-9824~~ do not use sesskey check when user already logged out in logout.php

Participants:	Jenny Gray and Petr Skoda
Security Level:	None
Resolved date:	06/Jun/07
Affected Branches:	MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE

Description

« Hide

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

All

Comments

Change History

Version Control

Sort Order:

Petr Skoda made changes - 06/Jun/07 01:30 AM

Field	Original Value	New Value
Link		This issue duplicates ~~MDL-9824~~ [ ~~MDL-9824~~ ]

[ Permalink | « Hide ]

Petr Skoda added a comment - 06/Jun/07 01:33 AM

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

Petr Skoda made changes - 06/Jun/07 01:33 AM

Status	Open [ 1 ]	Resolved [ 5 ]
Resolution		Duplicate [ 3 ]

[ Permalink | « Hide ]

Jenny Gray added a comment - 06/Jun/07 04:33 PM

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.

[ Permalink | « Hide ]

Petr Skoda added a comment - 06/Jun/07 05:08 PM

no problem, thanks for the report anyway

All content on this web site is made available under the GNU General Public License, unless otherwise stated.