Re: Re: Is There No Hope At All???

From: Date: Thu, 24 Aug 2000 17:44:05 +0000
Subject: Re: Re: Is There No Hope At All???
References: 1  Groups: php.general 
Request: Send a blank email to php-general+get-13475@lists.php.net to get a copy of this message
        while($counter < $total_rows):
<snip>
                        if ($genderee == $genderz) {
?> <a href="#a" onClick="horseWindow()"> <script language="Javascript1.2"> function horseWindow() {
       newWindow = window.open("Your_Horse.php3?name=<? echo
          $namee; ?>&color=<? echo $coloree; ?>&gender=<? echo $genderee;
?>&price=<? echo
          $pricee; ?>&birth=<? echo $birthee; ?>&performance_one=<? echo
          $performance_onee; ?>&performance_more=<? echo
$performance_moree; ?>&height=<?
          echo $heightee; ?>&description=<? echo $descriptionee;
?>","","toolbar=1,menubar=0,width=800,height=600,scrollbars=1,status=0,location=0,directories=0,left=0,top=0"); } </script> <?
                        echo mysql_result($result,$counter,"name");
<snip> Hi Ben, Jeremy's comment that you need to declare the function before you use it is correct. However, here is question I had when I looked at your code : - Why are you printing one <script> tag, declaring the same function over and over again for each link you create ? Can't you just create a form and hidden elements for each link and pass the link "id" or whatever you want to call it thru a onClick event ? That's just plain ugly, and I'm pretty sure that this is breaking your code all over the place. Now, I have no idea how this code could have even worked for you on another server, but I don't think that is the problem. Time to redesign your application in my opinion. Best Regards, Joao -- Joao Prado Maia Web Developer Kefta.com, Inc

« previous php.general (#13475) next »