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

[WordPress Trac]

#9798: TinyMCE Spellchecker not working behind firewall (google spellcheck)
--------------------------+-------------------------------------------------
 Reporter:  bforchhammer  |       Owner:  azaozz                      
     Type:  enhancement   |      Status:  new                         
 Priority:  normal        |   Milestone:  2.8                         
Component:  TinyMCE       |     Version:  2.8                         
 Severity:  normal        |    Keywords:  has-patch proxy spellchecker
--------------------------+-------------------------------------------------
Changes (by Denis-de-Bernardy):

  * keywords:  has-patch 2nd-opinion proxy spellchecker => has-patch proxy
               spellchecker
  * milestone:  Future Release => 2.8


Comment:

 Big +1 to the approach.

 One small tweak I'd suggest: dirname(dirname(...dirname(__FILE__)...))
 rather than the ../../.., as some servers don't behave well with relative
 include paths.

 Also, I took a quick glance at the original class and:

 http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/spellchecker#Spellchecker_PHP_options

 On the one hand side, I opened #9805, which presumably works since nobody
 is reporting spell checker issues. But the patch would need to be
 rewritten in such a way that these regex (and the one in the Spellchecker
 class) no longer use the /e modifier (see #8689).

 On the other side, the jargon option caught my attention. Could you look
 into bundling a fix to #7810 while we're at it? :-)

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