fix(cf): when previewing a number format, the conditional format styl…

…e does not take effect (#439)
dream-num · Mar 19, 2024 · d9e1e2a · d9e1e2a
1 parent 83599d3
commit d9e1e2a
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/packages/sheets-numfmt/src/controllers/numfmt.controller.ts b/packages/sheets-numfmt/src/controllers/numfmt.controller.ts
@@ -290,6 +290,7 @@ export class NumfmtController extends Disposable implements INumfmtController {
                                 priority: 99,
                                 handler: (cell, location, next) => {
                                     const { row, col } = location;
+                                    const defaultValue = next(cell) || {};
                                     if (
                                         selectionRanges.find(
                                             (range) =>
@@ -308,26 +309,26 @@ export class NumfmtController extends Disposable implements INumfmtController {
                                             type !== CellValueType.NUMBER ||
                                             this._previewPattern === null
                                         ) {
-                                            return next(cell);
+                                            return defaultValue;
                                         }
                                         const info = getPatternPreview(this._previewPattern, value as number, this._localeService.getCurrentLocale());
                                         if (info.color) {
                                             const colorMap = this._themeService.getCurrentTheme();
                                             const color = colorMap[`${info.color}500`];
                                             return {
-                                                ...cell,
+                                                ...defaultValue,
                                                 v: info.result,
                                                 t: CellValueType.STRING,
                                                 s: { cl: { rgb: color } },
                                             };
                                         }
                                         return {
-                                            ...cell,
+                                            ...defaultValue,
                                             v: info.result,
                                             t: CellValueType.STRING,
                                         };
                                     }
-                                    return next(cell);
+                                    return defaultValue;
                                 },
                             })
                         );