doesn't work for me at all. running 4.4 i click on "check spelling", and it brings up a new window (presumably with a spellcheck window) with the site in it. in Mozilla it brings up the window properly, but erroneously claims that spellcheck found no errors.

j.

CommentFileSizeAuthor
#4 spellcheck.module-fix-rel-url.udiff612 bytesSiv-1

Comments

TDobes’s picture

TDobes commented
Category: bug » support
Priority: Critical » Normal

It works just fine for me in IE. Can you give us some further details about your configuration? What other modules do you have enabled? You are using the 4.4 release of spellcheck, right? (the second line of spellcheck.module should say: /* $Id: spellcheck.module,v 1.9 2003/12/07 23:04:47 mathias Exp $ */>) What specific release(s) of IE did you test with and under what OS?

Anonymous’s picture

(not verified) commented
Category: support » bug

I'm also having this problem

IE version is 6.0.2800.1106.xpsp2.030422-1633
windows XP home

mods installed
aggregator
archive
bbcode
block
blog
blogapi
bookmarks
comment {not patched}
filestore
fixentities
forum
glossary
help
image
node
page
ping
poll
privatemsg
profile
queue
search
smileys
spellcheck
statistics
story
syndication
taxonomy
throttle
title
tracker
weather

useing ispell rather then aspell {don't have aspell on the server}
server is debian woody
php version is 4.1.2 {old I know}

mozilla client is 1.6 brings up the spell check window... which is partly themed but lacking background color ... and does not properly indacate that there is misspelled words
/* $Id: spellcheck.module,v 1.10 2004/03/09 01:36:47 mathias Exp $ */

jcosters’s picture

jcosters commented
Title: doesn't work in IE » clean url's

I think clean url's are the problem here. With them it doesn't work, without them it does (on my configuration - 4.4.1, spellcheck cvs, that is).

Siv-1’s picture

Siv-1 commented
Assigned: Unassigned » Siv-1
StatusFileSize
new spellcheck.module-fix-rel-url.udiff612 bytes

The attached patch works around differences in the interpretation of "relative URL" in IE 6x and Mozilla, and allows spellcheck to work on both platforms. Opera works now too.

The exact change included in the patch is:

-      $url = url("spellcheck&c=$crypt");
+      $url = url($base_url.'/'."spellcheck&c=$crypt");

Review and apply the attached patch.

Christopher Herberte’s picture

Christopher Herberte commented
Status: Needs review » Closed (fixed)

closed