Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [MySQL][PHP] system newsow
Forum PHP.pl > Forum > Przedszkole
zomb
6.11.2009, 21:06:48
Witam,
Piszę system newsów na swoja stronę, mam już dodawanie, wyświetlanie, edycje. Nie działa to jeszcze w 100% sprawnie. Link: http://91.121.10.107/busines/index.php
Po zalogowaniu jako admin mam pod każdym newsem przycisk Edytuj news. Jeżeli go nacisnę pokazują mi sie wszystkie newsy do edycji, oraz gdy coś edytuje to edytują się także wszystkie newsy.
Więc tutaj pytanie: jak zrobić aby było tak jak powinno? Nie będę śmiecił i wklejał tutaj wszystkie pliki, myślę że wiecie o co mi chodzi. Moje zapytania do bazy wyglądają tak:
[PHP] pobierz, plaintext 
  1. mysql_query("UPDATE `news` SET `tytul` = '$tytul', `tresc` = '$tresc', `autor` = '$autor' ;"); 
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
  1. SELECT * FROM news
[PHP] pobierz, plaintext

wiem że trzeba dodać w zapytaniu WHERE (przynajmniej tak mi się wydaje) ale niestety nie wiem co i jak. Proszę o pomoc, jeżeli będzie potrzeba dodam wszystkie pliki.
Spawnm
6.11.2009, 21:09:31
daj where id=$id gdzie $id to id edytowanego newsa
zomb
7.11.2009, 10:43:40
jak dodam WHERE `id` = '$id' to niestety ale po kliknieciu w edytuj nic sie nie pokazuje. Może jednak dam pliki:
news_pokaz.php:
http://wklej.org/hash/d4dac7d52a/
news_edytuj.php:
http://wklej.org/hash/ef78afbfbf/
news_edytuje.php:
http://wklej.org/hash/7fa3455610/

Wiem że jest to dość opornie zrobione ale jak na mój pierwszy system news myśle że dobrze.
Blame
7.11.2009, 11:10:07
Do news_pokaz.php dodajesz sobie w linijce 42:
[PHP] pobierz, plaintext 
  1. if ($pokaz1['ranga'] == 'admin'){
  2. $zapytanie = "SELECT * FROM news"; 
  3.  
  4. $wykonaj = @mysql_query ($zapytanie); 
  5.  
  6. while($pokaz = @mysql_fetch_array($wykonaj)){
  7.  
  8. echo '<a href="news_edytuj.php?id='.$pokaz['id].'" >Edytuj news</a>';}
  9.  
  10. }}
[PHP] pobierz, plaintext


W news-edytuj.php 6 i 7 linijkę zmieniasz na:

[PHP] pobierz, plaintext 
  1. $id=$_GET['id'];
  2. $zapytanie = "SELECT * FROM news WHERE 'id'='$id'"; 
[PHP] pobierz, plaintext


Powinno działać. Aha, i na twoim miejscu, scaliłbym news_edytuj i news_edytuje.
zomb
7.11.2009, 11:31:43
dodane, lecz teraz pokazuje tylko 1 news oraz dalej nie pokazuje po naciśnięciu edytuj, admin testowy: test/test
Blame
7.11.2009, 11:40:06
Poprawione:
[PHP] pobierz, plaintext 
  1. if ($pokaz1['ranga'] == 'admin'){
  2.  
  3.       $zapytanie = "SELECT * FROM news";
  4.  
  5.  
  6.  
  7.       $wykonaj = @mysql_query ($zapytanie);
  8.  
  9.  
  10.  
  11.       while($pokaz = @mysql_fetch_array($wykonaj)){
  12.  
  13.  
  14.  
  15.       echo '<a href="news_edytuj.php?id='.$pokaz['id'].'" >Edytuj news</a>';}
  16.  
  17.  
  18.  
  19.       }}}
[PHP] pobierz, plaintext 


[PHP] pobierz, plaintext 
  1.       $id=$_GET['id'];
  2.  
  3.       $zapytanie = "SELECT * FROM news WHERE id='$id'"; 
[PHP] pobierz, plaintext

Przepraszam za błąd, skrypt pisany na szybko.
zomb
7.11.2009, 11:43:07
wiem, poprawiłem te błędy szybciej lecz dalej to samo
(link do strony http://91.121.10.107/busines login: test haslo: test )
potreb
7.11.2009, 11:50:08
W formaulrzu edycji do action należy także dodać ?id=$pokaz['id']

Popatrzyłem w źródło formularza i rzeczywiście nie przekazujesz dalej zmiennej id, to skąd skrypt ma wiedzieć co edytować.
zomb
7.11.2009, 12:17:41
wiec co mam jeszcze dodać w pliku news_edytuj i w news_edytuje ?
po tych poprawkach w news_pokaz pokazuje tylko 1 news oraz kilka napisów edytuj news
Blame
7.11.2009, 12:45:38
A poprawiłeś kod tak jak podałem w poprzednim poście? Chodzi mi o jeszcze jedną klamrę na końcu. Specjalnie dla ciebie prze edytowałem plik news_edytuj.php teraz nie potrzebny ci już news_edytuje.php. Mam nadzieję że będzie działać. Oto on:
[PHP] pobierz, plaintext 
  1. <?
  2. session_start();
  3. if(!empty($_SESSION['login'])){ 
  4.  
  5. include("config.php");
  6.  
  7.  
  8. $id=$_GET['id'];
  9.  
  10. $zapytanie = "SELECT * FROM news WHERE 'id'='$id'";  
  11.  
  12. $wykonaj = @mysql_query ($zapytanie);
  13. while($pokaz = @mysql_fetch_array($wykonaj)){
  14.  
  15. ?>
  16. <form action="news_edytuj.php?id=<?$id?>" method="post">
  17. <table>
  18. <tr>
  19. 	<td width="132"><b>id :</b></td>	
  20. 	<td width="139"><? echo $pokaz['id']; ?></td>
  21. </tr>
  22. <tr>
  23. 	<td><b>tytul :</b></td>	
  24. 	<td><input name="tytul" type="text" value="<? echo $pokaz['tytul']; ?>" /></td>
  25. </tr>
  26. <tr>
  27. 	<td><b>tresc :</b></td>	<td><input name="tresc" type="text" value="<? echo $pokaz['tresc']; ?>" /></td>
  28. </tr>
  29. <tr>
  30. 	<td><b>autor :</b></td>	<td><input name="autor" type="text" value="<? echo $pokaz['autor']; ?>" /></td>
  31. </tr>
  32. <tr>
  33. 	<td><b>data:</b></td>	<td><? echo $pokaz['data']; ?></td>
  34. </tr>
  35.  
  36. <tr>
  37. 	<td></td>	
  38. 	<td>
  39. 	</td>
  40. </tr>
  41. </table>
  42. <input type="submit" value="Zmien dane" />
  43. </form>
  44.  
  45. <? 
  46. }
  47.  
  48. }else{ header("location:index.php"); } 
  49.  
  50. $id=$_GET['id'];
  51. $tytul = $_POST['tytul'];
  52. $tresc = $_POST['tresc'];
  53. $autor = $_POST['autor'];
  54.  
  55.  
  56.  
  57. mysql_query("UPDATE `news` SET `tytul` = '$tytul', `tresc` = '$tresc', `autor` = '$autor' WHERE id='$id'"); 
  58.  
  59. }
  60.  
  61. ?>
[PHP] pobierz, plaintext
zomb
7.11.2009, 16:54:38
tak, dodałem tą klamrę na końcu. Teraz nie potrzeba tej klamry w linii 59. Wgrałem to lecz niestety znowu nic się nie pokazuje:
http://91.121.10.107/busines/news_edytuj.php?id=9
oraz co z plikiem news_pokaz.php pokazuje się tylko 1 news i kilka przycisków Edytuj news.
potreb
7.11.2009, 17:32:26
Pomaganie w kodzie nie oznacza, że ktoś ma ci dawać do wklejenia całego przerobionego kodu. Trochę własnej inwencji. Pokazuj to co do tej zrobiłeś. To, że podasz nam url nie oznacza, że wszystko jest dla nas wiadome.

I najważniejsze, mam dane administratora, preparuje formularz i mogę zrobić sql injection lub nawet przez zwykłego geta. Przydałoby się przynajmniej minimalnie zabezpieczyć przed takimi rzeczami.


[PHP] pobierz, plaintext 
  1.  
  2. if(isset($_POST['submit'])) {
  3. $id=$_GET['id'];
  4. $tytul = $_POST['tytul'];
  5. $tresc = $_POST['tresc'];
  6. $autor = $_POST['autor'];
  7.  
  8.  
  9.  
  10. mysql_query("UPDATE `news` SET `tytul` = '$tytul', `tresc` = '$tresc', `autor` = '$autor' WHERE id='$id'"); 
  11. }
[PHP] pobierz, plaintext


Nie trzeba klamry :/ zauważ, że musisz mieć przynajmniej jakiś warunek dla edycji, bo edycja za każdym razem będzie się robiła, tyle że nie ma wartości, a parametr id przekazywałbym poprzez formularz i sprawdzałbym przy okazji czy jest to liczba.
zomb
7.11.2009, 19:30:46
zapomniałem dodać ale to chyba jest jasne że jestem początkujący ... może mi powiecie jak to zabezpieczyć? Naniosłem kilka poprawek w tym co mi podaliście typu 'id='$id' na id=.. i pokazywanie newsów działa już prawidłowo. Gdy kliknę Edytuj news pojawia mi się formularz lecz gdy zatwierdzę zmiany dane się nie zmieniają ;/ oto news_edytuj.php:
[PHP] pobierz, plaintext 
  1. <?php
  2. include("config.php");
  3.  
  4.  
  5. $id=$_GET['id'];
  6.  
  7. $zapytanie = "SELECT * FROM news WHERE id='$id'";  
  8.  
  9. $wykonaj = @mysql_query ($zapytanie);
  10. while($pokaz = @mysql_fetch_array($wykonaj)){
  11.  
  12. ?>
  13. <form action="news_edytuj.php?id=<?$id?>" method="post">
  14. <table>
  15. <tr>
  16. 	<td width="132"><b>id :</b></td>	
  17. 	<td width="139"><? echo $pokaz['id']; ?></td>
  18. </tr>
  19. <tr>
  20. 	<td><b>tytul :</b></td>	
  21. 	<td><input name="tytul" type="text" value="<? echo $pokaz['tytul']; ?>" /></td>
  22. </tr>
  23. <tr>
  24. 	<td><b>tresc :</b></td>	<td><input name="tresc" type="text" value="<? echo $pokaz['tresc']; ?>" /></td>
  25. </tr>
  26. <tr>
  27. 	<td><b>autor :</b></td>	<td><input name="autor" type="text" value="<? echo $pokaz['autor']; ?>" /></td>
  28. </tr>
  29. <tr>
  30. 	<td><b>data:</b></td>	<td><? echo $pokaz['data']; ?></td>
  31. </tr>
  32.  
  33. <tr>
  34. 	<td></td>	
  35. 	<td>
  36. 	</td>
  37. </tr>
  38. </table>
  39. <input type="submit" value="Zmien dane" />
  40. </form>
  41.  
  42. <? 
  43. }
  44.  
  45. if(isset($_POST['submit'])) {
  46. $id=$_GET['id'];
  47. $tytul = $_POST['tytul'];
  48. $tresc = $_POST['tresc'];
  49. $autor = $_POST['autor'];
  50.  
  51.  
  52.  
  53. mysql_query("UPDATE news SET `tytul` = '$tytul', `tresc` = '$tresc', `autor` = '$autor' WHERE id='$id'"); 
  54. }
  55.  
  56.  
  57.  
  58. ?>
[PHP] pobierz, plaintext
potreb
7.11.2009, 20:11:09 
[HTML] pobierz, plaintext 
  1. <input type="submit" name="submit" value="Zmien dane" />
[HTML] pobierz, plaintext


Warunek isset($_POST['submit']) sprawdza czy jest taka tablica, więc trzeba było dodać do submit name="submit".

Zamień mysql_query("UPDATE ... poniższym kodem.
[PHP] pobierz, plaintext 
  1. $sql = "UPDATE news SET `tytul` = '$tytul', `tresc` = '$tresc', `autor` = '$autor' WHERE id='". $id."' ";
  2. echo $sql; // wyświetlę sobie zmienną $sql i wtedy zobaczę czy zapytanie jest okey
  3. mysql_query($sql);
  4. echo mysql_error(); // w razie wystąpienia błędu dostanę info o błędzie
[PHP] pobierz, plaintext


Przy okazji sprawdzę czy id jest liczbą:
[PHP] pobierz, plaintext 
  1. if(isset($_POST['submit']) && ctype_digit($_GET['id'])) {
[PHP] pobierz, plaintext


Przy okazji pousuwaj wszystkie małpy z przed funkcji.

Co to?? Skrótowy zapis?? Raczej nie.
[HTML] pobierz, plaintext 
  1. <form action="news_edytuj.php?id=<?$id?>" method="post">
[HTML] pobierz, plaintext


Raczej tak
[HTML] pobierz, plaintext 
  1. <form action="news_edytuj.php?id=<?php echo $id; ?>" method="post">
[HTML] pobierz, plaintext


Zrób o to co cię prosiłem i napisz o efektach.
zomb
7.11.2009, 20:25:19
UPDATE news SET `tytul` = '00000', `tresc` = '00000', `autor` = '00000' WHERE id='9'

W końcu wszystko działa jak należy winksmiley.jpg) dzięki wielkie. Mógłbyś mi jeszcze powiedzieć jak to zabezpieczyć ?
Blame
7.11.2009, 20:32:35
Poczytaj trochę o mysql_real_escape_string, addslashes i htmlspecialchars. Dodam tylko tyle, że musisz nimi potraktować wszystkie dane które użytkownik wpisuje do formularza.
zomb
7.11.2009, 21:07:19
ok, wiem o co z tym chodzi, w najbliższym czasie poprawię to. Teraz skrypt wygląda tak:
[PHP] pobierz, plaintext 
  1. <?php
  2. include("config.php");
  3.  
  4.  
  5. $id=$_GET['id'];
  6.  
  7. $zapytanie = "SELECT * FROM news WHERE id='$id'";  
  8.  
  9. $wykonaj = @mysql_query ($zapytanie);
  10. while($pokaz = @mysql_fetch_array($wykonaj)){
  11.  
  12. ?>
  13. <form action="news_edytuj.php?id=<?php echo $id; ?>" method="post">
  14. <table>
  15. <tr>
  16. 	<td width="132"><b>id :</b></td>	
  17. 	<td width="139"><? echo $pokaz['id']; ?></td>
  18. </tr>
  19. <tr>
  20. 	<td><b>tytul :</b></td>	
  21. 	<td><input name="tytul" type="text" value="<? echo $pokaz['tytul']; ?>" /></td>
  22. </tr>
  23. <tr>
  24. 	<td><b>tresc :</b></td>	<td><input name="tresc" type="text" value="<? echo $pokaz['tresc']; ?>" /></td>
  25. </tr>
  26. <tr>
  27. 	<td><b>autor :</b></td>	<td><input name="autor" type="text" value="<? echo $pokaz['autor']; ?>" /></td>
  28. </tr>
  29. <tr>
  30. 	<td><b>data:</b></td>	<td><? echo $pokaz['data']; ?></td>
  31. </tr>
  32.  
  33. <tr>
  34. 	<td></td>	
  35. 	<td>
  36. 	</td>
  37. </tr>
  38. </table>
  39. <input type="submit" name="submit" value="Zmien dane" />
  40. </form>
  41.  
  42. <? 
  43. }
  44.  
  45. if (isset($_POST['submit']) && ctype_digit($_GET['id'])) {
  46. $id=$_GET['id'];
  47. $tytul = $_POST['tytul'];
  48. $tresc = $_POST['tresc'];
  49. $autor = $_POST['autor'];
  50.  
  51. $sql = "UPDATE news SET `tytul` = '$tytul', `tresc` = '$tresc', `autor` = '$autor' WHERE id='". $id."' ";
  52. mysql_query($sql) or die("Wystąpił błąd" );
  53.  
  54. echo '<font style="size:14px;position:relative;margin:0px auto;top:300px;"><center><b>Dane zostały zmienione.</b> <br /> Zachwile zostaniesz przekierowany na strone główną.<br />Jeżeli sie to sie stanie kliknij <a href="index.php">tutaj</a>.</font></center><meta http-equiv="refresh" content="4; URL=index.php"> ';
  55.  
  56. }
  57. ?>
[PHP] pobierz, plaintext


teraz mam jeszcze jedno pytanie: jak dodać edycję tylko dla admina, bo każdy może sobie wejść i edytować to. Gdy dodaje do kodu " if ($pokaz['ranga'] == 'admin'){ " niestety znowu ale tylko białe tło ;/
thek
7.11.2009, 22:11:00
A od kiedy to w informacji o newsach wciskasz typ użytkownika?
$pokaz['ranga'] == 'admin' nie ma prawa działać, bo $pokaz przechowuje informacje o newsie, która to zmienna jest efektem zapytania: "SELECT * FROM news WHERE id='$id'"
Jak Tyś tam rangę wcisnął to nie wiem smile.gif
Musisz informację o użytkowniku wyciągnąć z bazy w locie, albo pobrać jego uprawnienia z sesji. Dopiero na tej podstawie decydujesz czy user ma prawo edytować czy nie.
zomb
11.11.2009, 10:38:45
ok, fakt, nie zauważyłem.
Jeszcze jedno pytanie: jak "odwrócić" kolejność pokazywania newsów. Chodzi o to, że jak dodam teraz jakiś news to pokazuje on się "na dole" a nie "na górze".
Czekam na pomoc
Blame
11.11.2009, 10:59:17
Przy $zapytanie dajesz:
[PHP] pobierz, plaintext 
  1. $zapytanie = "SELECT * FROM news WHERE id='$id' ORDER BY id desc";
[PHP] pobierz, plaintext
zomb
11.11.2009, 14:46:55
ok, dziek, jeżeli to nie będzie problemem to potrzebowałbym jeszcze jednej rzeczy: podział newsów na strony ... Czy ktoś mógłby mi powiedzieć jak to zrobić bo chciałbym już ukończyć tą strone.
Blame
11.11.2009, 15:08:04
To się nazywa stronicowanie poszukaj w google, tego jest od groma.
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.
Invision Power Board © 2001-2025 Invision Power Services, Inc.