There seems to be a problem with the Ajax version, it fails most of the time. When you click the stars the refresh loads the frontpage into the voting form. This is with cleanurls enabled. I think most likely the problem has to do with the path that the javascript refreshes too, but I don't know enough about this to fix the problem. This is checked against the CVS version directly from the repo.

CommentFileSizeAuthor
#3 patch_20.txt533 bytesVladekk

Comments

dr. gubó’s picture

dr. gubó commented

I got the same error, but only with firefox. With IE, everything works fine.
Firefox displays the + and - links in black, while IE in red (which is the default color for links in my css). Clicking + or - in firefox loads the front page in the content area (between the sidebars). I got no clue what this could be since I don't know javascript. Maybe firefox handles javasscript differently?

Vladekk’s picture

Vladekk commented

Yeah, i've got this one also with pushbutton and clean urls.
Installed in root.

Vladekk’s picture

Vladekk commented
Status: Active » Reviewed & tested by the community
StatusFileSize
new patch_20.txt533 bytes

Looks like I finally fixed this one (with almost no knowledge of javascript and php)
The problem was in url, as was correctly stated before me. AJAX request didn't work because Firefox and Opera treated path as relative, while it needed to be absolute, starting with "/". I think I fixed it for any setup, but who knows . . .

While researching I also found other bug, that prevented mediumvote work in Explorer and Opera (at least for me). The called function name was incorrect (GetTextContent instead of getTextContent)
Sorry if I made something wrong, this is first time a made a patch and first time I dealt with javascript

sami_k’s picture

sami_k commented

confirmed, the patch does work to fix this problem. i do wondering however why you change the capitalization of the function... was there a need for that?

Vladekk’s picture

Vladekk commented

Yes, funcion name was indeed a bug. JavaScript is case-sensitive, so the only browser that understood incorrect name was Firefox/Mozilla (bulleproof behavior, heh?). At least for me and some other people.

Christoph C. Cemper’s picture

Christoph C. Cemper commented

Sorry, I applied this patch and it doesn't help the "clean url" problem

applied the Ajax / JS bugfix, but the real problem is that mediumvote won't display correct with clean urls enabled

ie

http://www.dp472-cvsonly.com/?q=node/1 work

and

http://www.dp472-cvsonly.com/node/1

does STILL not!

Christoph C. Cemper’s picture

Christoph C. Cemper commented
Title: Ajax Problem » Still Ajax Problem with installations in subdirs
Status: Reviewed & tested by the community » Active

Notice that by http://drupal.org/node/55468 I got it to run on drupal sites running in the root...

however if the drupal installation is somewhere below, i.e. lying in /articles/ instead of / then the ajax/JS again raises a 404