Push input to array and clear input not working as expected

Question

I have a controller which fetches data from a factory, as follows:

.controller('ChatDetailCtrl', function($scope, $stateParams, Messages) {
  $scope.messages = Messages.get($stateParams.partnerId);
  $scope.send = function (input) {
    input.id = Math.random();
    input.sent = true;
    input.time = new Date();
    $scope.messages.data.push(input);
    console.log($scope.messages);
  }
})

I use an ng-repeat to display the messages on the template. I then have an input which uses ng-click to run send. The problem is, when you send it then its added to the array, however if you carry on typing then it updates the sent message, rather than allowing you to send a new one.

How can I pass the input to the array, in a way which allows me to repeat it many times?

Maybe also post the template to make more clear what it is about. The Problem is that by pushing input into the Array, you still keep a reference inside your model ($scope), you dont refresh it. — David Losert
– David Losert, Commented Jul 27, 2014 at 11:55

Srinath Mandava · Accepted Answer · 2014-07-27 11:55:29Z

1

try using angular.copy so that you are not pushing the same reference but an entirely new object similar to input

.controller('ChatDetailCtrl', function($scope, $stateParams, Messages) {
  $scope.messages = Messages.get($stateParams.partnerId);
  $scope.send = function (input) {
    input.id = Math.random();
    input.sent = true;
    input.time = new Date();
    $scope.messages.data.push(angular.copy(input));
    console.log($scope.messages);
  }
})

answered Jul 27, 2014 at 11:55

Srinath Mandava

3,4722 gold badges27 silver badges37 bronze badges

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

Comments

user1364910 · Accepted Answer · 2014-07-27 11:54:48Z

0

Without having seen your markup and/or the structure of input, this here does the trick:

JsBin: http://jsbin.com/xevabevi/10/edit

Like Charminbear said, you need to clear the input data some way, after pushing to the messages array.

Alternatively, you could do the following in the push:

$scope.arr.push(angular.copy(input));

That way you wouldn't push the input data to the array, rather you would push a copy of it. This wouldn't clear your input fields however, as the original input would stay intact.

answered Jul 27, 2014 at 11:54

user1364910

Collectives™ on Stack Overflow

Push input to array and clear input not working as expected

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related