è giusta una sintassi del genere?

codice:
<select name="tipo" OnChange="doRedirect2(this, this.value, <%=idprop%>, <%=categoria%>, <%=tipo%>)">
codice:
<script language="javascript">
				  function doRedirect2(myObject, idcategoria, idprop, categoria, tipo) {
				 	  if (myObject.options[myObject.selectedIndex].value!="") {
						  top.location.href="edit-property.asp?nvcat=" + idcategoria + "&idprop=" + idprop + "&categoria=" + categoria + "&tipo=" + tipo
					  }	  else {
						  return false;
					  }
				  }
				</script>
grazie mille...ho una pagina che mi da errore su un javascript ma non capisco quale...