[wp-trac] Re: [WordPress Trac] #9798: TinyMCE Spellchecker not working behind firewall (google spellcheck)

WordPress Trac wp-trac at lists.automattic.com
Thu May 21 07:35:56 GMT 2009


#9798: TinyMCE Spellchecker not working behind firewall (google spellcheck)
--------------------------+-------------------------------------------------
 Reporter:  bforchhammer  |       Owner:  azaozz                 
     Type:  enhancement   |      Status:  new                    
 Priority:  normal        |   Milestone:  Future Release         
Component:  TinyMCE       |     Version:  2.8                    
 Severity:  normal        |    Keywords:  has-patch needs-testing
--------------------------+-------------------------------------------------

Comment(by bforchhammer):

 I think at least the "proxy support" bit should definitely go into 2.8 in
 some form! I repeat my argument from above:
 > If people start using proxy support in 2.8 there's a high probability
 that they will run into this issue.

 Also, as far as I can see the TinyMCE config file only allows to set a
 "general.engine" parameter which is used for both including the class-file
 as well as instantiating the class.

 [http://core.trac.wordpress.org/browser/trunk/wp-
 includes/js/tinymce/plugins/spellchecker/includes/general.php#L18
 general.php, l. 18-19]: this is where class file gets included.
 {{{
 if (isset($config['general.engine']))
         require_once(dirname(__FILE__) . "/../classes/" .
 $config["general.engine"] . ".php");
 }}}

 [http://core.trac.wordpress.org/browser/trunk/wp-
 includes/js/tinymce/plugins/spellchecker/rpc.php#L96 rpc.php l. 96-99]:
 this is where the SpellChecker object is created.
 {{{
 if (isset($config['general.engine'])) {
         $spellchecker = new $config['general.engine']($config);
         $result = call_user_func_array(array($spellchecker,
 $input['method']), $input['params']);
 }
 }}}

 I am not sure there's a way to use a class from a different location -
 without changing any TinyMCE files. Please correct me if I'm wrong. I am
 open for suggestions!

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/9798#comment:22>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list