Pushing values from multiple inputs from ng-repeat into array with AngularJS

Question

I have the following array:

$scope.variations = [
    {'nom':'',
     'preu':0,
     'grams':''
    },
    {'nom':'',
     'preu':0,
     'grams':''
    },
    {'nom':'',
     'preu':0,
     'grams':''
    }
];

And I display it in the frontend like so:

<div ng-repeat="vari in variations">
  <input type="text" class="inputmodal" ng-model="varinom">
  <input type="text" class="inputmodal" ng-model="varipreu">
  <input type="text" class="inputmodal" ng-model="varigrams">
</div>

Once I fill the 3 inputs in all 3 items of the array, I'm trying to push each one of them into another array, like so:

<a ng-click="test()">TEST</a>

And the JS:

 $scope.test = function(){
   $scope.singleorder = [];
   for(var i = 0; i < $scope.variations.length; i++)
     $scope.singleorder.push({
       'nom': $scope.variations[i].varinom,
       'grams': $scope.variations[i].varigrams,
       'preu': $scope.variations[i].varipreu,
     });

  console.log($scope.singleorder);
};

On my results in the console, I have 3 arrays in the console, but the values of each object are undefined.

What am I missing?

It should be ng-model="vari.nom" not ng-model="varinom"

Satpal
– Satpal

2017-02-25 20:36:56 +00:00
Commented Feb 25, 2017 at 20:36 — Satpal
– Satpal, Commented Feb 25, 2017 at 20:36
Put it as an answer and i vote u up!

Eric Mitjans
– Eric Mitjans

2017-02-25 20:38:26 +00:00
Commented Feb 25, 2017 at 20:38 — Eric Mitjans
– Eric Mitjans, Commented Feb 25, 2017 at 20:38

Satpal · Accepted Answer · 2017-02-25 20:40:28Z

1

You have set ngModel correctly. use reference variable vari like

<div ng-repeat="vari in variations">
  <input type="text" class="inputmodal" ng-model="vari.nom">
  <input type="text" class="inputmodal" ng-model="vari.preu">
  <input type="text" class="inputmodal" ng-model="vari.grams">
</div>

and use

 $scope.singleorder.push({
   'nom': $scope.variations[i].nom,
   'grams': $scope.variations[i].grams,
   'preu': $scope.variations[i].preu,
 });

answered Feb 25, 2017 at 20:40

Satpal

134k13 gold badges168 silver badges171 bronze badges

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

Collectives™ on Stack Overflow

Pushing values from multiple inputs from ng-repeat into array with AngularJS

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related