Skip to content
This repository was archived by the owner on Jul 28, 2025. It is now read-only.

Commit bc46c59

Browse files
committed
preparing types inside filters
1 parent 66b4c8a commit bc46c59

File tree

6 files changed

+58
-27
lines changed

6 files changed

+58
-27
lines changed

src/cdm/ComponentsModel.ts

Lines changed: 17 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import { CellContext, Table } from "@tanstack/react-table";
22
import { RowDataType } from "cdm/FolderModel";
33
import { Link, Literal } from "obsidian-dataview/lib/data-model/value";
4-
import { AtomicFilter } from "cdm/SettingsModel";
4+
import { AtomicFilter, FilterGroup } from "cdm/SettingsModel";
55

66
export type ColumnOption = {
77
value: string;
@@ -37,5 +37,19 @@ export type AtomicFilterComponentProps = {
3737
recursiveIndex: number[];
3838
level: number;
3939
atomicFilter: AtomicFilter;
40-
possibleColumns: string[];
41-
} & DataviewFiltersProps;
40+
possibleColumns: ColumnFilterOption[];
41+
} & DataviewFiltersProps;
42+
43+
export type ColumnFilterOption = {
44+
key: string;
45+
enabled: boolean;
46+
type: string;
47+
}
48+
49+
export type GroupFilterComponentProps = {
50+
group: FilterGroup;
51+
recursiveIndex: number[];
52+
level: number;
53+
table: Table<RowDataType>;
54+
possibleColumns: ColumnFilterOption[];
55+
};

src/components/modals/filters/handlers/DataviewFiltersComponent.tsx

Lines changed: 22 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,10 @@
1-
import { DataviewFiltersProps } from "cdm/ComponentsModel";
1+
import { ColumnFilterOption, DataviewFiltersProps } from "cdm/ComponentsModel";
22
import { obtainColumnsFromRows } from "components/Columns";
33
import React, { useEffect, useState } from "react";
44
import { FilterGroupCondition } from "cdm/SettingsModel";
55
import GroupFilterComponent from "components/modals/filters/handlers/GroupFilterComponent";
66
import DnDComponent from "components/behavior/DnDComponent";
7+
import { InputType } from "helpers/Constants";
78

89
const DataviewFiltersComponent = (props: DataviewFiltersProps) => {
910
const { table } = props;
@@ -13,8 +14,10 @@ const DataviewFiltersComponent = (props: DataviewFiltersProps) => {
1314
const filters = tableState.configState((state) => state.filters);
1415
const columnsInfo = tableState.columns((state) => state.info);
1516

16-
const [possibleColumns, setPossibleColumns] = useState([] as string[]);
17-
17+
const [possibleColumns, setPossibleColumns] = useState(
18+
[] as ColumnFilterOption[]
19+
);
20+
const DbColumns = columnsInfo.getAllColumns();
1821
useEffect(() => {
1922
new Promise<string[]>((resolve) => {
2023
// Empty conditions to refresh the dataview
@@ -25,11 +28,25 @@ const DataviewFiltersComponent = (props: DataviewFiltersProps) => {
2528
view.file.parent.path,
2629
configInfo.getLocalSettings(),
2730
emptyFilterConditions,
28-
columnsInfo.getAllColumns()
31+
DbColumns
2932
)
3033
);
3134
}).then((columns) => {
32-
setPossibleColumns(columns.sort((a, b) => a.localeCompare(b)));
35+
const columnOptions: ColumnFilterOption[] = [];
36+
columns.forEach((column) => {
37+
const possibleColumn = DbColumns.find(
38+
(dbColumn) => dbColumn.key === column
39+
);
40+
const isColumnExistent = possibleColumn !== undefined;
41+
columnOptions.push({
42+
enabled: isColumnExistent,
43+
key: column,
44+
type: isColumnExistent ? possibleColumn.input : InputType.TEXT,
45+
});
46+
});
47+
setPossibleColumns(
48+
columnOptions.sort((a, b) => a.key.localeCompare(b.key))
49+
);
3350
});
3451
}, []);
3552

src/components/modals/filters/handlers/ExistedColumnSelectorComponent.tsx

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import FormControl from "@mui/material/FormControl";
22
import Select, { SelectChangeEvent } from "@mui/material/Select";
3+
import { ColumnFilterOption } from "cdm/ComponentsModel";
34
import { StyleVariables } from "helpers/Constants";
45
import React from "react";
56

@@ -11,7 +12,7 @@ const ExistedColumnSelectorComponent = (selectorProps: {
1112
conditionIndex: number[],
1213
level: number
1314
) => (event: SelectChangeEvent<string>, child: React.ReactNode) => void;
14-
possibleColumns: string[];
15+
possibleColumns: ColumnFilterOption[];
1516
}) => {
1617
const { currentCol, recursiveIndex, level, onchange, possibleColumns } =
1718
selectorProps;
@@ -40,13 +41,13 @@ const ExistedColumnSelectorComponent = (selectorProps: {
4041
}}
4142
>
4243
<optgroup label="Fields from notes">
43-
{possibleColumns.map((key, index) => {
44+
{possibleColumns.map((columnOption, index) => {
4445
return (
4546
<option
46-
value={key}
47-
key={`MenuItem-${index}-existedColumnSelector-${key}--${level}-${recursiveIndex[level]}}`}
47+
value={columnOption.key}
48+
key={`MenuItem-${index}-existedColumnSelector-${columnOption.key}--${level}-${recursiveIndex[level]}}`}
4849
>
49-
{key}
50+
{columnOption.key}
5051
</option>
5152
);
5253
})}

src/components/modals/filters/handlers/FiltersHelper.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import { ColumnFilterOption } from "cdm/ComponentsModel";
12
import { AtomicFilter, FilterGroup, FilterGroupCondition } from "cdm/SettingsModel";
23
import { ConditionFiltersOptions, OperatorFilter } from "helpers/Constants";
34

@@ -15,7 +16,7 @@ export enum ModifyFilterOptionsEnum {
1516
}
1617

1718
const modifyRecursiveFilterGroups = (
18-
possibleColumns: string[],
19+
possibleColumns: ColumnFilterOption[],
1920
filterGroups: FilterGroup[],
2021
recursiveIndex: number[],
2122
level: number,
@@ -38,7 +39,7 @@ const modifyRecursiveFilterGroups = (
3839
case ModifyFilterOptionsEnum.ADD:
3940
(filterGroups[recursiveIndex[level]] as FilterGroupCondition)
4041
.filters.push({
41-
field: possibleColumns[0],
42+
field: possibleColumns[0].key,
4243
operator: OperatorFilter.CONTAINS[0],
4344
value: "",
4445
});
@@ -50,7 +51,7 @@ const modifyRecursiveFilterGroups = (
5051
disabled: false,
5152
filters: [
5253
{
53-
field: possibleColumns[0],
54+
field: possibleColumns[0].key,
5455
operator: OperatorFilter.CONTAINS[0],
5556
value: "",
5657
},

src/components/modals/filters/handlers/GroupFilterComponent.tsx

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -24,13 +24,8 @@ import LabelComponent from "components/modals/filters/handlers/LabelComponent";
2424
import { Setting } from "obsidian";
2525
import { t } from "lang/helpers";
2626
import { Db } from "services/CoreService";
27-
type GroupFilterComponentProps = {
28-
group: FilterGroup;
29-
recursiveIndex: number[];
30-
level: number;
31-
table: Table<RowDataType>;
32-
possibleColumns: string[];
33-
};
27+
import { GroupFilterComponentProps } from "cdm/ComponentsModel";
28+
3429
const GroupFilterComponent = (groupProps: GroupFilterComponentProps) => {
3530
const { group, recursiveIndex, level, table, possibleColumns } = groupProps;
3631
const { tableState } = table.options.meta;

src/components/reducers/ColumnFilters.tsx

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -71,12 +71,15 @@ export function TextFilter(headerProps: DatabaseHeaderProps) {
7171

7272
export function NumberFilter(headerProps: DatabaseHeaderProps) {
7373
const { column } = headerProps;
74-
const min = Number(column.getFacetedMinMaxValues()?.[0] ?? undefined);
75-
const max = Number(column.getFacetedMinMaxValues()?.[1] ?? undefined);
74+
const minRaw = Number(column.getFacetedMinMaxValues()?.[0] ?? undefined);
75+
const maxRaw = Number(column.getFacetedMinMaxValues()?.[1] ?? undefined);
7676
const minValue =
7777
(column.getFilterValue() as [number, number])?.[0] ?? undefined;
7878
const maxValue =
7979
(column.getFilterValue() as [number, number])?.[1] ?? undefined;
80+
81+
const min = Number.isNaN(minRaw) ? Number.MIN_SAFE_INTEGER : minRaw;
82+
const max = Number.isNaN(maxRaw) ? Number.MAX_SAFE_INTEGER : maxRaw;
8083
return (
8184
<>
8285
<div className="flex space-x-2" key={`${column.id}-number-filter`}>
@@ -103,8 +106,8 @@ export function NumberFilter(headerProps: DatabaseHeaderProps) {
103106
/>
104107
<DynamicDebouncedInput
105108
type="number"
106-
min={Number.isNaN(min) ? undefined : min}
107-
max={Number.isNaN(max) ? undefined : max}
109+
min={min}
110+
max={max}
108111
value={maxValue}
109112
onChange={(value) => {
110113
if (value === "" || value === null) {

0 commit comments

Comments
 (0)