PHP, rozwijana lista i przesyłanie danych

Question

Witam,

Staram się stworzyć listę która będzie łączyć się z bazą danych a następnie dane wybrane z tej bazy będą przesyłane do innej bazy, problem mam taki że po wybraniu danych z listy i przesłaniu ich do bazy one nie wyświetlają się w bazie, w bazie danych jest pusty rekord.

tak wygląda mój kod 

<form action="test.php" method="POST">
<select name="opcja_sel" style="width:100px" >
        <?php
        include "db1.php";

        $kat_query = mysqli_query($db,"SELECT Nazwa FROM zamowienie");

        while($zamowienie = mysqli_fetch_array($kat_query))
        {
        echo'
        <option value="opcja">'.$zamowienie['Nazwa'].'
        </option>';
        }
        ?>
        </select>

<button type="submit" name="submit" >Submit</button>
</form>

a tak wygląda plik test.php

<?php
include "db1.php";
if(isset($_POST['submit']))
{
    $opcja = $_POST['opcja_sel'];
    $insert = mysqli_query($db,"INSERT INTO `Home`(`opcja`) VALUES ('$opcja_sel')");
    if(!$insert)
    {
        echo mysqli_error();
    }
    else
    {
         mysqli_close($db);
         header("location:home.php");
         exit;
    }
}
mysqli_close($db);
?>

 

Answer 1

Tu masz jeszcze błąd

$opcja = $_POST['opcja'];

powinno być

$opcja = $_POST['opcja_sel'];

bo masz

<select name="opcja_sel" style="width:100px" >

 

<?php
include "db1.php";
if(isset($_POST['submit']))
{
    $opcja = $_POST['opcja_sel'];
    $insert = mysqli_query($db,"INSERT INTO `Home`(`opcja`) VALUES ('$opcja')");
    if(!$insert)
    {
        echo mysqli_error();
    }
    else
    {
         mysqli_close($db);
         header("location:home.php");
         exit;
    }
}
mysqli_close($db);
?>

 

Comments

Ale to w takim razie w tym miejscu też powinien dodać opcja_sel

    $insert = mysqli_query($db,"INSERT INTO `Home`(`opcja`) VALUES ('$opcja')");

Chodzi mi o VALUES żeby opcja zmienić na opcja_sel

---

$opcja_sel= $_POST['opcja_sel'];
    $insert = mysqli_query($db,"INSERT INTO `Home`(`opcja`) VALUES ('$opcja_sel')");

 

Answer 2

W sensie, że jak zaglądasz do bazy danych, to tam wpis jest pusty? Wygląda na to, że tutaj masz złą nazwę zmiennej:

$insert = mysqli_query($db,"INSERT INTO `Home`(`opcja`) VALUES ('$opcja_sel')");

bo wyżej masz pobraną wartość do zmiennej $opcja, a nie $opcja_sel. 

A tak przy okazji to brakuje tu zabezpieczenia chociażby np. przed sql injection.

Comments

<?php
include "db1.php";
if(isset($_POST['submit']))
{
    $opcja = $_POST['opcja'];
    $insert = mysqli_query($db,"INSERT INTO `Home`(`opcja`) VALUES ('$opcja')");
    if(!$insert)
    {
        echo mysqli_error();
    }
    else
    {
         mysqli_close($db);
         header("location:home.php");
         exit;
    }
}
mysqli_close($db);
?>

zmieniłem na ten sposób 

ale nadal nie działą

---

Sprawdź co jest "przysyłane" do test.php

<?php
  var_dump($_POST);
?>

 

Answer 3

Jest progress xd 

Teraz przynajmniej jest coś dodawane do bazy danych, ale niestety nie jest to wartość wybierana z listy 

 

Comments

Do PHP i bazy trafi value z wybranej opcji, które u Ciebie zawsze jest ustawione na "opcja"

echo'
        <option value="opcja">'.$zamowienie['Nazwa'].'
        </option>';

 

---

@Grozmo,

TO co mam tam wpisać?

teraz usunąłem z value "opcja" i jest teraz tak

<option value="">'.$zamowienie['Nazwa'].'
        </option>';

Ale teraz znów nic nie jest dodawane do bazy, jest dodawany pusty rekord

 

---

To dlatego, że masz puste value ;) Jak chcesz, aby było zapisywane i dodawane bezpośrednio to samo, co jest wyświetlane jako nazwa opcji, to wstaw to samo w atrybut value.

---

Nadal nie wiem co w końcu tam wstawić

---

No tak jak napisałem - jak chcesz, aby do PHP i finalnie bazy trafiały wartości wprost takie, jakie są na liście, to trzeba po prostu w value podać te nazwy, coś w rodzaju:

echo '<option value="'.$zamowienie['Nazwa'].'">'.$zamowienie['Nazwa'].'</option>';

---

O kuźwa działa

Dzięki mordo  masz piwo u mnie

---

Powiedz mi o co jeszcze chodziło tobie z tym? "A tak przy okazji to brakuje tu zabezpieczenia chociażby np. przed sql injection"

---

Ktoś z odrobiną technicznej wiedzy i dostępem do tego formularza może próbować manipulować zapytaniem do bazy, a więc finalnie wyciągnąć więcej danych, usunąć coś itd. Nawet na Wikipedii jest to całkiem sensownie pokazane: https://pl.wikipedia.org/wiki/SQL_injection#Niedostateczne_filtrowanie_danych Aby się zabezpieczyć należałoby bindować wartości, a nie po prostu wstawiać je do zapytania sklejając tekst.