Bueno, yo no soy mucho de javascript pero creo que aquí se tiene un error grave de concepto.
Cuando uno invoca una función de javascript generalmente ocurre a partir de un evento (on click, on submit,on load etc) y en tu caso no lo estas llamando asi:
Yo lo primero que haria para verificar es que por aparte crearas la función y con un simple botón trataras de quitar el atributo para verificar que la función funciona correctamente. En caso positivo ya tocaria que te pasaran a la sección de javascript para que alguien te diera mas luces de como hacerlo.
De igual forma cuando yo uso funciones javascript dentro de PHP no las coloco con un echo sino de la siguiente forma:
Código PHP:
?>//cierro el tag de php
//coloco la funcion javascript
<?php//vuelvo a abrir el php