Angular Cannot read property 'push' of undefined

Question

I have created a simple news ticker which seems to work fine on my codepen example. It seems i having a problem with integration - code structure. As it stands I am receiving the following error.

Cannot read property 'push' of undefined

Here is my a code snippet of my controller.

"use strict";

 var app = angular.module('portlandApp');

// WEB SERVICE GET REQUEST
var NewsService = function ($http){
this.$http = $http;
}

NewsService.prototype.getarticles = function () {
return this.$http.get('app/data/news.json').then(function (response){
    return response.data;
});
  };

app.service("newsService", NewsService);

angular
 .module('portlandApp')
 .controller('newsCtrl', ['$scope', 'newsService', '$location', '$interval',   '$timeout', function($scope, newsService, $location, $timeout, $interval) {

 var promise = newsService.getarticles();
  promise.then(function (data){
  $scope.articles = data.news.map(function(item) {
    //item.date = moment().format('Do MMMM YYYY');
    return item;
  });
console.log(data)
 });
 // AMOUNT OF ARTICLES
   $scope.articleLimit = 4;

// NEWS TICKER FUNCTION
$scope.moving = false;

$scope.moveLeft = function() {
    $scope.moving = true;
    $timeout($scope.switchFirst, 1000);
};
$scope.switchFirst = function () {
    $scope.news.push($scope.newsLink.shift());
    $scope.moving = false;
    $scope.$apply();
};

$interval($scope.moveLeft, 2000);


 }]);

Initialise '$scope.news = []' first... before you access it.. I dont see anywhere such declaration. — Vish
– Vish, Commented Aug 26, 2016 at 10:23

wilovent · Accepted Answer · 2016-08-26 09:33:40Z

2

You never intialize $scope.news, just add $scope.news = [] at the beginning of the controller

answered Aug 26, 2016 at 9:33

wilovent

1,3841 gold badge12 silver badges15 bronze badges

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

2 Comments

NewBoy Over a year ago

I don't this that's the case . I have made a edit to my code snippet

wilovent Over a year ago

In your updated snippet, there still not initialization of $scope.news

Nitya Kumar · Accepted Answer · 2016-08-26 09:40:03Z

0

You can try

// NEWS TICKER FUNCTION
$scope.moving = false;


$scope.news = []  // need to define array before push value.

enter code here

$scope.moveLeft = function() {
   $scope.moving = true;
   $timeout($scope.switchFirst, 1000);
 };
$scope.switchFirst = function () {
   $scope.news.push($scope.newsLink.shift());
   $scope.moving = false;
   $scope.$apply();
 };

answered Aug 26, 2016 at 9:40

Nitya Kumar

9688 silver badges14 bronze badges

2 Comments

NewBoy Over a year ago

I don't this that's the case . I have made a edit to my code snippet. I have already defined my array before pushing my value, using a web service

Nitya Kumar Over a year ago

you have to define array in JavaScript also

DB.Null · Accepted Answer · 2016-08-26 09:33:40Z

0

At the very beggining of your controller, declare that $scope.news= []

When you add values to your $scope.news, this variable is undefined

answered Aug 26, 2016 at 9:33

DB.Null

1,2048 silver badges11 bronze badges

Collectives™ on Stack Overflow

Angular Cannot read property 'push' of undefined

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related