[PHP][MYSQL]problem z UPDATE, UPDATE nie przekazuje zmiennych do bazy danych Opcje

[Vodecki]

Witam!

mam pewien (dość istotny) problem z instrukcją UPDATE z poziomu skryptu php.
Otóż jeżeli chcę update'ować pole i wstawić w nie wartość zmiennej to do bazy przekazywana jest wartość pusta zamiast prawdziwej wartości zmiennej.
skrypt wyglada tak

[PHP] pobierz, plaintext 
<?php
$id=$_GET['id_kat'];
$nazwa_kat_nowy=$_POST['nazwa_kat_nowy'];
			if (is_string($nazwa_kat_nowy))
			{
			echo 'zmienna ok<br>';
			}
 
			 $zapytanie="UPDATE kategorie SET nazwa='$nazwa_kat_nowy'  WHERE id_kat= '$id'";
 
			 echo $zapytanie;
			 mysql_query($zapytanie);
			 echo '<p>'.$nazwa_kat_nowy.'</p>';
			 echo mysql_error();
?>
[PHP] pobierz, plaintext 

instrukcja UPDATE sama w sobie działa - jeżeli w miejsce $nazwa_kat_nowy wstawie cokolwiek np.

[SQL] pobierz, plaintext 
UPDATE kategorie SET nazwa='zmieniona kategoria'  WHERE id_kat= '$id'
[SQL] pobierz, plaintext 

to instrukcja się wykona i w bazie zostanie zapisana "zmieniona kategoria" w wierszu z id_kat odpowiadającym $id, jednak jeżeli jest zmienna to w bzie zapisuje się puste pole.
wszystkie instrukcje echo które są w kodzie mają na celu sprawdzenie jaka jest wartość zmiennej $nazwa_kat_nowy - jest jak najbardziej prawidłowa tzn taka jak podałem w formularzu.

będe wdzięczny za jakąkolwiek pomoc.

[nospor]

a mozesz pokazac co wyswietla
echo $zapytanie;
dla przykladowych danych co ci nie dzialaja?

[webdice]

Dodaj:

[PHP] pobierz, plaintext 
<?php
mysql_query($zapytanie) or die (mysql_error ());
?>
[PHP] pobierz, plaintext 

i jeśli wyświetla Ci się błąd to podaj jaki.

[lewy87r]

witam,

pomin ta linijke:

$nazwa_kat_nowy=$_POST['nazwa_kat_nowy'];

reszte pozostaw bez zmian php/mysql domyslnie odwoluje sie do posta i geta i nie trzeba tej zmiennej deklarowac ;]
i powinno dzialac ;]

ewentualnie sprawdz jakie dane zapisujesz do bazy a jaki typ pola w bazie masz zadeklarowany

Ten post edytował lewy87r 12.05.2008, 12:57:20

[Shili]

@up
Ale wypada nie pisać w ten sposób, tylko jednak z frazą $_POST, imo to nie ta linijka powoduje błąd.
Straszliwie nieładnie i nieprofesjonalnie jest zczytywać w ten sposób na marginesie. Poza tym w ten sposób jest prościej kontrolować dane pochodzące od użytkowniika. Ba, niech ktoś w get doda zmienną o takiej samej nazwie i bez filtrowania danych będzie trochę zamieszania pewnie.

Kiedyś miałam podobny błąd, tylko z innego typu zapytaniem. Bodajże pomogło mi niewpisywanie zapytania w jednym łańcuchu, tylko łączenie łańcuchów, ale bądź co bądź przychylam się do prośby o wstawienie tego, co wypluwa mysql_error

Ten post edytował Shili 12.05.2008, 13:04:47

[Vodecki]

a mozesz pokazac co wyswietla
echo $zapytanie;
dla przykladowych danych co ci nie dzialaja?

dla $nazwa_kat_nowy='test'
wyświetla

[SQL] pobierz, plaintext 
UPDATE kategorie SET nazwa='test' WHERE id_kat= '4'
[SQL] pobierz, plaintext 

i jeśli wyświetla Ci się błąd to podaj jaki.

nie wyświetla błędu i dlatego nie mogę znaleźć rozwiązania

reszte pozostaw bez zmian php/mysql domyslnie odwoluje sie do posta i geta i nie trzeba tej zmiennej deklarowac ;]
i powinno dzialac ;]

nie działa

ewentualnie sprawdz jakie dane zapisujesz do bazy a jaki typ pola w bazie masz zadeklarowany

typ pola - VARCHAR

[nospor]

dla $nazwa_kat_nowy='test'
wyświetla

SQL
UPDATE kategorie SET nazwa='test' WHERE id_kat= '4'

hmmm, dziwne. To mowisz ze gdy zamiast zmiennej dasz poprostu 'test' to wowczas zadziala? To moze w takim razie jakis dziwne białe znaki idą postem. spróbuj dac trim na zmiennej

[PHP] pobierz, plaintext 
<?php
$nazwa_kat_nowy=trim($_POST['nazwa_kat_nowy']);
?>
[PHP] pobierz, plaintext 

[Vodecki]

To mowisz ze gdy zamiast zmiennej dasz poprostu 'test' to wowczas zadziala?

dokładnie tak

spróbuj dac trim na zmiennej

niesttey , bez zmian

[Shili]

[PHP] pobierz, plaintext 
<?php
$zapytanie="UPDATE kategorie SET nazwa=".${nazwa_kat_nowy}."  WHERE id_kat=".${id};
?>
[PHP] pobierz, plaintext 

Spróbuj może zastosować taką składnię; nie mam pojęcia co prawda czy zadziała (skłaniam się ku nie nawet); w razie czego pomiń nawiasy klamrowe jeszcze.

@edit
Oczywiście zeżarłam apostrofy przy edycji; tak na marginesie.

Ten post edytował Shili 12.05.2008, 13:44:01

[Vodecki]

też nie :/
echo jest ok ale nadal wysyła pustą zmienną.

[Pilsener]

Waliduj wszystko, co idzie postem, np. taką funkcją:

[PHP] pobierz, plaintext 
<?php
function tresc_wejscie($tresc)
{
   $tresc = HTMLSpecialChars($tresc);
   if (!get_magic_quotes_gpc())
	  {
	  $tresc=addslashes($tresc);
	  }
	$tresc = trim($tresc);
   return $tresc;
}
?>
[PHP] pobierz, plaintext 

- jeśli są znaki nowej linii, to można też je od razu zamienić na <br /> (czy paragrafy, zależy jak leży).

[potreb]

Pokaż cały kod wraz z formularzem wtedy będzie łatwiej wszystkich przeanalizować

[Vodecki]

Waliduj wszystko, co idzie postem, np. taką funkcją:

waliduje się ładnie tylko nadal bez efektu w momencie zapisu do bazy danych

Pokaż cały kod wraz z formularzem wtedy będzie łatwiej wszystkich przeanalizować

Formularz wygląda tak :

[HTML] pobierz, plaintext 
<form action="index.php?name=admin_kat_edycja&id_kat=<?php echo $id_kat ?>" method="post">
<input name="nazwa_kat_nowy" type="text" size="20" maxlength="40">
<input name="submit" type="submit" value="zapisz">
</form>
[HTML] pobierz, plaintext 

[lewy87r]

wnioskuje ze przepisales to z bledem ;] ze "echo" konczy sie srednikiem ";"
a method="post" proponuje pisać method="POST" drukowanymi jak jest poprawnie ;] (moze jakies serwery inaczej interpretuja)

[Pilsener]

[PHP] pobierz, plaintext 
...id_kat=<?php echo $id_kat ?>" >
[PHP] pobierz, plaintext 

- nie używaj takich zapisów to unikniesz problemów. Tak jest lepiej:

Kod

echo '<form action="index.php?name=admin_kat_edycja&id_kat="'.$id_kat.'" method="post">';

- a w ogóle najlepiej kod HTML trzymać w templatkach a zmienne tylko podstawiać. Nie zwróciłeś uwagi, że w adresie nie masz zmiennnej?

[Vodecki]

wnioskuje ze przepisales to z bledem ;] ze "echo" konczy sie srednikiem ";"

fakt, mój błąd

a method="post" proponuje pisać method="POST" drukowanymi jak jest poprawnie ;] (moze jakies serwery inaczej interpretuja)

też nie działa. Swoją drogą testowałem to na 3 serwerach z takim samym skutkiem :/

Nie zwróciłeś uwagi, że w adresie nie masz zmiennnej?

jakiej zmiennej Przecież nowa nazwa kategorii jest z formularza

[mike]

a method="post" proponuje pisać method="POST" drukowanymi jak jest poprawnie ;] (moze jakies serwery inaczej interpretuja)

Skąd to wywróżyłeś? Z fusów?
A teraz czytamy specyfikację: Element and attribute names must be in lower case a później specyfikację znacznika form gdzie jak byk:

<!ATTLIST form
%attrs;
action %URI; #REQUIRED
method (get|post) "get"
enctype %ContentType; "application/x-www-form-urlencoded"
onsubmit %Script; #IMPLIED
onreset %Script; #IMPLIED
accept %ContentTypes; #IMPLIED
accept-charset %Charsets; #IMPLIED
>

Mi to wygląda na małe litery. A Tobie?

No chyba, że mówisz o przestarzałej specyfikacji HTML 4.01. Jest ona sprzed 9 lat, więc to trochę staroć.

[potreb]

Nie ma co gdybać kiedy nie znamy całości kodu. Pokazujesz części a błąd może leżeć gdzieś indziej.

[--lewy87r--]

Dobra, to ja bym spróbował zmienić nazwe pola nie "nazwa_nowy_kat" tylko cos takiego: "nazwaxx2" moze niskie spacje mają jakiś wpływ:/ (conajmniej dziwne by to bylo)
może z jakiś względów trzeba dodać do forma enctype??
enctype="application/x-www-form-urlencoded"

[Pilsener]

jakiej zmiennej Przecież nowa nazwa kategorii jest z formularza

- a to:

[PHP] pobierz, plaintext 
<?php
$id=$_GET['id_kat'];
?>
[PHP] pobierz, plaintext 

? Mi to wygląda na id pobierane z adresu...

$zapytanie = "selekt coś tam"
Weź teraz echo $zapytanie i się dowiesz, czy zapytanie jest poprawne albo co jest źle. Inaczej nigdy się nie dogadamy.