Tickets :: [#9703] Check spelling is introducing html code on the message body

6.0.0-alpha10

5/16/25

Summary	Check spelling is introducing html code on the message body
Queue	IMP
Queue Version	Git master
Type	Bug
State	No Feedback
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	goncalo.queiros (at) portugalmail (dot) net
Created	03/23/2011 (5168 days ago)
Due
Updated	04/12/2011 (5148 days ago)
Assigned
Resolved	04/11/2011 (5149 days ago)
Milestone
Patch	No

04/12/2011 04:26:14 PM	Michael Slusarz	Comment #8	Reply to this comment
And I still can't reproduce. So something is different on your setup than mine so you will need to track down the problem locally and report back.

04/12/2011 08:53:02 AM	goncalo (dot) queiros (at) portugalmail (dot) net	Comment #7	Reply to this comment
Tested again, with more recent code, cleared cache and all... The problem is still there. I didn't mention in the previous comment that you need to type errors to see the   strings to appear. I followed exactly the steps i mentioned in the previous comment and i could reproduce exactly what im saying

04/11/2011 07:49:24 PM	Michael Slusarz	State ⇒ No Feedback

03/28/2011 05:28:49 PM	Michael Slusarz	Comment #6	Reply to this comment
Works for me.

03/28/2011 12:00:27 PM	goncalo (dot) queiros (at) portugalmail (dot) net	Comment #5	Reply to this comment
Sorry Michael, but after all there's some problems with this yet. - Open the compose, and start composing in HTML - Write at least 2 white spaces together - Change to text composition - Ask for spellcheck and resume edition The whitespaces are now replaced by " "

03/24/2011 11:20:43 AM	goncalo (dot) queiros (at) portugalmail (dot) net	Comment #4	Reply to this comment
Fixed. Thanks

03/24/2011 05:25:04 AM	Michael Slusarz	Comment #3 State ⇒ Resolved Assigned to Michael Slusarz	Reply to this comment
Can't reproduce, but we shouldn't be adding ajaxui to any imple URL. This should fix.

03/24/2011 05:24:17 AM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #9703~~: Don't add ajaxui to spellcheck URL request 1 files changed, 1 insertions(+), 1 deletions(-) http://git.horde.org/horde-git/-/commit/5271c17e583c7e321f4465f27940fe28cbbe815c

03/23/2011 05:06:33 PM	goncalo (dot) queiros (at) portugalmail (dot) net	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ Check spelling is introducing html code on the message body Queue ⇒ IMP Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
Compose a new HTML message on IMP, type something with two or more white spaces together, then ask for spell check. The message will now have   instead of the white spaces. This seems to be related with the url of the ajax call for spell check, because it ends with &ajaxui=1/html and the code seems to be expecting the "/html" to be placed before the &ajaxui..