Php - ce este gresit aici

[Mafios_X]

$server_exist = @mysql_num_rows(@mysql_query("SELECT * FROM `servers` WHERE `ip`='$ip';"));

Poate ip-ul mai este, dar poate nu acelasi port? Deci si port trebuie ! :-??

[Anonymous117]

$server_exist = @mysql_num_rows(@mysql_query("SELECT * FROM `servers` WHERE `ip`='$ip';"));

Poate ip-ul mai este, dar poate nu acelasi port? Deci si port trebuie ! :-??

De ce căcat folosiţi "@" ?

DE CE NU FOLOSIŢI LA QUERY ŞI LOG.. şi ce căcat caută ";" în query ?

[Haddaway]

Pui

Code: Select all

$server_exist = @mysql_num_rows(@mysql_query("SELECT * FROM `servers` WHERE `ip`='$ip' AND `port`='$port';"));

[Mafios_X]

Code: Select all

<?php
include("source/config.php");
if(!isset($_GET['data']))header("Location: http://www.site.ro");
// $datas = mysql_real_escape_string($_GET['data']);
$datas = $_GET['data'];
$data = explode(':',$datas);
$ip = $data[0];
$port = $data[1];
include("header.php");
?>

<div class="blocknew blocknew666">
      <div class="blocknewhdr">
         Sterge server <?=$ip;?>:<?=$port;?>
      </div>
<?php 
if($user['logged']==0)
{
echo '<form method="post" action="account/login/">
<input title="Username" style="width: 53px;" name="username" value="username" onfocus="javascript:inputHintHide(this, event, \'username\');" onmouseout="javascript:hoverPopupHide();" onmousemove="javascript:hoverPopupShow(this, event, \'Username\');" type="text">
<input title="Password" style="width: 53px;" name="password" value="" onfocus="javascript:inputHintHide(this, event, \'password\');" onmouseout="javascript:hoverPopupHide();" onmousemove="javascript:hoverPopupShow(this, event, \'Password\');" type="password">
<input title="Remember Me" checked="checked" name="cookieuser" value="1" onmouseout="javascript:hoverPopupHide();" onmousemove="javascript:hoverPopupShow(this, event, \'Remember Me\');" type="checkbox">
<input title="Log In" style="padding-bottom: 1px;" name="submit" value="LOG IN" src="/images/buton_login.png" alt="Log In" type="image">
</form>
<a style="color: white;" href="http://www.gamescan.ro/account/create/">
<img title="Sign Up" src="/images/buton_register.png" alt="Sign Up">
</a>';
}else
{

?>

<?php

$server_exist = @mysql_num_rows(@mysql_query("SELECT * FROM `servers` WHERE `ip`='$ip' AND `port`='$port';"));
if($server_exist=='1')
{
$q_utilizator = @mysql_query("SELECT `author` FROM `servers` WHERE `ip`='$ip' AND `port`='$port';");
while($q = @mysql_fetch_array($q_utilizator))
{
$q_utilizator = strtolower($q['utilizator']);
}

// vezi aici daca ai asa ceva
$utilizator = strtolower($user['name']);

if($q_utilizator==$utilizator)
{
$ok = mysql_query("DELETE FROM `servers` WHERE `ip`='$ip' AND `port`='$port'");
if($ok) {
?>
<div class="acces" align="center">
Server-ul <?=$ip;?> a fost sters din baza noastra de date!
</div>
<meta http-equiv="refresh" content="2; URL=account/">
<?}else {?>
Nu am putut sterge serverul
<? }

}else{
echo "<center>Nu aveti acces la  stergerea acestui server !</center>";
}
}else{
echo "Serverul nu exista in baza de date";
}
?>

<?php 

}

?>
<div class="item_h05">
</div>
</div>

   
<?php
include("footer.php");
?>

Merge doar ca, sterge si serverul care nu este al tau, practic nu verifica daca AUTHOR esti TU adica $user['name'] !

[MaimutaVirtuala]

O mare ciorba , trebuia lucrat cu sesiuni.

Code: Select all

<?php
//pornire sesiune..??

     //Conexiunea la database, nu?
     include("source/config.php");
     
     //debug request, vezi ce date iti vin si daca sunt goale.
     var_dump($_REQUEST);
     
     //daca request-ul nu este unul de tip get
     if(!$_SERVER['REQUEST_METHOD'] == 'GET'){
       //tipul de request nu este care trebuie?
       header('Location: www.adresa.ro');
     }


       $data = (isset($_GET['data'])) ? is_numeric($_GET['data']) : "null";
       
             $byte = explode(":", $data);
             
             $type_of_data = array(
                   'ip'    => $byte[0],
                   'port'  => $byte[1]
             );


?>

   <div class="blocknew blocknew666">
      <div class="blocknewhdr">
         Sterge server <?php print $type_of_data['ip'].':'.$type_of_data['port']; ?>
      </div>
      
<?php

     if($user['logged'] == 0){
       /*
       Afisezi form-ul de logare.
       Nu inteleg de ce nu lucrezi cu sesiuni sau cookies? Catalinu ciorba style?
       */
     } else {
       
            //verificam serverul
            $server_existency_table = mysql_query("SELECT `ip`,`port` FROM `servers` WHERE
            `ip` = '".$type_of_data['ip']."' AND `port` = '".$type_of_data['port']."'") or die(mysql_error());
            //or die()? and why? God know...or mysql team.
            
                 if(mysql_num_rows($server_existency_table > 0){
                   //Server-ul exista si facem inca o interogare in tr-unul din tabele.
                   $ownerof_server = mysql_query("SELECT `author` FROM `servers` WHERE
                   `ip` = '".$type_of_data['ip']."' AND `port` = '".$type_of_data['port']."'") or die(mysql_error());
                   //Si din nou :or die()? and why? God know...or mysql team.
                   
                   //Nu inteleg de ce a pus while(loop) cand trebuie sa extragi datele doar la un singur user si nu mai multi..
                   $data_fetch = mysql_fetch_assoc($ownerof_server);
                   $utilizator_server = strtolower($data_fetch['utilizator']);
                   
                      if($utilizator_server = //Aici at trebui sesiunea userului dar nu cred ca a-ti auzit de asta sau cel putin programatorul acestei ciorbe){
                         $server_deleted = mysql_query("DELETE FROM `servers` WHERE `ip` = '".$type_of_data['ip']."' AND `port` = '".$type_of_data['port']."'") or die(mysql_error());
                      }

                 } else {
                   //Nu exista?
                   print 'Serverul nu exista in baza de date.';

                 }

?>