fix(workflow): workflow node variables do not display inherited collection fields (#5415)

* fix: missing fields in the inherited table

* fix(client): fix hidden collection

* refactor(client): add new api to get all fields on collection manager

---------

Co-authored-by: mytharcher <mytharcher@gmail.com>

packages/core/client/src/data-source/collection/Collection.ts

@@ -250,6 +250,10 @@ export class Collection {
     return predicate ? filter(this.fields, predicate) : this.fields;
   }
 
+  getAllFields(predicate?: GetCollectionFieldPredicate) {
+    return this.getFields(predicate);
+  }
+
   protected getFieldsMap() {
     if (!this.fieldsMap) {
       this.fieldsMap = this.getFields().reduce((memo, field) => {

packages/core/client/src/data-source/collection/CollectionManager.ts

@@ -140,6 +140,10 @@ export class CollectionManager {
     return this.getCollection(collectionName)?.getFields(predicate) || [];
   }
 
+  getCollectionAllFields(collectionName: string, predicate?: GetCollectionFieldPredicate) {
+    return this.getCollection(collectionName)?.getAllFields(predicate) || [];
+  }
+
   /**
    * @example
    * getFilterByTK('users', { id: 1 }); // 1

packages/core/client/src/schema-component/antd/appends-tree-select/AppendsTreeSelect.tsx

@@ -12,13 +12,7 @@ import { Tag, TreeSelect } from 'antd';
 import type { DefaultOptionType, TreeSelectProps } from 'rc-tree-select/es/TreeSelect';
 import React, { useCallback, useEffect, useMemo, useState } from 'react';
 import { useTranslation } from 'react-i18next';
-import {
+import { CollectionFieldOptions_deprecated, parseCollectionName, useApp, useCompile } from '../../..';
-  CollectionFieldOptions_deprecated,
-  parseCollectionName,
-  useApp,
-  useCollectionManager_deprecated,
-  useCompile,
-} from '../../..';
 
 export type AppendsTreeSelectProps = {
   value: string[] | string;
@@ -106,7 +100,11 @@ export const AppendsTreeSelect: React.FC<TreeSelectProps & AppendsTreeSelectProp
   const [dataSourceName, collectionName] = parseCollectionName(collectionString);
   const app = useApp();
   const { collectionManager } = app.dataSourceManager.getDataSource(dataSourceName);
-  const getCollectionFields = collectionManager.getCollectionFields.bind(collectionManager);
+  const getCollectionFields = (name, predicate) => {
+    const instance = collectionManager.getCollection(name);
+    // NOTE: condition for compatibility with hidden collections like "attachments"
+    return instance ? instance.getAllFields(predicate) : [];
+  };
   const treeData = Object.values(optionsMap);
   const value: string | DefaultOptionType[] = useMemo(() => {
     if (props.multiple) {

packages/plugins/@nocobase/plugin-workflow/src/client/variable.tsx

@@ -503,7 +503,7 @@ export function useGetCollectionFields(dataSourceName?) {
   const app = useApp();
   const { collectionManager } = app.dataSourceManager.getDataSource(dataSourceName);
 
-  return useCallback((collectionName) => collectionManager.getCollectionFields(collectionName), [collectionManager]);
+  return useCallback((collectionName) => collectionManager.getCollectionAllFields(collectionName), [collectionManager]);
 }
 
 export function WorkflowVariableInput({ variableOptions, ...props }): JSX.Element {