@nocobase/client-v2 2.1.0-alpha.34 → 2.1.0-alpha.35

package/src/flow/models/fields/DisplayTitleFieldModel.tsx

@@ -12,7 +12,8 @@ import { Typography } from 'antd';
 import { castArray } from 'lodash';
 import { css } from '@emotion/css';
 import React from 'react';
-import { FieldModel } from '../base/FieldModel';
+import { FieldModel } from '../base';
+import { hasDisplayValue, normalizeDisplayValue } from '../utils/displayValueUtils';
 
 export class DisplayTitleFieldModel extends FieldModel {
   get collectionField(): CollectionField {
@@ -55,13 +56,20 @@ export class DisplayTitleFieldModel extends FieldModel {
     };
     if (titleField) {
       const result = castArray(value).flatMap((v, idx) => {
-        const result = this.renderComponent(v?.[titleField]);
-        const node = v?.[titleField] ? result : 'N/A';
+        const titleCollectionField =
+          this.context.collectionField?.targetCollection?.getField?.(titleField) || this.context.collectionField;
+        const displayValue = normalizeDisplayValue(v?.[titleField], { collectionField: titleCollectionField });
+        const result = this.renderComponent(displayValue);
+        const node = hasDisplayValue(displayValue) ? result : 'N/A';
         return idx === 0 ? [node] : [<span key={`sep-${idx}`}>, </span>, node];
       });
       return <Typography.Text {...typographyProps}>{result}</Typography.Text>;
     } else {
-      const textContent = <Typography.Text {...typographyProps}>{this.renderComponent(value)}</Typography.Text>;
+      const textContent = (
+        <Typography.Text {...typographyProps}>
+          {this.renderComponent(normalizeDisplayValue(value, { collectionField: this.context.collectionField }))}
+        </Typography.Text>
+      );
       return textContent;
     }
   }

package/src/flow/models/fields/SelectFieldModel.tsx

@@ -12,25 +12,55 @@ import { Select, Tag, Tooltip } from 'antd';
 import React from 'react';
 import { FieldModel } from '../base/FieldModel';
 import { MobileSelect } from './mobile-components/MobileSelect';
+import { enumToOptions, getSelectedEnumLabels } from '../../internal/utils/enumOptionsUtils';
+
+const getOriginalEnumOptions = (model: SelectFieldModel) => {
+  const fromEnum = enumToOptions(model.context.collectionField?.uiSchema?.enum, (text) => text) || [];
+  if (fromEnum.length > 0) {
+    return fromEnum.map((option) => ({ ...option }));
+  }
+  const current = Array.isArray(model.props.options) ? model.props.options : [];
+  return current.map((option) => ({ ...option }));
+};
 
 export class SelectFieldModel extends FieldModel {
   render() {
+    const fallbackOptions = getOriginalEnumOptions(this);
     const options = this.props.options?.map((v) => {
       return {
         ...v,
         label: this.translate(v.label),
       };
     });
+    const selectedLabels = getSelectedEnumLabels(this.props.value, fallbackOptions).map((item) => ({
+      ...item,
+      label: this.translate(item.label),
+    }));
+    const value = Array.isArray(this.props.value)
+      ? selectedLabels
+      : selectedLabels.length > 0
+        ? selectedLabels[0]
+        : this.props.value;
 
     // TODO: 移动端相关的代码需迁移到单独的插件中
     if (this.context.isMobileLayout) {
-      return <MobileSelect {...this.props} options={options} />;
+      return <MobileSelect {...this.props} options={options} displayValue={value} />;
     }
 
     return (
       <Select
         {...this.props}
+        value={value}
+        labelInValue
+        onChange={(nextValue) => {
+          if (Array.isArray(nextValue)) {
+            this.props.onChange?.(nextValue.map((item: any) => item?.value));
+            return;
+          }
+          this.props.onChange?.(nextValue?.value);
+        }}
         options={options}
+        labelRender={(item) => item.label}
         maxTagCount="responsive"
         maxTagPlaceholder={(omittedValues) => (
           <Tooltip

package/src/flow/models/fields/__tests__/ClickableFieldModel.test.ts

@@ -9,7 +9,9 @@
 
 import { describe, expect, it, vi } from 'vitest';
 import { FlowEngine } from '@nocobase/flow-engine';
+import { render, screen } from '@testing-library/react';
 import { ClickableFieldModel } from '../ClickableFieldModel';
+import { DisplayTextFieldModel } from '../DisplayTextFieldModel';
 
 function createRolesFieldModel(sourceRecord: Record<string, any>) {
   const engine = new FlowEngine();
@@ -84,4 +86,25 @@ describe('ClickableFieldModel', () => {
       { debounce: true },
     );
   });
+
+  it('renders object title field values by configured target title field', () => {
+    const engine = new FlowEngine();
+    engine.registerModels({ DisplayTextFieldModel });
+
+    const model = engine.createModel<DisplayTextFieldModel>({
+      use: DisplayTextFieldModel,
+      uid: 'display-text-association-title',
+    });
+    model.context.defineProperty('collectionField', {
+      value: {
+        targetCollectionTitleFieldName: 'code',
+        isAssociationField: () => true,
+      },
+    });
+
+    render(model.renderInDisplayStyle({ id: 2, name: 'Sales', code: 'S-001' }, { id: 1 }, false));
+
+    expect(screen.getByText('S-001')).toBeInTheDocument();
+    expect(screen.queryByText('Sales')).not.toBeInTheDocument();
+  });
 });

package/src/flow/models/fields/__tests__/DisplayEnumFieldModel.test.tsx

@@ -0,0 +1,39 @@
+/**
+ * This file is part of the NocoBase (R) project.
+ * Copyright (c) 2020-2024 NocoBase Co., Ltd.
+ * Authors: NocoBase Team.
+ *
+ * This project is dual-licensed under AGPL-3.0 and NocoBase Commercial License.
+ * For more information, please refer to: https://www.nocobase.com/agreement.
+ */
+
+import { describe, expect, it } from 'vitest';
+import { FlowEngine } from '@nocobase/flow-engine';
+import { render, screen } from '@testing-library/react';
+import { DisplayEnumFieldModel } from '../DisplayEnumFieldModel';
+
+describe('DisplayEnumFieldModel', () => {
+  it('renders labels for multiple values (multipleSelect/checkboxGroup)', () => {
+    const engine = new FlowEngine();
+    engine.registerModels({ DisplayEnumFieldModel });
+
+    const model = engine.createModel<DisplayEnumFieldModel>({
+      use: DisplayEnumFieldModel,
+      uid: 'display-enum-multiple-values',
+    });
+
+    model.setProps({
+      value: ['a', 'b'],
+      options: [
+        { label: 'Option A', value: 'a' },
+        { label: 'Option B', value: 'b' },
+      ],
+    });
+
+    render(model.render());
+
+    expect(screen.getByText('Option A')).toBeInTheDocument();
+    expect(screen.getByText('Option B')).toBeInTheDocument();
+    expect(screen.queryByText('a, b')).not.toBeInTheDocument();
+  });
+});

package/src/flow/models/fields/mobile-components/MobileSelect.tsx

@@ -13,7 +13,7 @@ import { Button, CheckList, Popup, SearchBar } from 'antd-mobile';
 import React, { useEffect, useMemo, useState } from 'react';
 
 export function MobileSelect(props) {
-  const { value, onChange, onChangeComplete, disabled, options = [], mode } = props;
+  const { value, displayValue, onChange, onChangeComplete, disabled, options = [], mode } = props;
   const ctx = useFlowModelContext();
   const t = ctx.t;
   const [visible, setVisible] = useState(false);
@@ -44,6 +44,7 @@ export function MobileSelect(props) {
       <div onClick={() => !disabled && setVisible(true)}>
         <Select
           {...props}
+          value={displayValue ?? value}
           open={false}
           dropdownStyle={{ display: 'none' }}
           showSearch={false}

package/src/flow/models/fields/mobile-components/__tests__/MobileSelect.test.tsx

@@ -178,6 +178,13 @@ describe('MobileSelect', () => {
     clickTrigger();
     expect(screen.queryByTestId('popup')).not.toBeInTheDocument();
   });
+
+  it('prefers displayValue for trigger rendering', () => {
+    const displayValue = [{ label: 'Published', value: 'published' }];
+    renderMobileSelect({ value: ['published'], displayValue, mode: 'multiple' });
+
+    expect(mockState.selectProps?.value).toEqual(displayValue);
+  });
 });
 
 function SubTableCellHarness({ value, onCommit, mode }: { value: any; onCommit: (value: any) => void; mode?: string }) {

package/src/flow/models/utils/displayValueUtils.ts

@@ -0,0 +1,57 @@
+/**
+ * This file is part of the NocoBase (R) project.
+ * Copyright (c) 2020-2024 NocoBase Co., Ltd.
+ * Authors: NocoBase Team.
+ *
+ * This project is dual-licensed under AGPL-3.0 and NocoBase Commercial License.
+ * For more information, please refer to: https://www.nocobase.com/agreement.
+ */
+
+import { isValidElement } from 'react';
+
+export function hasDisplayValue(value: any) {
+  return value !== undefined && value !== null && value !== '';
+}
+
+function getTitleFieldName(collectionField: any) {
+  const targetCollection = collectionField?.targetCollection;
+  return (
+    collectionField?.targetCollectionTitleFieldName ||
+    targetCollection?.titleCollectionField?.name ||
+    targetCollection?.titleField ||
+    targetCollection?.options?.titleField
+  );
+}
+
+function getTitleCollectionField(collectionField: any, titleFieldName: string) {
+  return (
+    collectionField?.targetCollection?.getField?.(titleFieldName) ||
+    collectionField?.targetCollection?.titleCollectionField
+  );
+}
+
+export function normalizeDisplayValue(value: any, options: { collectionField?: any } = {}): any {
+  if (!hasDisplayValue(value) || isValidElement(value)) {
+    return value;
+  }
+  if (['string', 'number', 'boolean'].includes(typeof value)) {
+    return value;
+  }
+  if (Array.isArray(value)) {
+    const parts = value.map((item) => normalizeDisplayValue(item, options)).filter(hasDisplayValue);
+    return parts.length ? parts.map(String).join(', ') : undefined;
+  }
+  if (value instanceof Date) {
+    return value.toISOString();
+  }
+  if (typeof value === 'object') {
+    const titleFieldName = getTitleFieldName(options.collectionField);
+    if (titleFieldName) {
+      return normalizeDisplayValue(value[titleFieldName], {
+        collectionField: getTitleCollectionField(options.collectionField, titleFieldName),
+      });
+    }
+    return undefined;
+  }
+  return value;
+}

package/src/utils/globalDeps.ts

@@ -9,7 +9,9 @@
 
 import * as antdCssinjs from '@ant-design/cssinjs';
 import * as antdIcons from '@ant-design/icons';
+import * as antdStyle from 'antd-style';
 import * as emotionCss from '@emotion/css';
+import * as formilyAntdV5 from '@formily/antd-v5';
 import * as formilyCore from '@formily/core';
 import * as formilyReact from '@formily/react';
 import * as formilyReactive from '@formily/reactive';
@@ -19,6 +21,8 @@ import { dayjs } from '@nocobase/utils/client';
 import * as nocobaseFlowEngine from '@nocobase/flow-engine';
 import * as ahooks from 'ahooks';
 import * as antd from 'antd';
+import * as FileSaver from 'file-saver';
+import axios from 'axios';
 import * as i18next from 'i18next';
 import lodash from 'lodash';
 import React from 'react';
@@ -48,12 +52,14 @@ export function defineGlobalDeps(requirejs: RequireJS) {
   requirejs.define('antd', () => antd);
   requirejs.define('@ant-design/icons', () => antdIcons);
   requirejs.define('@ant-design/cssinjs', () => antdCssinjs);
+  requirejs.define('antd-style', () => antdStyle);
 
   // i18next
   requirejs.define('i18next', () => i18next);
   requirejs.define('react-i18next', () => reactI18next);
 
   // formily
+  requirejs.define('@formily/antd-v5', () => formilyAntdV5);
   requirejs.define('@formily/core', () => formilyCore);
   requirejs.define('@formily/react', () => formilyReact);
   requirejs.define('@formily/reactive', () => formilyReactive);
@@ -68,7 +74,9 @@ export function defineGlobalDeps(requirejs: RequireJS) {
 
   // utils
   requirejs.define('ahooks', () => ahooks);
+  requirejs.define('axios', () => axios);
   requirejs.define('dayjs', () => dayjs);
   requirejs.define('lodash', () => lodash);
   requirejs.define('@emotion/css', () => emotionCss);
+  requirejs.define('file-saver', () => FileSaver);
 }

package/src/utils/remotePlugins.ts

@@ -12,33 +12,16 @@ import type { PluginClass } from '../PluginManager';
 import type { PluginData } from '../PluginManager';
 import type { RequireJS } from './requirejs';
 
-/**
- * @internal
- */
-export function defineDevPlugins(plugins: Record<string, PluginClass>) {
-  Object.entries(plugins).forEach(([packageName, plugin]) => {
-    window.define(`${packageName}/client-v2`, () => plugin);
-  });
+function getClientV2ModuleId(packageName: string) {
+  return `${packageName}/client-v2`;
 }
 
 /**
  * @internal
  */
-export function definePluginClient(packageName: string) {
-  window.define(`${packageName}/client-v2`, ['exports', packageName], function (_exports: any, _pluginExports: any) {
-    Object.defineProperty(_exports, '__esModule', {
-      value: true,
-    });
-    Object.keys(_pluginExports).forEach(function (key) {
-      if (key === '__esModule') return;
-      if (key in _exports && _exports[key] === _pluginExports[key]) return;
-      Object.defineProperty(_exports, key, {
-        enumerable: true,
-        get: function () {
-          return _pluginExports[key];
-        },
-      });
-    });
+export function defineDevPlugins(plugins: Record<string, PluginClass>) {
+  Object.entries(plugins).forEach(([packageName, plugin]) => {
+    window.define(getClientV2ModuleId(packageName), () => plugin);
   });
 }
 
@@ -49,7 +32,7 @@ export function configRequirejs(requirejs: any, pluginData: PluginData[]) {
   requirejs.requirejs.config({
     waitSeconds: 120,
     paths: pluginData.reduce<Record<string, string>>((acc, cur) => {
-      acc[cur.packageName] = cur.url;
+      acc[getClientV2ModuleId(cur.packageName)] = cur.url;
       return acc;
     }, {}),
   });
@@ -85,10 +68,7 @@ export function processRemotePlugins(pluginData: PluginData[], resolve: (plugins
 export function getRemotePlugins(requirejs: any, pluginData: PluginData[] = []): Promise<Array<[string, PluginClass]>> {
   configRequirejs(requirejs, pluginData);
 
-  const packageNames = pluginData.map((item) => item.packageName);
-  packageNames.forEach((packageName) => {
-    definePluginClient(packageName);
-  });
+  const packageNames = pluginData.map((item) => getClientV2ModuleId(item.packageName));
 
   return new Promise((resolve, reject) => {
     requirejs.requirejs(packageNames, processRemotePlugins(pluginData, resolve), reject);