As described. Spellcheck module should be usable for checking spelling of node comments, too.
Need a callback?

Comments

matt westgate’s picture

Assigned: Unassigned » matt westgate
Category: bug » feature

This is not bug but rather a feature request. The reason that it doesn't work right now is because comments aren't really nodes. Pages, projects, stories and blogs are nodes, but comments are comments and this module doesn't recognize them.

I plan on implementing this in the near future, along with a user permission "access comments spellchecker".

Kyber’s picture

Gotcha. Thanks.
IMO, *everything* should be a node, with various attribute tables hanging off. But then, I have a fascination with graphs and networks.

matt westgate’s picture

Experimental support for spellchecking comments has been added to CVS head.

Kyber’s picture

Thanks for the fix. Updated to it. Also like the elimination of the check box. Using a button is much clearer.
Think you would mind adding a test for any Misspelled words and hiding "Reset Form" if there are none? Perhaps instead of explaining to click on Correct Spellings to continue with text unchanged, you could just hide both buttons if there are no misspelled words?

Of course, as mentioned in the other bug, clicking on Correct Spelling just sends me to a login page at the moment.

Kyber’s picture

Hm. Got excited too soon.
I've added permissions to access comment spellchecking across the board, but still don't see an option for it.
Do I need to update any module besides spellchecking?

matt westgate’s picture

To use the comments spellchecking, you must apply the drupal.diff file to comments.module. There is no other way that i know of to hook into the comment form.

Kyber’s picture

I applied the diff, but get an error due to spellcheck_button() being an unknown method.
And indeed, I don't see it in spellcheck.module or comment.module (latest revisions of either).
Where'd you put it?

Kyber’s picture

I'm slow. you fixed 1.6.2.3 not 1.9. Guess I need to restore 1.9 and merge in 1.6.2.3 change.

Kyber’s picture

Aaaand, further cluttering up this bug with pointless comments.
1.9 was the patched one I want where you did fix it. 1.8 was the one I had before.
Good thing these comments don't go out to the mailing list. (I hope)

Anonymous’s picture