O que seria "não funcionar" para você? Seja mais claro. Uma sugestão, utilize document.getElementById ou outra forma de seleção (document.getElementsByTagName talvez). Seu código fica menos acoplado ("amarrado"). Um abraço.
Sinceramente, você espera que alguém vai ler tanto código? Coloque apenas a parte referente ao erro. O erro é bem claro... Está sendo passado um valor onde era esperada uma referência. Um abraço.
Não confie em nenhuma entrada de usuário (inclusive via $_GET). Converta os valores informados pelo usuários para os tipos adequados e use prepared statements ao interagir com o banco de dados ... São cuidados pequenos que fazem a diferença. Um abraço.