Tickets :: [#4886] Spell Check in HTML cause invalid errors

6.0.0-alpha14

6/24/25

Summary	Spell Check in HTML cause invalid errors
Queue	IMP
Queue Version	4.1.3
Type	Bug
State	Not A Bug
Priority	2. Medium
Owners
Requester	technology (at) pmi-portland (dot) org
Created	01/13/2007 (6737 days ago)
Due
Updated	01/23/2007 (6727 days ago)
Assigned	01/14/2007 (6736 days ago)
Resolved	01/22/2007 (6728 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

01/23/2007 04:21:07 AM	technology (at) pmi-portland (dot) org	Comment #9	Reply to this comment
No, I would think it is a bug in any 4.1.3 release. It was 100% reporducible that I could see. Todd

01/22/2007 05:02:33 PM	Jan Schneider	Comment #8 State ⇒ Not A Bug	Reply to this comment
Seems to have been a local issue then.

01/21/2007 03:33:47 PM	technology (at) pmi-portland (dot) org	Comment #7	Reply to this comment
Never mind... Got it that was the area and the HTMLEncode needed: str = str.replace(/\xA0/g, " ") Simple... fixed

01/21/2007 05:34:44 AM	technology (at) pmi-portland (dot) org	Comment #6	Reply to this comment
I have determined this is not an apell issue and it appears to be a htmlarea problem. I cannot find the bug but it is placing a chr(160) instead of a   ( ) for the first of the two spaces. I was going to diff the various JS files in htmlarea with the ones I have to see if I could find the change (I think it is in getHTMLWrapper around the line html = /^script\|style$/i.test(root.parentNode.tagName) ? root.data : HTMLArea.htmlEncode(root.data);. My problem is I cannot find these files in CVS. Can you point me to them?

01/16/2007 05:57:42 PM	Jan Schneider	Comment #5	Reply to this comment
Is it possible you could test on the previous version of aspell (Aspell 0.50.5)? Unfortunately not.

01/16/2007 05:17:55 PM	technology (at) pmi-portland (dot) org	Comment #4	Reply to this comment
I tried to test the newer version of aspell last night but I do not have the tools I need on my hosted site to test that. Is it possible you could test on the previous version of aspell (Aspell 0.50.5)? Cheers, Todd

01/14/2007 10:25:32 PM	technology (at) pmi-portland (dot) org	Comment #3	Reply to this comment
1) My ISP's aspell version is: International Ispell Version 3.1.20 (but really Aspell 0.50.5) 2) Spell check "ignore all" does not seem to work. 3) Would any of my config files help? I plan on installing Version 1.0 of groupware tonight.

01/14/2007 12:54:22 PM	Jan Schneider	Comment #2 State ⇒ Feedback	Reply to this comment
I can't reproduce this. I see HTML tags in the spell checker, but only the text outside of the tags is checked, the errors are displayed correctly, and corrected text isn't duplicated. aspell --version @(#) International Ispell Version 3.1.20 (but really Aspell 0.60.3)

01/13/2007 08:24:57 PM	technology (at) pmi-portland (dot) org	Comment #1 Priority ⇒ 2. Medium Type ⇒ Bug Summary ⇒ Spell Check in HTML cause invalid errors Queue ⇒ IMP State ⇒ Unconfirmed	Reply to this comment
Mail (imp) H3 (4.1.3) Groupware RC2 Spell binary: aspell Browser IE: 6.0, Netscape 8.1 Settings: Compose messages with an HTML GUI by default (if browser supports the feature)?: Checked Create a new email. Enter text with two spaces (i.e. Groupware is great) One of the spaces causes a spell check error. The spelling error reads something like "is¢ in p>Groupware is¼/span>? great&l ". Attempting to correct the error causes text beyond the next paragraph marker to be duplicated in the email. Manually deleting all the spaces and entering just one will remove the spell check error.