Problem/Motivation

When editing pages using ckeditor the editor with run its detach function due to the ajax call to autologout_ahah_set_last. It looks like this is due to the timer element not being present on the page so drupal ajax is running detachBehaviors on the document and not scoped to the timer element.

Steps to recreated:

  • Set auto logout timeout to 60 seconds for ease of testing
  • Create or edit a basic page using the ckeditor
  • Wait for 60 seconds – a request to autologout_ahah_set_last should be made
  • When the response is received Drupal won’t find the an element with the id #timer
  • detachedBehaviours will run scoped to the document and ckeitor will disappear showing the raw markup

Proposed resolution

Insert correct markup using javascript to the page and modifying the response received to contain the correct markup

Remaining tasks

Create patch for review

Comments

MobliMic created an issue. See original summary.

MobliMic’s picture

Version: 8.x-1.0-rc2 » 8.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new1.35 KB

Here is the patch I've done to fix the issue.

MobliMic’s picture

Assigned: MobliMic » Unassigned
arijits.drush’s picture

Status: Needs review » Needs work

Working perfect if there is no message to reset timer.
But fails when message comes and user click on "YES" button

MobliMic’s picture

Status: Needs work » Needs review
StatusFileSize
new1.64 KB

I have updated it so it works with the dialog box now

arijits.drush’s picture

Looks ok to me.

arijits.drush’s picture

Status: Needs review » Reviewed & tested by the community
arijits.drush’s picture

arijits.drush’s picture

Status: Reviewed & tested by the community » Fixed
arijits.drush’s picture

nicrodgers’s picture

Status: Fixed » Reviewed & tested by the community

Changing status back to RTBC as I don't think this has been committed yet.

  • AjitS committed 09ff0cf on 8.x-1.x authored by MobliMic
    Issue #2798553 by MobliMic: Auto logout causes ckeditor to detach when...
ajits’s picture

Status: Reviewed & tested by the community » Fixed

Thank you for the patch and reviews! Committed and pushed to 8.x-1.x

ajits’s picture

Status: Fixed » Closed (fixed)

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