PHPhulp.nl PHPhulp.nl
35k+ leden Over PHPhulp Offerte opdracht Contact
S

Waar zit het probleem ?

Door Signs op 09-03-2007 12:41
275 views
Beste mensen,

Ik heb een probleem met mijn website.
Het is beter uit te leggen als jullie het probleem zien.

http://www.spelletjesplaats.com (geen SPAM)

1) Bij alle spellen loopt de URL via /spellen/spelnaam.html
2) Bij "Meest gespeeld" en "nieuwste spellen" gaat deze via
index.php?=game.php&id= blabla

Als ik een spel die via nummer 1 aanklik, dan opent zich een nieuw venster en de iframe blijft wit.

Als ik een spel die via nummer 2 loopt aanklikt, dan gebeurt er helemaal niets. Het enige wat veranderd is de url in de adresbalk van IE.

Het lijkt erop dat de vorige eigenaar de urls SEO vriendelijk wilde maken. Ik heb hem benaderd hierover maar hij gebruikte geen .htaccess dingen of iets dergelijks.

Overigens werkte de website op zijn hoster perfect

Of het nu via 1 of 2 loopt maakt mij niets uit, als het probleem maar opgelost wordt.


Wat kan het probleem zijn ?

Ik moet er wel bij vertellen dat ik deze website onlangs heb overgenomen. Ik heb een DirectAdmin backup zip bestand ontvangen en deze geupped naar mijn eigen account.

Hopelijk zijn jullie mij bereid te helpen. Mochten jullie stukken code willen zien dan regel ik dat

Met vriendelijke groet

Kevin
Joren de Wit
09-03-2007 12:56
Stukken code zijn natuurlijk altijd handig. Maar natuurlijk wel alleen de relevante stukken ;)
S
Signs
09-03-2007 13:03
hehe uiteraard

Code van config.php


<?php 
ob_start(); 

include "connect.php";

$select = ("SELECT * FROM game_config"); // Selecteren uit een tabel
$query = mysql_query($select); 
while($list = mysql_fetch_object($query)){
$webmaster_email = "$list->email_webmaster";
$website_naam = "$list->site_naam";
$website_adres = "$list->site_adres";

$screen = "$list->screenshot";
$preloader1 = "$list->preloader";
$berichtenbalk1 = "$list->berichtenbalk";
$poll1 = "$list->poll";
$maxgames1 = "$list->maxgames";
$titelbalk1 = "$list->titelbalk";

$login = "$list->gebruiker";
$pw = "$list->wachtwoord";
}

?>


Code van Game.php



<?php include"header.inc.php";?>
<td valign="top" class="links">

<?php
$select = ("SELECT * FROM game_games WHERE id ='$id' limit 1"); // Selecteren uit een tabel
$query = mysql_query($select); 
$aantal = mysql_num_rows($query);
if ( $aantal == "0" )
{
echo "Deze game id bestaat niet.";
}
else
{
while($list = mysql_fetch_object($query)){
echo "<div class='kop'>$list->name</div>";

?>

<table border="0" cellpadding="0" cellspacing="0" align="center" class="boxmainm">
<tr>
<td class="boxmainb"></td>
</tr>
<tr>
<td align="center" valign="top">
<a href="http://www.beltonenplaza.com/"><b>Beltonen</b></a><br />
<script type="text/javascript"><!--
google_ad_client = "pub-6658887532390474";
google_ad_width = 728;
google_ad_height = 90;
google_ad_format = "728x90_as";
google_ad_type = "text_image";
google_ad_channel ="";
google_color_border = "F0C581";
google_color_bg = "F0C581";
google_color_link = "412800";
google_color_url = "412800";
google_color_text = "000000";
//--></script>
<script type="text/javascript"
  src="http://pagead2.googlesyndication.com/pagead/show_ads.js">
</script>

<?
echo "<br />
<center><iframe src='laden.php?id=$list->id' name='game' align='middle' width='$list->width' height='$list->height' marginheight='0' marginwidth='0' frameborder='0' scrolling='no' style='background-color: red; '></iframe></center>";
?>

<script type="text/javascript"><!--
google_ad_client = "pub-6658887532390474";
google_ad_width = 728;
google_ad_height = 15;
google_ad_format = "728x15_0ads_al_s";
google_ad_channel ="";
google_color_border = "F0C581";
google_color_bg = "F0C581";
google_color_link = "412800";
google_color_url = "412800";
google_color_text = "000000";
//--></script>
<script type="text/javascript"
  src="http://pagead2.googlesyndication.com/pagead/show_ads.js">
</script>

</td>
</tr>
<tr>
<td class="boxmaino"></td>
</tr>
</table>

<div class='kop'>Statistieken</div>
<table border="0" cellpadding="0" cellspacing="0" align="center" class="boxmainm">
<tr>
<td class="boxmainb"></td>
</tr>
<tr>
<td align="center" valign="top">

<?

echo "Deze game is totaal <b>$list->views</b> keer gespeeld.<br />";
$cijfer = "$list->cijfer1";
$stem = "$list->stemmen";
$stem1=$cijfer/$stem;
$stem2=$stem1*10;

if ( $cijfer == "0" )
{
echo "Er is nog niet gestemd op deze game.<br />";
}
else
{
echo "Deze game heeft een rating van <b>".round($stem2,0)."</b> op 10.<br />";
}

$game1 = "game_$list->id";

$game1=$_COOKIE["$game1"]; 
if ( $send == "2" )
{
echo "Er is een bericht verstuurd naar de webmaster, met mededeling dat deze game niet werk!";
}
else
{
echo "Klik <a href='?p=game.php&amp;id=$list->id&amp;naam=$list->name&amp;send=2'>hier</a> als deze game niet werkt.<br />";
}

if ( $game1 == "1" )
{
echo "U hebt al gestemd op deze game.";
}

else
{
if ( $send == "1" )
{

echo "U stem is succesvol toegevoegd";
}
else
{
echo "<center><br /><form action='?p=game.php&amp;id=$id&amp;send=1' method='post'>1 <input type='radio' value='1' name='keuze' />  2 <input type='radio' value='2' name='keuze' />  3 <input type='radio' value='3' name='keuze' />  4 <input type='radio' value='4' name='keuze' />  5 <input type='radio' value='5' name='keuze' />  6 <input type='radio' value='6' name='keuze' />  7 <input type='radio' value='7' name='keuze' />  8 <input type='radio' value='8' name='keuze' />  9 <input type='radio' value='9' name='keuze' />  10 <input type='radio' value='10' name='keuze' /> <input type='submit' value='Stem' /></form></center>";
}
}

}
}
?>


</td>
</tr>
<tr>
<td class="boxmaino"></td>
</tr>
</table>

<div class='kop'>Advertenties</div>
<table border="0" cellpadding="0" cellspacing="0" align="center" class="boxmainm">
<tr>
<td class="boxmainb"></td>
</tr>
<tr>
<td align="center" valign="top">
<?php include "banner_routatie.php"; ?>
</td>
</tr>
<tr>
<td class="boxmaino"></td>
</tr>
</table>
</td>

<?php include"footer.inc.php";?>


Code van laden.php


<?php
ob_start();
?>
<body bgcolor='EBEBEB'>
<?php
include "config.php";
$select = ("SELECT * FROM game_games WHERE id ='$id' limit 1"); // Selecteren uit een tabel
$query = mysql_query($select); 
$aantal = mysql_num_rows($query);
while($list = mysql_fetch_object($query)){
$url = "$list->url";

if ( $preloader1 == "Aan" )
{
?>

<div align="center">
<script> 
getal=0; 
url='<?php echo "$list->url"; ?>'; // hier de url welke geopend moet worden
function load() 
{ 
if(getal==100) 
{ 
window.location=url; 
} 
else { 
getal+=1; 
tabel.width=getal; 
text.innerHTML=getal+"%"; 
setTimeout("load()","10"); 
} 
} 
setTimeout("load()","10"); 
</script> 
Loading... 
<div id=text></div> 
<table cellpadding=0 cellspacing=0 width=102 style="border:1 solid black"> 
<tr><td align=left> 
<table cellpadding=0 cellspacing=0 bgcolor=#0000CC height=11 id=tabel> 
<tr><td></tr></td></table></tr></td></table> 
</div><br><br>
<?php 
$select = ("SELECT * FROM game_adverteer WHERE plaats ='preloader'"); // Selecteren uit een tabel
$query = mysql_query($select); 
$aantal = mysql_num_rows($query);
while($list = mysql_fetch_object($query)){
echo "<center><a href='uitklik.php?id=$list->id' target='blank'><img src='$list->banner' border='0'></a>";
}

}
else
{
header ("Location: $url");
}
}
?>
Joren de Wit
09-03-2007 13:07
Zowel op game.php als laden.php gebruik je in je eerste query de variabele $id. Maar deze variabele heb je nog nergens gedeclareerd?

Hoogstwaarschijnlijk werkte de scripts wel bij de vorige host als daar register_globals aan stond. De GET variabele 'id' kan dan gewoon met $id uitgelezen worden. Op jouw server staat dat waarschijnlijk uit en zul je $_GET['id'] moeten gebruiken om die variabele uit te lezen.
J
Jacco Engel
09-03-2007 13:18
Het makkelijkste is .htaccess aanmaken met de code :

register_globals on


om te testen of dit het probleem oplost
F
Frank -
09-03-2007 13:18
Zet de volgende regels bovenaan de scripts en ga dan debuggen:
<?php
ini_set('display_errors', 1);
error_reporting(E_ALL | E_STRICT);

// rest van het script
?>

Verder ontbreekt iedere vorm van foutafhandeling, de basis van een goed script.
S
Signs
09-03-2007 13:21
je bent geweldig! Ik ben geen php /sql expert...dus het is voor mij wat lastig.

Ik heb nu een .htaccess bestand gemaakt met de code
php_flag register_globals on

nu werk mijn site wel! DANK!!!
Joren de Wit
09-03-2007 13:26
Ja en dit is dus de minst slimme actie die je had kunnen ondernemen! Register_globals staat standaard niet voor niets uit omdat het bij niet goed gescripte scripts een potentieel beveiligingslek oplevert.

Kortom, lekker met je handjes van die functie afblijven en je script op een goede manier scripten.
F
Frank -
09-03-2007 13:27
Ik heb nu een .htaccess bestand gemaakt met de code
php_flag register_globals on
En dat noem jij een oplossing? Hiermee gooi je je eigen ruiten in, het begrip veiligheid heb je nu met een grote boog de deur uitgegooid. Waarom zou dit toch zijn uitgezet? Google daar maar eens op en probeer dan nog eens vrolijk te kijken...

Ga netjes scripten, dat levert veel betere resultaten op.
S
Signs
09-03-2007 13:37
oke dan...mm misschien inderdaad niet zo'n slimme actie.

Ik ben een totale newbie op php/sql gebied, dit leek mij wel een makkelijke oplossing..

Blanche, in je topic vermelde je dat ik $_GET moet invoeren.
Heb je een stuk code als voorbeeld waar ik dit moet invoeren ?
J
Jurgen assaasas
09-03-2007 13:37
gewoon in replace all met een simpele editor $id veranderen in $_GET['id']

Reageren

Inloggen om te reageren