From 718838da198900dcb23810cb7a6bf0b2245ccb5e Mon Sep 17 00:00:00 2001 From: Akhil Narang Date: Wed, 6 Mar 2024 14:03:00 +0530 Subject: [PATCH] fix: TypeError: Cannot read properties of null (reading 'classList') Sentry PRESS-4KJ / Support 10958 TypeError: Cannot read properties of null (reading 'classList') at ? (../../../../../apps/frappe/node_modules/frappe-datatable/dist/frappe-datatable.cjs.js:2940:37) at Array.forEach() at Ye.clearSelection(../../../../../apps/frappe/node_modules/frappe-datatable/dist/frappe-datatable.cjs.js:2940:14) at Ye._selectArea(../../../../../apps/frappe/node_modules/frappe-datatable/dist/frappe-datatable.cjs.js:2877:14) at Ye.selectArea(../../../../../apps/frappe/node_modules/frappe-datatable/dist/frappe-datatable.cjs.js:2865:18) at HTMLDivElement.t(../../../../../apps/frappe/node_modules/frappe-datatable/dist/frappe-datatable.cjs.js:2723:18) at invokeFunc(../../../../../apps/frappe/node_modules/frappe-datatable/dist/frappe-datatable.cjs.js:632:19) at trailingEdge(../../../../../apps/frappe/node_modules/frappe-datatable/dist/frappe-datatable.cjs.js:681:14) at timerExpired(../../../../../apps/frappe/node_modules/frappe-datatable/dist/frappe-datatable.cjs.js:669:14) at r(../../../../../apps/frappe/node_modules/src/helpers.ts:98:1) Signed-off-by: Akhil Narang --- src/cellmanager.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/cellmanager.js b/src/cellmanager.js index 83b1781..b6599fe 100644 --- a/src/cellmanager.js +++ b/src/cellmanager.js @@ -391,8 +391,11 @@ export default class CellManager { clearSelection() { (this._selectedCells || []) - .forEach($cell => $cell.classList.remove('dt-cell--highlight')); - + .forEach($cell => { + if ($cell && $cell.classList) { + $cell.classList.remove('dt-cell--highlight'); + } + }); this._selectedCells = []; this.$selectionCursor = null; }