RE: php and javascript error

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: Brad Fuller [mailto:bfuller@xxxxxxxxxxxxxxxx]
> Sent: Wednesday, March 07, 2007 12:02 PM
> To: 'Ed Curtis'; 'php-general@xxxxxxxxxxxxx'
> Subject: RE:  php and javascript error
> 
> > -----Original Message-----
> > From: Ed Curtis [mailto:e_curtis@xxxxxxxxxxxxx]
> > Sent: Wednesday, March 07, 2007 11:47 AM
> > To: php-general@xxxxxxxxxxxxx
> > Subject: Re:  php and javascript error
> >
> >
> > >>><a href="#"
> onclick="javascript:window.open('web_forward.php?address=<?
> > >>>echo $web_url; ?>&agent=<? echo $agent; ?>&real_company_name=<? echo
> > >>>$real_company_name; ?>', 'Web Site');">
> > >>>
> > >>>This produces an "Error on Page" in IE 7, but works perfectly in
> > Firefox
> > >>>and Netscape.
> 
> 
> Is it because you have given the window the name 'Web Site' which contains
> a space character?  Try naming it 'WebSite' (without the space)
> 
> Hopefully that is the culprit.

You should also be making sure to use urlencode() on all data that goes into
that URL.

<a href="#" onclick="javascript:window.open('web_forward.php?address=<?=
urlencode($web_url) ?>&agent=<?= urlencode($agent) ?>&real_company_name=<?=
urlencode($real_company_name) ?>', 'WebSite')"> click here </a>


> > >>>I have several other javascript calls on the page where this call
> > >>>resides and all of them work perfectly except for this one. Does this
> > >>>error occur because I'm sending the variables to another script that
> > >>>does some logging then forwards the user to the URL or is it just a
> IE
> > >>>quirk?
> > >>>
> > >>
> > >>This is a JavaScript issue, but may have something to do with the data
> > you
> > >>are echoing out in PHP. Look at the source in the browser. My guess it
> > >>that
> >
> > >>you've got a stay quote mark in one of your PHP vars.
> >
> > Not it, I looked and there's nothing there.
> >
> > >>PS: If you want your code to validate, change the & to &amp;  Add the
> > >>closing </a> tag too.
> >
> > Thanks, the &'s are now all &amp;. The closing </a> is in the script,
> > just wasn't posted in the message.
> >
> > >>PPS: It's advisable not to use the short tags, use <?php instead of <?
> >
> > Can you tell me why? Is it just a personal preference?
> >
> > Another JS call directly in front of this one works perfectly.
> >
> > <a href="#" onclick="javascript:window.open('email.php?mls=<? echo $mls;
> > ?>&amp;toagent=<? echo $agent; ?>&amp;toagentemail=<? echo $agent_email;
> > ?>&amp;tocompanyname=<? echo $real_company_name; ?>', 'Email',
> > 'width=400,height=400,scrollbars=no,resizable=yes');">
> > Send Me Email
> > </a>
> >
> > The only difference is this call has the window options included the one
> > that doesn't work doesn't have them. I've added them just to check and
> > it didn't change anything.
> >
> > --
> > PHP General Mailing List (http://www.php.net/)
> > To unsubscribe, visit: http://www.php.net/unsub.php
> >

-- 
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php


[Index of Archives]     [PHP Home]     [Apache Users]     [PHP on Windows]     [Kernel Newbies]     [PHP Install]     [PHP Classes]     [Pear]     [Postgresql]     [Postgresql PHP]     [PHP on Windows]     [PHP Database Programming]     [PHP SOAP]

  Powered by Linux