fix: filter data value doesn't match desktop

Author: appflowy

src/application/database-yjs/database.type.ts

@@ -42,9 +42,9 @@ export enum SortCondition {
 }
 
 export enum FilterType {
-  Data = 0,
-  And = 1,
-  Or = 2,
+  And = 0,
+  Or = 1,
+  Data = 2,
 }
 
 export interface Filter {

src/application/database-yjs/dispatch.ts

@@ -5,7 +5,6 @@ import { useCallback, useMemo } from 'react';
 import { v4 as uuidv4 } from 'uuid';
 import * as Y from 'yjs';
 
-import { Log } from '@/utils/log';
 import {
   useCreateRow,
   useDatabase,
@@ -76,8 +75,9 @@ import {
   YSharedRoot,
 } from '@/application/types';
 import { DefaultTimeSetting } from '@/application/user-metadata';
-import { applyYDoc } from '@/application/ydoc/apply';
 import { isDatabaseContainer } from '@/application/view-utils';
+import { applyYDoc } from '@/application/ydoc/apply';
+import { Log } from '@/utils/log';
 
 export function useResizeColumnWidthDispatch() {
   const database = useDatabase();
@@ -3455,18 +3455,40 @@ export function useAddFilter() {
 
   return useCallback(
     (fieldId: string) => {
-      if (!view) return;
+      Log.debug('[useAddFilter] Creating filter', { fieldId });
+
+      // Guard: Don't create filter if fieldId is missing or empty
+      if (!view || !fieldId || fieldId.trim() === '') {
+        Log.warn('[useAddFilter] Skipping filter creation: view or fieldId is missing', {
+          hasView: !!view,
+          fieldId,
+        });
+        return;
+      }
+
       const id = `${nanoid(6)}`;
 
+      Log.debug('[useAddFilter] Generated filter id', { filterId: id, fieldId });
+
       executeOperations(
         sharedRoot,
         [
           () => {
             const field = fields.get(fieldId);
+
+            if (!field) {
+              Log.warn('[useAddFilter] Field not found for fieldId:', fieldId);
+              return;
+            }
+
             const fieldType = Number(field.get(YjsDatabaseKey.type));
+
+            Log.debug('[useAddFilter] Field info', { fieldId, fieldType });
+
             let filters = view.get(YjsDatabaseKey.filters);
 
             if (!filters) {
+              Log.debug('[useAddFilter] Creating new filters array');
               filters = new Y.Array() as YDatabaseFilters;
               view.set(YjsDatabaseKey.filters, filters);
             }
@@ -3477,7 +3499,18 @@ export function useAddFilter() {
             filter.set(YjsDatabaseKey.field_id, fieldId);
             const conditionData = getDefaultFilterCondition(fieldType);
 
-            if (!conditionData) return;
+            if (!conditionData) {
+              Log.warn('[useAddFilter] No default condition for fieldType:', fieldType);
+              return;
+            }
+
+            Log.debug('[useAddFilter] Setting filter data', {
+              filterId: id,
+              fieldId,
+              fieldType,
+              condition: conditionData.condition,
+              content: conditionData.content,
+            });
 
             filter.set(YjsDatabaseKey.condition, conditionData.condition);
             if (conditionData.content !== undefined) {
@@ -3488,6 +3521,8 @@ export function useAddFilter() {
             filter.set(YjsDatabaseKey.filter_type, FilterType.Data);
 
             filters.push([filter]);
+
+            Log.debug('[useAddFilter] Filter created successfully', { filterId: id, filter: filter.toJSON() });
           },
         ],
         'addFilter'
@@ -3565,10 +3600,13 @@ export function useUpdateFilter() {
               return;
             }
 
-            if (fieldId) {
-              filter.set(YjsDatabaseKey.field_id, fieldId);
+            if (!fieldId) {
+              Log.warn('[useUpdateFilter] FieldId is missing');
+              return;
             }
 
+            filter.set(YjsDatabaseKey.field_id, fieldId);
+
             if (condition !== undefined) {
               filter.set(YjsDatabaseKey.condition, condition);
             }