newRawData.forEach is not a function

[mikehuebner]

Having an issue with Angular 1.4 where the forEach returns a null or says its not a function. Not sure whats happening. Here is the code its happening.

Grid.prototype.modifyRows = function modifyRows(newRawData) {
    console.log(newRawData);
    var self = this;
    var oldRows = self.rows.slice(0);
    var oldRowHash = self.rowHashMap || self.createRowHashMap();
    self.rowHashMap = self.createRowHashMap();
    self.rows.length = 0;

    newRawData.forEach( function( newEntity, i ) {
      var newRow;
      if ( self.options.enableRowHashing ){
        // if hashing is enabled, then this row will be in the hash if we already know about it
        newRow = oldRowHash.get( newEntity );
      } else {
        // otherwise, manually search the oldRows to see if we can find this row
        newRow = self.getRow(newEntity, oldRows);
      }

      // if we didn't find the row, it must be new, so create it
      if ( !newRow ){
        newRow = self.processRowBuilders(new GridRow(newEntity, i, self));
      }

      self.rows.push( newRow );
      self.rowHashMap.put( newEntity, newRow );
    });

    self.assignTypes();

    var p1 = $q.when(self.processRowsProcessors(self.rows))
      .then(function (renderableRows) {
        return self.setVisibleRows(renderableRows);
      });

    var p2 = $q.when(self.processColumnsProcessors(self.columns))
      .then(function (renderableColumns) {
        return self.setVisibleColumns(renderableColumns);
      });

    return $q.all([p1, p2]);
  };

I'm still exploring, just wanted to bring it up.

[mikehuebner]

Note, happening in version 1.3.14 as well. Just tested it, can't figure out why though, its pull data just fine. It gets an undefined so it must be something I'm doing. It results in this:

and the error is:

[mikehuebner]

Consider this fixed, found it.

If you include data: {} it freaks out, this is recent because I've been doing this due to the start of the application. As for the shrunk columns it was using the minWidth over the width set. Got rid of the width attributes and it is fixed.

[PaulL1]

Probably you should do data: [] instead, and it'd work fine. It assumes that data is an array, and {} isn't an array.

[bpena]

hi PaulL1, if you use angular.forEach(newRawData, function( newEntity, i ) instead of newRawData.forEach( function( newEntity, i ) it's work fine.

This may be a possible solution.

P.S. sorry for my english. XD

[PaulL1]

That will be because angular is happy to iterate over objects. We prefer to use the built-in javascript array iterator, that means you need to provide an array, not an object.

[bpena]

Ok. Thanks!!!

[gabrielgreen]

I just encountered this issue when I switched from setting the data directly on gridOptions to the documentation's recommendation of $scope.data = data; $scope.gridOptions.data = 'data';

The first way worked fine, the other gets the error described here...

UPDATE (in case someone does what I did): I fixed my issue - it was that I was doing $scope.gridOptions.data = 'data'; only at the time my AJAX callback occured, however it needed to be done on the initial setup.

The following works:
$scope.gridOptions.data = 'data';
$http.get('/api/snippets')
.success(data => {
$scope.data = data; });