simplify diff for removal of single node, fixes #47

Author: johanneswilm

diffDOM.js

@@ -716,11 +716,14 @@
                 index = 0,
                 last, e1, e2, i;
 
-            if (subtrees.length > 1) {
-                /* Two or more groups have been identified among the childnodes of t1
+            if (subtrees.length > 0) {
+                /* One or more groups have been identified among the childnodes of t1
                  * and t2.
                  */
-                return this.attemptGroupRelocation(t1, t2, subtrees, route);
+                 diffs = this.attemptGroupRelocation(t1, t2, subtrees, route);
+                 if (diffs.length > 0) {
+                     return diffs;
+                 }
             }
 
             /* 0 or 1 groups of similar child nodes have been found

tests/basic.html

@@ -23,6 +23,9 @@
   <h1>Test for diffDOM</h1>
 
   <!-- Add all divs to be compared here two by two -->
+  <div><p>first paragraph</p><img><p>Another paragraph</p><p>A third paragraph</p><p>A fourth paragraph</p><p>A fifth paragraph</p></div>
+  <div><img><p>Another paragraph</p><p>A third paragraph</p><p>A fourth paragraph</p><p>A fifth paragraph</p></div>
+
   <div><h1>Foo</h1><h2>Bar</h2><h3>Baz</h3></div>
   <div><h1>Foo</h1><h2>Bar</h2></div>
 
@@ -594,7 +597,7 @@ <h1>Test for diffDOM</h1>
         reportDiv();
         print("diff operations for div #" + i + " → div #" + (i + 1));
         diffs = dd.diff(divs[i], divs[i + 1]);
-        console.log(diffs);
+        console.log(JSON.stringify(diffs));
 
         print("applying...");
         t1 = divs[i].cloneNode(true);
@@ -604,7 +607,7 @@ <h1>Test for diffDOM</h1>
           print('...success!');
         } else {
           testFailure();
-          console.log(diffs);
+          console.log(JSON.stringify(diffs));
           console.log(t1.outerHTML);
           console.log(divs[i + 1].outerHTML);
           throw 'Outputs not matching';
@@ -618,7 +621,7 @@ <h1>Test for diffDOM</h1>
           testSuccess();
         } else {
           testFailure();
-          console.log(diffs);
+          console.log(JSON.stringify(diffs));
           console.log(t1.outerHTML);
           console.log(divs[i].outerHTML);
           throw 'Outputs not matching';