fix(stagger): check for null/undefined on stagger parent

checks whether a parent exists and has a classList part of issue aurelia/templating-resources#54
aurelia · May 7, 2015 · cb5dfa3 · cb5dfa3
1 parent 85663cd
commit cb5dfa3
Show file tree

Hide file tree

Showing 6 changed files with 19 additions and 11 deletions.
diff --git a/dist/amd/animator.js b/dist/amd/animator.js
@@ -93,7 +93,7 @@ define(['exports'], function (exports) {
         var parent = element.parentElement,
             delay = 0;
 
-        if (parent.classList.contains('au-stagger')) {
+        if (parent !== null && parent !== undefined && parent.classList.contains('au-stagger')) {
           var elemPos = Array.prototype.indexOf.call(parent.childNodes, element);
           delay = _this.getElementAnimationDelay(parent) * elemPos;
 
@@ -150,7 +150,7 @@ define(['exports'], function (exports) {
         var parent = element.parentElement,
             delay = 0;
 
-        if (parent.classList.contains('au-stagger')) {
+        if (parent !== null && parent !== undefined && parent.classList.contains('au-stagger')) {
           var elemPos = Array.prototype.indexOf.call(parent.childNodes, element);
           delay = _this2.getElementAnimationDelay(parent) * elemPos;
 

diff --git a/dist/commonjs/animator.js b/dist/commonjs/animator.js
@@ -92,7 +92,7 @@ var CssAnimator = (function () {
       var parent = element.parentElement,
           delay = 0;
 
-      if (parent.classList.contains('au-stagger')) {
+      if (parent !== null && parent !== undefined && parent.classList.contains('au-stagger')) {
         var elemPos = Array.prototype.indexOf.call(parent.childNodes, element);
         delay = _this.getElementAnimationDelay(parent) * elemPos;
 
@@ -149,7 +149,7 @@ var CssAnimator = (function () {
       var parent = element.parentElement,
           delay = 0;
 
-      if (parent.classList.contains('au-stagger')) {
+      if (parent !== null && parent !== undefined && parent.classList.contains('au-stagger')) {
         var elemPos = Array.prototype.indexOf.call(parent.childNodes, element);
         delay = _this2.getElementAnimationDelay(parent) * elemPos;
 

diff --git a/dist/es6/animator.js b/dist/es6/animator.js
@@ -94,7 +94,9 @@ export class CssAnimator {
       var parent = element.parentElement,
           delay = 0;
 
-      if(parent.classList.contains('au-stagger')) {
+      if(parent !== null &&
+         parent !== undefined &&
+         parent.classList.contains('au-stagger')) {
         var elemPos = Array.prototype.indexOf.call(parent.childNodes, element);
         delay = this.getElementAnimationDelay(parent) * elemPos;
 
@@ -162,7 +164,9 @@ export class CssAnimator {
       var parent = element.parentElement,
         delay = 0;
 
-      if(parent.classList.contains('au-stagger')) {
+      if(parent !== null &&
+         parent !== undefined &&
+         parent.classList.contains('au-stagger')) {
         var elemPos = Array.prototype.indexOf.call(parent.childNodes, element);
         delay = this.getElementAnimationDelay(parent) * elemPos;
 

diff --git a/dist/system/animator.js b/dist/system/animator.js
@@ -96,7 +96,7 @@ System.register([], function (_export) {
             var parent = element.parentElement,
                 delay = 0;
 
-            if (parent.classList.contains('au-stagger')) {
+            if (parent !== null && parent !== undefined && parent.classList.contains('au-stagger')) {
               var elemPos = Array.prototype.indexOf.call(parent.childNodes, element);
               delay = _this.getElementAnimationDelay(parent) * elemPos;
 
@@ -153,7 +153,7 @@ System.register([], function (_export) {
             var parent = element.parentElement,
                 delay = 0;
 
-            if (parent.classList.contains('au-stagger')) {
+            if (parent !== null && parent !== undefined && parent.classList.contains('au-stagger')) {
               var elemPos = Array.prototype.indexOf.call(parent.childNodes, element);
               delay = _this2.getElementAnimationDelay(parent) * elemPos;
 

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "aurelia-animator-css",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "An implementation of the abstract Animator interface from templating which enables css-based animations.",
   "keywords": [
     "aurelia",

diff --git a/src/animator.js b/src/animator.js
@@ -94,7 +94,9 @@ export class CssAnimator {
       var parent = element.parentElement,
           delay = 0;
 
-      if(parent.classList.contains('au-stagger')) {
+      if(parent !== null &&
+         parent !== undefined &&
+         parent.classList.contains('au-stagger')) {
         var elemPos = Array.prototype.indexOf.call(parent.childNodes, element);
         delay = this.getElementAnimationDelay(parent) * elemPos;
 
@@ -162,7 +164,9 @@ export class CssAnimator {
       var parent = element.parentElement,
         delay = 0;
 
-      if(parent.classList.contains('au-stagger')) {
+      if(parent !== null &&
+         parent !== undefined &&
+         parent.classList.contains('au-stagger')) {
         var elemPos = Array.prototype.indexOf.call(parent.childNodes, element);
         delay = this.getElementAnimationDelay(parent) * elemPos;