На самом деле все работает!
Проблема заключается в имени функции. При inline обработчиках в разметке в качестве глобальных функций и свойств доступны поля и методы document,
В данном случае это document.all
:
Document.all (не стандартизировано, больше не поддерживается, но все еще может работать)
Обеспечивает доступ ко всем элементам с идентификаторами (id). Это нестандартный интерфейс, вместо него рекомендуется использовать метод Document.getElementById().
В msdn:
all is no longer supported. Starting with Internet Explorer 11, use getElementById.
В примере ниже, можно увидеть, что all
, доступный в inline обработчике - это действительно document.all
, а так же, что обработчик вполне работает.
Пример:
function myfunc(all) {
console.log("Hello, i'm document.all: ", all, document.all === all);
}
angular.module('app', []).controller('showBooks', function($scope) {
$scope.booklist = [{
id: 1,
book_rating: 0
}, {
id: 2,
book_rating: 0
}, {
id: 3,
book_rating: 0
}];
})
<script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.5.8/angular.js"></script>
<table class="table table-striped table-condensed table-hover" ng-app="app" ng-controller="showBooks">
<tr ng-repeat="book in booklist">
<td name={{book.id}} ng-if="book.book_rating==0">
<input type="button" onclick="myfunc(all)" value="all">
</td>
</tr>
</table>
С другой стороны, раз уж используется ангуляр - стоит использовать возможности ангуляра, в данном случае директиву ng-click, вместо добавления обработчиков вручную, так как в этом случае теряется доступ к данным хранящимся в скопе.