ngAnimate throws exception in cancelChildAnimations on deletion of repeated element when using ng-include

[gregolsky]

Here's plunker http://plnkr.co/edit/HrmBBmz949YnS7IcPyp7?p=preview
Please try to click one of the "Remove this" links.
I attach stack trace just in case.

TypeError: Object # has no method 'querySelectorAll'
at cancelChildAnimations (http://code.angularjs.org/1.2.0-rc.3/angular-animate.js:593:36)
at Object.leave (http://code.angularjs.org/1.2.0-rc.3/angular-animate.js:332:11)
at ngRepeatAction (http://code.angularjs.org/1.2.0-rc.3/angular.js:17770:24)
at Object.$watchCollectionAction as fn
at Scope.$digest (http://code.angularjs.org/1.2.0-rc.3/angular.js:10569:27)
at Scope.$apply (http://code.angularjs.org/1.2.0-rc.3/angular.js:10802:24)
at HTMLAnchorElement. (http://code.angularjs.org/1.2.0-rc.3/angular.js:16514:17)
at http://code.angularjs.org/1.2.0-rc.3/angular.js:2331:10
at Array.forEach (native)
at forEach (http://code.angularjs.org/1.2.0-rc.3/angular.js:213:11)

[gregolsky]

The error occurs, when the "element" argument passed to the cancelChildAnimations function contains comment element (at index 0), which does not have the querySelectorAll method. The fix may be to check, if the element supports the method in question.

[matsko]

I see what's going on here. I also got your email yesterday outlining the issue. Thanks for sending that over, I wouldn't have noticed this as quickly.

Checking for querySelectorAll does fix it, but you want the code to be as informative as possible. What's going on is that the comment node (which ngInclude uses) is being animated and that's where it breaks.

So just check for element[0].nodeType == ELEMENT_NODE.

Can you update your pull request to handle that check? Basically something like:

function cancelChildAnimations(element) {
  var ELEMENT_NODE = 1;
  var node = element[0];
  if(node.nodeType == ELEMENT_NODE) {
    angular.forEach(node.querySelectorAll('.' + NG_ANIMATE_CLASS_NAME), function(element) {
      element = angular.element(element);
      var data = element.data(NG_ANIMATE_STATE);
      if(data) {
        cancelAnimations(data.animations);
        cleanup(element);
      }
    });
  }
}

Once ready I can merge it in directly.

[matsko]

Landed as b9557b0

[thaoula]

Hi Guys,

This error is still happening in version 1.2.10 and every version since the issue was reported.

I always have to check if the following line is undefined -

function cancelChildAnimations(element) {
var node = extractElementNode(element);

Regards,
Tarek