Hey guys, i'm having a bit of trouble with this news updating system...I have a main manage page which lists all of the news entries and beside each one there is an "edit entry" link which brings it to a page where the user is able to update the information and then submit it.
Here is the code for the page that "Edit Entry" is linked too.

PHP Code:

<?

require_once('../../mysql_connect.php');
$query = "SELECT name, date, message FROM news WHERE id=$id";
$result = @mysql_query($query);
$row = mysql_fetch_array ($result, MYSQL_NUM);
echo '<div id="wrapper" align="center">
<span class="title"><b>Edit News Entry</b></span><br /><br />
<form method="post" action="editnews_handler.php">

<span>Date:</span><br />

<INPUT TYPE="TEXT" NAME="date" VALUE="' . $row[1] . '" size=20><br /><br>

<span>Message:</span><br />
<textarea name="message" cols="60" rows="5" wrap="physical">' . $row[2] . '</textarea>
<br />
<span style="font-size:10px">**Try to keep image\'s width smaller than 400 pixles!</span>
<br />
<br />

<span>Name:</span><br />

<div class="submit"><INPUT TYPE="TEXT" NAME="name" VALUE="' . $row[0] . '"  size=40 ></div><br /><br />

<INPUT TYPE="submit" name="submit" value="submit news">

</form>
</div>';

mysql_close();
?>

This page displays the information in the form perfectly. The problem is it's handler. Which is this:

PHP Code:

<? 
if($_POST['submit']) 
{
require_once('../../mysql_connect.php');
$query = "UPDATE news SET message='$a', name='$b', date='$c' WHERE id=$id";
$result = mysql_query($query);}
?>
Query Updated.
<br />
<a href="/manage" target="_self">Go Back</a>

Now I know i have not named these variables yet, but i'm not really sure how, or even if this method is correct.
Any help is very appreciative. Thanks!

What is the error message that you are getting?

Roj

Now im not getting an error, its just not updating...

I am trying to make it so when you click on "Edit Entry", it displays the entry in a new page and also the information in the correct form. Which it does. But when you click to submit the new information, it is soposed to update the old info with a handler page. The mySQL database has 4 tables. id, name, date, and message. I am not sure how to write the handler page for updated submited information.

I changed the handler's page to this (whcih is the page that should update the current information, but it just isn't having any effect.

PHP Code:

<?
if($_POST['submit'])
{
$name = $_POST['name'];
$date = $_POST['date'];
$message = $_POST['message'];
$id = $_GET['id'];

require_once('../../mysql_connect.php');
$query = "UPDATE news SET message='$message', name='$name', date='$date' WHERE id=$id";
$result = mysql_query($query);}
?>

Hi there,

You need to pass the $id variable in the form as it is only the form fields that will be passed to the handler.
Basically the handler is not picking up the $_GET['id'] variable as it is not passed on. You need to insert a hidden form field called id that has the value of the news item id you are trying to update.

Hope this helps

Ian

hmm interesting. Thanks for the help...
This is what I did, I'm probably doing something wrong.

PHP Code:

<?

require_once('../../mysql_connect.php');
$query = "SELECT name, date, message FROM news WHERE id=$id";
$result = @mysql_query($query);
$row = mysql_fetch_array ($result, MYSQL_NUM);
echo '<div id="wrapper" align="center">
<span class="title"><b>Edit News Entry</b></span><br /><br />
<form method="post" action="editnews_handler.php">

<span>Date:</span><br />

<INPUT TYPE="TEXT" NAME="date" VALUE="' . $row[1] . '" size=20><br /><br>
<input type="hidden" name="id" value="'.$row['1'].'" />

<span>Message:</span><br />
<textarea name="message" cols="60" rows="5" wrap="physical">' . $row[2] . '</textarea>
<br />
<span style="font-size:10px">**Try to keep image\'s width smaller than 400 pixles!</span>
<br />
<br />

<span>Name:</span><br />

<div class="submit"><INPUT TYPE="TEXT" NAME="name" VALUE="' . $row[0] . '"  size=40 ></div><br /><br />

<INPUT TYPE="submit" name="submit" value="submit news">

</form>
</div>';

mysql_close();
?>

I only added a hidden field for the date to test. You can also see it in the source code on the edit entry page. I kind of understand what your saying, but I need a little assistance. Thanks alot.

I guess I don't know where to put the hidden field. The handler or the page with the form?

Hi there again,

Sorry its a bit late, but I've been kind of busy. The hidden field goes into the form where you have put it, but your coding style is different to mine and I don't follow it that well. What I have noticed is that you are using the same row element for the date field AND the id field. This cannot be right as the two row elements for id and date must be distinct from each other (ie different columns in your table.

If I assume that your table is set out as follows:

col[0] = name, col[1] = date, col[2] = message and col[3] = id, the following code should work:

PHP Code:

<? 

require_once('../../mysql_connect.php'); 
$query = "SELECT name, date, message FROM news WHERE id=$id"; 
$result = @mysql_query($query); 
$row = mysql_fetch_array ($result, MYSQL_NUM); 
echo '<div id="wrapper" align="center"> 
<span class="title"><b>Edit News Entry</b></span><br /><br /> 
<form method="post" action="editnews_handler.php"> 

<span>Date:</span><br /> 

<INPUT TYPE="TEXT" NAME="date" VALUE="' . $row[1] . '" size=20><br /><br> 
<input type="hidden" name="id" value="' . $row[3] . '" /> 

<span>Message:</span><br /> 
<textarea name="message" cols="60" rows="5" wrap="physical">' . $row[2] . '</textarea> 
<br /> 
<span style="font-size:10px">**Try to keep image\'s width smaller than 400 pixles!</span> 
<br /> 
<br /> 

<span>Name:</span><br /> 

<div class="submit"><INPUT TYPE="TEXT" NAME="name" VALUE="' . $row[0] . '"  size=40 ></div><br /><br /> 

<INPUT TYPE="submit" name="submit" value="submit news"> 

</form> 
</div>'; 

mysql_close(); 
?>

Let me know how you get on.
Regards
Ian

Still not updating...I'm starting to think it may be that the handler page is wrong. Which is this now.

PHP Code:

<?
if($_POST['submit'])
{
$name = $_POST['name'];
$date = $_POST['date'];
$message = $_POST['message'];
$id = $_GET['id'];

require_once('../../mysql_connect.php');
$query = "UPDATE news SET message='$message', name='$name', date='$date' WHERE id=$id";
$result = mysql_query($query);
} else {
echo 'Query did not update.;';
}
?>

It's weird because i'm not getting any errors.

Doh! - I'm a clutz!!

You're right about the handler! You need to change the $id variable from $_GET to $_POST.

Ian

So close!!! I think I figered out the problem. The hidden feild isn't picking up the id number. Like

PHP Code:

<input type="hidden" name="id" value="' . $row[3] . '" /> 


is just showing a blank value in the source code.

I got it! Thanks so much for the help. The problem was also in the form page.

PHP Code:

$query = "SELECT name, date, message FROM news WHERE id=$id"; 


forgot to select "id" which was teh reason why the source code was displaying a blank value.