Selector: add jQuery.uniqueSort; deprecate jQuery.unique

Fixes gh-2228

diff --git a/src/selector-sizzle.js b/src/selector-sizzle.js
index 7d3926b1a989436e07b8815e852eda32f6ba4f60..c091f1821690cc211755ba8e7399b2d677ccbf09 100644 (file)
--- a/src/selector-sizzle.js
+++ b/src/selector-sizzle.js
@@ -6,7 +6,7 @@ define([
 jQuery.find = Sizzle;
 jQuery.expr = Sizzle.selectors;
 jQuery.expr[":"] = jQuery.expr.pseudos;
-jQuery.unique = Sizzle.uniqueSort;
+jQuery.uniqueSort = jQuery.unique = Sizzle.uniqueSort;
 jQuery.text = Sizzle.getText;
 jQuery.isXMLDoc = Sizzle.isXML;
 jQuery.contains = Sizzle.contains;

diff --git a/src/traversing.js b/src/traversing.js
index d9ff0aeac4aa8b4e0907daed675ebf60278bc977..19f8d24e752038a45ed071ccc4b2f61694bd3eb8 100644 (file)
--- a/src/traversing.js
+++ b/src/traversing.js
@@ -85,7 +85,7 @@ jQuery.fn.extend({
                        }
                }
 
-               return this.pushStack( matched.length > 1 ? jQuery.unique( matched ) : matched );
+               return this.pushStack( matched.length > 1 ? jQuery.uniqueSort( matched ) : matched );
        },
 
        // Determine the position of an element within the set
@@ -111,7 +111,7 @@ jQuery.fn.extend({
 
        add: function( selector, context ) {
                return this.pushStack(
-                       jQuery.unique(
+                       jQuery.uniqueSort(
                                jQuery.merge( this.get(), jQuery( selector, context ) )
                        )
                );
@@ -182,7 +182,7 @@ jQuery.each({
                if ( this.length > 1 ) {
                        // Remove duplicates
                        if ( !guaranteedUnique[ name ] ) {
-                               jQuery.unique( matched );
+                               jQuery.uniqueSort( matched );
                        }
 
                        // Reverse order for parents* and prev-derivatives

diff --git a/src/traversing/findFilter.js b/src/traversing/findFilter.js
index eee3b6beef0a78db19114db3dc1d8ccb39ed6e9f..128df2cb42483e0c472e472b801de0429a1df570 100644 (file)
--- a/src/traversing/findFilter.js
+++ b/src/traversing/findFilter.js
@@ -72,7 +72,7 @@ jQuery.fn.extend({
                        jQuery.find( selector, self[ i ], ret );
                }
 
-               return this.pushStack( len > 1 ? jQuery.unique( ret ) : ret );
+               return this.pushStack( len > 1 ? jQuery.uniqueSort( ret ) : ret );
        },
        filter: function( selector ) {
                return this.pushStack( winnow(this, selector || [], false) );

diff --git a/test/unit/selector.js b/test/unit/selector.js
index 12c26f1cb06fb5fb71e7e20c80819177cb502ce2..ee117ca8f35fd56cdd34247f6810d3190034072b 100644 (file)
--- a/test/unit/selector.js
+++ b/test/unit/selector.js
@@ -386,7 +386,7 @@ test( "jQuery.contains", function() {
        ok( !jQuery.contains(document, detached), "document container (negative)" );
 });
 
-test("jQuery.unique", function() {
+test("jQuery.uniqueSort", function() {
        expect( 14 );
 
        function Arrayish( arr ) {
@@ -460,8 +460,8 @@ test("jQuery.unique", function() {
 
        jQuery.each( tests, function( label, test ) {
                var length = test.length || test.input.length;
-               deepEqual( jQuery.unique( test.input ).slice( 0, length ), test.expected, label + " (array)" );
-               deepEqual( jQuery.unique( new Arrayish(test.input) ).slice( 0, length ), test.expected, label + " (quasi-array)" );
+               deepEqual( jQuery.uniqueSort( test.input ).slice( 0, length ), test.expected, label + " (array)" );
+               deepEqual( jQuery.uniqueSort( new Arrayish(test.input) ).slice( 0, length ), test.expected, label + " (quasi-array)" );
        });
 });
 

diff --git a/test/unit/traversing.js b/test/unit/traversing.js
index b80b673ff6d5bef29ab7ef139c2b3cc35cb48fbb..08b384a7e353024017b71accce13f92a8387de6b 100644 (file)
--- a/test/unit/traversing.js
+++ b/test/unit/traversing.js
@@ -696,7 +696,7 @@ test("sort direction", function() {
 
        jQuery.each( methodDirections, function( method, reversed ) {
                var actual = elems[ method ]().get(),
-                       forward = jQuery.unique( [].concat( actual ) );
+                       forward = jQuery.uniqueSort( [].concat( actual ) );
                deepEqual( actual, reversed ? forward.reverse() : forward, "Correct sort direction for " + method );
        });
 });