Tanta complicação para algo tão simples...
onclick="if (this.style.display == 'none'){ this.style.display = 'block';} else{ this.style.display = 'none';}
aplica isto no div que queres esconder
podes também criar uma função para simplificar as coisas
<script>
function esconder_div (idiv){
var obj = document.getElementById(idiv);
if (obj .style.display == 'none'){ obj .style.display = 'block';} else{ obj .style.display = 'none';}
}
</script>
chamas a função com onclick="esconder_div(this.id)"
abraço