I've got a form, and I have a function that [is supposed to] takes the value of part of the form, and sets it as the hidden value called "user". If the user value is blank, for form will return false, and give an alert. For some reason, my function won't do anything, and I think it's because I use a variable in my document call... Here's my codes...

The Function...

<script language="JavaScript">
function refresh_user(formname, fieldname){
	var user = document.formname.fieldname.value
	document.formname.user.value = user
	if (user.length < 1){
		alert("Provide A User Value In The Form!")
		return false;
	} else {
		return true;
	}
}
</script>

And the form syntax...

<form onSubmit="return refresh_user(addadmin, id);" name="addadmin" action="'.$currentquery.'&sub3=add" method=post>
<input type=hidden name=user value="">
<table><td width=100>User ID:</td><td><input type=text name=id size=3></td></table>

Those are the parts that're needed to help out... Thanks, in advance.

Try this instead

<script type="text/javascript">
function refresh_user(formname, fieldname)
{
	var user = document.forms[formname][fieldname].value;
	document.forms[formname].user.value = user;

	if (user.length < 1)
    {
		alert("Provide A User Value In The Form!");
		return false;
	}
    else
		return true;
}
</script>

formname and fieldname were being treated as strings, so you can't use them the way you were using them. Since you can use document.forms as an array containing all the forms on the page, you can use the string formname as the index. And you can also use document.<formobj> as an array for every element within the form, so you can use fieldname as an index.

:-/ Still does the same thing... I have no clue what's wrong.

Problem Fixed!

The Function... (The one Chroder altered for me, Thanks)

<script language="JavaScript">
function refresh_user(formname, fieldname)
{
	var user = document.forms[formname][fieldname].value;
	document.forms[formname].user.value = user;

	if (user.length < 1){
		alert("Provide A User Value In The Form!");
		return false;
	} else {
		return true;
	}
}
</script>

And the form... (Single-Quoted Arguments)

<form onSubmit="return refresh_user(\'addadmin\', \'id\');" name="addadmin" action="'.$currentquery.'&sub3=add" method=post>
<input type=hidden name=user value="">
<table><td width=100>User ID:</td><td><input type=text name=id size=3></td></table>