Javascript help
Discussion
I'm trying (and failing!) to get the confirm function working under anything other than IE. My code is as follows:
This works fine under IE but the form won't sumbit under Netscape or Firefox.
Any suggestions?
I wrote said:I've had to put a space infront of all the html tags as PH seems to strip them off
< script language="Javascript">
function AreYouSure() {
if (confirm("Do you really want to do this?"))
DeleteForm.submit();
else
return false;
}
< /script>
< form action="deletestuff.php" method="post" name="DeleteForm">
.
.
.
.
.
.
< table border="0">
< tr>
< td width="4" background="but4.gif">< /td>
< td background="but5.gif" class="PicButton" onclick="AreYouSure();">< font color="#222222">Delete selected< /font>
< /td>
< td width="4" background="but6.gif">< /td>
< /tr>
< /table>
< /form>
This works fine under IE but the form won't sumbit under Netscape or Firefox.
Any suggestions?
Rather than having the javascript submit the form, why not instead have it redirect to the page with a URL paramater ($_GET in php) set, which would be the confirmation to delete.
For example:
DeleteDtuff.php would pop up the message when called.
Clicking yes would redirect to:
deleteStuff.php?confirm=1
PHP will then know it's ok to delete.
For example:
DeleteDtuff.php would pop up the message when called.
Clicking yes would redirect to:
deleteStuff.php?confirm=1
PHP will then know it's ok to delete.
Stick an alert before and after the submit to make sure it is running the code you think it is.
You function returns false in one case and nothing in the other, they can't both be right? I don't see any need for it to return anything.
As a matter of principle, put curly braces round every if/else clause to avoid control logic flaws when you add debug statements.
It may be that the reference to DeleteForm is not resolving correctly so you are getting a null. You could check for this explicitly. I tend to look elements up by ID, it isn't always necessary but I've got into the habbit because the name references don't always work.
You function returns false in one case and nothing in the other, they can't both be right? I don't see any need for it to return anything.
As a matter of principle, put curly braces round every if/else clause to avoid control logic flaws when you add debug statements.
It may be that the reference to DeleteForm is not resolving correctly so you are getting a null. You could check for this explicitly. I tend to look elements up by ID, it isn't always necessary but I've got into the habbit because the name references don't always work.
Excellent! That's what the problem was! Many thanks 
Gassing Station | Computers, Gadgets & Stuff | Top of Page | What's New | My Stuff