Merge pull request #187 from frappe/nextchamp-saqib-patch-1

fix: `data` is undefined
frappe · Jan 2, 2024 · ae07ce3 · ae07ce3
2 parents 98835d8 + b3ff486
commit ae07ce3
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/filterRows.js b/src/filterRows.js
@@ -35,9 +35,11 @@ function getFilterMethod(rows, data, filter) {
         let formatter = CellManager.getCustomCellFormatter(cell);
         if (data && data.constructor.name === 'DataManager') {
             data = data.getData(cell.rowIndex);
+        } else {
+            data = rows[cell.rowIndex]
         }
         if (formatter && cell.content) {
-            cell.html = formatter(cell.content, rows[cell.rowIndex], cell.column, rows[cell.rowIndex], filter, data);
+            cell.html = formatter(cell.content, rows[cell.rowIndex], cell.column, data, filter);
             return stripHTML(cell.html);
         }
         return cell.content || '';