[PHP] System news - edycja po wpisaniu 'hassla'? Opcje

[Forti]

[PHP] pobierz, plaintext 
<?php
include("config.php");
        $result = mysql_query("SELECT * FROM news ORDER BY newsid DESC",$connect);
        while($myrow = mysql_fetch_assoc($result))
             {
               echo "<b>Title: ";
               echo $myrow['title'];
               echo "</b><br>On: <i>";
               echo $myrow['dtime'];
               echo "</i><hr align=left width=160>";
               echo $myrow['text1'];
 
echo "<form method="POST">";
 
echo "<input type=text name="login"/>";
 
echo "<input type=submit value="Wyślij"/>";
 
echo "</form>";
if ($_POST['login'] == 10){
 
               echo "<br><a href=\"read_more.php?newsid=$myrow[newsid]\">Read More...</a>
 
                || <a href=\"edit_news.php?newsid=$myrow[newsid]\">Edit</a>
 
                 || <a href=\"delete_news.php?newsid=$myrow[newsid]\">Delete</a><br><hr>";
 
				     || <a href=\"add_news.php\">add</a><br><hr>";
			}
 
             }
 
?>
 
[PHP] pobierz, plaintext 

Jak już pewnie widzicie, problem jest od linijki echo "<form method="POST">"; - napisałem to sam i już przed wiedziałem że będzie źle.
Co chce osiągnąć - chce by 'newsy' były wyświetlane normalnie dla każdego odwiedzającego. U dołu powinno być pole do wpisania tekstu i przycisk - po wpisaniu hasła chce aby ukazałi mi się opcje: read more, edit, delete, add.

Wpadłem na ten pomysł bo wydaje mi się najprostszy a nie potrafie jeszcze ogarnąć systemu rejestracji, logowania itp. Robie prosty system newsów - na razie wychodzi, jednak problem w tym że każdy będzie mógł go edytować wg. tego co powyżej.

Jakieś rozwiązania praktyczne?

[Turson]

Masz błedy w liniach 13, 15, 17 - powinieneś dodać backslash przed ". 28 - <a href poza " " jest
Włącz raportowane błędów!

[kartin]

[PHP] pobierz, plaintext 
$myrow[newsid]
[PHP] pobierz, plaintext 

i podobne są niepoprawne

Wyświetlanie kodu HTML za pomocą echo jest niewłaściwe (należy oddzielać kod aplikacji od jej wyglądu). Jeśli upierasz się przy trzymaniu wszystkiego razem to zamiast:

[PHP] pobierz, plaintext 
echo "<form method="POST">";
 
echo "<input type=text name="login"/>";
 
echo "<input type=submit value="Wyślij"/>";
 
echo "</form>";
[PHP] pobierz, plaintext 

możesz dać:

[PHP] pobierz, plaintext 
?>
<form method="POST">
    <input type=text name="login"/>
    <input type=submit value="Wyślij"/>
</form>
<?php
[PHP] pobierz, plaintext 

[Forti]

Panowie, dziękuje za pomoc przy wyświetleniu kodu, rozumiem że trzeba poprawić wiele kwestii. Jednak nadal nie rozwiązuje to mojego problemu - jak zrobić, aby po wpisaniu np. 10 i kliknięciu przycisku wyświetliło mi opcje edit, delete, add new itp.

[kartin]

Jak teraz wygląda skrypt i jaki kod HTML otrzymujesz po jego uruchomieniu?

[Forti]

Witam ponownie.

Nie chce zakładać nowego tematu, więc napisze tutaj - może ktoś mi pomoże.

Na stronie głównej pod "newsami" mam takie cos:

[HTML] pobierz, plaintext 
<br><a href=\"edit_news.php?newsid=$myrow[newsid]\">Edit</a>
[HTML] pobierz, plaintext 

Następnie wyświetla mi się strona o takim kodzie:

[PHP] pobierz, plaintext 
<?php
 
include("config.php");
 
     if(isset($_POST['submit']))
 
  {
      $title = mysql_escape_string($_POST['title']);
 
      $text1 = mysql_escape_string($_POST['text1']);
 
      $text2 = mysql_escape_string($_POST['text2']);
 
         $result = mysql_query("UPDATE news SET title='$title', text1='$text1', text2='$text2' WHERE newsid='$newsid' ",$connect);
 
          echo "<b>Thank you! News UPDATED Successfully!<br>You'll be redirected to Home Page after (4) Seconds";
 
          echo "<meta http-equiv=Refresh content=4;url=index.php>";
 
}
 
elseif(isset($_GET['newsid']))
 
{
 
        $result = mysql_query("SELECT * FROM news WHERE newsid='$_GET[newsid]' ",$connect);
 
        while($myrow = mysql_fetch_assoc($result))
 
             {
 
                $title = $myrow["title"];
 
                $text1 = $myrow["text1"];
 
                $text2= $myrow["text2"];
 
?>
 
<br>
 
<h3>::Edit News</h3>
 
 
 
<form method="post" action="<?php echo $PHP_SELF ?>">
 
	<input type="hidden" name="newsid" value="<? echo $myrow['newsid']?>">
		Tutuł:<input name="title" size="40" maxlength="255" value="<? echo $title; ?>">
			<br>	
		Tekst:<textarea name="text1"  rows="7" cols="30"><? echo $text1; ?></textarea>
			<br>
		Tekst2:<textarea name="text2" rows="7" cols="30"><? echo $text2; ?></textarea>
			<br>
	<input type="submit" name="submit" value="Update News">
</form>
 
 
 
[PHP] pobierz, plaintext 

I wszystko ładnie pięknie, jednak "news" nie edytuje się - wraca do swojej poprzedniej treści. Co może być nie tak w tym kodzie? Żadnych błędów nie zwraca.

[johny_s]

pewnie nie ma newsid
ps.
tak Ciężko wyświetlić zapytanie i z palca do bazy wrzucić?

[kartin]

[PHP] pobierz, plaintext 
$result = mysql_query("UPDATE news SET title='$title', text1='$text1', text2='$text2' WHERE newsid='$newsid' ",$connect);
[PHP] pobierz, plaintext 

Pewnie dla zmiennej $newsid powinieneś przypisać wartość pobraną z $_POST.
W jakim celu do zmiennej $result pobierasz wynik wykonania zapytania, jak z niej nie korzystasz?

Zrób coś w rodzaju:

[PHP] pobierz, plaintext 
if ($result) {
    //komunikat zapisano poprawnie
}
else {
  //komunikat błąd zapisu do bazy
}
[PHP] pobierz, plaintext 

[Forti]

Tyle że delete dla newsid (tabela jest w mysql) czy przycisk dodaj (podobne jak edit ale dodaje nowy) działa poprawnie.
Link z index.php do edit i delete jest w tym samym miejscu, wygląda tak samo praktycznie jeżeli chodzi o kwestie NEWSID

Coś jest źle w kodzie aplikacji z edit, tylko nie potrafie stwierdzić co.

Zadanie: pobrać NEWSID i wyświetlić to, co jest w bazie w tabeli (tytuł, tekst1, tekst2) a następnie nadpisać to.

[kartin]

Wyświetl sobie treść zapytania SQL (to pewnie wszystko wyjasni) i dodatkowo sprawdzaj czy zostało wykonane prawidłowo.

[Forti]

hm.. chyba jeszcze troche nie wszystko ogarniam. Jak sprawdzić treść zapytania SQL?

[kartin]

Np. wyświetlić przez przy użyciu echo a w nim zapytanie skopiowane z mysql_query().

[Forti]

Nie ogarniam hehe

wstawiłem coś takiego:

[PHP] pobierz, plaintext 
 <?php
 echo         $result = mysql_query("SELECT * FROM news WHERE newsid='$_GET[newsid]' ",$connect);
 echo          $result = mysql_query("UPDATE news SET title='$title', text1='$text1', text2='$text2' WHERE newsid='$newsid' ",$connect);
 
 ?>
[PHP] pobierz, plaintext 

i wyświetla mi na stronie: Resource id #41 - a powinno pobrać np. id #5 (artykuł ma taki id) (samo pierwsze echo wyświetla Resource id #1)




ps, może ktoś będzie wiedział (nie chce kolejnego tematu robić)

mam coś takiego:

[HTML] pobierz, plaintext 
<br><a href=\"read_more.php?newsid=$myrow[newsid]\">Więcej</a></div>
[HTML] pobierz, plaintext 

i to jest na stronie news.php która jest zaincludowana w index.php (index.php?inc=news)
przy pomocy tej funkcji:

[PHP] pobierz, plaintext 
<?PHP
 
@$strona=$_GET['inc'];
 
 
if (isset($_GET['inc'])) {
    if (file_exists('inc/'.$strona.'.php'))
        include('inc/'.$strona.'.php');
    else
        include('inc/error.php');
}
else {
    //załaduj stronę główną
    $strona="news";
    include('inc/'.$strona.'.php');
}
 
 
?>
[PHP] pobierz, plaintext 

Jak sprawić aby plik read_more wraz z pobranym ID $_get wyświetlił mi się w miejscu news.php przy pomocy include? Próbowałem już róznych warunków, siedze generalnie od wczoraj nad tym problemem. Nawet próbowałem coś z input name="xx" i warunek co zrobić gdy się go naciśnie. Albo wywala error.php albo pustą strone.

Ten post edytował Forti 17.05.2014, 10:43:27

[Turson]

Zapytanie masz wyświetlić w ten sposób

[PHP] pobierz, plaintext 
$sql = "SELECT * FROM news WHERE newsid='$_GET[newsid]' ";
$result = mysql_query($sql,$connect);
echo $sql;
[PHP] pobierz, plaintext 

[Forti]

wyświetliło: SELECT * FROM news WHERE newsid='6'

Czyli w teorii prawidłowo, newsid pobiera prawidłowo (link: edit_news.php?newsid=6). Jednak po edycji dalej pozostaje wersja z przed.



edit:

naprawiłem ;]

było: $result = mysql_query("UPDATE news SET title='$title', text1='$text1', text2='$text2' WHERE newsid='$newsid' ",$connect);
a powinno być : $result = mysql_query("UPDATE news SET title='$title', text1='$text1', text2='$text2' WHERE newsid='$_GET[newsid]' ",$connect);


W postach wyżej zapytałem się odnośnie funkcji include, aby było coś takiego: index.php?inc=read_more.php?newsid=$myrow[newsid] - tylko mam problem bo moja funkcja sama pobiera read_more, wstawia .php a ze wszystkim potem jest problem (obecnie pobiera wszystko i na końcu wstawia .php przez co jest error.

Ten post edytował Forti 17.05.2014, 10:52:05

[Turson]

daj

[PHP] pobierz, plaintext 
print_r($_GET);
[PHP] pobierz, plaintext 

poza tym, to jest bez sensu

[PHP] pobierz, plaintext 
@$strona=$_GET['inc'];
if (isset($_GET['inc'])) {
}
[PHP] pobierz, plaintext 

Nie wygłuszaj błędów.

[PHP] pobierz, plaintext 
if (isset($_GET['inc'])) {
$strona = $_GET['inc'];
}
[PHP] pobierz, plaintext 

[Forti]

Tutaj pokażę jak bardzo zielony jestem, nie wiem co z tym zrobić.

[PHP] pobierz, plaintext 
print_r($_GET);
[PHP] pobierz, plaintext 

[Turson]

Wklej do swojego kodu najlepiej na sam początek i odśwież stronę, następnie to co tam sie wyswietli skopiuj i wklej na forum

[Forti]

array ()
na stronie index.php, pytanie tylko po co to miałem wstawić?

Jezeli chodzi o edycje wpisu to rozwiązałem problem.

Pytanie kolejne: na stronie index.php jest kod na include linków z menu (podany wyzej). domyślnie ładuje się inc/news.php, w nim jest link do pliku read_more.php?newsid=$myrow[newsid]. I chce aby ten link wraz z odpowiednią wartością newsid wyświetlił się w polu include na index.php. Do tego nie mogę dojść.