On a current project we've had a client request spell checking when creating nodes and relying on the browser isn't an option, so I thought it'd be a good opportunity to try out this module. In testing content-heavy sites we frequently use "lorem ipsum" text to test layout and presentation. I understand there might be some caveats with using Latin to test English-based but we've run in to a few errors which I think can be better handled by the module.

Firstly, it's possible for spellcheck_suggestions_render() to be called with a null argument, causing implode() to throw errors.

Secondly, I've run in to a problem which I'm finding difficult to explain but shouldn't be too hard to reproduce. From the looks of things, the regex is matching substrings of an error. This in turn is generating some malformed HTML output. Sorry this is a little vague but hopefully if you attempt to reproduce this you'll see what I mean (we use http://www.lipsum.com/feed/html to generate the text).

Let me know if there's any more information I can supply.

Comments

xano’s picture

Status: Active » Postponed (maintainer needs more info)

Hi there! Thanks for the report.

1) Could you stick to one particular topic per issue? This way things don't get mixed up accidentally.
2) Also, could you provide step by step instructions on how to reproduce the problem using a fresh installation of Drupal and Spell check dev?

xano’s picture

Status: Postponed (maintainer needs more info) » Closed (won't fix)

The code has been reworked to use Pspell and to integrate with Text Review. This issue is now obsolete.