fix: change autorun params to IPublicModelSettingField

liujuping · JackLian · commit 4f7a0b984c50 · 2023-04-03T14:57:52.000+08:00
diff --git a/packages/designer/jest.config.js b/packages/designer/jest.config.js
@@ -17,6 +17,8 @@ const jestConfig = {
   // testMatch: ['(/tests?/.*(test))\\.[jt]s$'],
   // testMatch: ['**/document/node/node.add.test.ts'],
   // testMatch: ['**/setting-field.test.ts'],
+  // testMatch: ['**/node.test.ts'],
+  // testMatch: ['**/builtin-hotkey.test.ts'],
   transformIgnorePatterns: [
     `/node_modules/(?!${esModules})/`,
   ],
diff --git a/packages/designer/src/designer/setting/utils.ts b/packages/designer/src/designer/setting/utils.ts
@@ -2,7 +2,7 @@
 import { isValidElement } from 'react';
 import { IPublicTypeFieldConfig, IPublicTypeSetterConfig } from '@alilc/lowcode-types';
 import { isSetterConfig, isDynamicSetter } from '@alilc/lowcode-utils';
-import { SettingField } from './setting-field';
+import { ISettingField } from './setting-field';
 
 function getHotterFromSetter(setter) {
   return setter && (setter.Hotter || (setter.type && setter.type.Hotter)) || []; // eslint-disable-line
@@ -35,7 +35,7 @@ export class Transducer {
 
   context: any;
 
-  constructor(context: SettingField, config: { setter: IPublicTypeFieldConfig['setter'] }) {
+  constructor(context: ISettingField, config: { setter: IPublicTypeFieldConfig['setter'] }) {
     let { setter } = config;
 
     // 1. validElement
diff --git a/packages/designer/src/document/node/node.ts b/packages/designer/src/document/node/node.ts
@@ -30,7 +30,6 @@ import type { IExclusiveGroup } from './exclusive-group';
 import { includeSlot, removeSlot } from '../../utils/slot';
 import { foreachReverse } from '../../utils/tree';
 import { NodeRemoveOptions, EDITOR_EVENT } from '../../types';
-import { Prop as ShellProp } from '@alilc/lowcode-shell';
 
 export interface NodeStatus {
   locking: boolean;
@@ -432,7 +431,7 @@ export class Node<Schema extends IPublicTypeNodeSchema = IPublicTypeNodeSchema>
     }
     this.autoruns = autoruns.map((item) => {
       return autorun(() => {
-        item.autorun(ShellProp.create(this.props.get(item.name, true))!);
+        item.autorun(this.props.getNode().settingEntry.get(item.name)?.internalToShellField());
       });
     });
   }
diff --git a/packages/designer/tests/document/node/node.test.ts b/packages/designer/tests/document/node/node.test.ts
@@ -1,8 +1,13 @@
 // @ts-nocheck
 import '../../fixtures/window';
 import { set } from '../../utils';
-import { Editor } from '@alilc/lowcode-editor-core';
+import {
+  Editor,
+  globalContext,
+  Setters as InnerSetters,
+} from '@alilc/lowcode-editor-core';
 import { Project } from '../../../src/project/project';
+import { Workspace as InnerWorkspace } from '@alilc/lowcode-workspace';
 import { DocumentModel } from '../../../src/document/document-model';
 import {
   isRootNode,
@@ -23,6 +28,7 @@ import rootContentMetadata from '../../fixtures/component-metadata/root-content'
 import rootFooterMetadata from '../../fixtures/component-metadata/root-footer';
 import { shellModelFactory } from '../../../../engine/src/modules/shell-model-factory';
 import { isNode } from '@alilc/lowcode-utils';
+import { Setters } from '@alilc/lowcode-shell';
 
 describe('Node 方法测试', () => {
   let editor: Editor;
@@ -35,6 +41,9 @@ describe('Node 方法测试', () => {
     designer = new Designer({ editor, shellModelFactory });
     project = designer.project;
     doc = new DocumentModel(project, formSchema);
+    editor.set('setters', new Setters(new InnerSetters()));
+    !globalContext.has(Editor) && globalContext.register(editor, Editor);
+    !globalContext.has('workspace') && globalContext.register(new InnerWorkspace(), 'workspace');
   });
 
   afterEach(() => {
diff --git a/packages/engine/src/inner-plugins/builtin-hotkey.ts b/packages/engine/src/inner-plugins/builtin-hotkey.ts
@@ -176,10 +176,9 @@ function getSuitablePlaceForNode(targetNode: IPublicModelNode, node: IPublicMode
   if (node?.componentMeta?.isModal) {
     return { container: focusNode, ref };
   }
-  const canDropInFn = document.checkNesting;
 
   if (!ref && focusNode && targetNode.contains(focusNode)) {
-    if (canDropInFn(focusNode, dragNodeObject)) {
+    if (document.checkNesting(focusNode, dragNodeObject)) {
       return { container: focusNode };
     }
 
@@ -191,7 +190,7 @@ function getSuitablePlaceForNode(targetNode: IPublicModelNode, node: IPublicMode
       if (!c.isContainerNode) {
         return false;
       }
-      if (canDropInFn(c, dragNodeObject)) {
+      if (document.checkNesting(c, dragNodeObject)) {
         return true;
       }
       return false;
@@ -201,15 +200,15 @@ function getSuitablePlaceForNode(targetNode: IPublicModelNode, node: IPublicMode
       return { container: dropElement, ref };
     }
 
-    if (canDropInFn(targetNode, dragNodeObject)) {
+    if (document.checkNesting(targetNode, dragNodeObject)) {
       return { container: targetNode, ref };
     }
 
     return null;
   }
 
   if (targetNode.isContainerNode) {
-    if (canDropInFn(targetNode, dragNodeObject)) {
+    if (document.checkNesting(targetNode, dragNodeObject)) {
       return { container: targetNode, ref };
     }
   }
diff --git a/packages/types/src/shell/type/metadata.ts b/packages/types/src/shell/type/metadata.ts
@@ -1,6 +1,6 @@
 import { MouseEvent } from 'react';
 import { IPublicTypePropType, IPublicTypeComponentAction } from './';
-import { IPublicModelNode, IPublicModelProp, IPublicModelSettingField } from '../model';
+import { IPublicModelNode, IPublicModelSettingField } from '../model';
 
 /**
  * 嵌套控制函数
@@ -102,7 +102,7 @@ export interface IPublicTypeFilterItem {
 }
 export interface IPublicTypeAutorunItem {
   name: string;
-  autorun: (prop: IPublicModelProp) => any;
+  autorun: (target: IPublicModelSettingField | null) => any;
 }
 
 // thinkof Array

Original file line number	Diff line number	Diff line change
`@@ -176,10 +176,9 @@ function getSuitablePlaceForNode(targetNode: IPublicModelNode, node: IPublicMode`
`176`	`176`	`if (node?.componentMeta?.isModal) {`
`177`	`177`	`return { container: focusNode, ref };`
`178`	`178`	`}`
`179`		`- const canDropInFn = document.checkNesting;`
`180`	`179`
`181`	`180`	`if (!ref && focusNode && targetNode.contains(focusNode)) {`
`182`		`- if (canDropInFn(focusNode, dragNodeObject)) {`
	`181`	`+ if (document.checkNesting(focusNode, dragNodeObject)) {`
`183`	`182`	`return { container: focusNode };`
`184`	`183`	`}`
`185`	`184`
`@@ -191,7 +190,7 @@ function getSuitablePlaceForNode(targetNode: IPublicModelNode, node: IPublicMode`
`191`	`190`	`if (!c.isContainerNode) {`
`192`	`191`	`return false;`
`193`	`192`	`}`
`194`		`- if (canDropInFn(c, dragNodeObject)) {`
	`193`	`+ if (document.checkNesting(c, dragNodeObject)) {`
`195`	`194`	`return true;`
`196`	`195`	`}`
`197`	`196`	`return false;`
`@@ -201,15 +200,15 @@ function getSuitablePlaceForNode(targetNode: IPublicModelNode, node: IPublicMode`
`201`	`200`	`return { container: dropElement, ref };`
`202`	`201`	`}`
`203`	`202`
`204`		`- if (canDropInFn(targetNode, dragNodeObject)) {`
	`203`	`+ if (document.checkNesting(targetNode, dragNodeObject)) {`
`205`	`204`	`return { container: targetNode, ref };`
`206`	`205`	`}`
`207`	`206`
`208`	`207`	`return null;`
`209`	`208`	`}`
`210`	`209`
`211`	`210`	`if (targetNode.isContainerNode) {`
`212`		`- if (canDropInFn(targetNode, dragNodeObject)) {`
	`211`	`+ if (document.checkNesting(targetNode, dragNodeObject)) {`
`213`	`212`	`return { container: targetNode, ref };`
`214`	`213`	`}`
`215`	`214`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`import { MouseEvent } from 'react';`
`2`	`2`	`import { IPublicTypePropType, IPublicTypeComponentAction } from './';`
`3`		`-import { IPublicModelNode, IPublicModelProp, IPublicModelSettingField } from '../model';`
	`3`	`+import { IPublicModelNode, IPublicModelSettingField } from '../model';`
`4`	`4`
`5`	`5`	`/**`
`6`	`6`	`* 嵌套控制函数`
`@@ -102,7 +102,7 @@ export interface IPublicTypeFilterItem {`
`102`	`102`	`}`
`103`	`103`	`export interface IPublicTypeAutorunItem {`
`104`	`104`	`name: string;`
`105`		`- autorun: (prop: IPublicModelProp) => any;`
	`105`	`+ autorun: (target: IPublicModelSettingField \| null) => any;`
`106`	`106`	`}`
`107`	`107`
`108`	`108`	`// thinkof Array`