#93 - Fix QP.drawLines

JustinPealing · Mar 1, 2021 · 9283acd · 9283acd
1 parent f542b71
commit 9283acd
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 1 deletion.
diff --git a/examples/xslt_render.html b/examples/xslt_render.html
@@ -0,0 +1,19 @@
+<!DOCTYPE html>
+<html lang="en">
+<head>
+    <meta charset="UTF-8">
+    <title>Example for html-query-plan</title>
+    <link rel="stylesheet" type="text/css" href="../css/qp.css" />
+    <script src="../dist/qp.js"></script>
+</head>
+<body>
+    <div id="container">
+        <!-- XSLT output goes here -->
+   </div>
+</div>
+</div>
+<script>
+    QP.drawLines(document.getElementById("container"));
+</script>
+</body>
+</html>
diff --git a/src/node.ts b/src/node.ts
@@ -120,13 +120,15 @@ class Node {
      * Gets the xml for the whole query plan.
      */
     get queryPlanXml(): Element {
-        return findAncestor(this.element, "qp-root").parentElement["xml"];
+        const root = findAncestor(this.element, "qp-root");
+        return root == null ? null : root.parentElement["xml"];
     }
 
     /**
      * Gets the xml element corresponding to this node from the query plan xml.
      */
     get nodeXml(): Element {
+        if (this.queryPlanXml == null) return null;
         return getNodeXml(this.queryPlanXml, this.statementId, this.nodeId);
     }
 

diff --git a/test/nodes_test.ts b/test/nodes_test.ts
@@ -112,6 +112,16 @@ describe("Node", () => {
 
         });
 
+        it("Returns null if the nodeXml is not set", () => {
+
+            let element = document.createElement("div");
+            element.className = "qp-node";
+            let node = new Node(element);
+
+            assert.isNull(node.nodeXml);
+
+        });
+
     });
 
 });