Lo que esta equivocado no es el codigo en si, es la logica aplicada para tu necesidad.
El codigo original funciona si tienes fijos algunos parametros (com las url's dentro del select)
Pero en tu caso los parametros son dinamicos, pues son consultados a una BBDD
Te recomiendo visites el
foro de AJAX
Suerte
P.D. Sigues teniendo lineas de mas (el punto 3)