Forms not working right, please help.

[ajcether]

I know I posted this code in another post, but since then I've decided that that post doesn't describe my problem. Here is the code:

function setSustainability( $pid, $psust, $conn){ 
	 if(isset($_POST['sustainability'.$pid ])){ // means the checkbox is checked
		mysql_query(..change db value to 1);
	}else { // the checkbox isn't checked
		mysql_query(...change db value to 0);
	}	
}

<? while ( $p = mysql_fetch_assoc($res) ): ?>
	 <form method="POST" name="sustForm<?=$pid?>">
         <label for="sustainability<?=$pid?>">Sustainability:</label>
         <input type="checkbox" name="sustainability<?=$pid?>" <?php if ($p['sustainability'] == 1)  echo 'checked="true"'; ?> /> (reload page from sidebar to update checkbox)
	 <input type="submit"  name="submit_sust<?=$pid?>" value="Save" onClick=<?= setSustainability($pid, $p['sustainability'], $conn); ?>/>
         </form>
<? endwhile; ?>

This looks to me like it should do exactly what I want it to do. It should let me use a checkbox to change a value to 1 if checked, 0 if unchecked, in a database. It does...but for only 1 form. It appears that the setSustainability() method is being called when the page loads instead of only when the "Save" buttons are clicked. (If I check a box, save it, and then reload the page, the values are correctly changed in the db table, and the checkbox is checked as it should be...but when I reload it again, then the values are set back to 0 and the box is unchecked). Note: I am not "refreshing" the page and therefore resending post data..I am just clicking a link to the page.

This makes absolutely no sense to me, I've given each form a different name using the id of the db item, so shouldn't the method only get called when a "Save" is clicked, and only for that button's form in particular?

Please help me out here!
Thanks

[Piranha]

If that post doesn't describe your problem you can edit the post to describe it. Or, now that you already have posted in another thread, edit the post and write a comment on the top of the post that this was not the problem. Then people don't have to read all the post before realising that you don't need help with it after all.

[ajcether]

I added a comment at the top of the post, sorry about that...I didn't think it made sense to edit and put this in a thread where the topic is checkbox values

[ajcether]

Okay, so I guess i can't call a php function with onClick...I need to use a javascript function instead....I have no idea how this works. Could anyone explain how the following works?(its from another part of this site, but I think it does something similar to what I want):

profile.js

function setReleased(id, strReleased) {
	new Ajax.Request(url+'profile_release_handler.php', {
		method: 'post',
		onSuccess: function(xhr) {
			$('releaseSwitch['+id+']').innerHTML = xhr.responseText.replace(/<\/?span.*?>/g, '');
		},
		onFailure: function(xhr) {
			alert("Error "+xhr.status+": "+xhr.statusText+'\n'+xhr.responseText);
		},
		parameters: {id: id, setrelease: strReleased}
	});
}

profile_release_handler.php

function printReleaseSwitch ($id, $strReleased) {
	echo "<span id='releaseSwitch[$id]'>";
	if ($strReleased == "1") {
		echo "Released ";
		echo "<a href='javascript:setReleased($id, 0);'>Unrelease</a>";
	} else {
		echo "<a href='javascript:setReleased($id, 1);'>Release</a> ";
		echo "Unreleased";
	}
	echo "</span>";
}

function doReleaseSwitch ($id, $strReleased) {
	global $conn;
 	mysql_query(change the released value in db);
	return ($strReleased == '0') ? '0' : '1';
 }

if (isset($_POST['setrelease'], $_POST['id'])) {
	$newVal = doReleaseSwitch($_POST['id'], $_POST['setrelease']);
	printReleaseSwitch($_POST['id'], $newVal);
}

This uses links to select which value the database entry has...how could I change it around to use a checkbox?