[MySQL] & [PHP] updating rows

[X-DraGoN]

Ik heb een lijst van namen die ik uit mysql haal via PHP


Ik presenteer die in een selectable lijst op de volgende manier

PHP:

<form method="post" action="update_rows.php">
  <table>
    <thead>
      <tr>
        <th>Select</th>
        <th>Naam</th>
        <th>Beurten</th>
      </tr>
    </thead>
    <tbody>
      <?php
        // Connect to the MySQL database
        $db = new mysqli('***knip***');

        // Retrieve the rows from the database
        $result = $db->query("SELECT leden_id, leden_naam, leden_beurten_left FROM leden_tbl ORDER BY leden_naam ASC");

        // Loop through the rows and create a checkbox for each one
        while ($row = $result->fetch_assoc()) {
          echo '<tr>';
          echo '<td><input type="checkbox" name="selected_rows[]" value="' . '"></td>';
          echo '<td>' . $row['leden_naam'] . '</td>';
          echo '<td>' . $row['leden_beurten_left'] . '</td>';
          echo '</tr>';
        }
      ?>
    </tbody>

Ik stuur deze lijst naar update_rows.php met de volgende inhoud:

PHP:

<?php
  // Connect to the MySQL database
  $db = new mysqli('l***knip***');

  // Retrieve the selected rows from the form data
  $selected_rows = $_POST['selected_rows'];

  // Loop through the selected rows and update the desired column
  foreach ($selected_rows as $row_id) {
    $db->query("UPDATE leden_tbl SET leden_beurten_left = leden_beurten_left -1 WHERE id = $row_id$");
  }

  // Display a confirmation message
  echo count($selected_rows) . " row(s) have been updated.";
?>

Ik krijg mooi een schermpje dat zegt x aantal rows updated, maar hij update niets in de tabel.
Hoe kan ik er voor zorgen dat hij de juiste rijen die ik aanduid, effectief gaat updaten?

Ik heb ook een 'leden_id' in mijn tabel, kan in die gebruiken om de juiste aangeduide regel door te geven?
Hoe zou die dan in de code passen?

[ Voor 0% gewijzigd door X-DraGoN op 31-03-2023 16:35 . Reden: fout mee gekopieerd ]

[Groentjuh]

Geen idee of autocommit aanstaat, maar daar zou ik even naar kijken.
Is de input array wel gevuld met IDs?
En misschien een lesje prepared sql statements volgen?

[ Voor 42% gewijzigd door Groentjuh op 31-03-2023 16:22 ]

[X-DraGoN]

Groentjuh schreef op vrijdag 31 maart 2023 @ 16:18:
Geen idee of autocommit aanstaat, maar daar zou ik even naar kijken.
En misschien een lesje prepared sql statements volgen?

Autocommit staat op "TRUE", dat lijkt me geen probleem te zijn

[Groentjuh]

var_dump($_POST['selected_rows']) ; eens? Staan daar de ids in? En geeft de query functie true terug?

[D4NG3R]

Je telt overigens hoe veel entries er in 'selected_rows' zitten, niet hoe vaak de gegeven query ook daadwerkelijk goed gaat. Je echo met X aantal updates zegt dan ook helemaal noppes.

[3raser]

code:

WHERE id = $row_id$

Ik werk niet regelmatig meer in PHP, maar waar is die laatste $ voor? Is de query niet gewoon fout?

[CodeCaster]

Je print het aantal items dat er gePOST wordt, maar je negeert het resultaat van de query. Print de query eens? Er staat een dollar achter.

En ja, je kunt ook andere kolommen opnemen in de WHERE, maar dan moet je de waarden daarvan POSTen in plaats van de IDs.

Edit: de value van je input is leeg; je zet er niets in (value="' . '").

Maar zoals gezegd: ga vooral prepared statements gebruiken.

[ Voor 51% gewijzigd door CodeCaster op 31-03-2023 16:33 ]

[Juup]

Potentieel gevaarlijke code met die $row_id zo in de string.
Beter placeholders gebruiken.

[RobIII]

3raser schreef op vrijdag 31 maart 2023 @ 16:27:

code:

WHERE id = $row_id$

Ik werk niet regelmatig meer in PHP, maar waar is die laatste $ voor? Is de query niet gewoon fout?

Ik gok dat TS een cat foo.php heeft gedaan en dat de regel daar op de terminal afgebroken is en TS dat heeft mee-gekopieerd

Edit: Oh, wacht, daar staat nog een "); achter, dus dat is 't niet. Dat is 't (mogelijk) wél bij ORDER BY leden_naam ASC$.

[ Voor 16% gewijzigd door RobIII op 31-03-2023 16:31 ]

[X-DraGoN]

RobIII schreef op vrijdag 31 maart 2023 @ 16:30:
[...]

Ik gok dat TS een cat foo.php heeft gedaan en dat de regel daar op de terminal afgebroken is en TS dat heeft mee-gekopieerd

Edit: Oh, wacht, daar staat nog een "); achter, dus dat is 't niet. Dat is 't (mogelijk) wél bij ORDER BY leden_naam ASC$.

Dat is idd fout gekopieerd, ik pas het even aan in de TS zelf

[StevenK]

Wat stuur je precies naar die update_rows.php? Is dat wel een lijst van ID's?

[X-DraGoN]

StevenK schreef op vrijdag 31 maart 2023 @ 16:38:
Wat stuur je precies naar die update_rows.php? Is dat wel een lijst van ID's?

Ik vermoed van niet, maar hoe stuur ik dan welke rijen ik heb aangeduid door naar update_rows.php?
En moet ik daar de 'leden_id' die ik mijn DB hebben zitten voor gebruiken?

[Cyphax]

X-DraGoN schreef op vrijdag 31 maart 2023 @ 16:42:
[...]

Ik vermoed van niet, maar hoe stuur ik dan welke rijen ik heb aangeduid door naar update_rows.php?
En moet ik daar de 'leden_id' die ik mijn DB hebben zitten voor gebruiken?

Dat kan je doen, ze op basis van id identificeren, maar dan moet je die id's nog ergens plaatsen, op de plek die @CodeCaster al aanwees. De value van de input zou vervolgens een lijst van id's moeten vormen die je met $_POST['selected_rows']; weer uitleest. Momenteel is die lijst leeg, omdat de value van de input in eerste instantie niet wordt gevuld.

[brambo123]

Sowieso heeft je checkbox geen value, en in je sql update gebruik je 'id' i.p.v. 'leden_id'
Checkbox moet natuurlijk zijn:

PHP:

echo '<td><input type="checkbox" name="selected_rows[]" value="' . $row['leden_id'] . '"></td>';

En gebruik even netjes prepare en execute voor je SQL.
Dan krijg je zoiets:

PHP:

$sql = $db->prepare("UPDATE `leden_tbl` SET `leden_beurten_left` = `leden_beurten_left`-1 WHERE `leden_id` = :rowid");
foreach ($selected_rows as $row_id) {
    $sql->execute([':rowid' => $row_id]);
}
$sql->closeCursor();

[X-DraGoN]

Bedankt voor de suggesties, ik ga er mee aan de slag

[Josk79]

Dit is een goede source om dingen niet fout te gaan doen: https://phptherightway.com/#mysql_extension