AngularJS Uncaught ReferenceError: ng-repeat ng-click

Question

I do not know why an error. When you run the ng-click in the ng-repaet, Uncaught ReferenceError occurs. Are you sure you will not know the cause of the click event does not occur?

test.js

var MyApp = angular.module('MyApp', []);

MyApp.controller('MyCtrl', ['$scope', function ($scope) {
    $scope.add = function() {
        alert("add click");
    };

    $scope.list = function() {
        return [{"1": {id: 1, name: "aaa"},
                "2": {id: 2, name: "bbb"},
                "3": {id: 3, name: "ccc"}}];
    }
}]);

test.html

<html>
<head>
    <meta charset="utf-8">
    <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.0/css/bootstrap.min.css">
    <script type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/angularjs/1.3.1/angular.min.js"></script>
    <script src="http://ajax.googleapis.com/ajax/libs/jquery/2.0.3/jquery.min.js"></script>
    <script type="text/javascript" src="test.js"></script>
    <title>Sample</title>
</head>
<body ng-app="MyApp">
    <div class="container" ng-controller="MyCtrl">
        <table class="table table-striped" ng-init="list = list()">
            <tr ng-repeat="element in list">
                <td ng-repeat="nake in element" onclick="add()">{{nake.id}}</td>
            </tr>
        </table>
    </div>
</body>
</html>

Error

Uncaught ReferenceError: add is not defined

Use ng-click instead of onclick , like: ng-click="add()" — Ivan Chernykh
– Ivan Chernykh, Commented Nov 6, 2014 at 7:07

Kutyel · Accepted Answer · 2014-11-06 07:17:52Z

1

You should use the ng-click angular tag, like this:

<td ng-repeat="nake in element" ng-click="add(nake.name)">{{nake.id}}</td>

Check this fiddle to see that it works properly.

answered Nov 6, 2014 at 7:17

Kutyel

9,1644 gold badges35 silver badges64 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Valentin Waeselynck · Accepted Answer · 2014-11-06 07:09:40Z

0

You need to use the ng-click directive instead of onclick.

answered Nov 6, 2014 at 7:09

Valentin Waeselynck

6,10130 silver badges48 bronze badges

Comments

RahulB · Accepted Answer · 2014-11-06 07:11:09Z

0

Use ng-click instead of onclick since any function binded to scope cannot be accessed through onclick event.

<td ng-repeat="nake in element" ng-click="add()">

answered Nov 6, 2014 at 7:11

RahulB

2,1101 gold badge21 silver badges28 bronze badges

Collectives™ on Stack Overflow

AngularJS Uncaught ReferenceError: ng-repeat ng-click

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related