[php][mysql] Aktualizacja tabel... Opcje

[Reiven]

Witam, wziąłem się za pisanie takiego swojego mini cms'a i mam problem z aktualizacją tabel. Dane wyświetla normalnie w formularzu ale żeby je uaktualnić to już nie chce.

[PHP] pobierz, plaintext 
<?php
function edit_news()
{
$id = $_GET['id'];
echo '<b>Edycja News'a:.</b>';
$query2 = "SELECT * FROM news WHERE id_n='$id'";
$result2 = mysql_query ($query2);
$row2 = mysql_fetch_array($result2);
 
$id = $row2['id_n'];
$tytul = $row2['tytul'];
$tresc = $row2['tresc'];
$autor = $row2['autor'];
 
echo '<form action="index.php" method="POST" style="border: 0px;"><table><tr><td>Tytul Newsa:</td><td><input type="text" name="tytul" value="'.$tytul.'"></td></tr><tr><td>Tresc:</td><td><textarea name="text" rows="15" cols="80" style="width: 100%">'.$tresc.'</textarea></td></tr>
 
 
<tr><td>Dodal:</td><td>'.$autor.'</td></tr>
<tr>
<td style="color:797979;" valign="top">akcje:</td>
<td valign="top">
<input type="hidden" name="id" value="'.$id.'">
<input type="hidden" name="cmd" value="edit_n_ok">
<input class="button" type="submit" style="font-weight: bold;" value="Edytuj">
</form></td>
</tr>
</table>';
}
 
function edit_n_ok()
{
global $tytul,$tresc,$autor,$id;
$tresc= nl2br($tresc);
$query = "UPDATE news SET tytul='$tytul', tresc='$tresc' WHERE id='$id'";
 
 
 $result = mysql_query($query);
echo "<script>self.location.href=" index.php?cmd=4";</script>";
}
?>
[PHP] pobierz, plaintext 

a tak wyglądają linki:

[PHP] pobierz, plaintext 
<?php
elseif ($cmd == 'edit_n') {
	include ("f/news.php");
	edit_news();
} elseif ($cmd == 'edit_n_ok') {
	include ("f/news.php");
	edit_n_ok();
}
?>
[PHP] pobierz, plaintext 

próbowałem też tak:

[PHP] pobierz, plaintext 
<?php
$query = "UPDATE `news` SET `tytul`='$_POST["tytul"]',  `tresc`='$_POST["tresc"]' WHERE `id`=$_POST["id"]'";
?>
[PHP] pobierz, plaintext 

i też nic to nie dało (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

[kossa]

masz błąd w tym kodzie:

$query = "UPDATE `news` SET `tytul`='$_POST["tytul"]', `tresc`='$_POST["tresc"]' WHERE `id`=$_POST["id"]'";

powino być:

[SQL] pobierz, plaintext 
$query = "UPDATE news SET tytul='".$_POST["tytul"]."',  tresc='".$_POST["tresc"]."' WHERE id=".$_POST["id"]." ";
[SQL] pobierz, plaintext 

Łukasz

[Reiven]

zmieniłem i nadal nic....

[Neojawor]

a czy nie powinno być $_POST['tytul'] zamiast $_POST["tytul"]?

ale przede wszystkim:
1) napisałeś funkcję edit_n_ok() i nigdzie jej nie wywołujesz, więc jak ma coś uaktualnić?
2) zmienne $tytul, $tresc i pozostałe w funkcji edit_n_ok() dostają atrybut global (nie wiem po co), a nie dostają żadnej wartości. Wyechuj w tej funkcji wszystkie te zmienne, to zobaczysz, że nic nie zawierają. Musisz im podać wartości wzięte z formularza, a więc:
$tytul = .... i tu bym coś wpisał, ale widzę, że przecież Ty nie masz formularza, który by zmieniał jakąkolwiek wartość

ten Twój form może nas przenieść do jakiejś strony, gdzie będzie inny form z polami wypełnionymi już obecną zawartością danego rekordu i tam dopiero wpiszesz nowe wartości i klikniesz submit-a który wywoła update

[Reiven]

oj źle zauważyłeś (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) )

[PHP] pobierz, plaintext 
<?php
elseif ($cmd == 'edit_n_ok') {
	include ("f/news.php");
	edit_n_ok();
?>
[PHP] pobierz, plaintext 

funkcja jest wykonywana.


hmm co do tego co napisałeś w pkt2
zrobiłem tak:

[PHP] pobierz, plaintext 
<?php
global $tytul,$tresc,$id;
$tresc= nl2br($tresc);
$query = "UPDATE news SET tytul='".$_POST["tytul"]."', tresc='".$_POST["tresc"]."' WHERE id=".$_POST["id"]." ";
 
echo $tytul;
echo $tresc;
echo $id;
 $result = mysql_query($query);
?>
[PHP] pobierz, plaintext 

i mi wyświetliło zmieniony tytuł i ID lecz treści nie wyświetliło.. a w bazie danych i tak nawet tytułu nie zmieniło.

[Neojawor]

oki, widzę że nie mogę sie połapać w Twoim kodzie, ale wiem dlaczego. Nie zauważyłem że masz w jednej linijce całą zawartość form-a i tylko widziałem dwa pola typu hidden (linia 15) (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Mam jednak pytania, żeby się połapać w plikach:
- czy ten pierwszy kod który podałeś to jest index.php ?
- w jakim pliku zawierają się te wywołania edit_news() i edit_n_ok() ?
- zastanawiam się, skąd się mają wziąć wartości zmiennych $tytul, $tresc, $autor, $id w funkcji edit_n_ok()? - przekazujesz je metodą POST z tego formularza, który masz w 15 linii? jeśli tak, to na początku funkcji zrobiłbym coś takiego:
$tytul=$_POST['tytul];
$tresc=$_POST['text'];
$id = $_POST['id];
natomiast zmienną $autor bym pominął bo do update nie jest potrzebna
a co do zapytania to dałbym je w takiej wersji jak podałeś na początku, z małą zmianą - id nie dajemy w ciapki, bo to liczba, a nie tekst:

[PHP] pobierz, plaintext 
<?php
$query = "UPDATE news SET tytul='$tytul', tresc='$tresc' WHERE id=$id";
?>
[PHP] pobierz, plaintext 

i mała uwaga:
- jeśli już używasz tablic referencyjnych, to identyfikatory bierz w apostrofy, a nie cudzysłowy, czyli daj $_POST['tytul'] zamiast $_POST["tytul"] - nie jestem pewien czy to ma znaczenie, ale na wszelki wypadek lepiej trzymać się standardu.

[Reiven]

nie ten 1 kod to nie index.php
to jest część pliku: f/news.php

w index.php mam ten kod:

[PHP] pobierz, plaintext 
<?php
elseif ($cmd == 'edit_n') {
	include ("f/news.php");
	edit_news();
} elseif ($cmd == 'edit_n_ok') {
	include ("f/news.php");
	edit_n_ok();
}
?>
[PHP] pobierz, plaintext 

co do kolejnego pytania... to się trochę rozpędziłem i dodałem $autor w globalach. Chciałbym dodać że jestem początkujący z php =) więc proszę o cierpliwość (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) dodałem na początku funkcji to co napisałeś, zmieniłem zapytanie i dalej nic...

już zrobiłem, miałem konflikt ze zmiennymi =) lecz teraz nie chce mi pobierać ID w innym skrypcie :| też związane z update.

Oto skrypt:
f/strony.php

[PHP] pobierz, plaintext 
<?php
function strony_ok() {
global $cmd,$zaloguj_login,$tytul,$text;
$query = "INSERT INTO strony VALUES(NOT NULL, '$tytul', '$text')";
mysql_query($query);
echo "<script>self.location.href=" index.php?cmd=2";</script>";
}
 
function strony_edit() {
 
global $ids, $titles, $teksts;
$id = $_GET['id'];
$query2 = "SELECT * FROM strony WHERE id_s='$id'";
$result2 = mysql_query ($query2);
$r2 = mysql_fetch_array($result2);
 
$id = $r2['id_s'];
$titles = $r2['tytul'];
$teksts = $r2['text'];
 
 
echo '<form name="form_1" method="POST" action="index.php">
<table align="center" border="0" cellspacing="3" cellpadding="0">
<tr>
<td align="right" valign="top" class="uni_01">tytuł</td>
<td class="next">
<input type="text" name="tytul" size="35" value="'.$titles.'">
</td>
</tr>
<tr>
<td align="right" valign="top" class="uni_01">zawartość</td>
<td class="next">
<textarea name="text" rows="15" cols="80" style="width: 100%">'.$teksts.'</textarea>
</td>
</tr>
<tr>
<td align="right" valign="top" class="uni_01">&nbsp;</td>
<td class="next">
<input type="hidden" name="id" value="'.$id.'">
<INPUT type="hidden" name="cmd" value="strony_up">
<input style="font-weight: bold;" type="submit" name="ok" value="zapisz">
<input type="reset" name="Reset" value="wyczyść">
</td>
</tr>
</table>
</form>
';
}
 
function strony_up() {
 
 
 
$titles = $_POST['tytul'];
$teksts = $_POST['text'];
$id = $_GET['id_s'];
 
 
 
$query = "UPDATE strony SET tytul='$titles', text='$teksts' WHERE id_s=$id";
 
echo $query;
 $result = mysql_query($query);
 
}
?>
[PHP] pobierz, plaintext 

a w index.php mam tak:

[PHP] pobierz, plaintext 
<?php
elseif ($cmd == '2') {
	 include ("f/strony.php");
	echo 'Strony Serwisu';
	echo '<table align="center" width="100%"><tr><td height="20%" bgcolor="#F4F4F4">ID</td><td height="20%" bgcolor="#F4F4F4">Tytuł</td><td height="20%" bgcolor="#F4F4F4">Akcja</td></tr>';
   strony();
   echo '</table>';
   dodaj_s();
} elseif ($cmd == 'strony_ok') {
	 include ("f/strony.php");
	echo 'Strony Serwisu - Dodaj Informacje';
   strony_ok();
} elseif ($cmd == 'strony_edit') {
	 include ("f/strony.php");
	echo '<BR>Aktualizacja danych<BR>';
   strony_edit();
} elseif ($cmd == 'strony_up') {
	 include ("f/strony.php");
	echo '<BR>Aktualizacja danych<BR>';
   strony_up();
}
?>
[PHP] pobierz, plaintext 

jak widzicie w 1 skrypcie umieściłem echo $query; żeby wyswietliło mi jakie zapytanie z tego wychodzi i tylko ID nie chce pobrać....

-------------

temat do zamknięcia, problem rozwiązałem.