Skip to content
Snippets Groups Projects
Unverified Commit 950bc988 authored by Anton Kulyk's avatar Anton Kulyk Committed by GitHub
Browse files

Fix crash when opening filter popover's expression editor (#35022)

parent 4252af1c
Branches
Tags
No related merge requests found
import {
filter,
getNotebookStep,
openOrdersTable,
popover,
restore,
} from "e2e/support/helpers";
describe("issue 34794", () => {
beforeEach(() => {
restore();
cy.signInAsNormalUser();
});
it("should not crash when navigating to filter popover's custom expression section (metabase#34794)", () => {
openOrdersTable({ mode: "notebook" });
filter({ mode: "notebook" });
popover().within(() => {
cy.findByText("Created At").click();
cy.icon("chevronleft").click(); // go back to the main filter popover
cy.findByText("Custom Expression").click();
cy.findByLabelText("Expression").type("[Total] > 10").blur();
cy.button("Done").click();
});
getNotebookStep("filter")
.findByText("Total is greater than 10")
.should("be.visible");
});
});
......@@ -165,10 +165,14 @@ export function FilterPopover({
};
if (editingFilter) {
const filterMBQL = filter?.raw();
const expression = isExpression(filterMBQL)
? (filterMBQL as Expression)
: undefined;
return (
<ExpressionWidget
query={query}
expression={filter?.raw() as Expression | undefined}
expression={expression}
startRule="boolean"
header={<ExpressionWidgetHeader onBack={handleExpressionWidgetClose} />}
onChangeExpression={handleExpressionChange}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment