Lo que no entiendo es la razón de copiar el valor del <select/> al <input/> cuando puedes recibir el mismo valor del <select/> directamente con PHP.
Porque lo que haces es eso, copiar el valor, nada especial.
Además el evento del <select/> debería ser onchange y no onclick.