fix some zero-width space weirdness
[fnpeditor.git] / src / smartxml / fragments.js
index 62a7a4c..f15a9e8 100644 (file)
@@ -19,7 +19,7 @@ $.extend(Fragment.prototype, {
 var NodeFragment = function(document, params) {
     Fragment.call(this, document);
     this.node = params.node;
-    this.nodePath = params.node.getPath();
+    this.nodePath = this.isValid() ? params.node.getPath() : null;
 };
 NodeFragment.prototype = Object.create(Fragment.prototype);
 $.extend(NodeFragment.prototype, {
@@ -27,14 +27,16 @@ $.extend(NodeFragment.prototype, {
         return this.document.containsNode(this.node);
     },
     restoreFromPaths: function() {
-        this.node = this.document.getNodeByPath(this.nodePath);
+        if(this.nodePath) {
+            this.node = this.document.getNodeByPath(this.nodePath);
+        }
     }
 });
 
 
 var CaretFragment = function(document, params) {
-    NodeFragment.call(this, document, params);
     this.offset = params.offset;
+    NodeFragment.call(this, document, params);
 
 };
 CaretFragment.prototype = Object.create(NodeFragment.prototype);
@@ -74,17 +76,20 @@ $.extend(RangeFragment.prototype, {
         this.startNode = this.document.getNodeByPath(this.startNodePath);
         this.endNode = this.document.getNodeByPath(this.endNodePath);
     },
-    hasSiblingBoundries: function() {
+    hasSiblingBoundaries: function() {
         return this.isValid() && this.startNode.isSiblingOf(this.endNode);
     },
-    boundriesSiblingParents: function() {
+    hasSameBoundaries: function() {
+        return this.isValid() && this.startNode.sameNode(this.endNode);
+    },
+    boundariesSiblingParents: function() {
         return this.startNode.document.getSiblingParents({
             node1: this.startNode,
             node2: this.endNode
         });
     },
     getCommonParent: function() {
-        var siblingParents = this.boundriesSiblingParents();
+        var siblingParents = this.boundariesSiblingParents();
         if(siblingParents) {
             return siblingParents.node1.parent();
         }