Argggggg!!!!
Que fea SQL!!
Yo la simplificaría...
Por un lado, eliminar TODOS los UCase()... no se para que los necesitás ahí. Además, hacerle un UCase al dato, valla y pase... pero al nombre de campo? qué sentido tiene?
Por el otro, meter TODOS los Request() en variables y luego utilizarlas en la SQL y en el AddNew.
Fijate, se va a hacer más simple de leer