SpellChecker: Pspell Option?

[brashquido]

Using Beta 2 and think it is great. I really like the SpellChecker? option but would like to be able to configure it to use the PHP Pspell function instead of having to connect to an external service such as Google. This should speed up spellchecking considerably, especially in environments where bandwidth to the Internet is limited. It also gives options to situations where privacy/security comes into play, as some places I do work for won't allow e-mail content to be spellchecked by an external service due to confidentiality. Great work overall though. I'm yet to demo RoundCube to anyone who is not inpressed by its slick interface.

[Harold]

Since Firefox 2.0+ supports spell checking on the client, I'd vote for not to include spell checking in the RoundCube backend...

[jpingle]

If security and having it be local are the only reasons, You can also setup a NOX Spell Server (http://orangoo.com/labs/GoogieSpell/Download/Nox%20Spell%20Server/) and then point your config to it like this:

$rcmail_config['spellcheck_uri'] = '127.0.0.1:14003/?lang=';

Even so, it would be nice to have a pure-PHP solution. Not everyone is lucky enough to have a pure-Firefox userbase. :/

[till]

If you want a pure PHP solution (I yet have to see a good reason), just port NoxSpell. Or write your own server from scratch.

For the moment though, I don't see this being a Roundcube issue, nor a valid feature request. We have hooks in place, it's up to someone else to contribute.

[thomasb]

Since Pspell is quite common, I'd leave this open with a low priority. Also TinyMCE comes up with some Pspell bindings which could propabyl adapted.

[dan]

seems as though TinyMCE has a standard plugin for spelling that supports

* pspell

* aspell

* google spell checker

http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/spellchecker located in /program/js/tiny_mce/plugins/spellchecker/

[dan]

though rough hacks i've got the Pspell working in html edit mode steps for the crude test:

1. download http://tinymce.moxiecode.com/download.php tinymce_spellchecker_php_2_0_2.zip
2. unpack in the directory /program/js/tiny_mce/plugins/spellchecker
3. edit ./program/js/tiny_mce/plugins/spellchecker/config.php to use engine 'Pspell'
4. install the php5-pspell and aspell dictionaries

5. apply a patch like

--- program/js/editor.js        (revision 1597)
+++ program/js/editor.js        (working copy)
@@ -15,6 +15,7 @@

 // Initialize the message editor

+//function rcmail_editor_init(skin_path, editor_lang, spellcheck_languages)
 function rcmail_editor_init(skin_path, editor_lang)
   {
   tinyMCE.init({ mode : "textareas",
@@ -23,12 +24,13 @@
                  apply_source_formatting : true,
                  theme : "advanced",
                  language : editor_lang,
-                 plugins : "emotions,media,nonbreaking,table,searchreplace,visualchars,directionality",
-                 theme_advanced_buttons1 : "bold,italic,underline,separator,justifyleft,justifycenter,justifyright,justifyfull,separator,bullist,numlist,outdent,indent,separator,link,unlink,emotions,charmap,code,forecolor,backcolor,fontselect,fontsizeselect, separator,undo,redo,image,media,ltr,rtl",
+                 plugins : "emotions,media,nonbreaking,table,searchreplace,visualchars,directionality,spellchecker",
+                 theme_advanced_buttons1 : "bold,italic,underline,separator,justifyleft,justifycenter,justifyright,justifyfull,separator,bullist,numlist,outdent,indent,separator,link,unlink,emotions,charmap,code,forecolor,backcolor,fontselect,fontsizeselect, separator,undo,redo,image,media,ltr,rtl,spellchecker",
                  theme_advanced_buttons2 : "",
                  theme_advanced_buttons3 : "",
                  theme_advanced_toolbar_location : "top",
                  theme_advanced_toolbar_align : "left",
+                 spellchecker_languages : "+English=en,Swedish=sv",
                  extended_valid_elements : "font[face|size|color|style],span[id|class|align|style]",
                  content_css : skin_path + "/editor_content.css",
                  external_image_list_url : "editor_images.js"


change to html edit mode when writing a new email and there will be a working spell icon.
If you have safemode problem the ./classes/PSpell.php has to have safe mode permissions to the dictionary. All pspell error occur in dialog boxes so its pretty easy to diagnose problems.

Question really is what is an acceptable patch? non-exclusive options:

1. replace the current googie implementation with TinyMCE's API to google?

1a. Is there value supporting googie and the TinyMCE google access?

2. Use the TRUE/FALSE value of 'enable_spellcheck' configuration options still apply to the google spell checker add a 'PSpell' value to make it do PSpelling

4. Is having a large spell button still needed. Is having the link for spelling above the text pane needed?

5. Is there value supporting PSpellShell?

[ksteinhoff]

My patch (pspell.patch) implements Pspell for the plain text spell checker.

It adds a configuration option ('spellcheck_engine').

In this patch, the contents of spell.inc have been moved to spell_googie.inc, and spell_pspell.inc has been add. spell.inc invokes googie or pspell, depending on the configurations option.

Using pspell requires the Pspell extension for PHP.

[till]

I am very much in favour of spinning off the translation stuff into "drivers".

Thanks for the patch Kris!

[thomasb]

Patch committed to trunk in r1649.

Now we still need to fix the html mode...

[thomasb]

OK, I'll take care of the html editor...

[thomasb]

tinyMCE spellchecker fixed in r1651

[dan]

thankyou, very much appreciated thomasb, Kris and till. This works for me and well done on the depth of integration keeping roundcube configation consistant with tinymce.