One button click call function more than once

Question

I have a function I call from a button (onClick) which
inserts a record in mySQL via a PHP page. For some odd reason it
tends to insert more than 1 record. The extra records tend to be
duplicates of previous records. I do not know of SPRY is calling
the function multiple times, if stuff is cached somewhere and it
calls the cached data and the new data. I perform record INSERTS
and UPDATES on many web pages via PHP which get called thousands of
times but only this SPRY application is doing this. Please see my
code below.
Spry.Utils.loadURL function: (not that I want to use POST but
having an issue and so using GET)
function addcommentget(recordID)
var newcomment = document.form1.tescomment.value
Spry.Utils.loadURL("GET","addTEScomment.php?id="+recordID+"&comment="+newcomment,true,Upda teRecordCallback);
FORM code (where button and textarea are)
<form action="" method="post" name="form1">
Add Comment: </br>
<textarea name="tescomment" cols="40"
rows="2"></textarea></br>
<input name="addcomment" type="button" value="Submit
Comment" onClick="addcomment('{@id}');">
</form>
PHP code:
<?php session_start();
require_once('Connections/ppg_conn.php');
$id = $_GET['id'];
$comment = $_GET['comment'];
$user = $_SESSION['username'];
$sql = mysql_query("INSERT INTO
claimsmanager_tesedits_comments (Ticket_ID, Commenter_email,
Comment)
VALUES('$id', '$user', '$comment')
") or die (mysql_error());
?>
Is there anything you see that might cause this strange
behavior?
Thanks,
Lee

Answer 1

Spry.Utils.loadURL() only fires off a single request when you
call it.
Aside from the fact that your button calls "addcomment" as
opposed to the "addcommentget" function you show ;-), I don't see
any problem. Of course I'm not seeing the big picture, only what
you post. Do your data sets load URLs that actually perform DB
updates? Or are they pure DB query URLS?
I would put a line like this:
Spry.Debug.trace("AddCommentGet() triggered!");
in your addcommentget() function and see if you get a single
trace statement every time you press the button.
--== Kin ==--