zrobiłem formularz edycji newsa czyli wczytałem do inputów rekordy z bazy

pod adresem ...../edit_news.php?newsid=1 wyświetla się właśnie ten news do edycji o podanym id

teraz jak napisać zapytanie UPDATE żeby właśnie aktualizowało news o id podanym w adresie

[PHP] pobierz, plaintext 
<?php
$zapytanie = "UPDATE news SET title='".$title."', newspartone='".$newspartone."', newsparttwo='".$newsparttwo."'" ;
?>
[PHP] pobierz, plaintext 

ja mam powyższe zapytanie i aktualizuje wszystkie rekordy a tabeli ;/

W zapytaniu dodaj WHERE np. jeśli chcesz pierwszy news to mmyśle, że dobrze będzie jeśli wybierzesz numer id z paska adresu i wpiszesz właśnie w WHERE.

[PHP] pobierz, plaintext 
<?php
$zapytanie = "UPDATE news SET title='".$title."', newspartone='".$newspartone."', newsparttwo='".$newsparttwo."' WHERE id = '".$id."'" ;
?>
[PHP] pobierz, plaintext 

@Fantazyn podał dobre rozwiązanie, tylko w zmienną ID wsadź sobie $_GET['newsid'].

jak dam $_GET['newsid'] wywala błąd a jak dam $id to tez klapa bo nie jest zdeklarowana
Parse error: syntax error, unexpected T_VARIABLE in......

~Qss, proponuję zacząć od jakiegoś kursu W komunikacie błędu masz napisane co jest źle poza tym.

Pozdrawiam!

może rzucę na to nico więcej światła

plik edit_news.php (z formularzem od edycji)

[PHP] pobierz, plaintext 
<?php
require('db_connect.php');
 
 
if(intval($_GET['newsid']))
{
 
  echo '<p><a href="index.php">Najnowsze newsy</a></p>';
 
 
  $sql="SELECT dodano, title, newspartone, newsparttwo FROM news WHERE id='".intval($_GET['newsid'])."'";
  $wynik=mysql_query($sql);
  if($r=mysql_fetch_assoc($wynik))
  {
  
  echo '<form method="post" action="edit_przetworz.php">';
  
echo ' Temat:<br /><input class="t_news" name="title" type="text" size="19" maxlength="200" value="'.$r['title'].'"><br />
Pierwsza część newsa:<br /><textarea class="first_news" name="newspartone" type="text" cols="" rows="" >'.$r['newspartone'].'</textarea><br />
Druga część newsa:<br />
<textarea class="second_news" name="newsparttwo" type="text" cols="" rows="" >'.$r['newsparttwo'].'</textarea><br />
<input name="addnews" type="submit" value="Dodaj"><input name="resetnews" type="reset" value="Wyczyść wszystko"></form>';
 
 
   }}
        else
        {
          $sql="SELECT id, title FROM news";
  $wynik=mysql_query($sql);
  while($r=mysql_fetch_assoc($wynik))
  {
    echo $r['title'].'<br />';
    echo '<a href="edit_news.php?newsid='.$r['id'].'">edytuj</a><br />';
 
  }
        }
?>
[PHP] pobierz, plaintext 

i drugi to przetwarzania tego co wpisane w inputach... edit_przetworz.php

[PHP] pobierz, plaintext 
<?php
$title = $_POST['title'];
$newspartone = $_POST['newspartone'];
$newsparttwo = $_POST['newsparttwo'];
 
require('db_connect.php'); 
$zapytanie = "UPDATE news SET title='".$title."', newspartone='".$newspartone."', newsparttwo='".$newsparttwo."' 
WHERE id ='"$_GET['newsid']"'";
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$title = $_POST['title'];
$newspartone = $_POST['newspartone'];
$newsparttwo = $_POST['newsparttwo'];
 
require('db_connect.php'); 
$zapytanie = "UPDATE news SET title='$title', newspartone='$newspartone', newsparttwo='$newsparttwo' WHERE id='{$_GET['newsid']}'";
?>
[PHP] pobierz, plaintext 

to wyżej to już wogole niedziała
bląd unexpected '{ w lini 8 więc {} z get nie mają sensu

[PHP] pobierz, plaintext 
<?php
$title = $_POST['title'];
$newspartone = $_POST['newspartone'];
$newsparttwo = $_POST['newsparttwo'];
$id=$_GET['newsid'];
 
require('db_connect.php');
$zapytanie = "UPDATE news SET title='".$title."', newspartone='".$newspartone."', newsparttwo='".$newsparttwo."' WHERE id='$id'";
?>
[PHP] pobierz, plaintext 

może tak?

to nic niedaje mam pytanie czy wogole moja koncepcja edytowania postów jest poprawna?

a co ci teraz wyskakuje ?
jeśli post mogą edytować wszyscy to filtruj dane:
mysql_real_escape_string()" title="Zobacz w manualu PHP" target="_manual

$title =mysql_real_escape_string($_POST['title']); itp...

Spróbuj napisać zapytanie do sql w ten sposób :

[SQL] pobierz, plaintext 
$zapytanie = "UPDATE news SET title='$title', newspartone='$newspartone', newsparttwo='$newsparttwo' WHERE id='$_GET[newsid]'";
[SQL] pobierz, plaintext 

czy w tych zmiennych masz wartości? czy łączysz się poprawnie z bazą danych? i przetestuj coś takiego?

[PHP] pobierz, plaintext 
<?php
$zapytanie = mysql_query("UPDATE news SET title='".$title."', newspartone='".$newspartone."', newsparttwo='".$newsparttwo."' WHERE id='$id'") or die(mysql_error());
?>
[PHP] pobierz, plaintext 

i powiedz czy zwraca jakieś błędy.

zechuj to zapytanie także zebysmy widzieli czy jak wyzej juz sie spytal piotro masz w nich wartosc np tak:

[PHP] pobierz, plaintext 
<?php
echo ("UPDATE news SET title='".$title."', newspartone='".$newspartone."', newsparttwo='".$newsparttwo."' WHERE id='$id'");
?>
[PHP] pobierz, plaintext 

ciężko szukać dziury w całym . Najlepiej wrzuć sobie linijkę na początku skryptu <?php
error_reporting(E_ALL);
ini_set('display_errors', 1);
?> . ayeo juz o tym wspomniał . Nam jest cięzko wychwycić błąd który jest mało widoczny .

Twoja koncepcja nie jest zła . Ja bym jednak te dwa pliki połączył w jeden . Więcej plików , większy młyn , więcej poprawiania i strata czasu .
Nie wiem po co to wrzuciłeś intval($_GE.......) , nie może być samo $_GET

~Qss, przesadzasz. Wyskakuje Ci, że masz błąd w linii 8, masz napisane dokładnie co jest nie tak! Tak trudno chwilkę pomyśleć? Szybciej sam to rozwiążesz zanim napiszesz kolejnego posta na forum. Piszesz o każdą durną literówkę! Myśl i czytaj komunikaty błędów!

Pozdrawiam!

wielkie dzięki za chęć pomocy ale już sobie sam poradziłem dzieki za rady i sorry za zaśmiecanie forum czasem ma po prostu dość i pisze ale znalazłem noew chęci i znalazłem błedy ;p