Active
Project:
Instant messenger
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
13 Apr 2010 at 16:58 UTC
Updated:
7 May 2010 at 08:12 UTC
Jump to comment: Most recent file
I reported the issue with clearn URLs a while back in #653104: Seems to expect using Clean URLs is okay and some of the other issues related to sound in #653436: No sound when message receive for non-current conversation that were related to IE not being able to use the indexOf method in javascript as well as IE not running the oninit event (or at least not doing it at the right time).
Attached is the patch that fixes these. The fix for the clean urls issue should allow the module to now work with both sites that are and aren't using clean URLs.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | im.patch | 3.49 KB | wdouglascampbell |
| #1 | im.patch | 3.04 KB | wdouglascampbell |
| im.patch | 2.99 KB | wdouglascampbell |
Comments
Comment #1
wdouglascampbell commentedSlight error in patch. Here is the correct one.
Comment #2
wdouglascampbell commentedMethod I used for correcting the use of indexOf in IE was causing me problems with Wysiwyg module so I have modified things. Attached is the new working patch.
Comment #3
pahariwalla commentedThanks @wdouglascampbell for these fixes, and also for taking the time to create a patch. Most useful things, patches. It will take a little time to test this out and get the fix in ( I don't have access to a Windows box at the moment , but I sure appreciate your work.