core: cancel checks for type string before it's used

draggable-test: changed cancel: true test: this should cancel on all elements, but simply do nothing, since true is a unsupported value.

1 files changed, 3 insertions, 3 deletions

diff --git a/ui/ui.sortable.js b/ui/ui.sortable.js
index 84eb0df93..9718dd1bc 100644
--- a/ui/ui.sortable.js
+++ b/ui/ui.sortable.js
@@ -65,9 +65,7 @@ $.widget("ui.sortable", $.extend($.ui.mouse, {
 		if(!noPropagation) this.element.triggerHandler(n == "sort" ? n : "sort"+n, [e, this.ui(inst)], this.options[n]);
 	},
 	serialize: function(o) {
-		
-		
-		
+
 		var items = ($.isFunction(this.options.items) ? this.options.items.call(this.element) : $(this.options.items, this.element)).not('.ui-sortable-helper'); //Only the items of the sortable itself
 		var str = []; o = o || {};
 		
@@ -80,11 +78,13 @@ $.widget("ui.sortable", $.extend($.ui.mouse, {
 		
 	},
 	toArray: function(attr) {
+		
 		var items = ($.isFunction(this.options.items) ? this.options.items.call(this.element) : $(this.options.items, this.element)).not('.ui-sortable-helper'); //Only the items of the sortable itself
 		var ret = [];
 
 		items.each(function() { ret.push($(this).attr(attr || 'id')); });
 		return ret;
+		
 	},
 	/* Be careful with the following core functions */
 	intersectsWith: function(item) {