[SOLVED] is this right?

[Reaper0167]

if (!form2.secure.value || !form2.secure_conf.value || !form2.secure.length = 4)

 

want to make sure the user has used 4 digits in the text box

 

!form2.secure.length = 4  ???

[celsoendo]

When you are comparing values you use == (2 equals)

When you are setting values you use = (1 equal)

 

In your case you must use 2 equals sign.

[Reaper0167]

So how would I got about adding the minimum requirement to this part of the script. Couldn something be added after the if statement where the //?// line is

<?php // what tags do i use in the javascript area to have it colored ??
<script type="text/javascript" language="javascript">
    function validate(form2)
    {
    var valid = true;
if (!form2.secure.value || !form2.secure_conf.value  || //?//)
    {
        document.getElementById('secure_error').innerHTML = 'You must enter a 4-digit security code.';
        valid = false;
    }
    else if (form2.secure.value != form2.secure_conf.value)
    {
        document.getElementById('secure_error').innerHTML = 'Security numbers do not match.';
        valid = false;
    }
    else
    {
        document.getElementById('amount_error').innerHTML = '';
    }
    return valid;
    }
</script>

[celsoendo]

Change this:

 

if (!form2.secure.value || !form2.secure_conf.value  || //?//)

 

for this:

 

if (!form2.secure.value || !form2.secure_conf.value  || form2.secure.value.length != 4)

[Reaper0167]

thanks bud.