А с чего бы должно работать? Глобальные переменные в JS объявляются так:
window.somevariable = "somecontent"
Это во-первых. Во-вторых - зачем "javascript
:" в обработчике? Пиши просто вызов функции.
Но я бы делал в функции return false в случае неуспешного выполнения, а в обработчике onclick соответствующую проверку. То есть,
<script>
function myFunc()
{
if (1 > 5)
return false;
else
return true;
}
</script>
<a href="#" onclick="if (myFunc()) {alert('Функция выполнена успешно');} else {alert('Что-то не так...');}">Ссылка</a>