aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTimmy Willison <timmywillisn@gmail.com>2015-05-04 15:27:12 -0700
committerTimmy Willison <timmywillisn@gmail.com>2015-05-04 15:28:44 -0700
commitd9d930f79e9e4edb7339199cd758dc654e053a3b (patch)
tree494cbf375b95ba24bcd168fbbb7a6316b61f9a6e
parentc916aef84f12843d20c4e39d89e79cccd2547950 (diff)
downloadjquery-d9d930f79e9e4edb7339199cd758dc654e053a3b.tar.gz
jquery-d9d930f79e9e4edb7339199cd758dc654e053a3b.zip
Selector: add jQuery.uniqueSort; deprecate jQuery.unique
Fixes gh-2228
-rw-r--r--src/selector-sizzle.js2
-rw-r--r--src/traversing.js6
-rw-r--r--src/traversing/findFilter.js2
-rw-r--r--test/unit/selector.js6
-rw-r--r--test/unit/traversing.js2
5 files changed, 9 insertions, 9 deletions
diff --git a/src/selector-sizzle.js b/src/selector-sizzle.js
index 7d3926b1a..c091f1821 100644
--- 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 9f3de7af1..c427792c1 100644
--- a/src/traversing.js
+++ b/src/traversing.js
@@ -84,7 +84,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
@@ -109,7 +109,7 @@ jQuery.fn.extend({
add: function( selector, context ) {
return this.pushStack(
- jQuery.unique(
+ jQuery.uniqueSort(
jQuery.merge( this.get(), jQuery( selector, context ) )
)
);
@@ -183,7 +183,7 @@ jQuery.each({
if ( this.length > 1 ) {
// Remove duplicates
if ( !guaranteedUnique[ name ] ) {
- ret = jQuery.unique( ret );
+ ret = jQuery.uniqueSort( ret );
}
// Reverse order for parents* and prev-derivatives
diff --git a/src/traversing/findFilter.js b/src/traversing/findFilter.js
index 2c2f46161..0039b95b0 100644
--- 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 673a86f85..19b2c2210 100644
--- a/test/unit/selector.js
+++ b/test/unit/selector.js
@@ -385,7 +385,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 ) {
@@ -459,8 +459,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 1be1992cd..db58beec3 100644
--- 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 );
});
});