Commit 4ca0ef5f authored by Paul Brown's avatar Paul Brown

fix backwards compatibility issues with new filters by not requiring 'select2'...

fix backwards compatibility issues with new filters by not requiring 'select2' data_type for filters with options
parent f0986130
...@@ -332,8 +332,6 @@ class FilterConverter(filters.BaseFilterConverter): ...@@ -332,8 +332,6 @@ class FilterConverter(filters.BaseFilterConverter):
@filters.convert('enum') @filters.convert('enum')
def conv_enum(self, column, name, options=None, **kwargs): def conv_enum(self, column, name, options=None, **kwargs):
# set all operations to select2
kwargs['data_type'] = "select2"
if not options: if not options:
options = [ options = [
(v, v) (v, v)
......
...@@ -98,7 +98,7 @@ class BaseBooleanFilter(BaseFilter): ...@@ -98,7 +98,7 @@ class BaseBooleanFilter(BaseFilter):
super(BaseBooleanFilter, self).__init__(name, super(BaseBooleanFilter, self).__init__(name,
(('1', lazy_gettext(u'Yes')), (('1', lazy_gettext(u'Yes')),
('0', lazy_gettext(u'No'))), ('0', lazy_gettext(u'No'))),
data_type="select2") data_type)
def validate(self, value): def validate(self, value):
return value in ('0', '1') return value in ('0', '1')
......
...@@ -47,7 +47,10 @@ var AdminFilters = function(element, filtersElement, filterGroups, activeFilters ...@@ -47,7 +47,10 @@ var AdminFilters = function(element, filtersElement, filterGroups, activeFilters
// generate HTML for filter input - allows changing filter input type to one with options or tags // generate HTML for filter input - allows changing filter input type to one with options or tags
function createFilterInput(inputContainer, filterValue, filter) { function createFilterInput(inputContainer, filterValue, filter) {
if (filter.type == "select2") { if (filter.type == "select2-tags") {
var $field = $('<input type="hidden" class="filter-val form-control" />').attr('name', makeName(filter.arg));
$field.val(filterValue);
} else if (filter.options) {
var $field = $('<select class="filter-val" />').attr('name', makeName(filter.arg)); var $field = $('<select class="filter-val" />').attr('name', makeName(filter.arg));
$(filter.options).each(function() { $(filter.options).each(function() {
...@@ -60,9 +63,6 @@ var AdminFilters = function(element, filtersElement, filterGroups, activeFilters ...@@ -60,9 +63,6 @@ var AdminFilters = function(element, filtersElement, filterGroups, activeFilters
.val(this[0]).text(this[1])); .val(this[0]).text(this[1]));
} }
}); });
} else if (filter.type == "select2-tags") {
var $field = $('<input type="hidden" class="filter-val form-control" />').attr('name', makeName(filter.arg));
$field.val(filterValue);
} else { } else {
var $field = $('<input type="text" class="filter-val form-control" />').attr('name', makeName(filter.arg)); var $field = $('<input type="text" class="filter-val form-control" />').attr('name', makeName(filter.arg));
$field.val(filterValue); $field.val(filterValue);
...@@ -90,8 +90,10 @@ var AdminFilters = function(element, filtersElement, filterGroups, activeFilters ...@@ -90,8 +90,10 @@ var AdminFilters = function(element, filtersElement, filterGroups, activeFilters
// save tag options as json on data attribute // save tag options as json on data attribute
field.attr('data-tags', JSON.stringify(options)); field.attr('data-tags', JSON.stringify(options));
} }
} }
faForm.applyStyle(field, filter.type);
} else if (filter.options) {
filter.type = "select2";
faForm.applyStyle(field, filter.type); faForm.applyStyle(field, filter.type);
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment