delete row from database - syntax error

Question

I found this code that does exactly what i need

http://www.daveismyname.com/tutorials/php-tutorials/delete-rows-from-a-mysql-database-with-a-confirmation/

When i copy the code into a new dreamweaver php file, it states that there is a syntax error in this part:

<?php
$result = mysql_query("SELECT * FROM news")or die(mysql_error());
while($row = mysql_fetch_object($result))
{
echo "<ul>n";
echo "<li>$row->newsTitle <a href="javascript:delnews('$row->newsID','$row->newsTitle')">Delete</a></li>n";
echo "</ul>n";
}
?>

more specifically in the echo <li>$row line.

if I try to upload it anyways it says:

Parse error: syntax error, unexpected T_STRING, expecting ',' or ';' in /home/asbj1076/public_html/testdel/admin.php on line 39

i can save it as a html5 file without that syntax error, but the code still aint working properly.

Any suggestions?

I know there has been a lot of similar questions already, sorry for that. I'm just a noob who's in over my head, so i need really specific help.

Thanks. Asbjørn

try echo "<li>$row->newsTitle <a href=\"javascript:delnews('$row->newsID','$row->newsTitle')\">Delete</a></li>n"; — M Khalid Junaid
– M Khalid Junaid, Commented Jul 18, 2013 at 19:25

Bad Wolf · Accepted Answer · 2013-07-18 19:40:00Z

2

Because of the double quotes " in your second echo statement PHP is throwing an error because it is trying to evaluate the HTML that is inside of those quotes.

echo "<li>$row->newsTitle <a href="javascript:delnews('$row->newsID','$row->newsTitle')">Delete</a></li>n";
                                   ^                                                  ^

Everything between those marks is trying to be evaulated as php and errors. Escape your quotes like this instead.

echo "<li>$row->newsTitle <a href=\"javascript:delnews('$row->newsID','$row->newsTitle')\">Delete</a></li>n";

answered Jul 18, 2013 at 19:40

Bad Wolf

8,3495 gold badges36 silver badges44 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

user2596886 Over a year ago

will try that out in a minute, its time for hotdogs now. thanks so much for the fast reply.

David Level Over a year ago

The answer is good but maybe you could seperate each part instead of writing php,html and js on the same line. Writing like this make the code really difficult to debug. After that he should remove mysql_* statement because it is deprecated now.

user2596886 Over a year ago

i can see what you say. fortunately it seems like fred already did that for me. thanks.

The Alpha · Accepted Answer · 2013-07-18 19:45:23Z

1

You may try (keep the ul out of loop)

echo "<ul>";
while($row = mysql_fetch_object($result))
{
    echo "<li>" . $row->newsTitle . " <a href='javascript:delnews(\'" . $row->newsID ."\', \'" . $row->newsTitle . "\')>Delete</a></li>";
}
echo "</ul>";

Output will be :

Hello Delete

answered Jul 18, 2013 at 19:45

The Alpha

147k30 gold badges294 silver badges313 bronze badges

Collectives™ on Stack Overflow

delete row from database - syntax error

2 Answers 2

3 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related