Witam.
Stworzyłem sobie formularz który wyświetla dane użytkowników. Chcę by po zmianie w polach i naciśnięciu wyslij aktualizował wszystkie rekordy. Jak na razie aktualizuje ostatni rekord. Myślę że błąd robię w zapytaniu update. Prosze o pomoc.

[PHP] pobierz, plaintext 
 
<?php require_once('../Connections/myconn.php'); ?>
<?php
if (!function_exists("GetSQLValueString")) {
function GetSQLValueString($theValue, $theType, $theDefinedValue = "", $theNotDefinedValue = "") 
{
  if (PHP_VERSION < 6) {
    $theValue = get_magic_quotes_gpc() ? stripslashes($theValue) : $theValue;
  }
 
  $theValue = function_exists("mysql_real_escape_string") ? mysql_real_escape_string($theValue) : mysql_escape_string($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']);
}
 
if ((isset($_POST["MM_update"])) && ($_POST["MM_update"] == "form1")) 
{
  $updateSQL = sprintf("UPDATE admin SET name=%s, email=%s WHERE id=%s",
                       GetSQLValueString($_POST['name'], "text"),
                       GetSQLValueString($_POST['email'], "text"),
                       GetSQLValueString($_POST['id'], "int"));
 
  mysql_select_db($database_myconn, $myconn);
  $Result1 = mysql_query($updateSQL, $myconn) or die(mysql_error());
 
  $updateGoTo = "reba.php";
  if (isset($_SERVER['QUERY_STRING'])) {
    $updateGoTo .= (strpos($updateGoTo, '?')) ? "&" : "?";
    $updateGoTo .= $_SERVER['QUERY_STRING'];
  }
  header(sprintf("Location: %s", $updateGoTo));
}
 
mysql_select_db($database_myconn, $myconn);
$query_Recordset1 = "SELECT * FROM `admin`";
$Recordset1 = mysql_query($query_Recordset1, $myconn) or die(mysql_error());
$row_Recordset1 = mysql_fetch_assoc($Recordset1);
$totalRows_Recordset1 = mysql_num_rows($Recordset1);
?>
<form name="form1" method="POST" action="<?php echo $editFormAction; ?>">
  <p>&nbsp;</p>
  <table width="200" border="0" align="center" cellpadding="0" cellspacing="0">
    <tr>
      <td align="center"><strong>Id</strong></td>
      <td align="center"><strong>Name</strong></td>
      <td align="center"><strong>Email</strong></td>
    </tr>
    <?php do { ?>
      <tr>
        <td><label for="id"></label>
        <input name="id" type="text" id="id" value="<?php echo $row_Recordset1['id']; ?>" size="15" maxlength="15"></td>
        <td><label for="name"></label>
        <input name="name" type="text" id="name" value="<?php echo $row_Recordset1['name']; ?>" size="15" maxlength="15"></td>
        <td><label for="email"></label>
        <input name="email" type="text" id="email" value="<?php echo $row_Recordset1['email']; ?>" size="15" maxlength="15"></td>
      </tr>
      <input name="id" type="hidden" id="id" value="<?php echo $row_Recordset1['id']; ?>">
      <?php } while ($row_Recordset1 = mysql_fetch_assoc($Recordset1)); ?>
  </table>
  <p>
    <input type="submit" name="button" id="button" value="Wyślij">
  </p>
  <p>&nbsp;</p>
  <input type="hidden" name="MM_update" value="form1">
</form>
<?php
mysql_free_result($Recordset1);
?>
 
 
[PHP] pobierz, plaintext 

No ale kazde pole ma miec swoje unikalne NAME tudziez NAME tablicowe a nie ty kazdemu ID przypisujesz NAME ID. PRzeciez one sie nadpisuja co latwo mozesz zobaczyc robiac print_r($_POST);

Czyli coś takiego zrobić name=$name, email=$email WHERE id=%s

W formularzu pola mają miec unikalne nazwy dla kazdego rekordu albo zrob te pola jako tablice
name="id[]"
itd

Zrobiłem coś takiego $updateSQL = sprintf("UPDATE admin SET name='name[]' , email='email[]' WHERE id=%s",
a w formularzu
<td><label for="name[]"></label>
<input name="name[]" type="text" id="name[]" value="<?php echo $row_Recordset1['name']; ?>" size="15" maxlength="15"></td>
<td><label for="email[]"></label>
<input name="email[]" type="text" id="email[]" value="<?php echo $row_Recordset1['email']; ?>" size="15" maxlength="15"></td>
</tr>
<input name="id" type="hidden" id="id" value="<?php echo $row_Recordset1['id']; ?>">
Ale teraz to juz nic nie update

@nospor: Proszę się poprawić: http://sjp.pl/tudzie%C5%BC
@woxala123: Popraw swój formularz, tak by w każdym wierszu znajdowały się pola formularza o nazwie przykładowo aktualizacja[%ID%][%POLE%], gdzie %ID% (z bazy danych) oraz %pole% (email/nazwa) są zmienne. Następnie po przesłaniu formularza daj sobie print_r($_POST) jak już zauważył @nospor - będziesz wiedział jak obchodzić się z danymi.

No przeciez dobrze napisalem.... no chyba ze patrzymy na dwa rozne wyrazy.

'name[]'
Przeciez ja ci to kazalem tylko w formie dawac a nie w zapytaniu.....
zrobze wkoncu print_r($_POST); i zobacz jak obsluzyc dane po zmanie co ci kazalem zrobic.

CZy napisac cos takiego
"UPDATE admin SET name=$name, email=$email WHERE id=$id" ,

W forularzu poprawiłem na

<?php do { ?>
<tr>
<td><label for="id[]"></label>
<input name="id[]" type="text" id="id[]" value="<?php echo $row_Recordset1['id']; ?>" size="15" maxlength="15"></td>
<td><label for="name[]"></label>
<input name="name[]" type="text" id="name[]" value="<?php echo $row_Recordset1['name']; ?>" size="15" maxlength="15"></td>
<td><label for="email[]"></label>
<input name="email[]" type="text" id="email[]" value="<?php echo $row_Recordset1['email']; ?>" size="15" maxlength="15"></td>
</tr>
<input name="id" type="hidden" id="id" value="<?php echo $row_Recordset1['id']; ?>">
<?php } while ($row_Recordset1 = mysql_fetch_assoc($Recordset1)); ?>
I po print_r($_POST); nie aktualizuje pól tylko robią się puste

Przy pomocy print_r($_POST); miales zobaczyc na wlasne oczy co idzie z forma a nie przy jego pomocy aktualizowac dane :/

Przenosze na przedszkole bo poziom twojej wiedzy jest na tym wlasnie poziomie

W takim razie dobrze zdefiniowałem pola?

[PHP] pobierz, plaintext 
<?php require_once('../Connections/myconn.php'); ?>
<?php
print_r($_POST);
if (!function_exists("GetSQLValueString")) {
function GetSQLValueString($theValue, $theType, $theDefinedValue = "", $theNotDefinedValue = "") 
{
  if (PHP_VERSION < 6) {
    $theValue = get_magic_quotes_gpc() ? stripslashes($theValue) : $theValue;
  }
 
  $theValue = function_exists("mysql_real_escape_string") ? mysql_real_escape_string($theValue) : mysql_escape_string($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']);
}
 
if ((isset($_POST["MM_update"])) && ($_POST["MM_update"] == "form1")) 
{
foreach ($_POST['id'] as $item) {
  $updateSQL = sprintf("UPDATE admin SET name=%s, email=%s WHERE id=%s",
                      GetSQLValueString($_POST['name'][$item], "text"),
                      GetSQLValueString($_POST['email'][$item], "text"),
                      GetSQLValueString($item, "int"));
 
  mysql_select_db($database_myconn, $myconn);
  $Result1 = mysql_query($updateSQL, $myconn) or die(mysql_error());
}
  $updateGoTo = "reba.php";
  if (isset($_SERVER['QUERY_STRING'])) {
    $updateGoTo .= (strpos($updateGoTo, '?')) ? "&" : "?";
    $updateGoTo .= $_SERVER['QUERY_STRING'];
  }
  header(sprintf("Location: %s", $updateGoTo));
}
 
mysql_select_db($database_myconn, $myconn);
$query_Recordset1 = "SELECT * FROM `admin`";
$Recordset1 = mysql_query($query_Recordset1, $myconn) or die(mysql_error());
$row_Recordset1 = mysql_fetch_assoc($Recordset1);
$totalRows_Recordset1 = mysql_num_rows($Recordset1);
?>
<form name="form1" method="POST" action="<?php echo $editFormAction; ?>">
  <p>&nbsp;</p>
  <table width="200" border="0" align="center" cellpadding="0" cellspacing="0">
    <tr>
      <td align="center"><strong>Id</strong></td>
      <td align="center"><strong>Name</strong></td>
      <td align="center"><strong>Email</strong></td>
    </tr>
    <?php do { ?>
      <tr>
        <td><label for="id[]"></label>
        <input name="id[]" type="text" id="id[]" value="<?php echo $row_Recordset1['id']; ?>" size="15" maxlength="15"></td>
        <td><label for="name[]"></label>
        <input name="name[<?php echo $row_Recordset1['id']; ?>]" type="text" id="name[]" value="<?php echo $row_Recordset1['name']; ?>" size="15" maxlength="15"></td>
        <td><label for="email[]"></label>
        <input name="email[<?php echo $row_Recordset1['id']; ?>]" type="text" id="email[]" value="<?php echo $row_Recordset1['email']; ?>" size="15" maxlength="15"></td>
      </tr>
      <input name="id_hd[]" type="hidden" id="id_hd[]" value="<?php echo $row_Recordset1['id']; ?>">
      <?php } while ($row_Recordset1 = mysql_fetch_assoc($Recordset1)); ?>
  </table>
  <p>
    <input type="submit" name="button" id="button" value="Wyślij">
  </p>
  <p>&nbsp;</p>
  <input type="hidden" name="MM_update" value="form1">
</form>
<?php
mysql_free_result($Recordset1);
?>
[PHP] pobierz, plaintext 

Super. Wielkie dziękuje i dodaje Ciebie do moich guru php.

Wyraz napisałeś poprawnie, tylko sensu on nie ma. Sprawdź sobie jeszcze raz tą stronę i swój post - jest to dosyć popularny błąd.

edit by nospor: no tak