Closed (fixed)
Project:
Video Chat
Version:
6.x-1.0
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
21 Aug 2009 at 14:34 UTC
Updated:
18 Nov 2009 at 10:54 UTC
The enter video chatroom button throws a javacript error and fails to open the window
N.B. This is only in IE . other browsers all seems OK
Comments
Comment #1
jerdiggity commentedThanks for the input... What version of IE?
Comment #2
medden commentedI'm getting a javascript error using I.E. 8.0.6
And again the pop up window wont open.
I changed line 36 to
$videochatblock .= "&setage=&setgender=&setroom='+roomid,'_blank'','resizable,width=1000,height=655');I.E. apparently doen not support variables for the string that specifies the name of window.
Use one of these instead:
* _blank The sURL is loaded into a new, unnamed window.
* _media The url is loaded in the Media Bar in Microsoft Internet Explorer 6. Microsoft Windows XP Service Pack 2 (SP2) and later. This feature is no longer supported. By default the url is loaded into a new browser window or tab.
* _parent The sURL is loaded into the current frame's parent. If the frame has no parent, this value acts as the value _self.
* _search Disabled in Windows Internet Explorer 7, see Security and Compatibility in Internet Explorer 7 for details. Otherwise, the sURL is opened in the browser's search pane in Internet Explorer 5 or later.
* _self The current document is replaced with the specified sURL.
* _top sURL replaces any framesets that may be loaded. If there are no framesets defined, this value acts as the value _self.
Comment #3
Mantriur commentedUsing "_blank" can cause users to accidently open several chat windows, which will disconnect all windows - the chat is limited to instance per client.
The problem isn't the window name itself, but using the room name for it, which is the domain name, which contains a dot, which IE doesn't like. ;)
As a quick workaround I'd recommend using any unique name other than "_blank" to ensure starting the chat twice just reloads it.
Comment #4
martinterryevans commentedProblem was first noticed in Internet Explorer - version 7.0.6001.18000
Was then tested using IE Tester (IE 5.5, IE 6, IE 7 and IE 8)
so I guess you can safely say problem is specific to Internet Explorer (all versions)
changing the window name to null seems to fix it
$videochatblock .= "&setage=&setgender=&setroom='+roomid, 'null','resizable,width=1000,height=655');
Comment #5
jerdiggity commentedThanks for all the input...
@martinterryevans: Good job! Worked for me after multiple tests and the roomid (domain) still remains intact.
Committed to CVS.
Closing...
Comment #7
francoud commentedI added this correction to version 5 and it solved the problem also there! :)