Forum PHP.pl > [mysql] edycja komentarzy

Pomoc - Szukaj - Użytkownicy - Kalendarz

Forum PHP.pl > Forum > Bazy danych > MySQL

Sadu2

11.06.2007, 13:43:35

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

11.06.2007, 14:38:42

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

11.06.2007, 14:43:15

hmm co masz na myśli?

bo ja nie widze róznicy ...

sticker

11.06.2007, 14:44:53

zmienna get act

Sadu2

11.06.2007, 14:50:44

Czy móglbyś jaśniej? Przepraszam ale raczkuje w tych sprawach.

//EDIT:

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

Norbas

12.06.2007, 12:57:04

Przede wszystkim przy indeksach tekstowych dla tablic $_GET oraz $_POST wstawiaj cudzysłowy lub apostrofy, czyli $_POST['nazwa'] albo $_GET["nazwa"]
W wierszu 4. w kodzie, który podałeś powinien być warunek:
Kod
if(!isset($_POST['zapisz'])) {
W wierszu 14. masz wstawiony parametr id2, który zawsze będzie równy $id, gdyż nie jest przetwarzany przez interpreter.
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

Sadu2

12.06.2007, 16:29:59

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.

Norbas

12.06.2007, 17:15:25

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

12.06.2007, 21:06:17

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

12.06.2007, 22:02:51

Po odczytaniu wyniku zapytania nie ustawiasz zmiennej $tresc (instrukcja powinna być w wierszu 24.)
W wierszu 31. usuń znaczniki <? i ?>
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);

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.