How do I detect if an object has been added using angular.forEach AngularJS

Question

I am trying to add objects to $scope.orderitems and if the object is already in the $scope.orderitems I want to change the quantity property of the object using angular.forEach instead of adding another object. Whenever I call the additem function I get an error that orderitems is not defined.

Here is my code

HTML

<input type="text" ng-model="item.name">
<button type="button" ng-click="additem(item)">

<ul>
    <li ng-repeat="orderitem in orderitems">
        <p>{{ orderitem.name }} | {{ orderitem.qty }}</p>
    </li>
</ul>

JS

app.controller('OrderCtrl', function($scope) {
    $scope.orderitems = [];
    $scope.additem = function(data) {

        angular.forEach(orderitems, function(orderitem) {
            if (orderitem.id === data.id) {
                orderitem.qty = orderitem.qty++;
            } else {
                data.qty = 1;
                $scope.orderitems.push(data);
            }
        });
    };
});

angular.forEach($scope.orderitems,function(orderItem...... — calebboyd
– calebboyd, Commented Dec 17, 2013 at 20:18

Rod Hartzell · Accepted Answer · 2013-12-17 21:07:18Z

7

Here's a working fiddle: http://jsfiddle.net/rodhartzell/hbN4G/

HTML

<body ng-app="app">
    <div ng-controller="OrderCtrl">
    <input type="text" ng-model="item.name">
        <button type="button" ng-click="additem()">add</button>

    <ul>
        <li ng-repeat="orderitem in orderitems">
            <p>{{ orderitem.name }} | {{ orderitem.qty }}</p>
        </li>
    </ul>
</div>

CONTROLLER

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

app.controller('OrderCtrl', function($scope) {

    $scope.orderitems = [{name: 'foo', id:1, qty:1}];
    $scope.additem = function() {
        var exists = false;
        var data = {
            id : $scope.orderitems.length + 1,
            name: $scope.item.name,
            qty: 1
        }
        angular.forEach($scope.orderitems, function (item) {
            if (item.name == data.name) {
                exists = true;
                item.qty++;
                return false;
            }
        });
        if(!exists){
            $scope.orderitems.push(data);
        }

    };
});

answered Dec 17, 2013 at 21:07

Rod Hartzell

4403 silver badges9 bronze badges

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

1 Comment

Rod Hartzell Over a year ago

I updated the fiddle -> Here's a working fiddle: jsfiddle.net/rodhartzell/hbN4G/2 . Just in case there is anyone out there still using Angular 1.

vittore · Accepted Answer · 2013-12-17 20:18:03Z

0

$scope.orderitems = [];
$scope.additem = function(data) {
    var item = null
    angular.forEach(orderitems, function(orderItem) {
        if (orderItem.id === data.id) {
            item = orderItem
            return false;
        }
    });
    if (item == null) {
       item = {
         id: data.id, qty : 0
       }
       $scope.orderitems.push(item)
    }

    item.qty++
};

answered Dec 17, 2013 at 20:18

vittore

17.6k6 gold badges46 silver badges84 bronze badges

Collectives™ on Stack Overflow

How do I detect if an object has been added using angular.forEach AngularJS

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related