Skip to content

Commit

Permalink
fix(b-table): properly handle empty included/excluded filter fields (c…
Browse files Browse the repository at this point in the history
…loses #5775) (#5780)

* fix(b-table): properly handle empty included/excluded filter fileds

* Update table-filtering.spec.js
  • Loading branch information
jacobmllr95 authored Sep 16, 2020
1 parent 34db9fc commit 78ac383
Show file tree
Hide file tree
Showing 4 changed files with 62 additions and 5 deletions.
2 changes: 1 addition & 1 deletion src/components/table/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -3000,7 +3000,7 @@ your app handles the various inconsistencies with events.
:current-page="currentPage"
:per-page="perPage"
:filter="filter"
:filterIncludedFields="filterOn"
:filter-included-fields="filterOn"
:sort-by.sync="sortBy"
:sort-desc.sync="sortDesc"
:sort-direction="sortDirection"
Expand Down
4 changes: 2 additions & 2 deletions src/components/table/helpers/mixin-filtering.js
Original file line number Diff line number Diff line change
Expand Up @@ -49,10 +49,10 @@ export default {
},
computed: {
computedFilterIgnored() {
return this.filterIgnoredFields ? concat(this.filterIgnoredFields).filter(identity) : null
return concat(this.filterIgnoredFields || []).filter(identity)
},
computedFilterIncluded() {
return this.filterIncludedFields ? concat(this.filterIncludedFields).filter(identity) : null
return concat(this.filterIncludedFields || []).filter(identity)
},
computedFilterDebounce() {
const ms = toInteger(this.filterDebounce, 0)
Expand Down
4 changes: 2 additions & 2 deletions src/components/table/helpers/sanitize-row.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,8 @@ const sanitizeRow = (row, ignoreFields, includeFields, fieldsObj = {}) => {
const allowedKeys = keys(formattedRow).filter(
key =>
!IGNORED_FIELD_KEYS[key] &&
!(isArray(ignoreFields) && arrayIncludes(ignoreFields, key)) &&
!(isArray(includeFields) && !arrayIncludes(includeFields, key))
!(isArray(ignoreFields) && ignoreFields.length > 0 && arrayIncludes(ignoreFields, key)) &&
!(isArray(includeFields) && includeFields.length > 0 && !arrayIncludes(includeFields, key))
)

return pick(formattedRow, allowedKeys)
Expand Down
57 changes: 57 additions & 0 deletions src/components/table/table-filtering.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -207,6 +207,63 @@ describe('table > filtering', () => {
wrapper.destroy()
})

it('`filter-ignored-fields` prop works', async () => {
const wrapper = mount(BTable, {
propsData: {
fields: testFields,
items: testItems,
filter: '',
filterIgnoredFields: []
}
})

expect(wrapper).toBeDefined()
await waitNT(wrapper.vm)

expect(wrapper.findAll('tbody > tr').length).toBe(3)

// Search for a value in "a" column
await wrapper.setProps({ filter: '3' })
await waitNT(wrapper.vm)
expect(wrapper.findAll('tbody > tr').length).toBe(1)

// Ignore "a" column from filtering
await wrapper.setProps({ filterIgnoredFields: ['a'] })
await waitNT(wrapper.vm)
expect(wrapper.findAll('tbody > tr').length).toBe(0)

wrapper.destroy()
})

it('`filter-included-fields` prop works', async () => {
const wrapper = mount(BTable, {
propsData: {
fields: testFields,
// Add a extra item with a duplicated value in another field
items: [...testItems, { a: 4, b: 'y', c: 'a' }],
filter: '',
filterIncludedFields: []
}
})

expect(wrapper).toBeDefined()
await waitNT(wrapper.vm)

expect(wrapper.findAll('tbody > tr').length).toBe(4)

// Search for "a"
await wrapper.setProps({ filter: 'a' })
await waitNT(wrapper.vm)
expect(wrapper.findAll('tbody > tr').length).toBe(2)

// Only include "a" and "b" fields
await wrapper.setProps({ filterIncludedFields: ['a', 'b'] })
await waitNT(wrapper.vm)
expect(wrapper.findAll('tbody > tr').length).toBe(1)

wrapper.destroy()
})

it('should filter for formatted values for keys which are not present in row', async () => {
const wrapper = mount(BTable, {
propsData: {
Expand Down

0 comments on commit 78ac383

Please sign in to comment.