[mysql] edycja komentarzy, select, update Opcje

[Sadu2]

Witam,
Jestem początkujący w tych sprawach i prosze o wyrozumiałość. :) Otóż mam taki kod:

[PHP] pobierz, plaintext 
// EDYCJA KOMENTARZY
else if($act == "edycja_kom") {
echo "<center><b>Edytuj komentarz</b></center></br>";
if(!$_POST[Zapisz]) {
$id = $_GET[id];
$id2 = $_GET[id2];
 
$query = "SELECT id, tresc FROM news_kom WHERE id = '$id'";
$result = mysql_query($query) or die(mysql_error());
$db = mysql_fetch_array($result);
$tresc = walidacja2($db[tresc]);
?>
 
<form method="post" action="admin.php?go=news&act=edytuj_kom&id=<? echo $id; ?>&id2=$id">
<b>Tresć:</b><br>
<textarea name="tresc" cols="50" rows="12"><? echo $tresc; ?></textarea><br>
<input type="submit" name="zapisz" value="Zapisz"></form>
<?
} else {
$tresc = walidacja($_POST[tresc]);
 
$query = "UPDATE news_kom SET tresc = '$tresc' WHERE id = '$id'";
$result = mysql_query($query) or die(mysql_error());
 
if($result) echo "Edycja zakończona !<br>
 <a href="admin.php?go=news"><img src="gfx/powrot.jpeg" alt="powrot"></a>";
if(!$result) echo "Edycja nie powiodła się !<br> 
<a href="admin.php?go=news"><img src="gfx/powrot.jpeg" alt="powrot"></a>";
}
}
}
 
?>
[PHP] pobierz, plaintext 

I wszystko jest dobrze, tylko że jak klikam na zapisz, to nie ma zadnej reakcji, nie wyskakuje zaden result ani nie zapisuje sie edycja komentarza. Co zrobiłem nie tak?
Pozdrawiam

[sticker]

moze to

[PHP] pobierz, plaintext 
else if($act == "edycja_kom") {
echo "<center><b>Edytuj komentarz</b></center></br>";
 
....
<form method="post" action="admin.php?go=news&act=edytuj_kom&id=<? echo $id; ?>&id2=$id">
[PHP] pobierz, plaintext 

[Sadu2]

hmm co masz na myśli? (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) bo ja nie widze róznicy ...

[sticker]

zmienna get act (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif)

[Sadu2]

(IMG:http://forum.php.pl/style_emoticons/default/sad.gif) Czy móglbyś jaśniej? Przepraszam ale raczkuje w tych sprawach.

//EDIT:

Nikt nie wie jak co może być źle?

Ten post edytował Sadu2 11.06.2007, 19:31:22

[Norbas]

1. Przede wszystkim przy indeksach tekstowych dla tablic $_GET oraz $_POST wstawiaj cudzysłowy lub apostrofy, czyli $_POST['nazwa'] albo $_GET["nazwa"]
2. W wierszu 4. w kodzie, który podałeś powinien być warunek:
   
   Kod
   if(!isset($_POST['zapisz'])) {
3. W wierszu 14. masz wstawiony parametr id2, który zawsze będzie równy $id, gdyż nie jest przetwarzany przez interpreter.
4. Przy wysyłaniu formularza wywołujesz: admin.php?go=news&act=edytuj_kom..., zaś przy sprawdzaniu jaka akcja została wywołana masz ($act == "edycja_kom"), moim zdaniem w wierszu 2. w kodzie który podałeś powinien być warunek:
   
   Kod
   else if($act == "edytuj_kom") {
   
   albo należy zmienić parametr act na edycja_kom w formularzu

Ten post edytował Norbas 12.06.2007, 12:58:18

[Sadu2]

O przepraszam, tyle razy na to patrzylem i nie zauwazyłem. No to teraz jak kilkam zapisz to żadnej reakcji nie mam, tylko jedynie zmienia sie link z:

admin.php?go=news&act=edytuj_kom&id=16&id2=39

na:

admin.php?go=news&act=edytuj_kom&id=16&id2=

id odpowiada id komentarza, a id2 odpowiada news_id czyli jakiego newsa to tyczy. Więc co nadal jest źle. (IMG:http://forum.php.pl/style_emoticons/default/dry.gif)

Ten post edytował Sadu2 12.06.2007, 16:30:44

[Norbas]

Wiersze 5. i 6. w kodzie, który podałeś wstaw przed warunek sprawdzający czy wysłano formularz, czyli:

Kod

//...
echo '<center><b>Edytuj komentarz</b></center></br>';
$id = intval($_GET['id']);
$id2 = intval($_GET['id2']);
if(!isset($_POST['zapisz']) {
    $query = 'SELECT id, tresc FROM news_kom WHERE id=' . $id;
    $result = mysql_query($query) or die(mysql_error());
    $db = mysql_fetch_array($result);
    echo '<form method="post" action="admin.php?go=news&act=edytuj_kom&id=' . $id . '&id2='. $id2 . '">' . PHP_EOL;
    echo '<b>Tresć:</b><br><textarea name="tresc" cols="50" rows="12">'
    echo htmlspecialchars($tresc) . '</textarea><br>' . PHP_EOL;
    echo '<input type="submit" name="zapisz" value="Zapisz"></form>' . PHP_EOL;
}
else {
    $tresc = walidacja($_POST['tresc']);
    $query = "UPDATE news_kom SET tresc = '$tresc' WHERE id = $id";
    $result = mysql_query($query) or die(mysql_error());
    echo 'Edycja zakończona !<br><a href="admin.php?go=news"><img src="gfx/powrot.jpeg" alt="powrot"></a>';
}
//..

[Sadu2]

A ja mam oto taki kod:

[PHP] pobierz, plaintext 
// TU EDYCJA
if ($act == "edycja_kom") {
if ($_POST['submit'] == 'Zapisz') {
//Sformatowanie danych
$id = $_POST['id'];
$tresc = walidacja($_POST['tresc']);
//Zapytanie
$query = "UPDATE news_kom SET tresc = '$tresc' WHERE id = '$id'";
$result = mysql_query($query) or die(mysql_error());
 
if (mysql_affected_rows() == 1) {
$success_msg = "<P>Twój komentarz został uaktualniony!</P>";
} else {
error_log(mysql_error());
$success_msg = "<P>Nie udalo się zaktualizować!</P>";
}
 
} else {
//Odczytanie komentarza wraz z tytułem
$id = $_GET['id'];
$query = "SELECT id, news_id, tresc FROM news_kom WHERE id = '$id'";
$result = mysql_query($query) or die(mysql_error());
$db = mysql_fetch_array($result);
 
}
$thispage = $SERVER['PHP_SELF'];
$form_page = <<< EOFORMPAGE
 
<form method="post" action="$this_page">
<b>Treść:</b><br>
<textarea name="tresc" cols="50" rows="12"><? $tresc ?></textarea><br>
<input TYPE="hidden" Name="id" value="$id">
<input type="submit" name="submit" value="Zapisz"></form>
 
$success_msg
EOFORMPAGE;
echo $form_page;
}
}
[PHP] pobierz, plaintext 

Tylko jest taka sprawa ze nie wyświetla mi treści komentarza w textarea, jest pusto, natomiast jesli wpisze cos i wezme zapisz, to wszystko jest ok. Jedyny problem to to ze nie widze tego komentarza którego edytuje.

[Norbas]

1. Po odczytaniu wyniku zapytania nie ustawiasz zmiennej $tresc (instrukcja powinna być w wierszu 24.)
2. W wierszu 31. usuń znaczniki <? i ?>
3. Moim zdaniem powinieneś zmienić wiersz 9., gdyż kod poniżej nie wykona się nigdy, gdy będą jakieś błędy w zapytaniu. Propozycja:
   
   Kod
   $result = @mysql_query($query);