fix(filter): should not filter out zero (#5106)

* fix(filter): should not filter out zero

* chore: make e2e test pass

packages/core/client/src/filter-provider/__tests__/utiles.test.ts

@@ -196,6 +196,32 @@ describe('transformToFilter', () => {
     expect(filter).toEqual(expectedFilter);
   });
 
+  it('should keep 0 value', () => {
+    const valuesWithZero = {
+      field1: 0,
+      field2: 'value2',
+    };
+
+    const expectedFilter = {
+      $and: [
+        {
+          field1: {
+            $eq: 0,
+          },
+        },
+        {
+          field2: {
+            $ne: 'value2',
+          },
+        },
+      ],
+    };
+
+    const filter = transformToFilter(valuesWithZero, operators, getCollectionJoinField, collectionName);
+
+    expect(filter).toEqual(expectedFilter);
+  });
+
   it('should handle null values', () => {
     const valuesWithNull = {
       field1: null,

packages/core/client/src/filter-provider/utils.ts

@@ -144,7 +144,7 @@ export const transformToFilter = (
           key = `${key}.${collectionField.targetKey || 'id'}`;
         }
 
-        if (!value) {
+        if (!value && value !== 0 && value !== false) {
           return null;
         }
 
@@ -167,7 +167,7 @@ export const useAssociatedFields = () => {
 };
 
 export const isAssocField = (field?: FieldOptions) => {
-  return ['o2o', 'oho', 'obo', 'm2o', 'createdBy', 'updatedBy', 'o2m', 'm2m', 'linkTo', 'chinaRegion'].includes(
+  return ['o2o', 'oho', 'obo', 'm2o', 'createdBy', 'updatedBy', 'o2m', 'm2m', 'linkTo', 'chinaRegion', 'mbm'].includes(
     field?.interface,
   );
 };

packages/core/client/src/schema-component/antd/filter/__tests__/useFilterActionProps.test.ts

@@ -65,4 +65,17 @@ describe('removeNullCondition', () => {
     const result = removeNullCondition(filter);
     expect(result).toEqual(expected);
   });
+
+  it('should keep 0 value', () => {
+    const filter = {
+      field1: 0,
+      field2: 'value2',
+    };
+    const expected = {
+      field1: 0,
+      field2: 'value2',
+    };
+    const result = removeNullCondition(filter);
+    expect(result).toEqual(expected);
+  });
 });