[sling-org-apache-sling-app-cms] branch master updated: fixed traversal check

[jeb]

This is an automated email from the ASF dual-hosted git repository.

jeb pushed a commit to branch master
in repository 
https://gitbox.apache.org/repos/asf/sling-org-apache-sling-app-cms.git


The following commit(s) were added to refs/heads/master by this push:
     new 15ef597  fixed traversal check
15ef597 is described below

commit 15ef59776739c49b67a6faaad95bfac80b857d5a
Author: JE Bailey <jason.bai...@sas.com>
AuthorDate: Mon Oct 15 12:53:03 2018 -0400

    fixed traversal check
---
 ui/src/main/frontend/src/js/cms.table.js |  5 +----
 ui/src/main/frontend/src/js/nomnom.js    | 22 ++++++++++------------
 2 files changed, 11 insertions(+), 16 deletions(-)

diff --git a/ui/src/main/frontend/src/js/cms.table.js 
b/ui/src/main/frontend/src/js/cms.table.js
index a6933b5..887ede6 100644
--- a/ui/src/main/frontend/src/js/cms.table.js
+++ b/ui/src/main/frontend/src/js/cms.table.js
@@ -31,9 +31,6 @@ nomnom.decorate(".table .load-versions", {
     
 });
 
-
-
-
 nomnom.decorate(".table", {
     initCallback: function(){
         var $table = $(this);
@@ -46,7 +43,7 @@ nomnom.decorate(".table", {
     }
 });
 
-nomnom.decorate("table tbody tr",{
+nomnom.decorate(".table tbody tr",{
     events :{
         click: function(event){
             var $current = $(this);
diff --git a/ui/src/main/frontend/src/js/nomnom.js 
b/ui/src/main/frontend/src/js/nomnom.js
index fa32f9c..8e59c65 100644
--- a/ui/src/main/frontend/src/js/nomnom.js
+++ b/ui/src/main/frontend/src/js/nomnom.js
@@ -31,7 +31,7 @@
     };
 
     var tagSelectors = {};
-    var debug = false;
+    var debug = true;
     var elementMap = new WeakMap();
 
     new MutationObserver(function(mutations) {
@@ -125,20 +125,18 @@
     };
 
     var checkAll = function(node) {
-        if (!node.querySelectorAll) {
-            return;
-        }
         var checkSet = new Set([node]);
-        checkSet.forEach(function(node){
-            let elements = node.children;
-            for (let i = 0; i < elements.length; i++) {
-                let element = elements[i];
-                if (element.querySelectorAll) {
-                    check(element);
-                    checkSet.add(element);
+        checkSet.forEach(function(element){
+            if (element.querySelectorAll) {
+                check(element);
+            }
+            let elements = element.children;
+            if (elements){
+                for (let i = 0; i < elements.length; i++) {
+                    checkSet.add(elements[i]);
                 }
             }
-            checkSet.delete(node);
+            checkSet.delete(element);
         });
     }