Opened 5 years ago

Closed 5 years ago

#5674 closed Bug (fixed)

SCAYT error on editor destroy

Reported by: SpellChecker.net Owned by: alfonsoml
Priority: Normal Milestone: CKEditor 3.3
Component: UI : Spell Checker Version:
Keywords: Confirmed Review+ Cc: SpellChecker.net

Description (last modified by alfonsoml)

Turn on scayt autostartup on sample page divreplace.html

Edit 1-st text block by double clicking, wait scayt run

Edit 2-nd text block:

error occurs:

plugin.getScayt() is null or not an object

Attachments (3)

5674.patch (1.3 KB) - added by SpellChecker.net 5 years ago.
add check of SCAYT control instance existing
5674_2.patch (1.1 KB) - added by SpellChecker.net 5 years ago.
debuging code remove
5674_3.patch (1.2 KB) - added by alfonsoml 5 years ago.
Proposed patch

Download all attachments as: .zip

Change History (9)

Changed 5 years ago by SpellChecker.net

add check of SCAYT control instance existing

Changed 5 years ago by SpellChecker.net

debuging code remove

comment:1 Changed 5 years ago by garry.yao

  • Cc SpellChecker.net added
  • Keywords Confirmed Review- added

The error is still there after the patch.

comment:2 Changed 5 years ago by SpellChecker.net

the error is different on trunk:

Error: iframe is null
Source File: http://hostname/lib/ckeditor.svn/_source/plugins/wysiwygarea/plugin.js
Line: 718

Changed 5 years ago by alfonsoml

Proposed patch

comment:3 Changed 5 years ago by alfonsoml

  • Description modified (diff)
  • Keywords Review? added; Review- removed
  • Owner set to alfonsoml
  • Status changed from new to assigned

The patch adds the protection and works for me both in trunk and 3.3 branch

comment:4 Changed 5 years ago by SpellChecker.net

works for us in trunk

comment:5 Changed 5 years ago by garry.yao

  • Keywords Review+ added; Review? removed

comment:6 Changed 5 years ago by alfonsoml

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed with [5502]

Note: See TracTickets for help on using tickets.
© 2003 – 2015 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy