I have un-checked the checkbox "Do not display the logout dialog" but dialog box is not displaying and redirect to login page. See attached settings

CommentFileSizeAuthor
#11 dialog_not_shown-3031020-11.patch1.48 KBjaperry
#9 dialog_not_shown-3031020-9.patch594 bytesbobcares
#8 dialog_not_shown-3031020-6.patch68 bytesbobcares
#6 dialog_not_shown-3031020-6.patch68 bytesbobcares
#2 logout_dialog-3031020-2.patch1.3 KBsergiu stici
autologoutpopup.PNG32.26 KBvernit

Comments

vernit created an issue. See original summary.

sergiu stici’s picture

sergiu stici commented
StatusFileSize
new logout_dialog-3031020-2.patch1.3 KB

I fixed the bug, here is the patch.

sergiu stici’s picture

sergiu stici commented
Status: Active » Needs review
ajits’s picture

ajits
he/him
Primary language Marathi
Location India
commented
Version: master » 8.x-1.x-dev

@Sergiu Stici: I am not able to reproduce this on my side. Can you please check again?

On a side note, please use the appropriate version. I am not tracking the master branch.

ajits’s picture

ajits
he/him
Primary language Marathi
Location India
commented
Status: Needs review » Postponed (maintainer needs more info)

Putting this on hold until there are proper steps to reproduce the error.

bobcares’s picture

bobcares commented
StatusFileSize
new dialog_not_shown-3031020-6.patch68 bytes

Hello,

In my case, the autologout_page_attachments() hook was not working. As a result, the autologout.js file was not loaded and hence nothing worked as expected.

Please find the attached patch which will change the hook autologout_page_attachments() to autologout_page_attachments_alter().

It might solve the issue.

bobcares’s picture

bobcares commented
bobcares’s picture

bobcares commented
StatusFileSize
new dialog_not_shown-3031020-6.patch68 bytes

In my case, the autologout_page_attachments() hook was not working. As a result, the autologout.js file was not loaded and hence nothing worked as expected.

Please find the attached patch which will change the hook autologout_page_attachments() to autologout_page_attachments_alter().

It might solve the issue.

Uploaded wrong file by mistake

bobcares’s picture

bobcares commented
StatusFileSize
new dialog_not_shown-3031020-9.patch594 bytes

Fixing the patch.

Sorry for the repeated posts.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
japerry’s picture

japerry
Location KOMK
commented
Status: Postponed (maintainer needs more info) » Reviewed & tested by the community
StatusFileSize
new dialog_not_shown-3031020-11.patch1.48 KB

Added to the 8.x-1.4 release #3293639: Issues for 8.x-1.4 release. Made some minor changes to the variables to match function method signatures.

japerry’s picture

japerry
Location KOMK
commented
Status: Reviewed & tested by the community » Fixed

  • japerry committed 9476a03 on 8.x-1.x authored by bobcares 
    Issue #3031020 by Sergiu Stici, Vernit: Logout Dialog is not displaying...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.