pt/dataflow
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix cleanLayout stripping expression columns from pivot layout save/restore

Browse Source 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Paul Trowbridge 2026-04-20 20:39:02 -04:00
parent 99b7b7d721
commit 7a7fd01285
1 changed files with 14 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
ui/src/pages/Pivot.jsx
View File

@ -173,11 +173,13 @@ export default function Pivot({ source }) {
function cleanLayout(cfg) {
if (!cfg) return cfg
const clean = { ...cfg }
if (clean.columns) clean.columns = clean.columns.filter(c => c == null || validCols.has(c))
if (clean.group_by) clean.group_by = clean.group_by.filter(c => validCols.has(c))
if (clean.split_by) clean.split_by = clean.split_by.filter(c => validCols.has(c))
if (clean.sort) clean.sort = clean.sort.filter(([c]) => validCols.has(c))
if (clean.filter) clean.filter = clean.filter.filter(([c]) => validCols.has(c))
const exprNames = new Set(Object.keys(clean.expressions || {}))
const valid = (c) => validCols.has(c) || exprNames.has(c)
if (clean.columns) clean.columns = clean.columns.filter(c => c == null || valid(c))
if (clean.group_by) clean.group_by = clean.group_by.filter(valid)
if (clean.split_by) clean.split_by = clean.split_by.filter(valid)
if (clean.sort) clean.sort = clean.sort.filter(([c]) => valid(c))
if (clean.filter) clean.filter = clean.filter.filter(([c]) => valid(c))
return clean
}
@ -257,11 +259,13 @@ export default function Pivot({ source }) {
function cleanLayout(cfg) {
if (!cfg) return cfg
const clean = { ...cfg }
if (clean.columns) clean.columns = clean.columns.filter(c => c == null || validCols.has(c))
if (clean.group_by) clean.group_by = clean.group_by.filter(c => validCols.has(c))
if (clean.split_by) clean.split_by = clean.split_by.filter(c => validCols.has(c))
if (clean.sort) clean.sort = clean.sort.filter(([c]) => validCols.has(c))
if (clean.filter) clean.filter = clean.filter.filter(([c]) => validCols.has(c))
const exprNames = new Set(Object.keys(clean.expressions || {}))
const valid = (c) => validCols.has(c) || exprNames.has(c)
if (clean.columns) clean.columns = clean.columns.filter(c => c == null || valid(c))
if (clean.group_by) clean.group_by = clean.group_by.filter(valid)
if (clean.split_by) clean.split_by = clean.split_by.filter(valid)
if (clean.sort) clean.sort = clean.sort.filter(([c]) => valid(c))
if (clean.filter) clean.filter = clean.filter.filter(([c]) => valid(c))
return clean
}
const cleaned = cleanLayout(layout.config)