Opened 6 years ago

Closed 6 years ago

#4411 closed Bug (fixed)

Bad configuration options for SCAYT

Reported by: alfonsoml Owned by:
Priority: Normal Milestone: CKEditor 3.3
Component: UI : Spell Checker Version: 3.0
Keywords: Confirmed Cc:


The initialization has this code

oParams.customDictionaryName = editor.config.scayt_customDictionaryName;
oParams.userDictionaryName = editor.config.scayt_userDictionaryName;
oParams.defLang = editor.scayt_defLang;

if ( CKEDITOR._scaytParams )
	for ( var k in CKEDITOR._scaytParams )
		oParams[ k ] = CKEDITOR._scaytParams[ k ];


scayt_customDictionaryName and scayt_userDictionaryName aren't documented anywhere.

editor.scayt_defLang isn't using the config object so it's harder to change its value. Also, while trying to test it I wasn't able to make it work (or at least, make it work as I expected)

The CKEDITOR._scaytParams object means that any such option is shared by all the editor instances, instead of being able to configure each instance in a different way. It can't be overriden also in a configuration file. It's also hard to know what can be used there. Luckily I was able to find that sLang worked and help this user:

Change History (4)

comment:1 Changed 6 years ago by

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

patch is submitted in #5145

comment:2 Changed 6 years ago by fredck

  • Resolution fixed deleted
  • Status changed from closed to reopened

It's ok to indicate that this one is supposed to be fixed by another tickets, but it should be closed only after confirming the fix one the relative ticket gets closed.

comment:3 Changed 6 years ago by

Completely addressed in #5287.

comment:4 Changed 6 years ago by garry.yao

  • Keywords Confirmed added
  • Milestone set to CKEditor 3.3
  • Resolution set to fixed
  • Status changed from reopened to closed

Fixed at #5287.

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