Witam. Mam skrypt który wyświetla, dodaje, usuwa i edytuje rekordy z bazy. Problem mam tylko z edycją. Kod nie wypluwa błędu ale też nie zatwierdzał zmian i nie wiem czemu. Prosił bym o sprawdzenie, a dla ułatwienia wstawiam tylko fragmenty dotyczące edycji.

Więc tak. Najpierw łączę się z bazą i wyświetlam listę rekordów za pomocą poniższej pętli:


Gdy kliknę w link "zmien" uruchamia się strona z formularzem:


Gdy wyślę nową nazwę wpisaną w formularzu otwiera się jeszcze raz ta sama strona z tym, że użyłem instrukcji warunkowej if która dokona zmian w rekordzie:


Dziwne jest, że gdy wpiszę nową nazwę dla kategorii w formularzu otrzymuje potwierdzenie, że została zmieniona a gdy sprawdzam rekord w bazie albo przez stronę nie zmienia się nie mam pojęcia co jest nie tak.

A pokaż mi kolego, gdzie w Twoim formularzu edycji jest ID kategorii Nie przesyłasz żadnego ID, skrypt nie ma pojęcia którą kategorię edytujesz. Dodaj do formularza pole typu "hidden" z wartością tego ID, daj mu name="id" i zadziała.

Ze swojej strony dodam jeszcze, że niepotrzebnie przesyłasz tutajnazwę. Po to tworzymy i przesyłamy unikalny identyfikator, aby tylko z niego korzystać. Odnośnie formularza, to luck zapomniał wspomnieć, abyś value tego ukrytego pola pobrał z tablicy $_GET.

Natomiast sam UPDATE odbywa się po wysłaniu formularza metodą post, zatem nie powinno być $_GET['nowa_nazwa'] tylko $_POST['nowa_nazwa'] i nie powinno być $_GET['id'] tylko $_POST['id'] (trzecia linijka w ostatnim listingu).

Chłopaki działa - dzięki wielkie:D Już podaję co zmieniłem. Zaraz nad formularzem utworzyłem zmienną i zapisałem w niej wartość 'id' pobierana z linka w momencie kliknięcia:

A w formularzu dodałem ukryte pole które przesyła numer identyfikatora kategorii która ma zostać zmieniona:

W "UPDATE..." pozmieniałem na $_POST[]

Ale mam jeszcze jedno pytanko. Mortus napisałeś, że niepotrzebnie przesyłam nazwę w linkach, że powinienem użyć identyfikatora i racja ale zrobiłem to tylko z jednego powodu. Przed formularzem mam echo który wyświetla informacje o tym, że poniższy formularz zmieni nazwę tej kategori: "tutaj_wyswietla_sie_nazwa_pobranej_kategorii" więc wystarczy, że wpisze $_GET['nazwa'] i problem z głowy. Gdy bym chciał to zrobić używając identyfikatora musiał bym się połączyć z bazą po raz drugi i pobrać nazwę za pomocą identyfikatora i wtedy ją zapisać w zmiennej i ją użyć. Moje rozwiązanie ogranicza się do przesłania wartości i odczytania jej co jest znacznie szybsze.

Jeśli jest jakieś inne wyjście to chętnie się dowiem

I to jest jedyne sensowne rozwiązanie. Dlaczego? Dlatego, że nazwę w linku możemy bez problemu zmienić i nie musi się ona wcale zgadzać z nazwą, jaka odpowiada przesyłanemu identyfikatorowi (który również możemy w linku zmienić). Dla przykładu weźmy tabelę:Adres możemy spreparować w taki sposób, że identyfikator będzie wskazywał na kategorię 1, ale nazwa będzie odpowiadać kategorii 2, przykładowy adres w przeglądarceUżytkownik zostanie wtedy poinformowany, że zmianie ulegnie nazwa 'kategoria 2', gdy w rzeczywistości będziemy zmieniać nazwę 'kategoria 1'. Zatem jeśli przesyłasz zarówno identyfikator, jak i nazwę, to wypadałoby zapytaniem do bazy danych sprawdzić, czy dany identyfikator odpowiada danej nazwie. A czy w związku z tym nie lepiej przesłać sam identyfikator i pobrać odpowiednią nazwę?

Rozumiem o co Ci chodzi i rzeczywiście sposób jest dobry ale boli mnie trochę to, że w jednym kodzie wykonuje dwa razy to samo (odczytuje te same wartości z tej samej tabeli) mam wrażenie, że kod taki był by strasznie lamerski. Wiem, że z moich ust to brzmi żałośnie bo sam się uczę i zabłysnąć z PHP to nie mogę ale sytuacja wygląda tak... ja ten kod napisałem dawno temu i był to mój pierwszy skrypt który posiadał ileś tam opcji z wartościami i bazami... Obecnie trochę lepiej już operuje językiem i gdy przypatrzyłem się temu skryptowi to choć sam jestem początkującym w PHP to widzę, że on płacze i prosi o przepisanie więc właśnie to robię dlatego też zależy mi aby był jak najbardziej krótki, przejrzysty, praktyczny i poprawny.

Zastanawiając się nad inną metodą zamiast ponownie pobierać wartości z bazy, to w miejscu gdzie po raz pierwszy je wyświetliłem jakoś zapisać je w do tabeli i jakoś przesłać a następnie odczytać albo stworzyć jakąś funkcję. Miało by to sens?