From 05ddbec1a70783f4efd060bba29abdae3b422911 Mon Sep 17 00:00:00 2001 From: Kevin Brown Date: Fri, 7 Nov 2014 11:13:53 -0500 Subject: [PATCH] Improve data array speeds This improves the speeds of creating the `'); + var option = document.createElement('option'); - $option.text(data.text); - $option.val(data.id); - $option.prop('disabled', data.disabled || false); - $option.prop('selected', data.selected || false); + option.value = data.id; - // Get any automatically generated data values - var detectedData = this.item($option); + if (data.disabled) { + option.disabled = true; + } - // Merge it with the already present data - var combinedData = $.extend({}, data, detectedData); + if (data.selected) { + option.selected = true; + } + + option.innerText = data.text; + + var $option = $(option); + + var normalizedData = this._normalizeItem(data); // Override the option's data with the combined data - $option.data('data', combinedData); + $.data($option, normalizedData); return $option; }; SelectAdapter.prototype.item = function ($option) { - var data = $option.data('data'); + var data = {}; - // If the data has already be generated, use it - if (data == null) { - if ($option.is('option')) { - data = { - id: $option.val(), - text: $option.html(), - disabled: $option.prop('disabled') - }; - } else if ($option.is('optgroup')) { - data = { - text: $option.attr('label'), - children: [] - }; + if ($.hasData($option)) { + data = $option.data('data'); - var $children = $option.children('option'); - var children = []; - - for (var c = 0; c < $children.length; c++) { - var $child = $($children[c]); - - var child = this.item($child); - - children.push(child); - } - - data.children = children; + if (data != null) { + return data; } - - if (data.id && this.container != null) { - data._resultId = this.generateResultId(this.container, data); - } - - data.selected = $option.prop('selected'); - - $option.data('data', data); } + if ($option.is('option')) { + data = { + id: $option.val(), + text: $option.html(), + disabled: $option.prop('disabled') + }; + } else if ($option.is('optgroup')) { + data = { + text: $option.attr('label'), + children: [] + }; + + var $children = $option.children('option'); + var children = []; + + for (var c = 0; c < $children.length; c++) { + var $child = $($children[c]); + + var child = this.item($child); + + children.push(child); + } + + data.children = children; + } + + data = this._normalizeItem(data); + + $option.data('data', data); + return data; }; + SelectAdapter.prototype._normalizeItem = function (item) { + var defaults = { + selected: false, + disabled: false + }; + + if (item._resultId == null && item.id && this.container != null) { + item._resultId = this.generateResultId(this.container, item); + } + + return $.extend({}, defaults, item); + }; + SelectAdapter.prototype.matches = function (params, data) { var matcher = this.options.get('matcher'); diff --git a/dist/js/select2.amd.js b/dist/js/select2.amd.js index 851cebd4..ac114e6b 100644 --- a/dist/js/select2.amd.js +++ b/dist/js/select2.amd.js @@ -1098,68 +1098,87 @@ define('select2/data/select',[ }; SelectAdapter.prototype.option = function (data) { - var $option = $(''); + var option = document.createElement('option'); - $option.text(data.text); - $option.val(data.id); - $option.prop('disabled', data.disabled || false); - $option.prop('selected', data.selected || false); + option.value = data.id; - // Get any automatically generated data values - var detectedData = this.item($option); + if (data.disabled) { + option.disabled = true; + } - // Merge it with the already present data - var combinedData = $.extend({}, data, detectedData); + if (data.selected) { + option.selected = true; + } + + option.innerText = data.text; + + var $option = $(option); + + var normalizedData = this._normalizeItem(data); // Override the option's data with the combined data - $option.data('data', combinedData); + $.data($option, normalizedData); return $option; }; SelectAdapter.prototype.item = function ($option) { - var data = $option.data('data'); + var data = {}; - // If the data has already be generated, use it - if (data == null) { - if ($option.is('option')) { - data = { - id: $option.val(), - text: $option.html(), - disabled: $option.prop('disabled') - }; - } else if ($option.is('optgroup')) { - data = { - text: $option.attr('label'), - children: [] - }; + if ($.hasData($option)) { + data = $option.data('data'); - var $children = $option.children('option'); - var children = []; - - for (var c = 0; c < $children.length; c++) { - var $child = $($children[c]); - - var child = this.item($child); - - children.push(child); - } - - data.children = children; + if (data != null) { + return data; } - - if (data.id && this.container != null) { - data._resultId = this.generateResultId(this.container, data); - } - - data.selected = $option.prop('selected'); - - $option.data('data', data); } + if ($option.is('option')) { + data = { + id: $option.val(), + text: $option.html(), + disabled: $option.prop('disabled') + }; + } else if ($option.is('optgroup')) { + data = { + text: $option.attr('label'), + children: [] + }; + + var $children = $option.children('option'); + var children = []; + + for (var c = 0; c < $children.length; c++) { + var $child = $($children[c]); + + var child = this.item($child); + + children.push(child); + } + + data.children = children; + } + + data = this._normalizeItem(data); + + $option.data('data', data); + return data; }; + SelectAdapter.prototype._normalizeItem = function (item) { + var defaults = { + selected: false, + disabled: false + }; + + if (item._resultId == null && item.id && this.container != null) { + item._resultId = this.generateResultId(this.container, item); + } + + return $.extend({}, defaults, item); + }; + SelectAdapter.prototype.matches = function (params, data) { var matcher = this.options.get('matcher'); diff --git a/dist/js/select2.full.js b/dist/js/select2.full.js index df985293..91d03973 100644 --- a/dist/js/select2.full.js +++ b/dist/js/select2.full.js @@ -10633,68 +10633,87 @@ define('select2/data/select',[ }; SelectAdapter.prototype.option = function (data) { - var $option = $(''); + var option = document.createElement('option'); - $option.text(data.text); - $option.val(data.id); - $option.prop('disabled', data.disabled || false); - $option.prop('selected', data.selected || false); + option.value = data.id; - // Get any automatically generated data values - var detectedData = this.item($option); + if (data.disabled) { + option.disabled = true; + } - // Merge it with the already present data - var combinedData = $.extend({}, data, detectedData); + if (data.selected) { + option.selected = true; + } + + option.innerText = data.text; + + var $option = $(option); + + var normalizedData = this._normalizeItem(data); // Override the option's data with the combined data - $option.data('data', combinedData); + $.data($option, normalizedData); return $option; }; SelectAdapter.prototype.item = function ($option) { - var data = $option.data('data'); + var data = {}; - // If the data has already be generated, use it - if (data == null) { - if ($option.is('option')) { - data = { - id: $option.val(), - text: $option.html(), - disabled: $option.prop('disabled') - }; - } else if ($option.is('optgroup')) { - data = { - text: $option.attr('label'), - children: [] - }; + if ($.hasData($option)) { + data = $option.data('data'); - var $children = $option.children('option'); - var children = []; - - for (var c = 0; c < $children.length; c++) { - var $child = $($children[c]); - - var child = this.item($child); - - children.push(child); - } - - data.children = children; + if (data != null) { + return data; } - - if (data.id && this.container != null) { - data._resultId = this.generateResultId(this.container, data); - } - - data.selected = $option.prop('selected'); - - $option.data('data', data); } + if ($option.is('option')) { + data = { + id: $option.val(), + text: $option.html(), + disabled: $option.prop('disabled') + }; + } else if ($option.is('optgroup')) { + data = { + text: $option.attr('label'), + children: [] + }; + + var $children = $option.children('option'); + var children = []; + + for (var c = 0; c < $children.length; c++) { + var $child = $($children[c]); + + var child = this.item($child); + + children.push(child); + } + + data.children = children; + } + + data = this._normalizeItem(data); + + $option.data('data', data); + return data; }; + SelectAdapter.prototype._normalizeItem = function (item) { + var defaults = { + selected: false, + disabled: false + }; + + if (item._resultId == null && item.id && this.container != null) { + item._resultId = this.generateResultId(this.container, item); + } + + return $.extend({}, defaults, item); + }; + SelectAdapter.prototype.matches = function (params, data) { var matcher = this.options.get('matcher'); diff --git a/dist/js/select2.full.min.js b/dist/js/select2.full.min.js index e13652f3..64071820 100644 --- a/dist/js/select2.full.min.js +++ b/dist/js/select2.full.min.js @@ -1,4 +1,4 @@ window.$=window.$||{},function(){$&&$.fn&&$.fn.select2&&$.fn.select2.amd&&(c=$.fn.select2.amd.define,b=$.fn.select2.amd.require);var a,b,c;!function(d){function e(a,b){return u.call(a,b)}function f(a,b){var c,d,e,f,g,h,i,j,k,l,m,n=b&&b.split("/"),o=s.map,p=o&&o["*"]||{};if(a&&"."===a.charAt(0))if(b){for(n=n.slice(0,n.length-1),a=a.split("/"),g=a.length-1,s.nodeIdCompat&&w.test(a[g])&&(a[g]=a[g].replace(w,"")),a=n.concat(a),k=0;k0&&(a.splice(k-1,2),k-=2)}a=a.join("/")}else 0===a.indexOf("./")&&(a=a.substring(2));if((n||p)&&o){for(c=a.split("/"),k=c.length;k>0;k-=1){if(d=c.slice(0,k).join("/"),n)for(l=n.length;l>0;l-=1)if(e=o[n.slice(0,l).join("/")],e&&(e=e[d])){f=e,h=k;break}if(f)break;!i&&p&&p[d]&&(i=p[d],j=k)}!f&&i&&(f=i,h=j),f&&(c.splice(0,h,f),a=c.join("/"))}return a}function g(a,b){return function(){return n.apply(d,v.call(arguments,0).concat([a,b]))}}function h(a){return function(b){return f(b,a)}}function i(a){return function(b){q[a]=b}}function j(a){if(e(r,a)){var b=r[a];delete r[a],t[a]=!0,m.apply(d,b)}if(!e(q,a)&&!e(t,a))throw new Error("No "+a);return q[a]}function k(a){var b,c=a?a.indexOf("!"):-1;return c>-1&&(b=a.substring(0,c),a=a.substring(c+1,a.length)),[b,a]}function l(a){return function(){return s&&s.config&&s.config[a]||{}}}var m,n,o,p,q={},r={},s={},t={},u=Object.prototype.hasOwnProperty,v=[].slice,w=/\.js$/;o=function(a,b){var c,d=k(a),e=d[0];return a=d[1],e&&(e=f(e,b),c=j(e)),e?a=c&&c.normalize?c.normalize(a,h(b)):f(a,b):(a=f(a,b),d=k(a),e=d[0],a=d[1],e&&(c=j(e))),{f:e?e+"!"+a:a,n:a,pr:e,p:c}},p={require:function(a){return g(a)},exports:function(a){var b=q[a];return"undefined"!=typeof b?b:q[a]={}},module:function(a){return{id:a,uri:"",exports:q[a],config:l(a)}}},m=function(a,b,c,f){var h,k,l,m,n,s,u=[],v=typeof c;if(f=f||a,"undefined"===v||"function"===v){for(b=!b.length&&c.length?["require","exports","module"]:b,n=0;n0&&b-1 in a}function e(a,b,c){if(bb.isFunction(b))return bb.grep(a,function(a,d){return!!b.call(a,d,a)!==c});if(b.nodeType)return bb.grep(a,function(a){return a===b!==c});if("string"==typeof b){if(ib.test(b))return bb.filter(b,a,c);b=bb.filter(b,a)}return bb.grep(a,function(a){return V.call(b,a)>=0!==c})}function f(a,b){for(;(a=a[b])&&1!==a.nodeType;);return a}function g(a){var b=pb[a]={};return bb.each(a.match(ob)||[],function(a,c){b[c]=!0}),b}function h(){_.removeEventListener("DOMContentLoaded",h,!1),a.removeEventListener("load",h,!1),bb.ready()}function i(){Object.defineProperty(this.cache={},0,{get:function(){return{}}}),this.expando=bb.expando+Math.random()}function j(a,b,c){var d;if(void 0===c&&1===a.nodeType)if(d="data-"+b.replace(vb,"-$1").toLowerCase(),c=a.getAttribute(d),"string"==typeof c){try{c="true"===c?!0:"false"===c?!1:"null"===c?null:+c+""===c?+c:ub.test(c)?bb.parseJSON(c):c}catch(e){}tb.set(a,b,c)}else c=void 0;return c}function k(){return!0}function l(){return!1}function m(){try{return _.activeElement}catch(a){}}function n(a,b){return bb.nodeName(a,"table")&&bb.nodeName(11!==b.nodeType?b:b.firstChild,"tr")?a.getElementsByTagName("tbody")[0]||a.appendChild(a.ownerDocument.createElement("tbody")):a}function o(a){return a.type=(null!==a.getAttribute("type"))+"/"+a.type,a}function p(a){var b=Lb.exec(a.type);return b?a.type=b[1]:a.removeAttribute("type"),a}function q(a,b){for(var c=0,d=a.length;d>c;c++)sb.set(a[c],"globalEval",!b||sb.get(b[c],"globalEval"))}function r(a,b){var c,d,e,f,g,h,i,j;if(1===b.nodeType){if(sb.hasData(a)&&(f=sb.access(a),g=sb.set(b,f),j=f.events)){delete g.handle,g.events={};for(e in j)for(c=0,d=j[e].length;d>c;c++)bb.event.add(b,e,j[e][c])}tb.hasData(a)&&(h=tb.access(a),i=bb.extend({},h),tb.set(b,i))}}function s(a,b){var c=a.getElementsByTagName?a.getElementsByTagName(b||"*"):a.querySelectorAll?a.querySelectorAll(b||"*"):[];return void 0===b||b&&bb.nodeName(a,b)?bb.merge([a],c):c}function t(a,b){var c=b.nodeName.toLowerCase();"input"===c&&zb.test(a.type)?b.checked=a.checked:("input"===c||"textarea"===c)&&(b.defaultValue=a.defaultValue)}function u(b,c){var d=bb(c.createElement(b)).appendTo(c.body),e=a.getDefaultComputedStyle?a.getDefaultComputedStyle(d[0]).display:bb.css(d[0],"display");return d.detach(),e}function v(a){var b=_,c=Pb[a];return c||(c=u(a,b),"none"!==c&&c||(Ob=(Ob||bb("