Problem z edycją rekordu na stronie. Opcje

[Play12]

Witajcie, krótko i na temat, napisałem skrypt i nie śmiga. Mógłby ktoś nakierować na błąd?

Wszystko elegancko, okno modalne wyświetla, dane z bazy danych pobiera ale gdy zmienie to co chce i klikam "zapisz" to nie zapisuje tego tylko informacja że "nie powiodło się", co źle robię? zapis nie kuma co ma zmienić, tzn. jaki rekord ? bo to logicznie przychodzi mi tylko do głowy, ale nie wiem, może ktoś kumaty by się wypowiedział by było fajnie

[PHP] pobierz, plaintext 
    <?php
 
    if(isset($_POST['save']))
 
 
 
        $id=$_POST['id'];
        $product_picture=$_POST['product_picture'];
        $product_name=$_POST['product_name'];
        $catalog_number=$_POST['catalog_number'];
        $additional_information=$_POST['additional_information'];
        $quantity=$_POST['quantity'];
        $price=$_POST['price'];
 
 
 
        $sql = mysql_query("UPDATE zdjecia SET id = '$id' product_picture = '$product_picture', product_name = '$product_name', catalog_number = '$catalog_number', additional_information = '$additional_information', quantity = '$quantity', price = '$price'");
 
 
            $wynik = mysql_query($sql);
            if ($wynik) {
                echo 'Prawidłowo dodano do bazy danych';   
            }
            else
                echo "nie powiodlo sie";
    }
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
    <button type="button" class="btn btn-primary" data-toggle="modal" data-target="#'.$tabela['id'].'">Edytuj</button>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
        <div class="modal fade" id="'.$tabela['id'].'" tabindex="-1" role="dialog" aria-labelledby="exampleModalCenterTitle" aria-hidden="true">
            <div class="modal-dialog modal-lg">
                <div class="modal-content">
                    <div class="modal-header">
                        <form name="form0" method="POST" action="">
                        <h5 class="modal-title" id="exampleModalLongTitle"><input class="form-control" name="product_name" id="product_name" value="'.$tabela['product_name'].'" /></h5>
                        <button type="button" class="close" data-dismiss="modal" aria-label="Close">
                        <span aria-hidden="true">&times;</span>
            </button>
                    </div>
                    <div class="modal-body">
                        <div class="row">
                            <div class="col-md-6">
                                <div class="form-group">
                                    <img src="'.$tabela['product_picture'].'" style="width: 350px;height: 250px;" class="img-responsive img-radio">
                                </div>
                            </div>
                            <div class="col-md-6">
                                <div class="form-group">
                                    Numer Katalogowy:
                                    <input class="form-control" name="catalog_number" id="catalog_number" value="'.$tabela['catalog_number'].'" />
                                </div>
                                <div class="form-group">
                                    Dodatkowe Informacje:
                                    <div class="input-group">
                                        <textarea class="form-control" name="additional_information" id="additional_information" rows="9" cols="25">'.$tabela['additional_information'].'</textarea>
                                    </div>
                                </div>
                                <div class="form-group">
                                    Ilość:
                                    <div class="input-group">
                                        <input class="form-control" name="quantity" id="quantity" value="'.$tabela['quantity'].'">
                                    </div>
                                </div>
                                <div class="form-group">
                                    Cena:
                                    <div class="input-group">
                                        <input class="form-control" name="price" id="price" value="'.$tabela['price'].'">
                                    </div>
                                </div>
                            </div>
                        </div>
 
                    </div>
                    <div class="modal-footer">
                    <input class="btn btn-primary" value="save" name="save" type="submit">
                        <button type="button" class="btn btn-secondary" data-dismiss="modal">Anuluj</button>
                    </div>
                    </form>
                </div>
            </div>
        </div>
[PHP] pobierz, plaintext 

[viking]

Myślisz że jak wykonasz 2x mysql_query gdzie drugi na wyniku to będzie lepiej? Do tego przestarzały kod i niebezpieczny.

[skleps]

mam nadzieję że to tylko jakaś nauka a nie kod na produkcję
bo aż skóra cierpnie..

po kolei:

$sql = mysql_query("UPDATE zdjecia SET id = '$id' product_picture = '$product_picture', product_name = '$product_name', catalog_number = '$catalog_number', additional_information = '$additional_information', quantity = '$quantity', price = '$price'");

powoduje że w $sql masz false lub true (abstrahuje od błędów w zapytaniu). chwilę później robisz:

$wynik = mysql_query($sql);

a w sumie nie wiem co zrobi baza danych, gdy dostanie jako zapytanie wartośc typu boolean

na moje oko możesz zrobić

[PHP] pobierz, plaintext 
       $wynik = mysql_query("UPDATE zdjecia SET id = '$id' product_picture = '$product_picture', product_name = '$product_name', catalog_number = '$catalog_number', additional_information = '$additional_information', quantity = '$quantity', price = '$price'") or die(mysql_error());
            if ($wynik) {
                echo 'Prawidłowo dodano do bazy danych';   
            }
            else {
                echo "nie powiodlo sie";
    }
 
 
[PHP] pobierz, plaintext 

z tym że i tak ci nie zadziała z powodu błędów w zapytaniu. Przypominam że:
UPDATE table_name
SET column1=value, column2=value2,...
WHERE some_column=some_value

[Play12]

Myślisz że jak wykonasz 2x mysql_query gdzie drugi na wyniku to będzie lepiej? Do tego przestarzały kod i niebezpieczny.

W takim razie słucham co mógłbym zmienić lub w czym się dokształcić

mam nadzieję że to tylko jakaś nauka a nie kod na produkcję
bo aż skóra cierpnie..

po kolei:

$sql = mysql_query("UPDATE zdjecia SET id = '$id' product_picture = '$product_picture', product_name = '$product_name', catalog_number = '$catalog_number', additional_information = '$additional_information', quantity = '$quantity', price = '$price'");

powoduje że w $sql masz false lub true (abstrahuje od błędów w zapytaniu). chwilę później robisz:

$wynik = mysql_query($sql);

a w sumie nie wiem co zrobi baza danych, gdy dostanie jako zapytanie wartośc typu boolean

na moje oko możesz zrobić

[PHP] pobierz, plaintext 
       $wynik = mysql_query("UPDATE zdjecia SET id = '$id' product_picture = '$product_picture', product_name = '$product_name', catalog_number = '$catalog_number', additional_information = '$additional_information', quantity = '$quantity', price = '$price'") or die(mysql_error());
            if ($wynik) {
                echo 'Prawidłowo dodano do bazy danych';   
            }
            else {
                echo "nie powiodlo sie";
    }
 
 
[PHP] pobierz, plaintext 

z tym że i tak ci nie zadziała z powodu błędów w zapytaniu. Przypominam że:
UPDATE table_name
SET column1=value, column2=value2,...
WHERE some_column=some_value

Uczę się No i powiem Ci że dużo mi pomogłeś ;D

Obecnie stoi mi na przeszkodzie jeszcze:

[SQL] pobierz, plaintext 
"Duplicate entry '0' for key 'PRIMARY'"
[SQL] pobierz, plaintext 

Ale myslę że tu dam radę ^.^

[viking]

W takim razie słucham co mógłbym zmienić lub w czym się dokształcić

https://prophp.pl/advice/show/14/jak_czytac...mentacje_php%3F
https://prophp.pl/advice/show/3/korzystam_z...ed._dlaczego%3F
https://prophp.pl/advice/show/11/filtrowani...moca_filter_var
https://prophp.pl/advice/show/23/o_czym_pow...onfiguracja_php
https://prophp.pl/advice/show/25/jak_zabezp...ql_injection%3F

Kluczy głównych AI się nie wypełnia - robi to baza.

[skleps]

Uczę się No i powiem Ci że dużo mi pomogłeś ;D

Obecnie stoi mi na przeszkodzie jeszcze:

[SQL] pobierz, plaintext 
"Duplicate entry '0' for key 'PRIMARY'"
[SQL] pobierz, plaintext 

Ale myslę że tu dam radę ^.^

problem zapewne jest w "SET id = '$id' " - strzelam że to pole w bazie jest Auto Increment (AI) więc baza sama uzupełnia to pole i przy Insertach i update należy nie modyfikować/aktualizować tej kolumny