fix: 明细表未指定 dataCfg.fields 配置时不应该渲染空数据占位 (#3003)

* fix: 明细表未指定 dataCfg.fields 配置时不渲染空数据占位

* fix: 优化写法

packages/s2-core/__tests__/spreadsheet/table-sheet-spec.ts

@@ -429,5 +429,30 @@ describe('TableSheet normal spec', () => {
 
       await expectEmptyPlaceholder(s2);
     });
+
+    test('should not render empty placeholder if all fields is empty', async () => {
+      const s2 = new TableSheet(
+        getContainer(),
+        { ...dataCfg, fields: {}, data: [] },
+        {
+          ...options,
+          frozen: {},
+          seriesNumber: {
+            enable: false,
+          },
+        },
+      );
+
+      await s2.render();
+      const [rect, icon, text] = (s2.facet as TableFacet).emptyPlaceholderGroup
+        .children;
+
+      expect(
+        (s2.facet as TableFacet).emptyPlaceholderGroup.children,
+      ).toHaveLength(0);
+      expect(rect).not.toBeDefined();
+      expect(icon).not.toBeDefined();
+      expect(text).not.toBeDefined();
+    });
   });
 });

packages/s2-core/src/facet/base-facet.ts

@@ -236,6 +236,10 @@ export abstract class BaseFacet {
     this.init();
   }
 
+  protected shouldRender() {
+    return !areAllFieldsEmpty(this.spreadsheet.dataCfg.fields);
+  }
+
   public getLayoutResult = (): LayoutResult => {
     return {
       ...this.layoutResult,
@@ -609,11 +613,8 @@ export abstract class BaseFacet {
     this.emitPaginationEvent();
   };
 
-  /**
-   * Start render, call from outside
-   */
   public render() {
-    if (areAllFieldsEmpty(this.spreadsheet.dataCfg.fields)) {
+    if (!this.shouldRender()) {
       return;
     }
 

packages/s2-core/src/facet/frozen-facet.ts

@@ -675,6 +675,10 @@ export abstract class FrozenFacet extends BaseFacet {
   }
 
   public render() {
+    if (!this.shouldRender()) {
+      return;
+    }
+
     this.calculateFrozenGroupInfo();
     this.renderFrozenPanelCornerGroup();
     super.render();

packages/s2-core/src/facet/table-facet.ts

@@ -83,6 +83,10 @@ export class TableFacet extends FrozenFacet {
   }
 
   public render() {
+    if (!this.shouldRender()) {
+      return;
+    }
+
     super.render();
     this.renderEmptyPlaceholder();
   }