fuxed onclick fired twice