mi sembra giusta, controlla che arrivino i dati alla pagina o metti un echo davanti alla query per vedere cosa esce.
PS un dato lo mandi in get e uno in post?