PHP Mysql UPDATE failure?

[Pikachu2000]

It looks like it should work as written. the only thing I see that may be an issue is you've used the short open <? tags in the table cells. It's better practice to always use the full <?php tag syntax, since short open tags are now off by default.

To determine if that is what's happening, look at the page's HTML source, and you'll see php code in there just as it is in the script.

[drdrew]

Thanks it was the short tags. Thanks for all your help, if you need auto parts I work for a AP company just ask.

[Pikachu2000]

Glad to help out. Please remember to mark the thread as solved (if it indeed is), using the button in the lower left corner.

 

If you're in the Dallas area, I may just take you up on the auto parts. I need a couple things soon . . .

[drdrew]

We ship, email me Drew@DrewToma.com OR Drew@Drdda.com. I recently shipped a o2 sensor to texas.

[Pikachu2000]

OK, I'll do that. I'm going to need a new brake booster soon. Mine is leaking P/S fluid all over everything (Ford diesel truck) . . .  Getting sick of seeing it all over the driveway, lol.

[drdrew]

Ya when you do need parts email me up w/ your car info or vin. If you have the part number cool in not I have a look up. We are rocking a AS/400.

[Pikachu2000]

I **LOVE** AS/400s. Haven't worked with one in quite a while, though. (Since V4R5 was brand new, LOL)

[drdrew]

Ya I like it to, in the many years we have had it only 6hrs of down time.

 

Quick Q This script below doesn't delete the record, any idea why?

<?php
include("../scripts/dbconnect.php");

mysql_select_db("rdb")or die("cannot select DB");

$sql="SELECT * FROM wo";
$result=mysql_query($sql);

$count=mysql_num_rows($result);

?>
<table width="400" border="0" cellspacing="1" cellpadding="0">
<tr>
<td><form name="form1" method="post" action="">
<table width="400" border="0" cellpadding="3" cellspacing="1" bgcolor="#CCCCCC">
<tr>
<td bgcolor="#FFFFFF"> </td>
<td colspan="4" bgcolor="#FFFFFF"><strong>Remove Wo</strong> </td>
</tr>
<tr>
<td align="center" bgcolor="#FFFFFF">|=|</td>
<td align="center" bgcolor="#FFFFFF"><strong>Won</strong></td>
<td align="center" bgcolor="#FFFFFF"><strong>CName</strong></td>
<td align="center" bgcolor="#FFFFFF"><strong>DWS</strong></td>
<td align="center" bgcolor="#FFFFFF"><strong>CEmail</strong></td>
</tr>
<?php
while($rows=mysql_fetch_array($result)){
?>
<tr>
<td align="center" bgcolor="#FFFFFF"><input name="checkbox[]" type="checkbox" id="checkbox[]" value="<?php echo $rows['won']; ?>"></td>
<td bgcolor="#FFFFFF"><?php echo $rows['won']; ?></td>
<td bgcolor="#FFFFFF"><?php echo $rows['cname']; ?></td>
<td bgcolor="#FFFFFF"><?php echo $rows['dws']; ?></td>
<td bgcolor="#FFFFFF"><?php echo $rows['cemail']; ?></td>
</tr>
<?php
}
?>
<tr>
<td colspan="5" align="center" bgcolor="#FFFFFF"><input name="delete" type="submit" id="delete" value="Delete"></td>
</tr>
<?php
if($delete){
for($i=0;$i<$count;$i++){
$del_id = $checkbox[$i];
$sql = "DELETE FROM wo WHERE id='$del_id'";
$result = mysql_query($sql);
}

if($result){
echo "Yes";
}
}
mysql_close();
?>
</table>
</form>
</td>
</tr>
</table>

[Pikachu2000]

Just looked it over quickly, but I think if($delete){ should be if( isset($_POST['delete']) ) {. $delete by itself would have no value assigned to it. Something else looks funny about it, but I'm far too tired to put my finger on it.

[drdrew]

Ty ill check it out tmrw, what looked funny was probably the else at the end.

[drdrew]

I tried that and the code didn't work. Do you think I should make a new post?