Ticket #1747: unique.patch

src/core.js

@@ -1093 +1093 @@
                 try {
 
                         for ( var i = 0, length = array.length; i < length; i++ ) {
-                                var id = jQuery.data( array[ i ] );
+                                var id = array[ i ];
 
+                                if ("object" == typeof array[ i ])
+                                        id = jQuery.data( array[ i ] );
+
                                 if ( !done[ id ] ) {
                                         done[ id ] = true;
                                         ret.push( array[ i ] );

test/unit/core.js

@@ -1074 +1074 @@
         // Disabled, randomly fails
         //ok( $("#iframe").contents()[0].body, "Check existance of IFrame body" );
 });
+
+test("$.unique(array)", function() {
+        expect(9);
+        isSet( $.unique([]), [], "Check .unique() with empty array" );
+        isSet( $.unique([1, 2, 3]), [1, 2, 3], "Check with array of ints, no duplicates." );
+        isSet( $.unique([1, 1, 2, 2, 3, 3]), [1, 2, 3], "Check with array of ints, duplicates." );
+        isSet( $.unique(['a', 'b', 'c']), ['a', 'b', 'c'], "Check with array of strings, no duplicates." );
+        isSet( $.unique(['a', 'a', 'b', 'b', 'c', 'c']), ['a', 'b', 'c'], "Check with array of strings, duplicates." );
+        var arr1 = [1, 2], arr2 = ['a', 'b'];
+        isSet( $.unique([arr1, arr2]), [arr1, arr2], "Check with array of arrays, no duplicates." );
+        isSet( $.unique([arr1, arr2, arr1, arr2]), [arr1, arr2], "Check with array of arrays, duplicates." );
+        var obj1 = {"":"obj1"}, obj2 = {"":"obj2"};
+        isSet( $.unique([obj1, obj2]), [obj1, obj2], "Check with array of objects, no duplicates." );
+        isSet( $.unique([obj1, obj2, obj1, obj2]), [obj1, obj2], "Check with array of objects, duplicates." );
+});