Функция у Вас лежит внутри $(document).ready
Из за этого onclick инициализируется раньше чем объявлена функция.
И если у Вас js подключен после элемента с onclick, то вынос функции за document ready не сработает.
Лучше вешать обработчик клика в js, а не в html елементе