Merge pull request #797 from Delgan/641-appendTo_prependTo

Fix PR #726 adding `appendTo()` and `prependTo()`

Readme.md

@@ -586,6 +586,20 @@ $.html()
 //    </ul>
 ```
 
+#### .appendTo( target )
+Insert every element in the set of matched elements to the end of the target.
+
+```js
+$('<li class="plum">Plum</li>').appendTo('#fruits')
+$.html()
+//=>  <ul id="fruits">
+//      <li class="apple">Apple</li>
+//      <li class="orange">Orange</li>
+//      <li class="pear">Pear</li>
+//      <li class="plum">Plum</li>
+//    </ul>
+```
+
 #### .prepend( content, [content, ...] )
 Inserts content as the *first* child of each of the selected elements.
 
@@ -600,6 +614,20 @@ $.html()
 //    </ul>
 ```
 
+#### .prependTo( target )
+Insert every element in the set of matched elements to the beginning of the target.
+
+```js
+$('<li class="plum">Plum</li>').prependTo('#fruits')
+$.html()
+//=>  <ul id="fruits">
+//      <li class="plum">Plum</li>
+//      <li class="apple">Apple</li>
+//      <li class="orange">Orange</li>
+//      <li class="pear">Pear</li>
+//    </ul>
+```
+
 #### .after( content, [content, ...] )
 Insert content next to each element in the set of matched elements.
 
@@ -614,7 +642,7 @@ $.html()
 //    </ul>
 ```
 
-#### .insertAfter( content )
+#### .insertAfter( target )
 Insert every element in the set of matched elements after the target.
 
 ```js
@@ -642,7 +670,7 @@ $.html()
 //    </ul>
 ```
 
-#### .insertBefore( content )
+#### .insertBefore( target )
 Insert every element in the set of matched elements before the target.
 
 ```js

lib/api/manipulation.js

@@ -102,6 +102,26 @@ var uniqueSplice = function(array, spliceIdx, spliceCount, newElems, parent) {
   return array.splice.apply(array, spliceArgs);
 };
 
+exports.appendTo = function(target) {
+  if (!target.cheerio) {
+    target = this.constructor.call(this.constructor, target, null, this._originalRoot);
+  }
+
+  target.append(this);
+
+  return this;
+};
+
+exports.prependTo = function(target) {
+  if (!target.cheerio) {
+    target = this.constructor.call(this.constructor, target, null, this._originalRoot);
+  }
+
+  target.prepend(this);
+
+  return this;
+};
+
 exports.append = _insert(function(dom, children, parent) {
   uniqueSplice(children, children.length, 0, dom, parent);
 });

test/api/manipulation.js

@@ -496,6 +496,66 @@ describe('$(...)', function() {
     });
   });
 
+  describe('.appendTo', function() {
+
+    it('(html) : should add element as last child', function() {
+      var $plum = $('<li class="plum">Plum</li>').appendTo(fruits);
+      expect($plum.parent().children().eq(3).hasClass('plum')).to.be.ok();
+    });
+
+    it('($(...)) : should add element as last child', function() {
+      $('<li class="plum">Plum</li>').appendTo($fruits);
+      expect($fruits.children().eq(3).hasClass('plum')).to.be.ok();
+    });
+
+    it('(Node) : should add element as last child', function() {
+      $('<li class="plum">Plum</li>').appendTo($fruits[0]);
+      expect($fruits.children().eq(3).hasClass('plum')).to.be.ok();
+    });
+
+    it('(selector) : should add element as last child', function() {
+      $('<li class="plum">Plum</li>').appendTo('#fruits');
+      expect($fruits.children().eq(3).hasClass('plum')).to.be.ok();
+    });
+
+    it('(Array) : should add element as last child of all elements in the array', function() {
+      var $multiple = $('<ul><li>Apple</li></ul><ul><li>Orange</li></ul>');
+      $('<li class="plum">Plum</li>').appendTo($multiple.get());
+      expect($multiple.first().children().eq(1).hasClass('plum')).to.be.ok();
+      expect($multiple.last().children().eq(1).hasClass('plum')).to.be.ok();
+    });
+  });
+
+  describe('.prependTo', function() {
+
+    it('(html) : should add element as first child', function() {
+      var $plum = $('<li class="plum">Plum</li>').prependTo(fruits);
+      expect($plum.parent().children().eq(0).hasClass('plum')).to.be.ok();
+    });
+
+    it('($(...)) : should add element as first child', function() {
+      $('<li class="plum">Plum</li>').prependTo($fruits);
+      expect($fruits.children().eq(0).hasClass('plum')).to.be.ok();
+    });
+
+    it('(Node) : should add node as first child', function() {
+      $('<li class="plum">Plum</li>').prependTo($fruits[0]);
+      expect($fruits.children().eq(0).hasClass('plum')).to.be.ok();
+    });
+
+    it('(selector) : should add element as first child', function() {
+      $('<li class="plum">Plum</li>').prependTo('#fruits');
+      expect($fruits.children().eq(0).hasClass('plum')).to.be.ok();
+    });
+
+    it('(Array) : should add element as first child of all elements in the array', function() {
+      var $multiple = $('<ul><li>Apple</li></ul><ul><li>Orange</li></ul>');
+      $('<li class="plum">Plum</li>').prependTo($multiple.get());
+      expect($multiple.first().children().eq(0).hasClass('plum')).to.be.ok();
+      expect($multiple.last().children().eq(0).hasClass('plum')).to.be.ok();
+    });
+  });
+
   describe('.after', function() {
 
     it('() : should do nothing', function() {
@@ -790,7 +850,7 @@ describe('$(...)', function() {
     });
 
     it('(Node) : should add element as previous sibling', function() {
-      var plum = $('<li class="plum">Plum</li>');
+      var plum = $('<li class="plum">Plum</li>')[0];
       $('.apple').before(plum);
       expect($('.apple').prev().hasClass('plum')).to.be.ok();
     });