Browse Source

Fixed DataSet.distinct requiring options.type to be configured

css_transitions
jos 10 years ago
parent
commit
1a2d82dd08
1 changed files with 13 additions and 6 deletions
  1. +13
    -6
      src/DataSet.js

+ 13
- 6
src/DataSet.js View File

@ -760,17 +760,18 @@ DataSet.prototype.min = function (field) {
* The returned array is unordered. * The returned array is unordered.
*/ */
DataSet.prototype.distinct = function (field) { DataSet.prototype.distinct = function (field) {
var data = this._data,
values = [],
fieldType = this._options.type[field],
count = 0;
var data = this._data;
var values = [];
var fieldType = this._options.type && this._options.type[field] || null;
var count = 0;
var i;
for (var prop in data) { for (var prop in data) {
if (data.hasOwnProperty(prop)) { if (data.hasOwnProperty(prop)) {
var item = data[prop]; var item = data[prop];
var value = util.convert(item[field], fieldType);
var value = item[field];
var exists = false; var exists = false;
for (var i = 0; i < count; i++) {
for (i = 0; i < count; i++) {
if (values[i] == value) { if (values[i] == value) {
exists = true; exists = true;
break; break;
@ -783,6 +784,12 @@ DataSet.prototype.distinct = function (field) {
} }
} }
if (fieldType) {
for (i = 0; i < values.length; i++) {
values[i] = util.convert(values[i], fieldType);
}
}
return values; return values;
}; };

Loading…
Cancel
Save