มันผิดตรงไหน อ่ะคับ นิดเดียวคับ รูปแบบมันเป็นยังไงคับ

<? echo"<a href='javascript:MM_openPrintsale(\"$idsale3\",\"$idmem\");'><img src='../PhotoWeb/print.png' border='0'></a>"?>

<script language="JavaScript" type="text/JavaScript">
function MM_openPrintsale(idsale,idmem) 
{
	window.open('PrintDataSale.php?idsale='+idsale+'&idmem='+idmem','width=700,*');
}
</script>