From 3e044a5ffacadd1f261d9f43e8b1d802dc166ae4 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Mon, 14 Jan 2019 13:34:04 +0530 Subject: [PATCH] =?UTF-8?q?feat:=20=F0=9F=8E=B8=20Add=20compareValue=20hoo?= =?UTF-8?q?k=20to=20control=20comparison?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit For custom comparison where the formatted value is different than the actual value, users will mostly compare using the formatted string. Use compareValue to return parsed cell value and keyword. --- cypress/integration/datatable_spec.js | 4 +-- index.html | 9 ++++++- src/body-renderer.js | 4 ++- src/filterRows.js | 36 +++++++++++++++++++++------ 4 files changed, 42 insertions(+), 11 deletions(-) diff --git a/cypress/integration/datatable_spec.js b/cypress/integration/datatable_spec.js index b0010e0..a6cc7c7 100644 --- a/cypress/integration/datatable_spec.js +++ b/cypress/integration/datatable_spec.js @@ -163,7 +163,7 @@ describe('DataTable', function () { }); }); - describe.only('Inline Filters', function () { + describe('Inline Filters', function () { before(function () { cy.visit('/'); }); @@ -229,7 +229,7 @@ describe('DataTable', function () { }); it('greater than for string type filters', function () { - cy.get('@filterInput6').type('> 2011/07/01'); + cy.get('@filterInput6').type('> 01/07/2011'); cy.wait(500); cy.get('.dt-row[data-row-index=0]').should('not.be.visible'); cy.get('.dt-row[data-row-index=1]').should('be.visible'); diff --git a/index.html b/index.html index 68ab6bc..5c04f2d 100644 --- a/index.html +++ b/index.html @@ -153,7 +153,14 @@ { name: "Position" }, { name: "Office" }, { name: "Extn." }, - { name: "Start Date", /*format: (value) => value.split('/').reverse().join('/')*/ }, + { + name: "Start Date", + format: (value) => (value || '').split('/').reverse().join('/'), + compareValue: (cell, keyword) => { + const keywordValue = keyword.split('/').reverse().join('/') + return [+new Date(cell.content), +new Date(keywordValue)]; + } + }, { name: "Salary", format: value => formatMoney(value) } ]; data = [["Tiger Nixon", { content: "System Architect
New line", editable: false }, "Edinburgh", 5421, "2011/04/25", 320800], ["Garrett Winters", "Accountant", "Tokyo", 8422, "2011/07/25", 170750], ["Ashton Cox", "Junior Technical Author", "San Francisco", 1562, "2009/01/12", 86000], ["Cedric Kelly", "Senior Javascript Developer", "Edinburgh", 6224, "2012/03/29", 433060], ["Airi Satou", "Accountant", "Tokyo", 5407, "2008/11/28", 162700], ["Brielle Williamson", "Integration Specialist", "New York", 4804, "2012/12/02", 372000], ["Herrod Chandler", "Sales Assistant", "San Francisco", 9608, "2012/08/06", 137500], ["Rhona Davidson", "Integration Specialist", "Tokyo", 6200, "2010/10/14", 327900], ["Colleen Hurst", "Javascript Developer", "San Francisco", 2360, "2009/09/15", 205500], ["Sonya Frost", "Software Engineer", "Edinburgh", 1667, "2008/12/13", 103600], ["Jena Gaines", "Office Manager", "London", 3814, "2008/12/19", 90560], ["Quinn Flynn", "Support Lead", "Edinburgh", 9497, "2013/03/03", 342000], ["Charde Marshall", "Regional Director", "San Francisco", 6741, "2008/10/16", 470600], ["Haley Kennedy", "Senior Marketing Designer", "London", 3597, "2012/12/18", 313500], ["Tatyana Fitzpatrick", "Regional Director", "London", 1965, "2010/03/17", 385750], ["Michael Silva", "Marketing Designer", "London", 1581, "2012/11/27", 198500], ["Paul Byrd", "Chief Financial Officer (CFO)", "New York", 3059, "2010/06/09", 725000], ["Gloria Little", "Systems Administrator", "New York", 1721, "2009/04/10", 237500], ["Bradley Greer", "Software Engineer", "London", 2558, "2012/10/13", 132000], ["Dai Rios", "Personnel Lead", "Edinburgh", 2290, "2012/09/26", 217500], ["Jenette Caldwell", "Development Lead", "New York", 1937, "2011/09/03", 345000], ["Yuri Berry", "Chief Marketing Officer (CMO)", "New York", 6154, "2009/06/25", 675000], ["Caesar Vance", "Pre-Sales Support", "New York", 8330, "2011/12/12", 106450], ["Doris Wilder", "Sales Assistant", "Sidney", 3023, "2010/09/20", 85600], ["Angelica Ramos", "Chief Executive Officer (CEO)", "London", 5797, "2009/10/09", 1200000], ["Gavin Joyce", "Developer", "Edinburgh", 8822, "2010/12/22", 92575], ["Jennifer Chang", "Regional Director", "Singapore", 9239, "2010/11/14", 357650], ["Brenden Wagner", "Software Engineer", "San Francisco", 1314, "2011/06/07", 206850], ["Fiona Green", "Chief Operating Officer (COO)", "San Francisco", 2947, "2010/03/11", 850000], ["Shou Itou", "Regional Marketing", "Tokyo", 8899, "2011/08/14", 163000], ["Michelle House", "Integration Specialist", "Sidney", 2769, "2011/06/02", 95400], ["Suki Burks", "Developer", "London", 6832, "2009/10/22", 114500], ["Prescott Bartlett", "Technical Author", "London", 3606, "2011/05/07", 145000], ["Gavin Cortez", "Team Leader", "San Francisco", 2860, "2008/10/26", 235500], ["Martena Mccray", "Post-Sales support", "Edinburgh", 8240, "2011/03/09", 324050], ["Unity Butler", "Marketing Designer", "San Francisco", 5384, "2009/12/09", 85675], ["Howard Hatfield", "Office Manager", "San Francisco", 7031, "2008/12/16", 164500], ["Hope Fuentes", "Secretary", "San Francisco", 6318, "2010/02/12", 109850], ["Vivian Harrell", "Financial Controller", "San Francisco", 9422, "2009/02/14", 452500], ["Timothy Mooney", "Office Manager", "London", 7580, "2008/12/11", 136200], ["Jackson Bradshaw", "Director", "New York", 1042, "2008/09/26", 645750], ["Olivia Liang", "Support Engineer", "Singapore", 2120, "2011/02/03", 234500], ["Bruno Nash", "Software Engineer", "London", 6222, "2011/05/03", 163500], ["Sakura Yamamoto", "Support Engineer", "Tokyo", 9383, "2009/08/19", 139575], ["Thor Walton", "Developer", "New York", 8327, "2013/08/11", 98540], ["Finn Camacho", "Support Engineer", "San Francisco", 2927, "2009/07/07", 87500], ["Serge Baldwin", "Data Coordinator", "Singapore", 8352, "2012/04/09", 138575], ["Zenaida Frank", "Software Engineer", "New York", 7439, "2010/01/04", 125250], ["Zorita Serrano", "Software Engineer", "San Francisco", 4389, "2012/06/01", 115000], ["Jennifer Acosta", "Junior Javascript Developer", "Edinburgh", 3431, "2013/02/01", 75650], ["Cara Stevens", "Sales Assistant", "New York", 3990, "2011/12/06", 145600], ["Hermione Butler", "Regional Director", "London", 1016, "2011/03/21", 356250], ["Lael Greer", "Systems Administrator", "London", 6733, "2009/02/27", 103500], ["Jonas Alexander", "Developer", "San Francisco", 8196, "2010/07/14", 86500], ["Shad Decker", "Regional Director", "Edinburgh", 6373, "2008/11/13", 183000], ["Michael Bruce", "Javascript Developer", "Singapore", 5384, "2011/06/27", 183000], ["Donna Snider", "Customer Support", "New York", 4226, "2011/01/25", 112000]]; diff --git a/src/body-renderer.js b/src/body-renderer.js index 84946b0..1c180c9 100644 --- a/src/body-renderer.js +++ b/src/body-renderer.js @@ -37,6 +37,9 @@ export default class BodyRenderer { const rowHTML = this.rowmanager.getRowHTML(row, row.meta); el.innerHTML = rowHTML; return el.children[0]; + }, + afterRender: () => { + this.restoreState(); } }; @@ -95,7 +98,6 @@ export default class BodyRenderer { } restoreState() { - this.rowmanager.showAllRows(); this.rowmanager.highlightCheckedRows(); this.cellmanager.selectAreaOnClusterChanged(); this.cellmanager.focusCellOnClusterChanged(); diff --git a/src/filterRows.js b/src/filterRows.js index 3a19874..d56b7e3 100644 --- a/src/filterRows.js +++ b/src/filterRows.js @@ -30,11 +30,31 @@ export default function filterRows(rows, filters) { }; function getFilterMethod(filter) { + const stringCompareValue = cell => + String(stripHTML(cell.html || '') || cell.content || '').toLowerCase(); + + const numberCompareValue = cell => parseFloat(cell.content); + + const getCompareValues = (cell, keyword) => { + if (cell.column.compareValue) { + const compareValues = cell.column.compareValue(cell, keyword); + if (compareValues && Array.isArray(compareValues)) return compareValues; + } + + // check if it can be converted to number + const float = numberCompareValue(cell); + if (!isNaN(float)) { + return [float, keyword]; + } + + return [stringCompareValue(cell), keyword]; + }; + let filterMethodMap = { contains(keyword, cells) { return cells .filter(cell => { - const hay = String(stripHTML(cell.html || '') || cell.content || '').toLowerCase(); + const hay = stringCompareValue(cell); const needle = (keyword || '').toLowerCase(); return !needle || hay.includes(needle); }) @@ -44,8 +64,8 @@ function getFilterMethod(filter) { greaterThan(keyword, cells) { return cells .filter(cell => { - const value = cell.content; - return value > keyword; + const [compareValue, keywordValue] = getCompareValues(cell, keyword); + return compareValue > keywordValue; }) .map(cell => cell.rowIndex); }, @@ -53,8 +73,8 @@ function getFilterMethod(filter) { lessThan(keyword, cells) { return cells .filter(cell => { - const value = cell.content; - return value < keyword; + const [compareValue, keywordValue] = getCompareValues(cell, keyword); + return compareValue < keywordValue; }) .map(cell => cell.rowIndex); }, @@ -71,8 +91,10 @@ function getFilterMethod(filter) { range(rangeValues, cells) { return cells .filter(cell => { - const value = cell.content; - return value >= rangeValues[0] && value <= rangeValues[1]; + const values1 = getCompareValues(cell, rangeValues[0]); + const values2 = getCompareValues(cell, rangeValues[1]); + const value = values1[0]; + return value >= values1[1] && value <= values2[1]; }) .map(cell => cell.rowIndex); }