страница редактирования новостей PHP

[Alexander1311]

здравствуйте!!!При написании странички редактирования новостей возникла проблема,планировалось что на страницу будут выводится все новости из БД, а при выборе конкретной(при клике) должна выводится новость в форме для редактирования.При этом все должно происходить на одной странице,но к сожалению функция if не срабатывает и при клике форма не выводится((
вот код который я прописал:

PHP код:

if (!isset ($id))
{
$result = mysql_query("SELECT title,id FROM news");      
$myrow = mysql_fetch_array($result);

do 
{
printf ("<p><a href='admin_edit_news.php?id=%s'>%s</a></p>",$myrow["id"],$myrow["title"]);
}

while ($myrow = mysql_fetch_array($result));

}

else
{

$result = mysql_query("SELECT * FROM news WHERE id=$id");      
$myrow = mysql_fetch_array($result);

print <<<HERE
                 <h2>Редактировать новость :</h2>
             <p><form action="update_news.php" method="POST">
             <p>Название :<br>
             <input name="title" id="title" type="text" size="50" value="$myrow [title]"> </p>
             <p>Краткое описание :<br>
             <input name="meta_d" id="meta_d" type="text" size="50" value="$myrow [meta_d]"> </p>
             <p>Ключевые слова :<br>
                <input name="meta_k" id="meta_k" type="text" size="50" value="$myrow [meta_k]"> </p>      
             <p>Дата :<br>
                <input name="date" id="date" type="text" value="$myrow [date]"></p>
             <p>Маленькая картинка для главной страницы (100x130 без тегов!) :<br>
                <input name="mini_img" type="text" size="50" value="$myrow [mini_img]"></p>
             <p>Маленькое описание для главной страницы:<br>
             <textarea name="discription_index" id="discription_index" cols="60" rows="5" maxlength="150" value="$myrow[discription_index]"></textarea> </p>
             <p>Краткое описание:<br>
             <textarea name="discription" id="discription" cols="60" rows="20" maxlength="1500" value="$myrow [discription]"></textarea> </p>
             <p>Tекст:<br>
             <textarea name="text" id="text" cols="60" rows="40" value="$myrow [text]"></textarea> </p>
             <input name="id" type="hidden" value="$myrow [id]">
             <input type="submit" value="внести изменения">
              </form> </p>
HERE;
} 


подскажите в чем проблема заранее благодарен!!!!

[ADSoft]

что есть $id откуда он здесь берется? ... судя по всему $id не существует при выполнении данного скрипта ...

[Andkorol]

Сообщение от Alexander1311

но к сожалению функция if не срабатывает и при клике форма не выводится((

printf ("<p><a href='admin_edit_news.php?id=%s'>%s</a></p>",$myrow["id"],$myrow["title"]);

Надо полагать, в вашем условии нужно использовать не просто $id - а $_GET['id'].

[Alexander1311]

я знаю что проблема в $id но я думал что она подставляется в эту строчку<a href='admin_edit_news.php?id=%s'>
извините что задаю глупые вопросы но я пока не силен в php.
подскажите как правильно прописать?

[ADSoft]

смотри предыдущий пост....
и рекоментовал бы $id = intval($_GET['id']) чтобы исключить SQL иньекции

[Alexander1311]

если вам не трудно можете прописать исправление прямо в коде а то у меня не получается))

вот примерно так???? if (!isset ($id)) заменить на if (!isset ( $_GET['id']))

[Andkorol]

Сообщение от Alexander1311

Andkorolесли вам не трудно можете прописать исправление прямо в коде а то у меня не получается))

Что там может не получиться???
Добавьте в начало скрипта вариант, предложенный ADSoft - и всё.
Детский сад, блин...

[Alexander1311]

извините,но при добавлении предложенного варианта срабатывает else и на страницу выводится форма.

[Andkorol]

!isset() в условии замените на empty():

PHP код:

$id = (!empty($_GET['id'])) ? intval($_GET['id']) : 0;
if(empty($id)){
// ваш дальнейший код в условии ... 


[Alexander1311]

все сделал, но снова выводится форма вместо заголовков