Javascript Problems

Anon

Hi,
I've tried to make an exit popup to work but nothing happens! Is this code wrong?
Below is a snippet from th body of the php.

-----------CODE START------------
</tr>
</table>
</form>
<script language="javascript">
OnUnLoad="window.popup('http://www.bandmart.com');"
</script>
<?
}
elseif (session_is_registered("valid_user")&&($activated == '0'))
{
echo "<p><b>Your account has be
-----------CODE END------------

Anon

is this a PHP related question?

rrijnders

I <i>hate</i> those things! When someone exits a site, they don't want to be bombarded with ads that pop ads that pop more ads everytime you close one! It's enough to make a person disable javascript all together!

Anon

It's not to create an annoying popup, its to open a php window to close the session for security reasons. The url I put in the code is just an example. I want it to open php script called "logout_small.php". The javascript has been placed after an html table between <? and ?> of an if statement.

The full code can be seen at:

http://www.brooky.com/mh/code.htm

I would really appreciate any help. I'm in the process of creating a php/MySQL on line repeat perscription service for a health center in the UK.

Anon

Yes, I've put a link to the code on a lower thread.

rrijnders

1) I don't see where you have defined the "popup" function anywhere. You need to define a function to pop a window.

2) You have to attach the "onUnload" event handler to an object.

Put this in your page header (between the <HEAD> and </HEAD> tags:

<b>
<SCRIPT LANGUAGE="Javascript">
function popup(pagesource) {
  var t-this.open(pagesource,"logout","toolbar=no,menubar=no,location=no,scrollbars=yes,resizable=no,width=400,height=425");
  t.focus();
}
</SCRIPT>
</b>

Then put the "onUnload" event handler into your <BODY> tag like so:

You can play with the parms in the window open command to get whatever results you need, but that should do it for your.

-- Rich

rrijnders

Change that big long javascript line to begin with:

<b>var t=this.open(</b>...

not

<b>var t-this.open(</b>...

-- Rich

Anon

Thanks so much, I'll give that ago.