2009-08-23 5 views
0

Les gars, je n'ai pas beaucoup de chance ... et j'ai essayé et essayé. Avoir inclus le code ci-dessous. Je fais cela dans Dreamweaver, d'où le code drôle. Ceci est la page d'édition. J'ai analysé avec succès la valeur 'bet_id' de la page 1 de cette page. Il remplit les champs de formulaire avec les valeurs correctes "bet_id" et "category_id" en fonction de la valeur analysée à partir de la page 1. Un problème survient lorsque je mets à jour les valeurs dans le formulaire. Si je mets à jour la valeur 'category_id' et clique sur le bouton Update bet, le script ne met pas à jour l'enregistrement de pari dans ma base de données. Toute aide très appréciée.PHP MYSQL Hyperliens et formulaires à nouveau

<?php require_once('../Connections/punters_c.php'); ?> 
<?php 
mysql_select_db($database_punters_c, $punters_c); 

$query_Recordset1  = "SELECT bet_id, punter_id,category_id FROM betslip where bet_id =".intval($_REQUEST['bet_id']); 
$Recordset1    = mysql_query($query_Recordset1, $punters_c) or die(mysql_error()); 
$row_Recordset1   = mysql_fetch_assoc($Recordset1); 
$totalRows_Recordset1 = mysql_num_rows($Recordset1); 


##the below function removes dodgy field values 

function GetSQLValueString($theValue, $theType, $theDefinedValue = "", $theNotDefinedValue = "") 
{ 
    $theValue = (!get_magic_quotes_gpc()) ? addslashes($theValue) : $theValue; 

    switch ($theType) { 
    case "text": 
     $theValue = ($theValue != "") ? "'" . $theValue . "'" : "NULL"; 
     break;  
    case "long": 
    case "int": 
     $theValue = ($theValue != "") ? intval($theValue) : "NULL"; 
     break; 
    case "double": 
     $theValue = ($theValue != "") ? "'" . doubleval($theValue) . "'" : "NULL"; 
     break; 
    case "date": 
     $theValue = ($theValue != "") ? "'" . $theValue . "'" : "NULL"; 
     break; 
    case "defined": 
     $theValue = ($theValue != "") ? $theDefinedValue : $theNotDefinedValue; 
     break; 
    } 
    return $theValue; 
} 
$editFormAction = $_SERVER['PHP_SELF']; 
if (isset($_SERVER['QUERY_STRING'])) { 
    $editFormAction .= "?" . htmlentities($_SERVER['QUERY_STRING']); 
} 
?> 
<? 
       // edit.php 
if ((isset($_POST["apply"])) && ($_POST["apply"] == "update_betslip_detail")){ 

     $query = sprintf(" UPDATE betslip 
          SET category_id = '%d' 
          WHERE bet_id = %d", 
          mysql_real_escape($_POST['category_id']), 
          mysql_real_escape($_POST['bet_id']) 
                   ); 

    mysql_select_db($database_punters_c, $punters_c); 
    $Result1 = mysql_query($query, $punters_c) or die('Connection error to MYSQL occurred: '.(mysql_error())); 

     header("Location: /update_betslip_test.php"); 

    } 
    else 
    { 
     echo "bet detail not updated"; 
    } 
    ?> 

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> 
<html xmlns="http://www.w3.org/1999/xhtml"> 
<head> 
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1" /> 
<title>Untitled Document</title> 
</head> 

<body> 
    <form action="<?php echo $editFormAction; ?>" method="POST" enctype="multipart/form-data" name="update_betslip_detail"> 

    <input type="text"  name="bet id"  id = "bet_id"  value="<?php echo $row_Recordset1['bet_id']; ?>"/> 
    <input type="text"  name="category_id" id = "category_id" value="<?php echo $row_Recordset1['category_id']; ?>"/> 


    <input type="hidden" name= "apply"  value="update_betslip_detail"/> 

    <input type="submit" value="Update bet"/> 
    </form> 

    <p><a href="update_betslip_test.php">Back to Update page </a></p> 
</body></html> 
<?php 
mysql_free_result($Recordset1); 
?> 
+0

vous devriez vraiment ranger cela, son orge lisible – 32423hjh32423

+0

Youre droite. Pardon. J'utilise un lien hypertexte sur un champ bet_id dans une table d'enregistrement de la page 1 pour afficher les détails de l'enregistrement (bet_id et category_id) dans un formulaire de la page 2. Je veux que ces champs de formulaire soient des champs modifiables peut modifier/mettre à jour dans la base de données en cliquant sur le bouton Mettre à jour le pari. J'espère que cela a du sens. –

+0

Doit également confirmer que c'est juste la mise à jour de la partie de la base de données avec laquelle j'ai des problèmes. –

Répondre

0

Tout d'abord, il n'y a pas de fonction appelée mysql_real_escape() si l'appelant devrait vous donner une erreur fatale (à moins que ce quelque chose que vous vous avez défini). Aussi: puisque vous utilisez sprintf(..., '%d'), en utilisant mysql_real_escape_string() (la fonction correcte pour échapper des chaînes) n'est pas nécessaire. Note (connexe mais pas liée à votre problème d'origine): vous ne devriez pas utiliser addslashes() dans votre fonction GetSQLValueString(). Cela aussi devrait être mysql_real_escape_string(). Le simple appel au addslashes() fonctionnera pour une entrée légitime (par exemple, tournez O'Brien en O\'Brien) mais cela ne fonctionnera pas pour certains types d'entrée malveillante.