Tek-Tips is the largest IT community on the Internet today!

Members share and learn making Tek-Tips Forums the best source of peer-reviewed technical information on the Internet!

Register Log in
  • Congratulations strongm on being selected by the Tek-Tips community for having the most helpful posts in the forums last week. Way to Go!

stack overflow error in IE?

Status
Not open for further replies.
raindogs

raindogs

Programmer
Nov 23, 2005
27
US
I'm writing a script that will cause all links in a pop-up window to open in the parent window. Here's what I have in the head:

<script language="JavaScript" type="text/javascript">
<!--
function checkPops() {
if (window.opener && !window.opener.closed) {
var x = document.getElementsByTagName('a');
for (var i=0;i<x.length;i++) {
x.target=window.opener.name;
}
}
}
//-->
</script>


I'm calling the function with an "onLoad" in the <body> tag, and everything works great in Firefox. The problem is, I get a "stack overflow" error in IE. The error points to the line that contains the <body> tag to call the function. I don't see anything that should be causing this problem, can anyone else?

Thanks,
Alex
 
Sort by date Sort by votes
You didn't show the <body> tag, so it's difficult to see if there are any errors there.

The stack overflow errors I've run into have been due to recursion that doesn't terminate, but your Javascript function that you've shown doesn't appear to do that.

Lee
 
Upvote 0 Downvote
Apologies for not including the body tag. It's the ever-exciting

<body onload="checkPops()">

Thanks,
Alex
 
Upvote 0 Downvote
If my above suggestion isn't relevant to this, then I'd suggest downloading and installing Microsoft's free script debugger. Then, place a "debugger" statement at the top of the function, step through 1 line at a time, and see where the stack overflow is happening.

Hope this helps,
Dan



[tt]Dan's Page [blue]@[/blue] Code Couch
[/tt]
 
Upvote 0 Downvote
Does this change it for you?
Code: 
function checkPops() {
    if (window.opener && !window.opener.closed) {
        var x = document.getElementsByTagName('a');
        var w = window.opener.name;
        for (var i=0;i<x.length;i++) {
            x[i].target=w;
        }
    }
}
From an optimisation perspective you could change your loop to run a bit faster (but it won't fix the problem):
Code: 
for (var i=0,max=x.length; i<max; i++) {
Cheers,
Jeff

[tt]Jeff's Page @ Code Couch
[/tt]

What is Javascript? FAQ216-6094
 
Upvote 0 Downvote
Status
Not open for further replies.

Similar threads

jomat
  • Locked
  • Question
JS to show the timer in html under title instead of in the status bar
Replies
2
Views
840
xwb
xwb
MDGarcia
  • Locked
  • Question
Java script blocked due to MIME type error
Replies
2
Views
1K
MDGarcia
MDGarcia
JScannell
  • Locked
  • Question
Programmatically inserting a newline in a &lt;textarea&gt;, &lt;span&gt;, or &lt;div&gt;
Replies
5
Views
1K
ggriffit
ggriffit
JPSeo
  • Locked
  • Question
simple SyntaxError: missing ) after argument list error
Replies
2
Views
251
JPSeo
JPSeo
dannyo12
  • Locked
  • Question
Async library error in terminal while running Ethereum Smart Contracts test with mocha?
Replies
0
Views
141
dannyo12
dannyo12

Part and Inventory Search

Sponsor

Back
Top