[molnij]

странно что не выдало сообщение: НЕ ВЕШАЙ ОБРАБОТЧИКИ ЧЕРЕЗ АТРИБУТЫ!!!
Используйте нормальный способ кодирования, JS отдельно, HTML отдельно. Зачем совать все воедино, это же говнокод.

Я в JS новичок и за бурей эмоций не очень понял, что именно вы рекомендуете?
Кстати гугл по "НЕ ВЕШАЙ ОБРАБОТЧИКИ ЧЕРЕЗ АТРИБУТЫ" не выдает ничего особо впечатляющего. Первая же ссылка ведет на http://javascript.ru/tutorial/events/intro, где написано, что варианты с document.getElementById('myElement').onclick = function() {} и атрибутом эквивалентны.
А учитывая, что в системе весь код генерируется, видимо автору было проще вешать обработчики по месту проживания элемента. И в чем-то я его могу понять.

ну и в целом хотелось бы понять, если код валидный, почему он не выполняется, а если нет - почему он выполняется