Re: mysql_affected_rows() function

From: "Tyler Longren" <tyler@jdhost.com> > That's not the exact code. The exact code is below. Multiple workers > are being selected from a MULTIPLE <select> form field. I just use a > while loop do go through the selected ones to delete them individually. > > PHP Version: 4.3.4 > MySQL Version: 4.0.17 > > $i="0"; Don't put quotes around numbers. $i = 0; is all you need. > while ($i <= "$selected") { You also don't need quotes around just a variable. ($i <= $selected) is all you need. > $delete_assignment = mysql_query("DELETE FROM webprojectassign WHERE > workerid='$assigned[$i]'"); > $i++; > } > if (mysql_affected_rows($delete_assignment) < "1") { > $tpl->newBlock('message'); > $tpl->assign("message","<b>ERROR:</b> There was an error deleting > those > workers from the project."); > } > else { > $tpl->newBlock('message'); > $tpl->assign("message","The workers have been deleted from the > project."); > } The problem you're having is that your IF check of mysql_affected_rows() is outside of your WHILE loop. So, you're executing a series of DELETE queries, yet only checking mysql_affected_rows() for the very last one (which evidently does not actually delete anything). You could probably benifit from a "DELETE FROM webprojectassign WHERE worderid IN (1,2,3,45)" syntax, too, so you only need to do a single query. Since it's coming from a MULTIPLE <select>, a simple $list = implode(',',$_POST['select_name']); will create the list. Although you'd be better off looping through each one making sure it's a number, though, to prevent any SQL injection attacks. Ask if you need anymore info. :) ---John Holmes...