[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