[PHP][MySQL] Nie chce zrobić update'u. Opcje

[Wojciechowy]

Mam dziwny problem i nie wiem jak go rozwiązać.

Chcę aktualizować rekordy w plik.php mam formularz, który po wypełnieniu przechodzi do index.php, a tam jest cały kod ws. UPDATE'u rekordu.
tylko jest mały problem, jeśli w pole tekstowe wpisze cyfry to wszystko jest OK (UPDATE wychodzi pomyślnie), ale jak juz wpisze się literki to nie robi update'u.
Sprawdzałem, w obu przypadkach typ zmiennej pozostaje 'STRING', w bazie danych ta kolumna ma typ TEXT.

Syntax'ow nie ma. Nie wiem jak temu zaradzić, proszę o pomoc!

[zordon]

kodu kodu kodu ....

[Wojciechowy]

[PHP] pobierz, plaintext 
$ntext = $_POST["text"];
$tekst_query = mysql_query("UPDATE `text` SET `TEXT`=$ntext WHERE `ID`=$id and `PD`=$pd");
[PHP] pobierz, plaintext 

a tak to w sumie nie wiem co dać - formularz jest dobrze zrobiony, wszystkie zmienne są dobrze przesyłane do index.php (wiem, bo dodawanie, usuwanie rekordów działa, aktualizacja zresztą też tylko mam problem z tymi tekstami)

jak wyswietlam $ntext przed mysql_query to wyświetla dobrze, tylko nie wiem dlaczego nie wykonuje update'u!.

Ten post edytował Wojciechowy 2.06.2010, 15:23:18

[zordon]

http://pl.php.net/manual/en/function.mysql-error.php
zobacz co wyświetla

ponadto upewnij się, że masz na pewno dobre warunki where zapytania -> czy wartosci kolumn ID i PD na pewno zgadzają się z tymi ze zmiennych(oba warunki muszą być spełnione jednocześnie! )

a najlepiej podaj kod zarówno formularza jak i skryptu przypisującego te zmienne - jesli wszystko byłoby 'dobrze' zrobione to nie byłoby problemu

Ten post edytował zordon 2.06.2010, 15:36:18

[Wojciechowy]

1064: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '`ID`=1 and `PD`=1' at line 1

tylko gdzie ten syntax? ;o

wartości kolumn się zgadzają, bo wszystkie inne wykonywanie działania na tych warunkach pracują bezproblemowo.
a kod, już wklejam.


index.php

[PHP] pobierz, plaintext 
 
                  case 'edit':
 
			$id = $_GET["id"];
			$pd = $_GET["pd"];
 
			$nnazwa = $_POST["nazwa"];
			$ntext = $_POST["text"];
 
			if ($_POST["chnazwa"]==1) {
 
			$q1 = "UPDATE `menu` SET `NAZWA`=$nnazwa `ID`=$id and `PD`=$pd";
			$ins1 = mysql_query($q1);
			echo mysql_errno().": ".mysql_error()."<BR>";
			}
 
			if ($_POST["chtekst"]==1) {
 
			echo 'chtekst';
			$tekst_update = '';
			$tekst_query = mysql_query("UPDATE `text` SET `TEXT`=$ntext WHERE `ID`=$id and `PD`=$pd");
 
                 break;
 
 
[PHP] pobierz, plaintext 

plik.php

[PHP] pobierz, plaintext 
<form action="index.php?action=edit&id= <?php echo $id."&pd=".$pd; ?> " method="post">
 
<input type="checkbox" name="chnazwa" value="1" onclick="this.form.elements['nazwa'].disabled = !this.checked" id="c1"/> 
<label for="c1">Nazwa:</label> 
<input name="nazwa" type="text" value="
 
<?php echo $nm; ?>
 
" disabled="disabled"/>
 
</td></tr><tr><td></td><td>
 
<input type="checkbox" name="chtekst" value="1" onclick="this.form.elements['text'].disabled = !this.checked" id="c2"/>
<label for="c2">Tekst:</label> 
 
<textarea name="text" disabled="disabled">
<?php echo $txt; ?>
</textarea>
 
</td></tr><tr>
<td></td><td>
<input type="submit" />
</form>
[PHP] pobierz, plaintext 

z grubsza wyczyściłem ten kod teraz, żeby nie zaśmiecać.

Ten post edytował Wojciechowy 2.06.2010, 15:46:56

[zordon]

wiesz co, najłatwiej dojść będzie do błędu w ten sposób:
zmień sobie fragment z pierwszego posta na:

[PHP] pobierz, plaintext 
$ntext = $_POST["text"];
 
$query = "UPDATE `text` SET `TEXT`=$ntext WHERE `ID`=$id and `PD`=$pd";
 
// var dump($query); die();
 
$tekst_query = mysql_query($query);
[PHP] pobierz, plaintext 

chwilowo odkomentuj tą linijkę i wywołaj skrypt dwukrotnie: z liczbą jako parametr z formularza i tekstem.
wklej tu zawartość zmiennej $query

PS . pokasuj sobie spacje w formularzu przy wstawianiu value do pola textowego 'nazwa' bo dane z tego pola dostajesz w formmie <spacja>twoja_zmienna<spacja>, ew odbieraj ja przez trim($_POST['nazwa'])

Ten post edytował zordon 2.06.2010, 17:29:55